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

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

Issue 2637003002: Add share_target field to Manifest. (Closed)
Patch Set: Doc update and clean up Created 3 years, 11 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 8e09aad2e71ea9aa86f681d7df267f2dbbddb09c..a854b02b8681d83e442bf9eec560078b31b874bd 100644
--- a/content/public/common/manifest.cc
+++ b/content/public/common/manifest.cc
@@ -18,19 +18,24 @@ Manifest::Icon::Icon() { }
Manifest::Icon::Icon(const Icon& other) = default;
-Manifest::Icon::~Icon() {
-}
+Manifest::Icon::~Icon() { }
bool Manifest::Icon::operator==(const Manifest::Icon& other) const {
return src == other.src && type == other.type && sizes == other.sizes;
}
-Manifest::RelatedApplication::RelatedApplication() {
-}
+Manifest::ShareTarget::ShareTarget() { }
Matt Giuca 2017/01/18 07:51:32 Has this been git cl formatted? I think there's su
constantina 2017/01/18 23:28:09 Done. There were inconsistencies, so I made all em
+
+Manifest::ShareTarget::~ShareTarget() { }
-Manifest::RelatedApplication::~RelatedApplication() {
+bool Manifest::ShareTarget::is_null() const {
+ return url_template.is_null();
}
+Manifest::RelatedApplication::RelatedApplication() { }
+
+Manifest::RelatedApplication::~RelatedApplication() { }
+
Manifest::Manifest()
: display(blink::WebDisplayModeUndefined),
orientation(blink::WebScreenOrientationLockDefault),
@@ -41,8 +46,7 @@ Manifest::Manifest()
Manifest::Manifest(const Manifest& other) = default;
-Manifest::~Manifest() {
-}
+Manifest::~Manifest() { }
bool Manifest::IsEmpty() const {
return name.is_null() &&
@@ -51,6 +55,7 @@ bool Manifest::IsEmpty() const {
display == blink::WebDisplayModeUndefined &&
orientation == blink::WebScreenOrientationLockDefault &&
icons.empty() &&
+ share_target.is_null() &&
related_applications.empty() &&
!prefer_related_applications &&
theme_color == Manifest::kInvalidOrMissingColor &&

Powered by Google App Engine
This is Rietveld 408576698