Run the later parts of startup as UI thread tasks

content/public/android/java/src/org/chromium/content/browser/StartupObserver.java

Index: content/public/android/java/src/org/chromium/content/browser/StartupObserver.java
diff --git a/content/public/android/java/src/org/chromium/content/browser/StartupObserver.java b/content/public/android/java/src/org/chromium/content/browser/StartupObserver.java
new file mode 100644
index 0000000000000000000000000000000000000000..b7e92ae2aef25f37f7f365237ece683f32e87b9e
--- /dev/null
+++ b/content/public/android/java/src/org/chromium/content/browser/StartupObserver.java
@@ -0,0 +1,14 @@
+// Copyright (c) 2012 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.
+
+

[joth, 2013/07/22 16:28:15]

nit: spurious \n

[aberent, 2013/07/22 19:37:40]

Done.

+package org.chromium.content.browser;
+
+import org.chromium.base.CalledByNative;
+
+public class StartupObserver {

[joth, 2013/07/22 16:28:15]

class comment - mostly mention which class you use this with

[aberent, 2013/07/22 19:37:40]

Done.

+    // Default version of allTasksRun does nothing
+    @CalledByNative
+    public void allTasksRun() {};

[joth, 2013/07/22 16:28:15]

nit: run sounds like a command here rather than past participle. maybe:-

didRunAllStartupTasks()

allSartupTasksRan()

[aberent, 2013/07/22 19:37:40]

Done.

+}

[joth, 2013/07/22 16:28:15]

(vaguely wonder if we need a class for this, or could just use Runnable )

[aberent, 2013/07/22 19:37:40]

I don't think one can call a Runnable's run method across the JNI. In addition
since StartupObserver objects are passed through a number of levels having an
explicit class seems clearer for the function arguments.