Add a basic linux client for Blimp.

blimp/client/blimp_startup.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 "blimp/client/blimp_startup.h"
+
+#include "base/files/file_path.h"
+#include "base/lazy_instance.h"
+#include "base/logging.h"
+#include "base/message_loop/message_loop.h"
+#include "base/path_service.h"
+#include "third_party/skia/include/core/SkGraphics.h"
+#include "ui/gl/gl_surface.h"
+
+namespace {
+base::LazyInstance<scoped_ptr<base::MessageLoopForUI>> g_main_message_loop =
+    LAZY_INSTANCE_INITIALIZER;

[Wez, 2016/01/05 19:15:02]

Add a comment to explain the purpose of this member.

+}
+
+namespace blimp {
+
+void InitializeLogging() {
+  logging::LoggingSettings settings;
+#if defined(OS_ANDROID)
+  settings.logging_dest = logging::LOG_TO_SYSTEM_DEBUG_LOG;
+#else
+  base::FilePath log_filename;
+  PathService::Get(base::DIR_EXE, &log_filename);
+  log_filename = log_filename.AppendASCII("blimp_client.log");
+  settings.logging_dest = logging::LOG_TO_ALL;
+  settings.log_file = log_filename.value().c_str();
+  settings.delete_old = logging::DELETE_OLD_LOG_FILE;
+#endif  // OS_ANDROID
+  logging::InitLogging(settings);
+  logging::SetLogItems(false,   // Process ID

[nyquist, 2015/12/17 03:00:21]

Why do we not want to log the process and thread ID on the engine?

[Wez, 2016/01/05 19:15:02]

This is blimp_client logging initialization - there is only one process, so we
probably want to log the thread Id and timestamp, not the process ID nor tick
count.

+                       false,   // Thread ID
+                       false,   // Timestamp
+                       false);  // Tick count
+  VLOG(0) << "Chromium logging enabled: level = " << logging::GetMinLogLevel()
+          << ", default verbosity = " << logging::GetVlogVerbosity();
+}
+
+bool InitializeMainMessageLoop() {
+  // TODO(dtrainor): Initialize ICU?

[Wez, 2016/01/05 19:15:02]

Clarify why we might need to initialize ICU here.

+

[Wez, 2016/01/05 19:15:02]

nit: Remove the spurious blank line.

+  if (!gfx::GLSurface::InitializeOneOff())
+    return false;
+  SkGraphics::Init();

[Wez, 2016/01/05 19:15:02]

The function name is InitializeMainMessageLoop, but we're also initializing some
global Skia and gfx state, so please add a comment to clarify what's going on
here.

+  g_main_message_loop.Get().reset(new base::MessageLoopForUI);
+  return true;
+}
+
+}  // namespace blimp