Chromium Code Reviews| Index: extensions/browser/updater/manifest_fetch_data.h |
| diff --git a/extensions/browser/updater/manifest_fetch_data.h b/extensions/browser/updater/manifest_fetch_data.h |
| index 0a31846d4eccdc4fa0b3488e7d2085a0247ebdd6..bafd6ca48782f09a68c235d4cf7d8332718bd9d4 100644 |
| --- a/extensions/browser/updater/manifest_fetch_data.h |
| +++ b/extensions/browser/updater/manifest_fetch_data.h |
| @@ -74,22 +74,26 @@ class ManifestFetchData { |
| int request_id, |
| const std::string& brand_code, |
| const std::string& base_query_params, |
| - PingMode ping_mode); |
| + PingMode ping_mode, |
| + bool foreground_check); |
|
Devlin
2017/03/24 15:14:47
I'd slightly prefer we use an enum for this, e.g.
Minh X. Nguyen
2017/03/27 23:05:29
Done.
|
| ~ManifestFetchData(); |
| // Returns true if this extension information was successfully added. If the |
| - // return value is false it means the full_url would have become too long, and |
| + // return value is false it means the full_url would have become too long or |
| + // the request type is not compatible the current request type, and |
| // this ManifestFetchData object remains unchanged. |
| bool AddExtension(const std::string& id, |
| const std::string& version, |
| const PingData* ping_data, |
| const std::string& update_url_data, |
| - const std::string& install_source); |
| + const std::string& install_source, |
| + bool foreground_check); |
| const GURL& base_url() const { return base_url_; } |
| const GURL& full_url() const { return full_url_; } |
| const std::set<std::string>& extension_ids() const { return extension_ids_; } |
| const std::set<int>& request_ids() const { return request_ids_; } |
| + bool foreground_check() const { return foreground_check_; } |
| // Returns true if the given id is included in this manifest fetch. |
| bool Includes(const std::string& extension_id) const; |
| @@ -132,6 +136,9 @@ class ManifestFetchData { |
| // (and possibly extra metrics) will be included in the fetch query. |
| const PingMode ping_mode_; |
| + // If true, this is an on-demand update. |
| + bool foreground_check_; |
| + |
| DISALLOW_COPY_AND_ASSIGN(ManifestFetchData); |
| }; |