Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(6135)

Unified Diff: chrome/browser/extensions/process_management_browsertest.cc

Issue 2850793005: Remove command line/field trial support and configs for Isolate Extensions. (Closed)
Patch Set: Remove more cases of "isolate.*extension" Created 3 years, 8 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chrome/browser/extensions/process_management_browsertest.cc
diff --git a/chrome/browser/extensions/process_management_browsertest.cc b/chrome/browser/extensions/process_management_browsertest.cc
index c446d02a0310698067466a7ec1888924299f3a70..14432bcfabaa17ce765fdc9d5208667deb7ef823 100644
--- a/chrome/browser/extensions/process_management_browsertest.cc
+++ b/chrome/browser/extensions/process_management_browsertest.cc
@@ -324,11 +324,9 @@ IN_PROC_BROWSER_TEST_F(ProcessManagementTest,
// Verify that the navigation transferred the contents to another renderer
// process.
- if (extensions::IsIsolateExtensionsEnabled()) {
- content::RenderProcessHost* new_process_host =
- web_contents->GetMainFrame()->GetProcess();
- EXPECT_NE(old_process_host, new_process_host);
- }
+ content::RenderProcessHost* new_process_host =
+ web_contents->GetMainFrame()->GetProcess();
+ EXPECT_NE(old_process_host, new_process_host);
}
IN_PROC_BROWSER_TEST_F(ChromeWebStoreProcessTest,
@@ -371,14 +369,8 @@ IN_PROC_BROWSER_TEST_F(ChromeWebStoreProcessTest,
EXPECT_TRUE(content::ExecuteScriptAndExtractBool(
web_contents, navigation_starting_script, &ignored_script_result));
- // When --isolate-extensions is enabled, the expectation is that the store
- // will be properly put in its own process, otherwise the renderer process
- // is going to be terminated.
- if (!extensions::IsIsolateExtensionsEnabled()) {
- crash_observer.Wait();
- return;
- }
-
+ // The expectation is that the store will be properly put in its own process,
+ // otherwise the renderer process is going to be terminated.
// Verify that the navigation succeeded.
nav_observer.Wait();
EXPECT_EQ(cws_web_url, web_contents->GetLastCommittedURL());

Powered by Google App Engine
This is Rietveld 408576698