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

Unified Diff: chrome/browser/extensions/extension_uninstall_dialog.h

Issue 1148323007: [Extensions] Introduce a ScopedExtensionDialogAutoConfirm (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Ben's Created 5 years, 7 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_uninstall_dialog.h
diff --git a/chrome/browser/extensions/extension_uninstall_dialog.h b/chrome/browser/extensions/extension_uninstall_dialog.h
index e8039c3a43c174ffc700d5bb27cbf083c2d61c52..9691906aabd4fa086070fd105c3a4b3a9aead1dd 100644
--- a/chrome/browser/extensions/extension_uninstall_dialog.h
+++ b/chrome/browser/extensions/extension_uninstall_dialog.h
@@ -34,24 +34,6 @@ class ExtensionUninstallDialog
CLOSE_ACTION_LAST = 3,
};
- // A setting to cause extension/app installs from the webstore skip the normal
- // confirmation dialog. This should only be used in tests.
- enum AutoConfirmForTests {
- NONE, // The prompt will show normally.
- ACCEPT, // The prompt will always accept.
- CANCEL, // The prompt will always cancel.
- };
- class ScopedAutoConfirm {
- public:
- explicit ScopedAutoConfirm(AutoConfirmForTests new_value);
- ~ScopedAutoConfirm();
-
- private:
- AutoConfirmForTests original_value_;
-
- DISALLOW_COPY_AND_ASSIGN(ScopedAutoConfirm);
- };
-
// TODO(devlin): For a single method like this, a callback is probably more
// appropriate than a delegate.
class Delegate {
@@ -111,8 +93,6 @@ class ExtensionUninstallDialog
const gfx::ImageSkia& icon() const { return icon_; }
private:
- static AutoConfirmForTests g_auto_confirm_for_testing;
-
// Handles the "report abuse" checkbox being checked at the close of the
// dialog.
void HandleReportAbuse();

Powered by Google App Engine
This is Rietveld 408576698