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

Unified Diff: extensions/browser/updater/manifest_fetch_data.h

Issue 2299203004: Attempt to repair corrupt enterprise policy force-installed extensions (Closed)
Patch Set: fix chromeos compile problem Created 4 years, 3 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: 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..eac419be43ea86f145a99afc971ada355445c9a8 100644
--- a/extensions/browser/updater/manifest_fetch_data.h
+++ b/extensions/browser/updater/manifest_fetch_data.h
@@ -70,6 +70,19 @@ class ManifestFetchData {
disable_reasons(reasons) {}
};
+ // A variety of parameters for special cases that aren't used for most
+ // requests.
+ struct ExtraParams {
+ // Additional data to be passed up in the update request.
+ std::string update_url_data;
+
+ // Indicates whether this is a policy force-installed extension that we're
+ // reinstalling due to corruption being detected in it.
lazyboy 2016/09/02 19:21:53 s/detected in it/detected in the extension as this
asargent_no_longer_on_chrome 2016/09/09 03:30:44 Comment simplified (Note that this struct moved to
+ bool is_corrupt_policy_reinstall;
lazyboy 2016/09/02 19:21:53 (again feel free to suggest something better) in o
asargent_no_longer_on_chrome 2016/09/09 03:30:44 This moved to extension_downloader.h, and I change
+
+ ExtraParams();
+ };
+
ManifestFetchData(const GURL& update_url,
int request_id,
const std::string& brand_code,
@@ -83,8 +96,8 @@ class ManifestFetchData {
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,
+ const ExtraParams* extra_params);
const GURL& base_url() const { return base_url_; }
const GURL& full_url() const { return full_url_; }

Powered by Google App Engine
This is Rietveld 408576698