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

Unified Diff: chrome/browser/ui/views/srt_prompt_dialog.h

Issue 2795133002: Chrome Cleaner UI: Add a dialog class for the new UI (Closed)
Patch Set: No longer subclassing View for the details section Created 3 years, 8 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/ui/views/srt_prompt_dialog.h
diff --git a/chrome/browser/ui/views/srt_prompt_dialog.h b/chrome/browser/ui/views/srt_prompt_dialog.h
new file mode 100644
index 0000000000000000000000000000000000000000..02a993165ae48595bbe34aac59a1bd3dfae0f9ff
--- /dev/null
+++ b/chrome/browser/ui/views/srt_prompt_dialog.h
@@ -0,0 +1,83 @@
+// Copyright 2017 The Chromium Authors. All rights reserved.
+// Use of this source code is governed by a BSD-style license that can be
+// found in the LICENSE file.
+
+#ifndef CHROME_BROWSER_UI_VIEWS_SRT_PROMPT_DIALOG_H_
+#define CHROME_BROWSER_UI_VIEWS_SRT_PROMPT_DIALOG_H_
+
+#include "base/macros.h"
+#include "third_party/skia/include/core/SkColor.h"
+#include "ui/gfx/image/image_skia.h"
+#include "ui/views/controls/button/button.h"
+#include "ui/views/controls/button/label_button.h"
+#include "ui/views/window/dialog_delegate.h"
+
+class Browser;
+
+namespace safe_browsing {
+class SRTPromptController;
+}
+
+// A modal dialog asking the user if they want to run the Chrome Cleanup
+// tool. The dialog will have the following sections:
+//
+// 1. Main section with general information about unwanted software that has
+// been found on the user's system.
+// 2. Expandable details section with more details about unwanted software that
+// will be removed and Chrome settings that will be reset.
+// 3. Checkbox asking for permissions to upload logs (not yet implemented).
+//
+// The strings and icons used in the dialog are provided by a
+// |SRTPromptController| object, which will also receive information about how
+// the user interacts with the dialog. The controller object owns itself and
+// will delete itself once it has received information about the user's
+// interaction with the dialog. See the |SRTPromptController| class's
+// description for more details.
+class SRTPromptDialog : public views::DialogDelegateView,
+ public views::ButtonListener {
+ public:
+ // The |controller| object manages its own lifetime and is not owned by
+ // |SRTPromptDialog|. See the description of the |SRTPromptController| class
+ // for details.
+ explicit SRTPromptDialog(safe_browsing::SRTPromptController* controller);
+ ~SRTPromptDialog() override;
+
+ void Show(Browser* browser);
+
+ // ui::DialogModel overrides.
+ bool ShouldDefaultButtonBeBlue() const override;
+
+ // views::WidgetDelegate overrides.
+ ui::ModalType GetModalType() const override;
+ base::string16 GetWindowTitle() const override;
+
+ // views::DialogDelegate overrides.
+ base::string16 GetDialogButtonLabel(ui::DialogButton button) const override;
+ bool Accept() override;
+ bool Cancel() override;
+
+ // views::View overrides.
+ gfx::Size GetPreferredSize() const override;
+
+ // views::ButtonListener overrides.
+ void ButtonPressed(views::Button* sender, const ui::Event& event) override;
+
+ private:
+ SkColor GetDetailsButtonColor();
+ void UpdateDetailsButton();
+
+ Browser* browser_;
+ safe_browsing::SRTPromptController* controller_;
+ // Is set to true when the user has interacted with the dialog or the dialog
+ // has been closed some other way. Used to ensure that |controller_| does not
+ // receives multiple notifications about user interaction or the dialog being
+ // closed.
+ bool interaction_done_;
+
+ views::View* details_view_;
+ views::LabelButton* details_button_;
+
+ DISALLOW_COPY_AND_ASSIGN(SRTPromptDialog);
+};
+
+#endif // CHROME_BROWSER_UI_VIEWS_SRT_PROMPT_DIALOG_H_

Powered by Google App Engine
This is Rietveld 408576698