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

Side by Side Diff: chrome/browser/ui/views/ime/ime_window_frame_view.h

Issue 2744463002: Add VectorIconButton functionality to ImageButton. (Closed)
Patch Set: fix cros build 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 unified diff | Download patch
OLDNEW
1 // Copyright 2016 The Chromium Authors. All rights reserved. 1 // Copyright 2016 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_IME_IME_WINDOW_FRAME_VIEW_H_ 5 #ifndef CHROME_BROWSER_UI_VIEWS_IME_IME_WINDOW_FRAME_VIEW_H_
6 #define CHROME_BROWSER_UI_VIEWS_IME_IME_WINDOW_FRAME_VIEW_H_ 6 #define CHROME_BROWSER_UI_VIEWS_IME_IME_WINDOW_FRAME_VIEW_H_
7 7
8 #include "chrome/browser/ui/ime/ime_window.h" 8 #include "chrome/browser/ui/ime/ime_window.h"
9 #include "ui/views/controls/button/button.h" 9 #include "ui/views/controls/button/image_button.h"
10 #include "ui/views/window/non_client_view.h" 10 #include "ui/views/window/non_client_view.h"
11 11
12 namespace views { 12 namespace views {
13 class ImageButton;
14 class ImageView; 13 class ImageView;
15 } 14 }
16 15
17 namespace ui { 16 namespace ui {
18 17
19 class ImeWindowView; 18 class ImeWindowView;
20 19
21 // The non-client frame view implementation for the IME window UI. 20 // The non-client frame view implementation for the IME window UI.
22 class ImeWindowFrameView : public views::NonClientFrameView, 21 class ImeWindowFrameView : public views::NonClientFrameView,
23 public views::ButtonListener { 22 public views::ImageButtonDelegate {
24 public: 23 public:
25 // According to the UX spec, the follow-cursor window needs to have the title 24 // According to the UX spec, the follow-cursor window needs to have the title
26 // bar on the side instead of on the top (because the follow-cursor window is 25 // bar on the side instead of on the top (because the follow-cursor window is
27 // majorly used as suggestion list which can be shown in horizontal). 26 // majorly used as suggestion list which can be shown in horizontal).
28 // TODO(shuchen): locate the title bar on the right in the RTL case. 27 // TODO(shuchen): locate the title bar on the right in the RTL case.
29 ImeWindowFrameView(ImeWindowView* ime_window_view, 28 ImeWindowFrameView(ImeWindowView* ime_window_view,
30 ImeWindow::Mode mode); 29 ImeWindow::Mode mode);
31 ~ImeWindowFrameView() override; 30 ~ImeWindowFrameView() override;
32 31
33 void Init(); 32 void Init();
(...skipping 40 matching lines...) Expand 10 before | Expand all | Expand 10 after
74 ImeWindow::Mode mode_; 73 ImeWindow::Mode mode_;
75 views::ImageButton* close_button_; 74 views::ImageButton* close_button_;
76 views::ImageView* title_icon_; 75 views::ImageView* title_icon_;
77 76
78 DISALLOW_COPY_AND_ASSIGN(ImeWindowFrameView); 77 DISALLOW_COPY_AND_ASSIGN(ImeWindowFrameView);
79 }; 78 };
80 79
81 } // namespace ui 80 } // namespace ui
82 81
83 #endif // CHROME_BROWSER_UI_VIEWS_IME_IME_WINDOW_FRAME_VIEW_H_ 82 #endif // CHROME_BROWSER_UI_VIEWS_IME_IME_WINDOW_FRAME_VIEW_H_
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698