[TimeZoneMonitor] Decouple renderer side impl from content to blink.

third_party/WebKit/public/platform/Platform.h

Index: third_party/WebKit/public/platform/Platform.h
diff --git a/third_party/WebKit/public/platform/Platform.h b/third_party/WebKit/public/platform/Platform.h
index 708d13a5988ae19bc6b5876588d6952df00122d6..091d47d84d5cc2649a2747c79120616b00cba017 100644
--- a/third_party/WebKit/public/platform/Platform.h
+++ b/third_party/WebKit/public/platform/Platform.h
@@ -141,6 +141,11 @@ class BLINK_PLATFORM_EXPORT Platform {
   static void initialize(Platform*);
   static void shutdown();
   static Platform* current();
+  // Used to get whether message loop is ready for current thread, to help
+  // blink::initialize() determining whether can initialize mojo stuff or not.
+  // TODO(leonhsl): http://crbug.com/660274 Remove this public API by ensuring
+  // a message loop before calling blink::initialize().
+  static bool isMessageLoopReady();

[kinuko, 2016/11/08 03:22:56]

(Putting aside the need of this method that seems to have discussed so long)

do we need this method to be exposed at Platform?  This seems to be only used in
Source/modules/ModulesInitializer.cpp or my text search is broken.  Could this
be placed, say, in platform/mojo/MojoHelper.h?  (It's orthogonal to mojo but we
wouldn't want to make it stick around for general use either)

[leonhsl(Using Gerrit), 2016/11/08 08:20:39]

Yes the only user is Source/modules/ModulesInitializer.cpp.
I'm sure ModulesInitializer.cpp is able to call Platform public APIs so I put
the API here.

And, I tried to put isMessageLoopReady() definition into
platform/mojo/MojoHelper.h, but got build error as bellow. Any ideas? Thanks.

"
In file included from
../../third_party/WebKit/Source/modules/imagecapture/ImageCapture.cpp:18:
../../third_party/WebKit/Source/platform/mojo/MojoHelper.h:36:15: error: unused
function 'isMessageLoopReady' [-Werror,-Wunused-function]
  static bool isMessageLoopReady() {
"

 
   // Used to switch the current platform only for testing.
   static void setCurrentPlatformForTesting(Platform*);