Fix nested GuestView's container bounds calculation.

chrome/browser/pdf/pdf_extension_util.cc

 // Copyright 2015 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 "chrome/browser/pdf/pdf_extension_util.h"
 
 #include "base/strings/string_util.h"
 #include "chrome/common/chrome_content_client.h"
 #include "chrome/common/chrome_switches.h"
 #include "chrome/grit/browser_resources.h"
+#include "content/public/test/browser_test_utils.h"
+#include "grit/component_extension_resources.h"
 #include "ui/base/resource/resource_bundle.h"
 
 namespace pdf_extension_util {
 
 namespace {
 
 // Tags in the manifest to be replaced.
 const char kNameTag[] = "<NAME>";
 const char kIndexTag[] = "<INDEX>";
 
@@ skipping 20 matching lines @@
   base::ReplaceFirstSubstringAfterOffset(
       &manifest_contents, 0, kNameTag, ChromeContentClient::kPDFPluginName);
 
   DCHECK(manifest_contents.find(kIndexTag) != std::string::npos);
   std::string index = switches::PdfMaterialUIEnabled() ?
       kMaterialIndex : kRegularIndex;
   base::ReplaceSubstringsAfterOffset(&manifest_contents, 0, kIndexTag, index);
   return manifest_contents;
 }
 
+bool EnsurePDFHasLoaded(content::WebContents* web_contents) {

[raymes, 2015/07/21 01:20:14]

This file is used in production code however this function should never be used
in production code and I wouldn't want someone to accidentally do that. Can we
move this function to a test-only file?

[lazyboy, 2015/07/21 03:55:06]

Oh I missed that seeing pdf_extension_util and pdf_extenion_test side by side, I
was reading it "pdf_extension_test_util" for some reason.

Moved stuff to pdf_extension_test_util.*

+  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;
+}
+
 }  // namespace pdf_extension_util