Force resize event to be sent to OOPIF subframes when main frame resizes.

content/browser/screen_orientation/screen_orientation_browsertest.cc

Index: content/browser/screen_orientation/screen_orientation_browsertest.cc
diff --git a/content/browser/screen_orientation/screen_orientation_browsertest.cc b/content/browser/screen_orientation/screen_orientation_browsertest.cc
index 253f16713fa36b45e7a5247bc8c0393ff3dd420d..badf77b97bce3c1c4fef3fe97090b7e502082c0f 100644
--- a/content/browser/screen_orientation/screen_orientation_browsertest.cc
+++ b/content/browser/screen_orientation/screen_orientation_browsertest.cc
@@ -6,6 +6,7 @@
 
 #include "base/command_line.h"
 #include "base/macros.h"
+#include "base/strings/stringprintf.h"
 #include "build/build_config.h"
 #include "content/browser/frame_host/frame_tree.h"
 #include "content/browser/frame_host/render_frame_host_impl.h"
@@ -355,4 +356,43 @@ IN_PROC_BROWSER_TEST_F(ScreenOrientationOOPIFBrowserTest, ScreenOrientation) {
   }
 }
 
+#ifdef OS_ANDROID
+IN_PROC_BROWSER_TEST_F(ScreenOrientationOOPIFBrowserTest,
+                       DISABLED_ScreenOrientationLock) {

[Charlie Reis, 2017/06/06 22:41:37]

Please include a comment for why this is disabled.

Is there any way that we'll be able to enable it in the future?  And is it
meaningful on any other platforms?

[lfg, 2017/06/07 22:30:57]

Done.
This test isn't meaningful (as of today) on other platforms, since we only
support the Screen Orientation lock APIs on Android (the other platforms do have
orientation change, but not lock).

As for enabling in the future, I've looked at a couple of options:

1. Use the renderer mock interfaces (MockScreenOrientationClient) that we
currently use to test this API. Those are used for testing this API with
LayoutTests.
  - The problem here is that there's no way to replicate the locking information
across OOPIF renderers, which makes it difficult.

2. Mock browser process interfaces.
  - I believe we would need to mock WebContentsViewAndroid and the
ScreenOrientationProvider classes, and possibly RenderWidgetHostViewnAndroid.
The problem here is that there's no framework to mock those classes so there's a
lot of boilerplate code that needs to be written.

+  GURL main_url(embedded_test_server()->GetURL(
+      "a.com", "/cross_site_iframe_factory.html?a(b)"));
+  EXPECT_TRUE(NavigateToURL(shell(), main_url));
+  WaitForResizeComplete(shell()->web_contents());
+
+  const char* types[] = {"portrait-primary", "portrait-secondary",
+                         "landscape-primary", "landscape-secondary"};
+
+  FrameTreeNode* root = web_contents()->GetFrameTree()->root();
+  FrameTreeNode* child = root->child_at(0);
+  RenderFrameHostImpl* frames[] = {root->current_frame_host(),
+                                   child->current_frame_host()};
+
+  EXPECT_TRUE(ExecuteScript(root->current_frame_host(),
+                            "document.body.webkitRequestFullscreen()"));
+  for (const char* type : types) {
+    std::string script =
+        base::StringPrintf("screen.orientation.lock('%s')", type);
+    EXPECT_TRUE(ExecuteScript(child->current_frame_host(), script));
+
+    for (auto* frame : frames) {
+      std::string orientation_type;
+      while (type != orientation_type) {
+        EXPECT_TRUE(ExecuteScriptAndExtractString(
+            frame,
+            "window.domAutomationController.send(screen.orientation.type)",
+            &orientation_type));
+      }
+    }
+
+    EXPECT_TRUE(ExecuteScript(child->current_frame_host(),
+                              "screen.orientation.unlock()"));
+  }
+}
+#endif  // OS_ANDROID
+
 } // namespace content