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/CAE-AwArTNGhABmJCw%3D_ArbCNAVx%2BVvt%3DkNp-ZQvb23uwudRubg%40mail.gmail.com.