Makes OOPIFs the default

components/html_viewer/html_frame_apptest.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 "base/bind.h"
 #include "base/command_line.h"
 #include "base/json/json_reader.h"
 #include "base/run_loop.h"
 #include "base/strings/stringprintf.h"
 #include "base/test/test_timeouts.h"
@@ skipping 14 matching lines @@
 using mandoline::Frame;
 using mandoline::FrameConnection;
 using mandoline::FrameTree;
 using mandoline::FrameTreeClient;
 
 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\");"
     "iframe.src = \"http://127.0.0.1:%u/files/empty_page.html\";"
     "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 expression?

+    return false;
+  return true;
 }
 
 mojo::ApplicationConnection* ApplicationConnectionForFrame(Frame* frame) {
   return static_cast<FrameConnection*>(frame->user_data())
       ->application_connection();
 }
 
 std::string GetFrameText(ApplicationConnection* connection) {
   html_viewer::TestHTMLViewerPtr test_html_viewer;
   connection->ConnectToService(&test_html_viewer);
@@ skipping 435 matching lines @@
         script_value_as_list->GetString(0u, &message_in_child);
       }
     }
   } while (message_in_child != "hello from parent" &&
            base::TimeTicks::Now() - start_time <
                TestTimeouts::action_timeout());
   EXPECT_EQ("hello from parent", message_in_child);
 }
 
 }  // namespace mojo