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

Unified Diff: chrome/browser/extensions/extension_install_prompt.cc

Issue 2668863002: [Extensions] Remove FeatureSwitch::easy_off_store_install (Closed)
Patch Set: Add TODO for histograms 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: chrome/browser/extensions/extension_install_prompt.cc
diff --git a/chrome/browser/extensions/extension_install_prompt.cc b/chrome/browser/extensions/extension_install_prompt.cc
index e4b81f782b90d4f51ca5c5759e2190604d214174..ff86a0a972e15c57ce09d46f78307e503d68840e 100644
--- a/chrome/browser/extensions/extension_install_prompt.cc
+++ b/chrome/browser/extensions/extension_install_prompt.cc
@@ -667,16 +667,9 @@ void ExtensionInstallPrompt::ShowDialog(
// We special-case themes to not show any confirm UI. Instead they are
// immediately installed, and then we show an infobar (see OnInstallSuccess)
// to allow the user to revert if they don't like it.
- //
- // We don't do this in the case where off-store extension installs are
- // disabled because in that case, we don't show the dangerous download UI, so
- // we need the UI confirmation.
- if (extension->is_theme()) {
- if (extension->from_webstore() ||
- extensions::FeatureSwitch::easy_off_store_install()->IsEnabled()) {
- base::ResetAndReturn(&done_callback_).Run(Result::ACCEPTED);
- return;
- }
+ if (extension->is_theme() && extension->from_webstore()) {
+ base::ResetAndReturn(&done_callback_).Run(Result::ACCEPTED);
+ return;
}
LoadImageIfNeeded();

Powered by Google App Engine
This is Rietveld 408576698