[Media,Remoting] Remove the disable remoting button on Android

third_party/WebKit/Source/core/html/shadow/MediaRemotingElements.cpp

Index: third_party/WebKit/Source/core/html/shadow/MediaRemotingElements.cpp
diff --git a/third_party/WebKit/Source/core/html/shadow/MediaRemotingElements.cpp b/third_party/WebKit/Source/core/html/shadow/MediaRemotingElements.cpp
index c53bc9d89a9bc70e4e54f558e73914a91e81f0cd..7944d4c8924e6af90fa52d15283f5d5f3f354a04 100644
--- a/third_party/WebKit/Source/core/html/shadow/MediaRemotingElements.cpp
+++ b/third_party/WebKit/Source/core/html/shadow/MediaRemotingElements.cpp
@@ -64,10 +64,18 @@ MediaRemotingExitButtonElement::MediaRemotingExitButtonElement(
 }
 
 void MediaRemotingExitButtonElement::OnShown() {
+  if (const ComputedStyle* style = NonLayoutObjectComputedStyle()) {
+    if (style->Display() == EDisplay::kNone)

[xjz, 2017/05/25 19:02:18]

Have you tested on this? I am not sure whether this works. IIUC, the
showing/hiding of the button is decided by the previous
RemoveInlineStyleProperty()/SetInlineStyleProperty() call:
https://cs.chromium.org/chromium/src/third_party/WebKit/Source/core/html/shad...

OnShown()/OnHidden() here are not to show/hide the button, but to
register/de-register the event handler.

[whywhat, 2017/05/25 20:26:29]

The hiding of the button happens via the mediacontrolsandroid.css that affects
the button only. The one you pointed to affects the whole interstitial, AFAIK.
Yes, my intention here is not to register/unregister the event handler in case
the button is not visible.
The names became a bit confusing though. It now mean more like
"OnInterstitialShown" rather than "OnExitButtonShown" :)

Mounir suggested that the event handlers won't get called for display:none
elements anyway, but still, registering them felt like a waste.

[xjz, 2017/05/25 21:20:34]

Yes, you are correct. When showing/hiding the interstitial, only the display
property of the whole interstitial (MediaRemotingInterstitial) is changed, not
the separate elements.

I still feel a little confusing to do the check here. Maybe we can do the check
in MediaRemotingInterstitial::Show()/Hide(). If the Display property of the
|exit_button_| is kNone (indicating that the button is not visible), don't call
this OnShown()/OnHidden(). WDYT?

[rune, 2017/05/26 11:17:57]

Querying the computed style requires that the computed style is up-to-date. I
don't think it is at this point because RemoveInlineStyleProperty() will cause
the computed style to be updated asynchronously.

It could be an idea to add/remove the event listener on
MediaRemotingExitButtonElement::AttachLayoutTree/DetachLayoutTree instead.

+      return;
+  }
   GetDocument().addEventListener(EventTypeNames::click, listener_, true);
 }
 
 void MediaRemotingExitButtonElement::OnHidden() {
+  if (const ComputedStyle* style = NonLayoutObjectComputedStyle()) {
+    if (style->Display() == EDisplay::kNone)
+      return;
+  }
   GetDocument().removeEventListener(EventTypeNames::click, listener_, true);
 }