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

Issues Search
    My Issues | Starred     Open | Closed | All

Unified Diff: components/html_viewer/html_frame_apptest.cc

Issue 1294473008: Makes OOPIFs the default (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: cleanup Created 5 years, 4 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
« components/html_viewer/html_document_application_delegate.cc ('K') | « components/html_viewer/html_document_application_delegate.cc ('k') | components/html_viewer/html_viewer_switches.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: components/html_viewer/html_frame_apptest.cc
diff --git a/components/html_viewer/html_frame_apptest.cc b/components/html_viewer/html_frame_apptest.cc
index 55688fc7c3870d43a6c430ea32025b0789699e1d..d220dea362dd7d6718e91af7d9701678bed55423 100644
--- a/components/html_viewer/html_frame_apptest.cc
+++ b/components/html_viewer/html_frame_apptest.cc
@@ -32,7 +32,7 @@ namespace mojo {
namespace {
// Switch to enable out of process iframes.
-const char kOOPIF[] = "oopifs";
+const char kDisableOOPIF[] = "disable--oopifs";
const char kAddFrameWithEmptyPageScript[] =
"var iframe = document.createElement(\"iframe\");"
@@ -40,7 +40,9 @@ const char kAddFrameWithEmptyPageScript[] =
"document.body.appendChild(iframe);";
bool EnableOOPIFs() {
- return base::CommandLine::ForCurrentProcess()->HasSwitch(kOOPIF);
+ if (base::CommandLine::ForCurrentProcess()->HasSwitch(kDisableOOPIF))
Fady Samuel 2015/08/14 21:32:36 like previous comment, simply return !this express
+ return false;
+ return true;
}
mojo::ApplicationConnection* ApplicationConnectionForFrame(Frame* frame) {
« components/html_viewer/html_document_application_delegate.cc ('K') | « components/html_viewer/html_document_application_delegate.cc ('k') | components/html_viewer/html_viewer_switches.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698