Chromium Code Reviews| 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 && |