Changes to the GAIA auth fetching to support one-click login.

chrome/common/net/gaia/gaia_auth_consumer.h

Index: chrome/common/net/gaia/gaia_auth_consumer.h
===================================================================
--- chrome/common/net/gaia/gaia_auth_consumer.h	(revision 123466)
+++ chrome/common/net/gaia/gaia_auth_consumer.h	(working copy)
@@ -8,9 +8,14 @@
 
 #include <string>
 #include <map>
+#include <vector>
 
 class GoogleServiceAuthError;
 
+namespace net {
+typedef std::vector<std::string> ResponseCookies;

[Andrew T Wilson (Slow), 2012/02/27 18:00:41]

Why is this in |net| but nothing else in this file is? Is there a downside to
having this outside net (or just no typedef?)

[Roger Tawa OOO till Jul 10th, 2012/02/29 18:46:53]

This is essentially a forward declare.  I moved the forward declare to here from
gaia_auth_fetcher.h

[Andrew T Wilson (Slow), 2012/02/29 21:57:17]

Yeah, I guess I'm just thinking that nothing breaks if we remove the net::
namespace here or just directly pass around std::vector<string>. But I don't
feel strongly about it so leave it if you like.

[Roger Tawa OOO till Jul 10th, 2012/03/01 17:55:48]

I investigated simply declaring a new string vector here, but there is code that
depends on this actually being net::ResponseCookies, see OnTokenAuthFetched(). 
So instead of having two string vector types in this code, I figured I would
just stick to one.

+}
+
 typedef std::map<std::string, std::string> UserInfoMap;
 
 // An interface that defines the callbacks for objects that
@@ -53,7 +58,8 @@
   virtual void OnGetUserInfoSuccess(const UserInfoMap& data) {}
   virtual void OnGetUserInfoFailure(const GoogleServiceAuthError& error) {}
 
-  virtual void OnTokenAuthSuccess(const std::string& data) {}
+  virtual void OnTokenAuthSuccess(const net::ResponseCookies& cookies,
+                                  const std::string& data) {}
   virtual void OnTokenAuthFailure(const GoogleServiceAuthError& error) {}
 
   virtual void OnUberAuthTokenSuccess(const std::string& token) {}