manifest: add theme_color value to the manifest

content/public/common/manifest.h

Index: content/public/common/manifest.h
diff --git a/content/public/common/manifest.h b/content/public/common/manifest.h
index 43bef5bee75f500d1233d3e80ba23a4a22c8e9c9..6f8f8f4eb405dd52b29cb7d471f71dfd06cb389e 100644
--- a/content/public/common/manifest.h
+++ b/content/public/common/manifest.h
@@ -94,6 +94,9 @@ struct CONTENT_EXPORT Manifest {
   // present.
   DisplayMode display;
 
+  // Set to -1 if parsing failed or field is not present.

[mlamouri (slow - plz ping), 2015/07/16 12:35:40]

Replace -1 by kInvalidOrMissingThemeColor.

[Lalit Maganti, 2015/07/16 14:49:01]

Done

+  int64_t theme_color;

[mlamouri (slow - plz ping), 2015/07/16 12:35:40]

nit: I'm not sure which order you are following here. It's not following the
spec order, nor the alphabetical order. Could you add that before gcm_sender_id?

[Lalit Maganti, 2015/07/16 14:49:01]

Done.

+
   // Set to blink::WebScreenOrientationLockDefault if the parsing failed or the
   // field was not present.
   blink::WebScreenOrientationLockType orientation;
@@ -121,6 +124,9 @@ struct CONTENT_EXPORT Manifest {
   // IPC. The renderer process should truncate the strings before sending the
   // Manifest and the browser process must do the same when receiving it.
   static const size_t kMaxIPCStringLength;
+
+  // Constant representing an invalid theme color. Set to -1.
+  static const long kInvalidThemeColor;

[mlamouri (slow - plz ping), 2015/07/16 12:35:40]

kInvalidOrMissingThemeColor maybe? Also, you might want to have the same type as
theme_color, which is int64_t.

[Lalit Maganti, 2015/07/16 14:49:01]

Done.

 };
 
 } // namespace content