signin: force web signin flow initiated visits to accounts.google.com to their own process.

chrome/browser/signin/signin_manager.h

Index: chrome/browser/signin/signin_manager.h
diff --git a/chrome/browser/signin/signin_manager.h b/chrome/browser/signin/signin_manager.h
index 2272c7acc0d17dc479fdf403ea2052941bd8b484..ac6ea203f4940d66e756b6d0a22e12c1a61e9f64 100644
--- a/chrome/browser/signin/signin_manager.h
+++ b/chrome/browser/signin/signin_manager.h
@@ -40,6 +40,10 @@ class GaiaAuthFetcher;
 class PrefService;
 class SigninGlobalError;
 
+namespace content {
+class RenderProcessHost;
+}
+
 namespace policy {
 class CloudPolicyClient;
 }
@@ -195,6 +199,9 @@ class SigninManager : public GaiaAuthConsumer,
   // ignored).
   bool IsSignoutProhibited() const;
 
+  void SetSigninProcess(const content::RenderProcessHost* process);
+  bool IsSigninProcess(const content::RenderProcessHost* process) const;

[Roger Tawa OOO till Jul 10th, 2013/02/28 04:07:42]

Any reason to pass RenderProcessHost* instead of just an int?

[Charlie Reis, 2013/02/28 17:33:02]

Yes, I think the ID would be sufficient here, and consistent with similar checks
elsewhere.

+
  protected:
   // Weak pointer to parent profile (protected so FakeSigninManager can access
   // it).
@@ -331,6 +338,7 @@ class SigninManager : public GaiaAuthConsumer,
 
   base::WeakPtrFactory<SigninManager> weak_pointer_factory_;
 
+  int signin_process_id_;
 
 #if defined(ENABLE_CONFIGURATION_POLICY) && !defined(OS_CHROMEOS)
   // CloudPolicyClient reference we keep while determining whether to create