Third Party Authentication for Android Part II - Java Native Interface plumbing

remoting/client/jni/chromoting_jni_runtime.h

Index: remoting/client/jni/chromoting_jni_runtime.h
diff --git a/remoting/client/jni/chromoting_jni_runtime.h b/remoting/client/jni/chromoting_jni_runtime.h
index bb356a13616e33bc814d28589bdefec7c486dea2..86931f6da82bc7d030212724ddbaf256eb30878c 100644
--- a/remoting/client/jni/chromoting_jni_runtime.h
+++ b/remoting/client/jni/chromoting_jni_runtime.h
@@ -96,6 +96,12 @@ class ChromotingJniRuntime {
   // Draws the latest image buffer onto the canvas. Call on the display thread.
   void RedrawCanvas();
 
+  // Pops up a third party login page to fetch token required for
+  // authentication.

[Lambros, 2014/06/07 01:38:12]

Add "Call on UI thread."
I think these methods are grouped together by which thread they are called on,
so perhaps move this just below CommitPairingCredentials()?

[kelvinp, 2014/06/09 18:42:32]

Done.

+  void FetchThirdPartyToken(const GURL& token_url,
+                            const std::string& client_id,
+                            const std::string& scope);

[Lambros, 2014/06/07 01:38:12]

Same here. Make sure the ordering of methods in .cc file follows the header file
(assuming it does so already).

[kelvinp, 2014/06/09 18:42:32]

Done.

+
  private:
   ChromotingJniRuntime();