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

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: Created 3 years, 9 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..e78517c969f0efcd5eca9bd1f3095f6062aa54a8
--- /dev/null
+++ b/chrome/browser/ui/views/srt_prompt_dialog.h
@@ -0,0 +1,71 @@
+// 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).
ftirelo 2017/04/05 18:03:40 Please explain the relationship with the controlle
alito 2017/04/05 19:23:49 Done.
+class SRTPromptDialog : public views::DialogDelegateView,
+ public views::ButtonListener {
+ public:
+ explicit SRTPromptDialog(safe_browsing::SRTPromptController* controller);
ftirelo 2017/04/05 18:03:40 Nit: please explain who owns the controller pointe
alito 2017/04/05 19:23:49 Done.
+ ~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:
+ class ExpandableMessageView;
+
+ Browser* browser_;
+ safe_browsing::SRTPromptController* controller_;
+ bool interaction_done_;
+
+ ExpandableMessageView* details_view_;
+ views::LabelButton* expand_details_button_;
+ SkColor expand_details_button_color_;
+ gfx::ImageSkia expand_icon_;
+ gfx::ImageSkia fold_icon_;
+
+ DISALLOW_COPY_AND_ASSIGN(SRTPromptDialog);
+};
+
+#endif // CHROME_BROWSER_UI_VIEWS_SRT_PROMPT_DIALOG_H_

Powered by Google App Engine
This is Rietveld 408576698