Connect Sky and Ganesh in SkyShell

sky/shell/sky_main.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 "sky/shell/sky_main.h"
 
 #include "base/android/jni_android.h"
 #include "base/android/jni_array.h"
 #include "base/android/jni_string.h"
 #include "base/at_exit.h"
@@ skipping 13 matching lines @@
 using base::LazyInstance;
 
 namespace sky {
 namespace shell {
 
 namespace {
 
 LazyInstance<scoped_ptr<base::MessageLoop>> g_java_message_loop =
     LAZY_INSTANCE_INITIALIZER;
 
-LazyInstance<base::android::ScopedJavaGlobalRef<jobject>> g_main_activiy =
-    LAZY_INSTANCE_INITIALIZER;
-
 LazyInstance<scoped_ptr<Shell>> g_shell = LAZY_INSTANCE_INITIALIZER;
 
 void InitializeLogging() {
   logging::LoggingSettings settings;
   settings.logging_dest = logging::LOG_TO_SYSTEM_DEBUG_LOG;
   logging::InitLogging(settings);
   // To view log output with IDs and timestamps use "adb logcat -v threadtime".
   logging::SetLogItems(false,   // Process ID
                        false,   // Thread ID
                        false,   // Timestamp
                        false);  // Tick count
 }
 
 }  // namespace
 
 static void Init(JNIEnv* env,
                  jclass clazz,
                  jobject activity) {
-  g_main_activiy.Get().Reset(env, activity);
-
   base::android::ScopedJavaLocalRef<jobject> scoped_activity(env, activity);

[eseidel, 2015/02/18 20:55:56]

You mentioned you wanted to rename this to "application" instead of "activity".

   base::android::InitApplicationContext(env, scoped_activity);
 
   base::CommandLine::Init(0, nullptr);
 
   InitializeLogging();
 
   g_java_message_loop.Get().reset(new base::MessageLoopForUI);
   base::MessageLoopForUI::current()->Start();
 
   base::i18n::InitializeICU();
   gfx::GLSurface::InitializeOneOff();
 
   g_shell.Get().reset(new Shell(g_java_message_loop.Get()->task_runner()));
-
-  g_java_message_loop.Get()->PostTask(
-      FROM_HERE,
-      base::Bind(&Shell::Init, base::Unretained(g_shell.Get().get())));
-}
-
-static jboolean Start(JNIEnv* env, jclass clazz) {
-  LOG(INFO) << "Native code started!";
-  return true;
+  g_shell.Get()->Init();

[eseidel, 2015/02/18 20:55:56]

This can just be Shell::Init(), which creates g_shell, no?

 }
 
 bool RegisterSkyMain(JNIEnv* env) {
   return RegisterNativesImpl(env);
 }
 
 }  // namespace sky
 }  // namespace mojo