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

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 try - fix test breakages in TranslateUIDelegateTest 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 36 matching lines...) Expand 10 before | Expand all | Expand 10 after
89 97
90 // views::LinkListener method. 98 // views::LinkListener method.
91 void LinkClicked(views::Link* source, int event_flags) override; 99 void LinkClicked(views::Link* source, int event_flags) override;
92 100
93 // content::WebContentsObserver method. 101 // content::WebContentsObserver method.
94 void WebContentsDestroyed() override; 102 void WebContentsDestroyed() override;
95 103
96 // Returns the current view state. 104 // Returns the current view state.
97 TranslateBubbleModel::ViewState GetViewState() const; 105 TranslateBubbleModel::ViewState GetViewState() const;
98 106
107 bool ShouldShowCloseButton() const override;
108
109 // views::StyledLabelListener method.
110 void StyledLabelLinkClicked(views::StyledLabel* label,
111 const gfx::Range& range,
112 int event_flags) override;
99 private: 113 private:
100 enum LinkID { 114 enum LinkID {
101 LINK_ID_ADVANCED, 115 LINK_ID_ADVANCED,
102 LINK_ID_LANGUAGE_SETTINGS, 116 LINK_ID_LANGUAGE_SETTINGS,
103 }; 117 };
104 118
105 enum ButtonID { 119 enum ButtonID {
106 BUTTON_ID_TRANSLATE, 120 BUTTON_ID_TRANSLATE,
107 BUTTON_ID_DONE, 121 BUTTON_ID_DONE,
108 BUTTON_ID_CANCEL, 122 BUTTON_ID_CANCEL,
109 BUTTON_ID_SHOW_ORIGINAL, 123 BUTTON_ID_SHOW_ORIGINAL,
110 BUTTON_ID_TRY_AGAIN, 124 BUTTON_ID_TRY_AGAIN,
111 BUTTON_ID_ALWAYS_TRANSLATE, 125 BUTTON_ID_ALWAYS_TRANSLATE,
112 }; 126 };
113 127
114 enum ComboboxID { 128 enum ComboboxID {
115 COMBOBOX_ID_DENIAL, 129 COMBOBOX_ID_DENIAL,
116 COMBOBOX_ID_SOURCE_LANGUAGE, 130 COMBOBOX_ID_SOURCE_LANGUAGE,
117 COMBOBOX_ID_TARGET_LANGUAGE, 131 COMBOBOX_ID_TARGET_LANGUAGE,
118 }; 132 };
119 133
120 friend class TranslateBubbleViewTest; 134 friend class TranslateBubbleViewTest;
121 friend void ::translate::test_utils::PressTranslate(::Browser*); 135 friend void ::translate::test_utils::PressTranslate(::Browser*);
122 FRIEND_TEST_ALL_PREFIXES(TranslateBubbleViewTest, TranslateButton); 136 FRIEND_TEST_ALL_PREFIXES(TranslateBubbleViewTest, TranslateButton);
137 FRIEND_TEST_ALL_PREFIXES(TranslateBubbleViewTest, TranslateButtonIn2016Q2UI);
138 FRIEND_TEST_ALL_PREFIXES(TranslateBubbleViewTest, CloseButtonIn2016Q2UI);
123 FRIEND_TEST_ALL_PREFIXES(TranslateBubbleViewTest, AdvancedLink); 139 FRIEND_TEST_ALL_PREFIXES(TranslateBubbleViewTest, AdvancedLink);
140 FRIEND_TEST_ALL_PREFIXES(TranslateBubbleViewTest, AdvancedLinkIn2016Q2UI);
124 FRIEND_TEST_ALL_PREFIXES(TranslateBubbleViewTest, ShowOriginalButton); 141 FRIEND_TEST_ALL_PREFIXES(TranslateBubbleViewTest, ShowOriginalButton);
125 FRIEND_TEST_ALL_PREFIXES(TranslateBubbleViewTest, TryAgainButton); 142 FRIEND_TEST_ALL_PREFIXES(TranslateBubbleViewTest, TryAgainButton);
126 FRIEND_TEST_ALL_PREFIXES(TranslateBubbleViewTest, 143 FRIEND_TEST_ALL_PREFIXES(TranslateBubbleViewTest,
127 AlwaysTranslateCheckboxAndCancelButton); 144 AlwaysTranslateCheckboxAndCancelButton);
128 FRIEND_TEST_ALL_PREFIXES(TranslateBubbleViewTest, 145 FRIEND_TEST_ALL_PREFIXES(TranslateBubbleViewTest,
129 AlwaysTranslateCheckboxAndDoneButton); 146 AlwaysTranslateCheckboxAndDoneButton);
130 FRIEND_TEST_ALL_PREFIXES(TranslateBubbleViewTest, DoneButton); 147 FRIEND_TEST_ALL_PREFIXES(TranslateBubbleViewTest, DoneButton);
131 FRIEND_TEST_ALL_PREFIXES(TranslateBubbleViewTest, 148 FRIEND_TEST_ALL_PREFIXES(TranslateBubbleViewTest,
132 DoneButtonWithoutTranslating); 149 DoneButtonWithoutTranslating);
133 FRIEND_TEST_ALL_PREFIXES(TranslateBubbleViewTest, 150 FRIEND_TEST_ALL_PREFIXES(TranslateBubbleViewTest,
(...skipping 42 matching lines...) Expand 10 before | Expand all | Expand 10 after
176 193
177 // Switches the view type. 194 // Switches the view type.
178 void SwitchView(TranslateBubbleModel::ViewState view_state); 195 void SwitchView(TranslateBubbleModel::ViewState view_state);
179 196
180 // Switches to the error view. 197 // Switches to the error view.
181 void SwitchToErrorView(translate::TranslateErrors::Type error_type); 198 void SwitchToErrorView(translate::TranslateErrors::Type error_type);
182 199
183 // Updates the advanced view. 200 // Updates the advanced view.
184 void UpdateAdvancedView(); 201 void UpdateAdvancedView();
185 202
203 void ReportUiAction(int action);
204
186 static TranslateBubbleView* translate_bubble_view_; 205 static TranslateBubbleView* translate_bubble_view_;
187 206
188 views::View* before_translate_view_; 207 views::View* before_translate_view_;
189 views::View* translating_view_; 208 views::View* translating_view_;
190 views::View* after_translate_view_; 209 views::View* after_translate_view_;
191 views::View* error_view_; 210 views::View* error_view_;
192 views::View* advanced_view_; 211 views::View* advanced_view_;
193 212
194 std::unique_ptr<ui::SimpleComboboxModel> denial_combobox_model_; 213 std::unique_ptr<ui::SimpleComboboxModel> denial_combobox_model_;
195 std::unique_ptr<LanguageComboboxModel> source_language_combobox_model_; 214 std::unique_ptr<LanguageComboboxModel> source_language_combobox_model_;
196 std::unique_ptr<LanguageComboboxModel> target_language_combobox_model_; 215 std::unique_ptr<LanguageComboboxModel> target_language_combobox_model_;
197 216
198 views::Combobox* denial_combobox_; 217 views::Combobox* denial_combobox_;
199 views::Combobox* source_language_combobox_; 218 views::Combobox* source_language_combobox_;
200 views::Combobox* target_language_combobox_; 219 views::Combobox* target_language_combobox_;
201 220
202 views::Checkbox* always_translate_checkbox_; 221 views::Checkbox* always_translate_checkbox_;
203 222
204 views::LabelButton* advanced_cancel_button_; 223 views::LabelButton* advanced_cancel_button_;
205 views::LabelButton* advanced_done_button_; 224 views::LabelButton* advanced_done_button_;
206 225
207 std::unique_ptr<TranslateBubbleModel> model_; 226 std::unique_ptr<TranslateBubbleModel> model_;
208 227
209 translate::TranslateErrors::Type error_type_; 228 translate::TranslateErrors::Type error_type_;
210 229
211 // Whether the window is an incognito window. 230 // Whether the window is an incognito window.
212 const bool is_in_incognito_window_; 231 const bool is_in_incognito_window_;
213 232
214 // Whether the translation is acutually executed. 233 // Whether one of denial buttons is clicked.
215 bool translate_executed_; 234 bool denial_button_clicked_;
235
236 // Use 2016 Q2 Translate UI
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