Index: content/browser/manifest/manifest_browsertest.cc |
diff --git a/content/browser/manifest/manifest_browsertest.cc b/content/browser/manifest/manifest_browsertest.cc |
index bcac10714853d01f9273f1f8445a3a49f103c464..cc6a90586026cb829a97dc57153bbf4e15fd6a3f 100644 |
--- a/content/browser/manifest/manifest_browsertest.cc |
+++ b/content/browser/manifest/manifest_browsertest.cc |
@@ -247,7 +247,7 @@ IN_PROC_BROWSER_TEST_F(ManifestBrowserTest, DynamicManifest) { |
std::string manifest_url = |
GetTestUrl("manifest", "dummy-manifest.json").spec(); |
ASSERT_TRUE(content::ExecuteScript( |
- shell()->web_contents(), "setManifestTo('" + manifest_url + "')")); |
+ shell(), "setManifestTo('" + manifest_url + "')")); |
GetManifestAndWait(); |
EXPECT_FALSE(manifest().IsEmpty()); |
@@ -260,7 +260,7 @@ IN_PROC_BROWSER_TEST_F(ManifestBrowserTest, DynamicManifest) { |
std::string manifest_url = |
GetTestUrl("manifest", "empty-manifest.json").spec(); |
ASSERT_TRUE(content::ExecuteScript( |
- shell()->web_contents(), "setManifestTo('" + manifest_url + "')")); |
+ shell(), "setManifestTo('" + manifest_url + "')")); |
GetManifestAndWait(); |
EXPECT_TRUE(manifest().IsEmpty()); |
@@ -290,8 +290,8 @@ IN_PROC_BROWSER_TEST_F(ManifestBrowserTest, CORSManifest) { |
std::string manifest_url = cors_embedded_test_server()->GetURL( |
"/manifest/dummy-manifest.json").spec(); |
- ASSERT_TRUE(content::ExecuteScript(shell()->web_contents(), |
- "setManifestTo('" + manifest_url + "')")); |
+ ASSERT_TRUE( |
+ content::ExecuteScript(shell(), "setManifestTo('" + manifest_url + "')")); |
GetManifestAndWait(); |
EXPECT_TRUE(manifest().IsEmpty()); |
@@ -308,8 +308,8 @@ IN_PROC_BROWSER_TEST_F(ManifestBrowserTest, CORSManifest) { |
// it is actually fully loaded. |
manifest_url = |
embedded_test_server()->GetURL("/manifest/dummy-manifest.json").spec(); |
- ASSERT_TRUE(content::ExecuteScript(shell()->web_contents(), |
- "setManifestTo('" + manifest_url + "')")); |
+ ASSERT_TRUE( |
+ content::ExecuteScript(shell(), "setManifestTo('" + manifest_url + "')")); |
GetManifestAndWait(); |
} |
@@ -330,8 +330,8 @@ IN_PROC_BROWSER_TEST_F(ManifestBrowserTest, CORSManifestWithAcessControls) { |
std::string manifest_url = cors_embedded_test_server()->GetURL( |
"/manifest/manifest-cors.json").spec(); |
- ASSERT_TRUE(content::ExecuteScript(shell()->web_contents(), |
- "setManifestTo('" + manifest_url + "')")); |
+ ASSERT_TRUE( |
+ content::ExecuteScript(shell(), "setManifestTo('" + manifest_url + "')")); |
GetManifestAndWait(); |
EXPECT_FALSE(manifest().IsEmpty()); |
@@ -360,8 +360,8 @@ IN_PROC_BROWSER_TEST_F(ManifestBrowserTest, MixedContentManifest) { |
std::string manifest_url = |
https_server->GetURL("/manifest/dummy-manifest.json").spec(); |
- ASSERT_TRUE(content::ExecuteScript(shell()->web_contents(), |
- "setManifestTo('" + manifest_url + "')")); |
+ ASSERT_TRUE( |
+ content::ExecuteScript(shell(), "setManifestTo('" + manifest_url + "')")); |
GetManifestAndWait(); |
EXPECT_TRUE(manifest().IsEmpty()); |
@@ -441,8 +441,7 @@ IN_PROC_BROWSER_TEST_F(ManifestBrowserTest, PushStateNavigation) { |
{ |
TestNavigationObserver navigation_observer(shell()->web_contents(), 1); |
ASSERT_TRUE(content::ExecuteScript( |
- shell()->web_contents(), |
- "history.pushState({foo: \"bar\"}, 'page', 'page.html');")); |
+ shell(), "history.pushState({foo: \"bar\"}, 'page', 'page.html');")); |
navigation_observer.Wait(); |
} |
@@ -470,7 +469,7 @@ IN_PROC_BROWSER_TEST_F(ManifestBrowserTest, AnchorNavigation) { |
{ |
TestNavigationObserver navigation_observer(shell()->web_contents(), 1); |
ASSERT_TRUE(content::ExecuteScript( |
- shell()->web_contents(), |
+ shell(), |
"var a = document.createElement('a'); a.href='#foo';" |
"document.body.appendChild(a); a.click();")); |
navigation_observer.Wait(); |