gdata: Expose authentication states from GDataDocumentsService.

chrome/browser/chromeos/gdata/gdata_documents_service.h

Index: chrome/browser/chromeos/gdata/gdata_documents_service.h
diff --git a/chrome/browser/chromeos/gdata/gdata_documents_service.h b/chrome/browser/chromeos/gdata/gdata_documents_service.h
index 81ca2626c496497774b5d53f61985eed587ee69d..d626cbf623dc0a191f947f0e129637febfff13f5 100644
--- a/chrome/browser/chromeos/gdata/gdata_documents_service.h
+++ b/chrome/browser/chromeos/gdata/gdata_documents_service.h
@@ -184,6 +184,12 @@ class DocumentsServiceInterface {
   virtual void AuthorizeApp(const GURL& resource_url,
                             const std::string& app_id,
                             const GetDataCallback& callback) = 0;
+
+  // True if OAuth2 auth token is retrieved and believed to be fresh.
+  virtual bool IsFullyAuthenticated() const = 0;
+
+  // True if OAuth2 refresh token is present.
+  virtual bool IsPartiallyAuthenticated() const = 0;

[Daniel Erat, 2012/07/24 20:31:08]

nit since you're just using the existing names in this change: Do "fully
authenticated" and "partially authenticated" have have well-known present/fresh
vs. present meanings when talking about OAuth2 tokens?  If not, names like
HasFreshAuthToken() and HasAuthToken() would probably be more descriptive.  I'm
also a bit concerned that people may think that IsPartiallyAuthenticated()
implies !IsFullyAuthenticated().

[satorux1, 2012/07/24 20:52:01]

That's a good suggestion. I'll create a patch to rename these methods.

 };
 
 // This class provides documents feed service calls.
@@ -248,6 +254,9 @@ class DocumentsService : public DocumentsServiceInterface {
   virtual void AuthorizeApp(const GURL& resource_url,
                             const std::string& app_id,
                             const GetDataCallback& callback) OVERRIDE;
+  virtual bool IsFullyAuthenticated() const OVERRIDE;
+  virtual bool IsPartiallyAuthenticated() const OVERRIDE;
+
  private:
   Profile* profile_;