Implement the 2016Q2 Translate UI designe spec out in

chrome/browser/ui/views/translate/translate_bubble_view.h

 // Copyright 2013 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_TRANSLATE_TRANSLATE_BUBBLE_VIEW_H_
 #define CHROME_BROWSER_UI_VIEWS_TRANSLATE_TRANSLATE_BUBBLE_VIEW_H_
 
 #include <map>
 #include <string>
 
 #include "base/gtest_prod_util.h"
 #include "base/macros.h"
+#include "base/strings/utf_string_conversions.h"
 #include "chrome/browser/translate/chrome_translate_client.h"
 #include "chrome/browser/ui/translate/language_combobox_model.h"
 #include "chrome/browser/ui/translate/translate_bubble_model.h"
 #include "chrome/browser/ui/translate/translate_bubble_test_utils.h"
 #include "chrome/browser/ui/views/location_bar/location_bar_bubble_delegate_view.h"
 #include "components/translate/core/common/translate_errors.h"
 #include "content/public/browser/web_contents_observer.h"
 #include "ui/views/controls/button/button.h"
 #include "ui/views/controls/combobox/combobox_listener.h"
 #include "ui/views/controls/link_listener.h"
+#include "ui/views/controls/styled_label_listener.h"
 
 class Browser;
 class PrefService;
 
+namespace gfx {
+class Range;
+}
+
 namespace views {
 class Checkbox;
 class GridLayout;
 class LabelButton;
 class Link;
 class View;
+class StyledLabel;
 }
 
 namespace ui {
 class SimpleComboboxModel;
 }
 
 class TranslateBubbleView : public LocationBarBubbleDelegateView,
                             public views::ButtonListener,
                             public views::ComboboxListener,
                             public views::LinkListener,
+                            public views::StyledLabelListener,
                             public content::WebContentsObserver {
  public:
   // Commands shown in the action-style combobox. The value corresponds to the
   // position in the combobox menu. Gaps will become separators.
   enum class DenialComboboxIndex {
     DONT_TRANSLATE = 0,
     NEVER_TRANSLATE_LANGUAGE = 1,
     NEVER_TRANSLATE_SITE = 3,
     MENU_SIZE = 4,
   };
@@ skipping 36 matching lines @@
 
   // views::LinkListener method.
   void LinkClicked(views::Link* source, int event_flags) override;
 
   // content::WebContentsObserver method.
   void WebContentsDestroyed() override;
 
   // Returns the current view state.
   TranslateBubbleModel::ViewState GetViewState() const;
 
+  bool ShouldShowCloseButton() const override;

[msw, 2016/04/29 20:00:02]

Order with the other WidgetDelegate method, WindowClosing, in decl order.

[ftang, 2016/04/29 22:37:02]

Done.

+
+  // views::StyledLabelListener method.
+  void StyledLabelLinkClicked(views::StyledLabel* label,

[msw, 2016/04/29 20:00:02]

Order just after the LinkListener override and before the WebContentsObserver
override to match the subclass ordering.

[ftang, 2016/04/29 22:37:02]

Done.

+                              const gfx::Range& range,
+                              int event_flags) override;
+
+  // Overridden from views::WidgetObserver:
+  void OnWidgetClosing(views::Widget* widget) override;

[msw, 2016/04/29 20:00:02]

Order before non-override function GetViewState.

[ftang, 2016/04/29 22:37:02]

Done.

+
  private:
   enum LinkID {
     LINK_ID_ADVANCED,
     LINK_ID_LANGUAGE_SETTINGS,
   };
 
   enum ButtonID {
     BUTTON_ID_TRANSLATE,
     BUTTON_ID_DONE,
     BUTTON_ID_CANCEL,
     BUTTON_ID_SHOW_ORIGINAL,
     BUTTON_ID_TRY_AGAIN,
     BUTTON_ID_ALWAYS_TRANSLATE,
   };
 
   enum ComboboxID {
     COMBOBOX_ID_DENIAL,
     COMBOBOX_ID_SOURCE_LANGUAGE,
     COMBOBOX_ID_TARGET_LANGUAGE,
   };
 
   friend class TranslateBubbleViewTest;
   friend void ::translate::test_utils::PressTranslate(::Browser*);
   FRIEND_TEST_ALL_PREFIXES(TranslateBubbleViewTest, TranslateButton);
+  FRIEND_TEST_ALL_PREFIXES(TranslateBubbleViewTest, TranslateButtonIn2016Q2UI);
+  FRIEND_TEST_ALL_PREFIXES(TranslateBubbleViewTest, CloseButtonIn2016Q2UI);
   FRIEND_TEST_ALL_PREFIXES(TranslateBubbleViewTest, AdvancedLink);
+  FRIEND_TEST_ALL_PREFIXES(TranslateBubbleViewTest, AdvancedLinkIn2016Q2UI);
   FRIEND_TEST_ALL_PREFIXES(TranslateBubbleViewTest, ShowOriginalButton);
   FRIEND_TEST_ALL_PREFIXES(TranslateBubbleViewTest, TryAgainButton);
   FRIEND_TEST_ALL_PREFIXES(TranslateBubbleViewTest,
                            AlwaysTranslateCheckboxAndCancelButton);
   FRIEND_TEST_ALL_PREFIXES(TranslateBubbleViewTest,
                            AlwaysTranslateCheckboxAndDoneButton);
   FRIEND_TEST_ALL_PREFIXES(TranslateBubbleViewTest, DoneButton);
   FRIEND_TEST_ALL_PREFIXES(TranslateBubbleViewTest,
                            DoneButtonWithoutTranslating);
   FRIEND_TEST_ALL_PREFIXES(TranslateBubbleViewTest,
@@ skipping 42 matching lines @@
 
   // Switches the view type.
   void SwitchView(TranslateBubbleModel::ViewState view_state);
 
   // Switches to the error view.
   void SwitchToErrorView(translate::TranslateErrors::Type error_type);
 
   // Updates the advanced view.
   void UpdateAdvancedView();
 
+  void ReportUiAction(int action);
+
   static TranslateBubbleView* translate_bubble_view_;
 
   views::View* before_translate_view_;
   views::View* translating_view_;
   views::View* after_translate_view_;
   views::View* error_view_;
   views::View* advanced_view_;
 
   std::unique_ptr<ui::SimpleComboboxModel> denial_combobox_model_;
   std::unique_ptr<LanguageComboboxModel> source_language_combobox_model_;
   std::unique_ptr<LanguageComboboxModel> target_language_combobox_model_;
 
   views::Combobox* denial_combobox_;
   views::Combobox* source_language_combobox_;
   views::Combobox* target_language_combobox_;
 
   views::Checkbox* always_translate_checkbox_;
 
   views::LabelButton* advanced_cancel_button_;
   views::LabelButton* advanced_done_button_;
 
   std::unique_ptr<TranslateBubbleModel> model_;
 
   translate::TranslateErrors::Type error_type_;
 
   // Whether the window is an incognito window.
   const bool is_in_incognito_window_;
 
-  // Whether the translation is acutually executed.
-  bool translate_executed_;
+  // Whether one of denial buttons is clicked.

[msw, 2016/04/29 20:00:02]

nit: 'the', 'was': "Whether one of the denial was clicked."

[ftang, 2016/04/29 22:37:02]

this is merge mistake. removed

+  bool denial_button_clicked_;
+
+  // Use 2016 Q2 Translate UI

[msw, 2016/04/29 20:00:02]

nit: trailing period; "Use the new (2016 Q2) translate UI."

[ftang, 2016/04/29 22:37:02]

Done.

+  bool use_2016_q2_ui_;
 
   DISALLOW_COPY_AND_ASSIGN(TranslateBubbleView);
 };
 
 #endif  // CHROME_BROWSER_UI_VIEWS_TRANSLATE_TRANSLATE_BUBBLE_VIEW_H_