+beccahughes@ because I've seen a CL that's not too far from the bug description.

Amy, would it make sense to move this as a P2? The cost may outweigh the benefits here as we would need to handle multiple hiding logic and there are a few edge cases that are not specified in the video mock such as what happens if the user clicks on the video.

I'd be open to negotiating on a different P1 to get this in, this is a major polish item. Without it, either all the controls stick around too long and the play button obscures the video, or the scrubber controls disappear too quickly. 

Additionally, it's a standard pattern that YT, Netflix, etc all use, so I'm concerned that breaking from that norm would appear broken.

(examples for context https://photos.app.goo.gl/XX8s99MvBiZYZ5cN2)

LMK if you need detailed specs for the logic handling and edge case specifics. 

I'm actually not sure to understand the rationale.

YouTube doesn't show a play button overlay on desktop. It shows a quick play/pause animation but clicking anywhere on the video will trigger play/pause. They do not have the same issue we have with hiding the button that will be needed to pause the playback.

On mobile (native app), YouTube shows a play/pause button overlay but it does not hide it quickly. I did not measure the time but it's fairly similar to what we are doing today as far as I can tell.

Netflix never appears to have a play/pause button overlay, either on desktop or on their native application.

Moving to P2 but beccahughes has a CL up so we can either close this or land it today. Happy to jump in a VC if needed.

The rationale is more about the scrubber than the play button. In each of those examples, the scrubber remains onscreen for a few seconds after the video has begun to play. (YT doesn't have our exact issue since their button is integrated with the scrubber)

The idea is that in order to keep the scrubber on screen for a few seconds (and normalize with the examples in comment 4), we'll need to hide our play button. 

Bumping the priority back up, feel free to add time to my calendar if I can offer more clarification.

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/35938c276b40b2fc55afb8050673c1a8d163a2aa

commit 35938c276b40b2fc55afb8050673c1a8d163a2aa
Author: Becca Hughes <beccahughes@chromium.org>
Date: Fri Apr 27 00:45:13 2018

Media Controls: Hide overlay play button early when playback

When playback starts immediately hide the play button.

BUG= 814512 

Change-Id: Id69c449f6830a5e6c6a0c2a332a72bf08cf88ae8
Reviewed-on: https://chromium-review.googlesource.com/1011215
Reviewed-by: Mounir Lamouri <mlamouri@chromium.org>
Reviewed-by: Tommy Steimel <steimel@chromium.org>
Commit-Queue: Becca Hughes <beccahughes@chromium.org>
Cr-Commit-Position: refs/heads/master@{#554228}
[add] https://crrev.com/35938c276b40b2fc55afb8050673c1a8d163a2aa/third_party/WebKit/LayoutTests/media/controls/modern/overlay-play-button-tap-to-hide.html
[modify] https://crrev.com/35938c276b40b2fc55afb8050673c1a8d163a2aa/third_party/blink/renderer/modules/media_controls/elements/media_control_overlay_play_button_element.cc
[modify] https://crrev.com/35938c276b40b2fc55afb8050673c1a8d163a2aa/third_party/blink/renderer/modules/media_controls/elements/media_control_overlay_play_button_element.h
[modify] https://crrev.com/35938c276b40b2fc55afb8050673c1a8d163a2aa/third_party/blink/renderer/modules/media_controls/media_controls_impl.cc
[modify] https://crrev.com/35938c276b40b2fc55afb8050673c1a8d163a2aa/third_party/blink/renderer/modules/media_controls/media_controls_impl.h
[modify] https://crrev.com/35938c276b40b2fc55afb8050673c1a8d163a2aa/third_party/blink/renderer/modules/media_controls/resources/modernMediaControls.css

The bug is marked as P3 or Feature. It should not be merged as M67 is in beta. 
Please contact the approriate milestone owner if you have questions.
Owners: cmasso@(Android), cmasso@(iOS), kbleicher@(ChromeOS), govind@(Desktop)

For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot

@govind - this is an important UI tweak for the media controls launching in M67.

How is the change listed at #9 looking canary?

The bug is marked as P3 or Feature. It should not be merged as M67 is in beta. 
Please contact the approriate milestone owner if you have questions.
Owners: cmasso@(Android), cmasso@(iOS), kbleicher@(ChromeOS), govind@(Desktop)

For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot

Changing to "Type:bug" as sheriffbot continue to reject merge for "Type:Feature".

This bug requires manual review: M67 has already been promoted to the beta branch, so this requires manual review
Please contact the milestone owner if you have questions.
Owners: cmasso@(Android), cmasso@(iOS), kbleicher@(ChromeOS), govind@(Desktop)

For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot

This looks good in canary

Approving merge to M67 branch 3396 based on comments #12 and #18. Please merge. Thank you.

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/eb11cf1d4b67c01c458dba9cad5746faa3cba57a

commit eb11cf1d4b67c01c458dba9cad5746faa3cba57a
Author: Becca Hughes <beccahughes@chromium.org>
Date: Fri Apr 27 21:59:50 2018

Media Controls: Hide overlay play button early when playback

When playback starts immediately hide the play button.

BUG= 814512 

Change-Id: Id69c449f6830a5e6c6a0c2a332a72bf08cf88ae8
Reviewed-on: https://chromium-review.googlesource.com/1011215
Reviewed-by: Mounir Lamouri <mlamouri@chromium.org>
Reviewed-by: Tommy Steimel <steimel@chromium.org>
Commit-Queue: Becca Hughes <beccahughes@chromium.org>
Cr-Original-Commit-Position: refs/heads/master@{#554228}(cherry picked from commit 35938c276b40b2fc55afb8050673c1a8d163a2aa)
Reviewed-on: https://chromium-review.googlesource.com/1033497
Reviewed-by: Becca Hughes <beccahughes@chromium.org>
Cr-Commit-Position: refs/branch-heads/3396@{#363}
Cr-Branched-From: 9ef2aa869bc7bc0c089e255d698cca6e47d6b038-refs/heads/master@{#550428}
[add] https://crrev.com/eb11cf1d4b67c01c458dba9cad5746faa3cba57a/third_party/WebKit/LayoutTests/media/controls/modern/overlay-play-button-tap-to-hide.html
[modify] https://crrev.com/eb11cf1d4b67c01c458dba9cad5746faa3cba57a/third_party/blink/renderer/modules/media_controls/elements/media_control_overlay_play_button_element.cc
[modify] https://crrev.com/eb11cf1d4b67c01c458dba9cad5746faa3cba57a/third_party/blink/renderer/modules/media_controls/elements/media_control_overlay_play_button_element.h
[modify] https://crrev.com/eb11cf1d4b67c01c458dba9cad5746faa3cba57a/third_party/blink/renderer/modules/media_controls/media_controls_impl.cc
[modify] https://crrev.com/eb11cf1d4b67c01c458dba9cad5746faa3cba57a/third_party/blink/renderer/modules/media_controls/media_controls_impl.h
[modify] https://crrev.com/eb11cf1d4b67c01c458dba9cad5746faa3cba57a/third_party/blink/renderer/modules/media_controls/resources/modernMediaControls.css

Issue 839112 has been merged into this issue.