[Push] Use proper accessors for PushMessagingApplicationId.

chrome/browser/services/gcm/push_messaging_application_id.h

Index: chrome/browser/services/gcm/push_messaging_application_id.h
diff --git a/chrome/browser/services/gcm/push_messaging_application_id.h b/chrome/browser/services/gcm/push_messaging_application_id.h
index 54f14807f2ce9650a0f4785128a9157ca985584e..ddb0af2c6ecf676c095bdd0d4d78f961a2018266 100644
--- a/chrome/browser/services/gcm/push_messaging_application_id.h
+++ b/chrome/browser/services/gcm/push_messaging_application_id.h
@@ -54,9 +54,11 @@ class PushMessagingApplicationId {
 
   bool IsValid() const;
 
-  const std::string app_id_guid;
-  const GURL origin;
-  const int64 service_worker_registration_id;
+  const std::string& app_id_guid() const { return app_id_guid_; }
+  const GURL& origin() const { return origin_; }
+  int64 service_worker_registration_id() const {
+    return service_worker_registration_id_;
+  }
 
  private:
   friend class PushMessagingApplicationIdTest;
@@ -67,6 +69,10 @@ class PushMessagingApplicationId {
   PushMessagingApplicationId(const std::string& app_id_guid,
                              const GURL& origin,
                              int64 service_worker_registration_id);
+
+  std::string app_id_guid_;

[johnme, 2015/02/17 12:42:05]

I'd rather you keep this immutable (const)

+  GURL origin_;
+  int64 service_worker_registration_id_;
 };
 
 }  // namespace gcm