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

Side by Side Diff: chrome/browser/ui/views/translate/translate_bubble_view.h

Issue 1923143003: Implement the 2016Q2 Translate UI designe spec out in (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: 2nd change according to msw review comments Created 4 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 unified diff | Download patch
OLDNEW
1 // Copyright 2013 The Chromium Authors. All rights reserved. 1 // Copyright 2013 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #ifndef CHROME_BROWSER_UI_VIEWS_TRANSLATE_TRANSLATE_BUBBLE_VIEW_H_ 5 #ifndef CHROME_BROWSER_UI_VIEWS_TRANSLATE_TRANSLATE_BUBBLE_VIEW_H_
6 #define CHROME_BROWSER_UI_VIEWS_TRANSLATE_TRANSLATE_BUBBLE_VIEW_H_ 6 #define CHROME_BROWSER_UI_VIEWS_TRANSLATE_TRANSLATE_BUBBLE_VIEW_H_
7 7
8 #include <map> 8 #include <map>
9 #include <string> 9 #include <string>
10 10
11 #include "base/gtest_prod_util.h" 11 #include "base/gtest_prod_util.h"
12 #include "base/macros.h" 12 #include "base/macros.h"
13 #include "base/strings/utf_string_conversions.h"
13 #include "chrome/browser/translate/chrome_translate_client.h" 14 #include "chrome/browser/translate/chrome_translate_client.h"
14 #include "chrome/browser/ui/translate/language_combobox_model.h" 15 #include "chrome/browser/ui/translate/language_combobox_model.h"
15 #include "chrome/browser/ui/translate/translate_bubble_model.h" 16 #include "chrome/browser/ui/translate/translate_bubble_model.h"
16 #include "chrome/browser/ui/translate/translate_bubble_test_utils.h" 17 #include "chrome/browser/ui/translate/translate_bubble_test_utils.h"
17 #include "chrome/browser/ui/views/location_bar/location_bar_bubble_delegate_view .h" 18 #include "chrome/browser/ui/views/location_bar/location_bar_bubble_delegate_view .h"
18 #include "components/translate/core/common/translate_errors.h" 19 #include "components/translate/core/common/translate_errors.h"
19 #include "content/public/browser/web_contents_observer.h" 20 #include "content/public/browser/web_contents_observer.h"
20 #include "ui/views/controls/button/button.h" 21 #include "ui/views/controls/button/button.h"
21 #include "ui/views/controls/combobox/combobox_listener.h" 22 #include "ui/views/controls/combobox/combobox_listener.h"
22 #include "ui/views/controls/link_listener.h" 23 #include "ui/views/controls/link_listener.h"
24 #include "ui/views/controls/styled_label_listener.h"
23 25
24 class Browser; 26 class Browser;
25 class PrefService; 27 class PrefService;
26 28
29 namespace gfx {
30 class Range;
31 }
32
27 namespace views { 33 namespace views {
28 class Checkbox; 34 class Checkbox;
29 class GridLayout; 35 class GridLayout;
30 class LabelButton; 36 class LabelButton;
31 class Link; 37 class Link;
32 class View; 38 class View;
39 class StyledLabel;
33 } 40 }
34 41
35 namespace ui { 42 namespace ui {
36 class SimpleComboboxModel; 43 class SimpleComboboxModel;
37 } 44 }
38 45
39 class TranslateBubbleView : public LocationBarBubbleDelegateView, 46 class TranslateBubbleView : public LocationBarBubbleDelegateView,
40 public views::ButtonListener, 47 public views::ButtonListener,
41 public views::ComboboxListener, 48 public views::ComboboxListener,
42 public views::LinkListener, 49 public views::LinkListener,
50 public views::StyledLabelListener,
43 public content::WebContentsObserver { 51 public content::WebContentsObserver {
44 public: 52 public:
45 // Commands shown in the action-style combobox. The value corresponds to the 53 // Commands shown in the action-style combobox. The value corresponds to the
46 // position in the combobox menu. Gaps will become separators. 54 // position in the combobox menu. Gaps will become separators.
47 enum class DenialComboboxIndex { 55 enum class DenialComboboxIndex {
48 DONT_TRANSLATE = 0, 56 DONT_TRANSLATE = 0,
49 NEVER_TRANSLATE_LANGUAGE = 1, 57 NEVER_TRANSLATE_LANGUAGE = 1,
50 NEVER_TRANSLATE_SITE = 3, 58 NEVER_TRANSLATE_SITE = 3,
51 MENU_SIZE = 4, 59 MENU_SIZE = 4,
52 }; 60 };
(...skipping 17 matching lines...) Expand all
70 78
71 // Returns the bubble view currently shown. This may return NULL. 79 // Returns the bubble view currently shown. This may return NULL.
72 static TranslateBubbleView* GetCurrentBubble(); 80 static TranslateBubbleView* GetCurrentBubble();
73 81
74 TranslateBubbleModel* model() { return model_.get(); } 82 TranslateBubbleModel* model() { return model_.get(); }
75 83
76 // views::BubbleDialogDelegateView methods. 84 // views::BubbleDialogDelegateView methods.
77 void Init() override; 85 void Init() override;
78 void ButtonPressed(views::Button* sender, const ui::Event& event) override; 86 void ButtonPressed(views::Button* sender, const ui::Event& event) override;
79 87
80 // views::WidgetDelegate method. 88 // views::WidgetDelegate methods.
89 bool ShouldShowCloseButton() const override;
81 void WindowClosing() override; 90 void WindowClosing() override;
82 91
83 // views::View methods. 92 // views::View methods.
84 bool AcceleratorPressed(const ui::Accelerator& accelerator) override; 93 bool AcceleratorPressed(const ui::Accelerator& accelerator) override;
85 gfx::Size GetPreferredSize() const override; 94 gfx::Size GetPreferredSize() const override;
86 95
87 // views::CombboxListener methods. 96 // views::CombboxListener methods.
88 void OnPerformAction(views::Combobox* combobox) override; 97 void OnPerformAction(views::Combobox* combobox) override;
89 98
90 // views::LinkListener method. 99 // views::LinkListener method.
91 void LinkClicked(views::Link* source, int event_flags) override; 100 void LinkClicked(views::Link* source, int event_flags) override;
92 101
102 // views::StyledLabelListener method.
103 void StyledLabelLinkClicked(views::StyledLabel* label,
104 const gfx::Range& range,
105 int event_flags) override;
106
93 // content::WebContentsObserver method. 107 // content::WebContentsObserver method.
94 void WebContentsDestroyed() override; 108 void WebContentsDestroyed() override;
95 109
110 // Overridden from views::WidgetObserver:
111 void OnWidgetClosing(views::Widget* widget) override;
112
96 // Returns the current view state. 113 // Returns the current view state.
97 TranslateBubbleModel::ViewState GetViewState() const; 114 TranslateBubbleModel::ViewState GetViewState() const;
98 115
99 private: 116 private:
100 enum LinkID { 117 enum LinkID {
101 LINK_ID_ADVANCED, 118 LINK_ID_ADVANCED,
102 LINK_ID_LANGUAGE_SETTINGS, 119 LINK_ID_LANGUAGE_SETTINGS,
103 }; 120 };
104 121
105 enum ButtonID { 122 enum ButtonID {
106 BUTTON_ID_TRANSLATE, 123 BUTTON_ID_TRANSLATE,
107 BUTTON_ID_DONE, 124 BUTTON_ID_DONE,
108 BUTTON_ID_CANCEL, 125 BUTTON_ID_CANCEL,
109 BUTTON_ID_SHOW_ORIGINAL, 126 BUTTON_ID_SHOW_ORIGINAL,
110 BUTTON_ID_TRY_AGAIN, 127 BUTTON_ID_TRY_AGAIN,
111 BUTTON_ID_ALWAYS_TRANSLATE, 128 BUTTON_ID_ALWAYS_TRANSLATE,
112 }; 129 };
113 130
114 enum ComboboxID { 131 enum ComboboxID {
115 COMBOBOX_ID_DENIAL, 132 COMBOBOX_ID_DENIAL,
116 COMBOBOX_ID_SOURCE_LANGUAGE, 133 COMBOBOX_ID_SOURCE_LANGUAGE,
117 COMBOBOX_ID_TARGET_LANGUAGE, 134 COMBOBOX_ID_TARGET_LANGUAGE,
118 }; 135 };
119 136
120 friend class TranslateBubbleViewTest; 137 friend class TranslateBubbleViewTest;
121 friend void ::translate::test_utils::PressTranslate(::Browser*); 138 friend void ::translate::test_utils::PressTranslate(::Browser*);
122 FRIEND_TEST_ALL_PREFIXES(TranslateBubbleViewTest, TranslateButton); 139 FRIEND_TEST_ALL_PREFIXES(TranslateBubbleViewTest, TranslateButton);
140 FRIEND_TEST_ALL_PREFIXES(TranslateBubbleViewTest, TranslateButtonIn2016Q2UI);
141 FRIEND_TEST_ALL_PREFIXES(TranslateBubbleViewTest, CloseButtonIn2016Q2UI);
123 FRIEND_TEST_ALL_PREFIXES(TranslateBubbleViewTest, AdvancedLink); 142 FRIEND_TEST_ALL_PREFIXES(TranslateBubbleViewTest, AdvancedLink);
143 FRIEND_TEST_ALL_PREFIXES(TranslateBubbleViewTest, AdvancedLinkIn2016Q2UI);
124 FRIEND_TEST_ALL_PREFIXES(TranslateBubbleViewTest, ShowOriginalButton); 144 FRIEND_TEST_ALL_PREFIXES(TranslateBubbleViewTest, ShowOriginalButton);
125 FRIEND_TEST_ALL_PREFIXES(TranslateBubbleViewTest, TryAgainButton); 145 FRIEND_TEST_ALL_PREFIXES(TranslateBubbleViewTest, TryAgainButton);
126 FRIEND_TEST_ALL_PREFIXES(TranslateBubbleViewTest, 146 FRIEND_TEST_ALL_PREFIXES(TranslateBubbleViewTest,
127 AlwaysTranslateCheckboxAndCancelButton); 147 AlwaysTranslateCheckboxAndCancelButton);
128 FRIEND_TEST_ALL_PREFIXES(TranslateBubbleViewTest, 148 FRIEND_TEST_ALL_PREFIXES(TranslateBubbleViewTest,
129 AlwaysTranslateCheckboxAndDoneButton); 149 AlwaysTranslateCheckboxAndDoneButton);
130 FRIEND_TEST_ALL_PREFIXES(TranslateBubbleViewTest, DoneButton); 150 FRIEND_TEST_ALL_PREFIXES(TranslateBubbleViewTest, DoneButton);
131 FRIEND_TEST_ALL_PREFIXES(TranslateBubbleViewTest, 151 FRIEND_TEST_ALL_PREFIXES(TranslateBubbleViewTest,
132 DoneButtonWithoutTranslating); 152 DoneButtonWithoutTranslating);
133 FRIEND_TEST_ALL_PREFIXES(TranslateBubbleViewTest, 153 FRIEND_TEST_ALL_PREFIXES(TranslateBubbleViewTest,
(...skipping 42 matching lines...) Expand 10 before | Expand all | Expand 10 after
176 196
177 // Switches the view type. 197 // Switches the view type.
178 void SwitchView(TranslateBubbleModel::ViewState view_state); 198 void SwitchView(TranslateBubbleModel::ViewState view_state);
179 199
180 // Switches to the error view. 200 // Switches to the error view.
181 void SwitchToErrorView(translate::TranslateErrors::Type error_type); 201 void SwitchToErrorView(translate::TranslateErrors::Type error_type);
182 202
183 // Updates the advanced view. 203 // Updates the advanced view.
184 void UpdateAdvancedView(); 204 void UpdateAdvancedView();
185 205
206 void ReportUiAction(int action);
207
186 static TranslateBubbleView* translate_bubble_view_; 208 static TranslateBubbleView* translate_bubble_view_;
187 209
188 views::View* before_translate_view_; 210 views::View* before_translate_view_;
189 views::View* translating_view_; 211 views::View* translating_view_;
190 views::View* after_translate_view_; 212 views::View* after_translate_view_;
191 views::View* error_view_; 213 views::View* error_view_;
192 views::View* advanced_view_; 214 views::View* advanced_view_;
193 215
194 std::unique_ptr<ui::SimpleComboboxModel> denial_combobox_model_; 216 std::unique_ptr<ui::SimpleComboboxModel> denial_combobox_model_;
195 std::unique_ptr<LanguageComboboxModel> source_language_combobox_model_; 217 std::unique_ptr<LanguageComboboxModel> source_language_combobox_model_;
196 std::unique_ptr<LanguageComboboxModel> target_language_combobox_model_; 218 std::unique_ptr<LanguageComboboxModel> target_language_combobox_model_;
197 219
198 views::Combobox* denial_combobox_; 220 views::Combobox* denial_combobox_;
199 views::Combobox* source_language_combobox_; 221 views::Combobox* source_language_combobox_;
200 views::Combobox* target_language_combobox_; 222 views::Combobox* target_language_combobox_;
201 223
202 views::Checkbox* always_translate_checkbox_; 224 views::Checkbox* always_translate_checkbox_;
203 225
204 views::LabelButton* advanced_cancel_button_; 226 views::LabelButton* advanced_cancel_button_;
205 views::LabelButton* advanced_done_button_; 227 views::LabelButton* advanced_done_button_;
206 228
207 std::unique_ptr<TranslateBubbleModel> model_; 229 std::unique_ptr<TranslateBubbleModel> model_;
208 230
209 translate::TranslateErrors::Type error_type_; 231 translate::TranslateErrors::Type error_type_;
210 232
211 // Whether the window is an incognito window. 233 // Whether the window is an incognito window.
212 const bool is_in_incognito_window_; 234 const bool is_in_incognito_window_;
213 235
214 // Whether the translation is acutually executed. 236 // Use the new (2016 Q2) translate UI.
215 bool translate_executed_; 237 bool use_2016_q2_ui_;
216 238
217 DISALLOW_COPY_AND_ASSIGN(TranslateBubbleView); 239 DISALLOW_COPY_AND_ASSIGN(TranslateBubbleView);
218 }; 240 };
219 241
220 #endif // CHROME_BROWSER_UI_VIEWS_TRANSLATE_TRANSLATE_BUBBLE_VIEW_H_ 242 #endif // CHROME_BROWSER_UI_VIEWS_TRANSLATE_TRANSLATE_BUBBLE_VIEW_H_
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698