Include viewport visibility checks for autoplay experiment.

third_party/WebKit/Source/core/testing/Internals.idl

Index: third_party/WebKit/Source/core/testing/Internals.idl
diff --git a/third_party/WebKit/Source/core/testing/Internals.idl b/third_party/WebKit/Source/core/testing/Internals.idl
index 821bba1bc55d21f44b1cfda24fb9994147d20554..634a96cc0dea6bb3e29bd664f3d0640d8c3f89db 100644
--- a/third_party/WebKit/Source/core/testing/Internals.idl
+++ b/third_party/WebKit/Source/core/testing/Internals.idl
@@ -353,4 +353,6 @@
     bool setScrollbarVisibilityInScrollableArea(Node node, boolean visible);
 
     void forceRestrictIFramePermissions();
+
+    [TypeChecking=Interface] void triggerAutoplayViewportCheck(HTMLMediaElement mediaElement);

[ojan, 2015/10/01 06:14:26]

We typically try to avoid adding one-off APIs like this. Unfortunately, since we
don't have a great track record of unittests, that means we handle things via
regression tests (i.e. layout tests), which aren't very friendly to exposing the
guts of the system. As an aside, feel free to write unittests whenever you can.
We're happy to have more unittests and fewer layout tests where it's not too
onerous to get setup.

Since this will go away when we remove the experiment code, I'm OK with taking
on this small bit of technical debt temporarily. But, please add a
TODO(liberato) to remove this once the experiment is done in all the headers
this method is defined in and in the calling code (i.e. in the layout test).

[esprehn, 2015/10/01 07:04:15]

You should be able to write a Sim test for this, see
DocumentLoadingRenderingTest. that would remove the need for this

[liberato (no reviews please), 2015/10/01 22:49:03]

added the TODO.

[liberato (no reviews please), 2015/10/01 22:49:03]

will look at this test, but for now leaving with TODO.

 };