[Blimp] Adds blimp engine browser test framework and LoadUrl test.

blimp/test/browser_tests/blimp_browser_test.cc

Index: blimp/test/browser_tests/blimp_browser_test.cc
diff --git a/blimp/test/browser_tests/blimp_browser_test.cc b/blimp/test/browser_tests/blimp_browser_test.cc
new file mode 100644
index 0000000000000000000000000000000000000000..76372058461189b0acf443b0a8e868815b5af7c8
--- /dev/null
+++ b/blimp/test/browser_tests/blimp_browser_test.cc
@@ -0,0 +1,122 @@
+// Copyright 2016 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 "blimp/test/browser_tests/blimp_browser_test.h"
+
+#include <string>
+
+#include "base/base_switches.h"
+#include "base/command_line.h"
+#include "base/files/file_path.h"
+#include "base/memory/ptr_util.h"
+#include "base/message_loop/message_loop.h"
+#include "base/run_loop.h"
+#include "blimp/client/app/blimp_client_switches.h"
+#include "blimp/client/session/assignment_source.h"
+#include "blimp/engine/app/blimp_browser_main_parts.h"
+#include "blimp/engine/app/blimp_content_browser_client.h"
+#include "blimp/engine/app/blimp_content_main_delegate.h"
+#include "blimp/engine/app/blimp_engine_config.h"
+#include "blimp/engine/app/switches.h"
+#include "blimp/engine/session/blimp_engine_session.h"
+#include "content/public/browser/browser_thread.h"
+#include "content/public/browser/render_process_host.h"
+#include "content/public/common/url_constants.h"
+#include "ui/gl/gl_switches.h"
+
+namespace blimp {
+namespace {
+
+const char kTestDataFilePath[] = "blimp/engine/test/data";
+const char kClientTokenFilePath[] = "blimp/test/data/test_client_token";
+}  // namespace

[mmenke, 2016/05/05 14:45:01]

nit:  should be consistent here - either a blank line both after the start and
before the end of the anonymous namespace, or in neither spot.

[haibinlu, 2016/05/05 18:06:30]

Done.

+
+BlimpBrowserTest::BlimpBrowserTest() {
+  base::FilePath test_data(FILE_PATH_LITERAL(kTestDataFilePath));
+  CreateTestServer(test_data);
+}
+
+BlimpBrowserTest::~BlimpBrowserTest() {}
+
+void BlimpBrowserTest::SetUp() {
+  base::CommandLine* command_line = base::CommandLine::ForCurrentProcess();
+  SetUpCommandLine(command_line);
+  BrowserTestBase::SetUp();
+}
+
+engine::BlimpEngineSession* BlimpBrowserTest::GetEngineSession() {
+  return engine::BlimpContentMainDelegate::GetInstanceForTesting()
+      ->GetBlimpContentBrowserClient()
+      ->blimp_browser_main_parts()
+      ->GetBlimpEngineSession();
+}
+
+void BlimpBrowserTest::OnGetEnginePort(uint16_t port) {
+  DCHECK_GT(port, 0);
+  // Update the command line so that Client will use this port to connect.
+  base::CommandLine::ForCurrentProcess()->AppendSwitchASCII(
+      blimp::switches::kEnginePort, std::to_string(port));

[mmenke, 2016/05/05 14:45:01]

Modifying the command line after we've started other threads doesn't look
threadsafe to me. 
https://code.google.com/p/chromium/codesearch#chromium/src/base/command_line....

[haibinlu, 2016/05/05 18:06:30]

Thanks for catching another bug!  

Changed to not updating command line here. Client, which starts after
SetUpOnMainThread, will use assignment directly without consulting command line.

+  QuitRunLoop();
+}
+
+void BlimpBrowserTest::SetUpOnMainThread() {
+  GetEngineSession()->GetEnginePortForTesting(
+      base::Bind(&BlimpBrowserTest::OnGetEnginePort, base::Unretained(this)));
+  RunUntilQuit();
+}
+
+void BlimpBrowserTest::TearDownOnMainThread() {
+  content::BrowserThread::GetMessageLoopProxyForThread(
+      content::BrowserThread::UI)
+      ->PostTask(FROM_HERE, base::MessageLoop::QuitWhenIdleClosure());
+}
+
+void BlimpBrowserTest::SetUpCommandLine(base::CommandLine* command_line) {
+  command_line->AppendSwitchASCII(::switches::kVModule, "blimp*=2");
+
+  // Engine switches
+  blimp::engine::SetUpCommandLine(command_line);
+  // Use dynamically allocated port for browser test.
+  command_line->AppendSwitchASCII(blimp::switches::kEnginePort, "0");
+  command_line->AppendSwitchASCII(blimp::engine::kClientTokenPath,
+                                  kClientTokenFilePath);
+
+  // Client switches
+  // Client's AssignmentSource uses engine IP&port from command line and
+  // defaults to TCP client.
+  command_line->AppendSwitchASCII(blimp::switches::kEngineIP, "127.0.0.1");
+  command_line->AppendSwitchASCII(blimp::switches::kEngineTransport, "tcp");
+}
+
+void BlimpBrowserTest::RunTestOnMainThreadLoop() {
+  DCHECK(content::BrowserThread::CurrentlyOn(content::BrowserThread::UI));
+
+  // Pump startup related events.
+  base::MessageLoop::current()->RunUntilIdle();
+
+  SetUpOnMainThread();
+  RunTestOnMainThread();
+  TearDownOnMainThread();
+
+  for (content::RenderProcessHost::iterator i(
+           content::RenderProcessHost::AllHostsIterator());
+       !i.IsAtEnd(); i.Advance()) {
+    i.GetCurrentValue()->FastShutdownIfPossible();
+  }
+}
+
+void BlimpBrowserTest::RunUntilQuit() {
+  base::MessageLoop::ScopedNestableTaskAllower nestable_allower(
+      base::MessageLoop::current());
+  EXPECT_FALSE(run_loop_);
+  run_loop_ = base::WrapUnique(new base::RunLoop());
+  run_loop_->Run();
+  run_loop_ = nullptr;
+}
+
+void BlimpBrowserTest::QuitRunLoop() {
+  run_loop_->Quit();
+}
+
+}  // namespace blimp