[Media controls] Add rotate-to-fullscreen gesture behind flag

third_party/WebKit/Source/modules/media_controls/MediaControlsOrientationLockDelegate.cpp

Index: third_party/WebKit/Source/modules/media_controls/MediaControlsOrientationLockDelegate.cpp
diff --git a/third_party/WebKit/Source/modules/media_controls/MediaControlsOrientationLockDelegate.cpp b/third_party/WebKit/Source/modules/media_controls/MediaControlsOrientationLockDelegate.cpp
index b6f4ab8ff80bd3d25fff272a1db1151047b8a84e..b2fec921133c5f93724b7421ce0bdcd27e2c15f5 100644
--- a/third_party/WebKit/Source/modules/media_controls/MediaControlsOrientationLockDelegate.cpp
+++ b/third_party/WebKit/Source/modules/media_controls/MediaControlsOrientationLockDelegate.cpp
@@ -64,8 +64,6 @@ class DummyScreenOrientationCallback : public WebLockOrientationCallback {
 MediaControlsOrientationLockDelegate::MediaControlsOrientationLockDelegate(
     HTMLVideoElement& video)
     : EventListener(kCPPEventListenerType), video_element_(video) {
-  if (VideoElement().isConnected())
-    Attach();

[mlamouri (slow - plz ping), 2017/04/19 17:10:46]

debugging code?

[johnme, 2017/04/19 18:21:28]

No - it turns out this is redundant, and leads to Attach getting called twice.
The constructor is called from MediaControlsImpl::Create, and immediately after
constructing the delegate this calls shadow_root.AppendChild(controls) which
calls Attach (ContainerNode::AppendChild calls ContainerNode::InsertNodeVector<>
which calls ContainerNode::NotifyNodeInsertedInternal which calls
MediaControlsImpl::InsertedInto which calls
MediaControlsOrientationLockDelegate::Attach).

But I'll leave it in this file as discussed offline, and filed
https://crbug.com/713275 to track it instead.

 }
 
 void MediaControlsOrientationLockDelegate::Attach() {