Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(1093)

Unified Diff: content/public/common/manifest.cc

Issue 1235883007: manifest: add theme_color value to the manifest (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Fix nasko's comments Created 5 years, 5 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: content/public/common/manifest.cc
diff --git a/content/public/common/manifest.cc b/content/public/common/manifest.cc
index 1345e076b4114f66aaebeaac4dd253df894b5982..ed19c470073b435008580c28e83da985666b65bb 100644
--- a/content/public/common/manifest.cc
+++ b/content/public/common/manifest.cc
@@ -7,6 +7,7 @@
namespace content {
const double Manifest::Icon::kDefaultDensity = 1;
+const int64_t Manifest::kInvalidOrMissingThemeColor = -1;
const size_t Manifest::kMaxIPCStringLength = 4 * 1024;
Manifest::Icon::Icon()
@@ -25,7 +26,8 @@ Manifest::RelatedApplication::~RelatedApplication() {
Manifest::Manifest()
: display(DISPLAY_MODE_UNSPECIFIED),
orientation(blink::WebScreenOrientationLockDefault),
- prefer_related_applications(false) {
+ prefer_related_applications(false),
+ theme_color(Manifest::kInvalidOrMissingThemeColor) {
}
Manifest::~Manifest() {
@@ -40,6 +42,7 @@ bool Manifest::IsEmpty() const {
icons.empty() &&
related_applications.empty() &&
!prefer_related_applications &&
+ theme_color == Manifest::kInvalidOrMissingThemeColor &&
gcm_sender_id.is_null();
}

Powered by Google App Engine
This is Rietveld 408576698