LGTM3 On Fri, Mar 15, 2024 at 1:21 PM Mike Taylor <miketa...@chromium.org> wrote:
> Thank you! LGTM2 > On 3/15/24 1:13 PM, Tommy Steimel wrote: > > Got it, thanks! Adding one in crrev.com/c/5375045 > > On Fri, Mar 15, 2024 at 7:51 AM Mike Taylor <miketa...@chromium.org> > wrote: > >> If a human can observe the expected behavior by clicking on a button and >> noticing the lack of the return to tab button, then effectively yes. >> On 3/15/24 10:40 AM, Tommy Steimel wrote: >> >> I didn't realize you could do that. So if I make a manual WPT test, I can >> make it depend on code in //chrome? >> >> On Fri, Mar 15, 2024 at 6:15 AM Mike Taylor <miketa...@chromium.org> >> wrote: >> >>> Is there any reason to not convert >>> https://steimelchrome.github.io/document-pip/hide-back-to-tab-button.html >>> into a manual WPT >>> <https://web-platform-tests.org/writing-tests/manual.html>? Manual >>> tests aren't ideal, but something seems better than nothing. >>> On 3/14/24 9:42 PM, Domenic Denicola wrote: >>> >>> Awesome, thanks for the quick turnaround time! With that change, LGTM1. >>> >>> On Fri, Mar 15, 2024 at 2:16 AM Tommy Steimel <stei...@google.com> >>> wrote: >>> >>>> Thanks for the feedback! Updated in >>>> https://github.com/WICG/document-picture-in-picture/pull/116 >>>> >>>> On Wed, Mar 13, 2024 at 6:53 PM Domenic Denicola <dome...@chromium.org> >>>> wrote: >>>> >>>>> I found an issue with the API design here that might result in a >>>>> backward-incompatible change: >>>>> https://github.com/WICG/document-picture-in-picture/issues/115 >>>>> >>>>> With my spec mentor hat on, sorry for not catching it sooner! >>>>> >>>>> On Thu, Mar 14, 2024 at 2:12 AM 'Tommy Steimel' via blink-dev < >>>>> blink-dev@chromium.org> wrote: >>>>> >>>>>> Contact emails stei...@chromium.org, liber...@chromium.org >>>>>> >>>>>> Explainer None >>>>>> >>>>>> Specification >>>>>> https://github.com/WICG/document-picture-in-picture/pull/114 >>>>>> >>>>>> Summary >>>>>> >>>>>> This adds a new parameter ("allowReturnToOpener") to the document >>>>>> picture-in-picture API that, when set to false, hints to the user agent >>>>>> that they should not show a button in the picture-in-picture window that >>>>>> allows the user to return to the opener. While having a button to return >>>>>> content to the opener always makes sense in the video picture-in-picture >>>>>> case (the video stream can be returned to the video element in the opener >>>>>> tab), this is not always the case for document picture-in-picture >>>>>> experiences. This gives developers more control over the user experience >>>>>> when they determine that such a button does not make sense for their use >>>>>> case. >>>>>> >>>>>> >>>>>> Blink component Blink>Media>PictureInPicture >>>>>> <https://bugs.chromium.org/p/chromium/issues/list?q=component:Blink%3EMedia%3EPictureInPicture> >>>>>> >>>>>> TAG review >>>>>> https://github.com/w3ctag/design-reviews/issues/798#issuecomment-1967916721 >>>>>> >>>>>> TAG review status Pending >>>>>> >>>>>> Risks >>>>>> >>>>>> >>>>>> Interoperability and Compatibility >>>>>> >>>>>> None >>>>>> >>>>>> >>>>>> *Gecko*: No signal ( >>>>>> https://github.com/mozilla/standards-positions/issues/670#issuecomment-1967919675) >>>>>> Added comment to existing standards position issue for document >>>>>> picture-in-picture. No response yet >>>>>> >>>>>> *WebKit*: No signal ( >>>>>> https://github.com/WebKit/standards-positions/issues/41#issuecomment-1967918830) >>>>>> Added comment to existing standards position issue for document >>>>>> picture-in-picture. No response yet >>>>>> >>>>>> *Web developers*: Positive ( >>>>>> https://github.com/WICG/document-picture-in-picture/issues/113) We >>>>>> have received feature requests for the ability to hide the "back to tab" >>>>>> button from the document picture-in-picture window. >>>>>> >>>>>> *Other signals*: >>>>>> >>>>>> Ergonomics >>>>>> >>>>>> N/A >>>>>> >>>>>> >>>>>> Activation >>>>>> >>>>>> N/A >>>>>> >>>>>> >>>>>> Security >>>>>> >>>>>> N/A >>>>>> >>>>>> >>>>>> WebView application risks >>>>>> >>>>>> Does this intent deprecate or change behavior of existing APIs, such >>>>>> that it has potentially high risk for Android WebView-based applications? >>>>>> >>>>>> N/A, as this is not available on Android >>>>>> >>>>>> >>>>>> Debuggability >>>>>> >>>>>> N/A >>>>>> >>>>>> >>>>>> Will this feature be supported on all six Blink platforms (Windows, >>>>>> Mac, Linux, ChromeOS, Android, and Android WebView)? No >>>>>> >>>>>> The document picture-in-picture API is not supported on Android >>>>>> >>>>>> >>>>>> Is this feature fully tested by web-platform-tests >>>>>> <https://chromium.googlesource.com/chromium/src/+/main/docs/testing/web_platform_tests.md> >>>>>> ? No >>>>>> >>>>>> The document picture-in-picture feature itself is fully tested on >>>>>> WPT, but this additional parameter isn't since it's a hint to the user >>>>>> agent and therefore any actual changes happen in the embedder >>>>>> >>>>>> >>>>>> Flag name on chrome://flags None >>>>>> >>>>>> Finch feature name None >>>>>> >>>>>> Non-finch justification >>>>>> >>>>>> Small change to existing API >>>>>> >>>>>> >>>>>> Requires code in //chrome? False >>>>>> >>>>>> Sample links >>>>>> >>>>>> https://steimelchrome.github.io/document-pip/hide-back-to-tab-button.html >>>>>> >>>>>> Estimated milestones >>>>>> Shipping on desktop 124 >>>>>> >>>>>> Anticipated spec changes >>>>>> >>>>>> Open questions about a feature may be a source of future web compat >>>>>> or interop issues. Please list open issues (e.g. links to known github >>>>>> issues in the project for the feature specification) whose resolution may >>>>>> introduce web compat/interop risk (e.g., changing to naming or structure >>>>>> of >>>>>> the API in a non-backward-compatible way). >>>>>> N/A >>>>>> >>>>>> Link to entry on the Chrome Platform Status >>>>>> https://chromestatus.com/feature/6223347936657408 >>>>>> >>>>>> This intent message was generated by Chrome Platform Status >>>>>> <https://chromestatus.com/>. >>>>>> -- >>>>>> You received this message because you are subscribed to the Google >>>>>> Groups "blink-dev" group. >>>>>> To unsubscribe from this group and stop receiving emails from it, >>>>>> send an email to blink-dev+unsubscr...@chromium.org. >>>>>> To view this discussion on the web visit >>>>>> https://groups.google.com/a/chromium.org/d/msgid/blink-dev/CAE-AwAqR%2BNBOJT4h9YRkdOB9ksbPYgFCfP5JvmTKuCbFA-4-cQ%40mail.gmail.com >>>>>> <https://groups.google.com/a/chromium.org/d/msgid/blink-dev/CAE-AwAqR%2BNBOJT4h9YRkdOB9ksbPYgFCfP5JvmTKuCbFA-4-cQ%40mail.gmail.com?utm_medium=email&utm_source=footer> >>>>>> . >>>>>> >>>>> -- >>> You received this message because you are subscribed to the Google >>> Groups "blink-dev" group. >>> To unsubscribe from this group and stop receiving emails from it, send >>> an email to blink-dev+unsubscr...@chromium.org. >>> To view this discussion on the web visit >>> https://groups.google.com/a/chromium.org/d/msgid/blink-dev/CAM0wra-v7hqnNZrXm866Vq4MJxGX0eCxxEz4k3SYxyLSrDQ%3D1g%40mail.gmail.com >>> <https://groups.google.com/a/chromium.org/d/msgid/blink-dev/CAM0wra-v7hqnNZrXm866Vq4MJxGX0eCxxEz4k3SYxyLSrDQ%3D1g%40mail.gmail.com?utm_medium=email&utm_source=footer> >>> . >>> >>> -- > You received this message because you are subscribed to the Google Groups > "blink-dev" group. > To unsubscribe from this group and stop receiving emails from it, send an > email to blink-dev+unsubscr...@chromium.org. > To view this discussion on the web visit > https://groups.google.com/a/chromium.org/d/msgid/blink-dev/1de2b46b-b43d-4ecf-b1e3-afaf87bdb02c%40chromium.org > <https://groups.google.com/a/chromium.org/d/msgid/blink-dev/1de2b46b-b43d-4ecf-b1e3-afaf87bdb02c%40chromium.org?utm_medium=email&utm_source=footer> > . > -- You received this message because you are subscribed to the Google Groups "blink-dev" group. To unsubscribe from this group and stop receiving emails from it, send an email to blink-dev+unsubscr...@chromium.org. To view this discussion on the web visit https://groups.google.com/a/chromium.org/d/msgid/blink-dev/CAOMQ%2Bw_zF_WTCJ8CF2fH2ZW1rDJNUA4G-AyKB4bj71n0QfS6FA%40mail.gmail.com.