Fix nested GuestView's container bounds calculation.

chrome/browser/pdf/pdf_extension_test.cc

 // Copyright 2014 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "base/base_paths.h"
 #include "base/files/file_enumerator.h"
 #include "base/files/file_util.h"
 #include "base/hash.h"
 #include "base/path_service.h"
 #include "chrome/browser/extensions/component_loader.h"
 #include "chrome/browser/extensions/extension_apitest.h"
 #include "chrome/browser/extensions/extension_service.h"
+#include "chrome/browser/pdf/pdf_extension_test_util.h"
+#include "chrome/browser/pdf/pdf_extension_util.h"
 #include "chrome/browser/ui/tabs/tab_strip_model.h"
 #include "chrome/common/chrome_paths.h"
 #include "chrome/common/chrome_switches.h"
 #include "chrome/test/base/ui_test_utils.h"
 #include "content/public/browser/browser_plugin_guest_manager.h"
 #include "content/public/browser/plugin_service.h"
 #include "content/public/test/browser_test_utils.h"
 #include "extensions/browser/extension_registry.h"
 #include "extensions/common/manifest_handlers/mime_types_handler.h"
 #include "extensions/test/result_catcher.h"
-#include "grit/component_extension_resources.h"
 #include "net/test/embedded_test_server/embedded_test_server.h"
-#include "ui/base/resource/resource_bundle.h"
 
 const int kNumberLoadTestParts = 10;
 
 class PDFExtensionTest : public ExtensionApiTest,
                          public testing::WithParamInterface<int> {
  public:
   ~PDFExtensionTest() override {}
 
   void SetUpCommandLine(base::CommandLine* command_line) override {
     command_line->AppendSwitch(switches::kDisablePdfMaterialUI);
@@ skipping 67 matching lines @@
 
   // Load the PDF at the given URL and use the PDFScriptingAPI to ensure it has
   // finished loading. Return true if it loads successfully or false if it
   // fails. If it doesn't finish loading the test will hang. This is done from
   // outside of the BrowserPlugin guest to ensure the PDFScriptingAPI works
   // correctly from there.
   bool LoadPdf(const GURL& url) {
     ui_test_utils::NavigateToURL(browser(), url);
     content::WebContents* web_contents =
         browser()->tab_strip_model()->GetActiveWebContents();
-    std::string scripting_api_js =
-        ResourceBundle::GetSharedInstance()
-            .GetRawDataResource(IDR_PDF_PDF_SCRIPTING_API_JS)
-            .as_string();
-    CHECK(content::ExecuteScript(web_contents, scripting_api_js));
-
-    bool load_success = false;
-    CHECK(content::ExecuteScriptAndExtractBool(
-        web_contents,
-        "var scriptingAPI = new PDFScriptingAPI(window, "
-        "    document.getElementsByTagName('embed')[0]);"
-        "scriptingAPI.setLoadCallback(function(success) {"
-        "  window.domAutomationController.send(success);"
-        "});",
-        &load_success));
-    return load_success;
+    return pdf_extension_test_util::EnsurePDFHasLoaded(web_contents);
   }
 
   // Load all the PDFs contained in chrome/test/data/<dir_name>. This only runs
   // the test if base::Hash(filename) mod kNumberLoadTestParts == k in order
   // to shard the files evenly across values of k in [0, kNumberLoadTestParts).
   void LoadAllPdfsTest(const std::string& dir_name, int k) {
     base::FilePath test_data_dir;
     ASSERT_TRUE(PathService::Get(chrome::DIR_TEST_DATA, &test_data_dir));
     base::FileEnumerator file_enumerator(test_data_dir.AppendASCII(dir_name),
                                          false, base::FileEnumerator::FILES,
@@ skipping 93 matching lines @@
 
 IN_PROC_BROWSER_TEST_F(MaterialPDFExtensionTest, ZoomManager) {
   RunTestsInFile("zoom_manager_test.js", "test.pdf");
 }
 
 IN_PROC_BROWSER_TEST_F(MaterialPDFExtensionTest, ElementsTest) {
   // Although this test file does not require a PDF to be loaded, loading the
   // elements without loading a PDF is difficult.
   RunTestsInFile("material_elements_test.js", "test.pdf");
 }