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

Side by Side Diff: ui/app_list/views/app_list_view.h

Issue 2952763002: SearchBoxView now enables/disables cursor based on user interaction. (Closed)
Patch Set: Addressed Comments, refactored. Created 3 years, 5 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 (c) 2012 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2012 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 UI_APP_LIST_VIEWS_APP_LIST_VIEW_H_ 5 #ifndef UI_APP_LIST_VIEWS_APP_LIST_VIEW_H_
6 #define UI_APP_LIST_VIEWS_APP_LIST_VIEW_H_ 6 #define UI_APP_LIST_VIEWS_APP_LIST_VIEW_H_
7 7
8 #include <memory> 8 #include <memory>
9 9
10 #include "base/callback.h" 10 #include "base/callback.h"
(...skipping 78 matching lines...) Expand 10 before | Expand all | Expand 10 after
89 void ShowWhenReady(); 89 void ShowWhenReady();
90 90
91 void UpdateBounds(); 91 void UpdateBounds();
92 92
93 // Enables/disables a semi-transparent overlay over the app list (good for 93 // Enables/disables a semi-transparent overlay over the app list (good for
94 // hiding the app list when a modal dialog is being shown). 94 // hiding the app list when a modal dialog is being shown).
95 void SetAppListOverlayVisible(bool visible); 95 void SetAppListOverlayVisible(bool visible);
96 96
97 views::Widget* search_box_widget() const { return search_box_widget_; } 97 views::Widget* search_box_widget() const { return search_box_widget_; }
98 98
99 SearchBoxView* search_box_view() const { return search_box_view_; }
oshima 2017/06/29 16:58:50 non const if returning non const internal object (
newcomer 2017/06/30 21:47:23 Done! I only did this because the widget's getter
100
99 // Overridden from views::View: 101 // Overridden from views::View:
100 gfx::Size CalculatePreferredSize() const override; 102 gfx::Size CalculatePreferredSize() const override;
101 void OnPaint(gfx::Canvas* canvas) override; 103 void OnPaint(gfx::Canvas* canvas) override;
102 const char* GetClassName() const override; 104 const char* GetClassName() const override;
103 105
104 // WidgetDelegate overrides: 106 // WidgetDelegate overrides:
105 bool ShouldHandleSystemCommands() const override; 107 bool ShouldHandleSystemCommands() const override;
106 bool ShouldDescendIntoChildForEventHandling( 108 bool ShouldDescendIntoChildForEventHandling(
107 gfx::NativeView child, 109 gfx::NativeView child,
108 const gfx::Point& location) override; 110 const gfx::Point& location) override;
(...skipping 112 matching lines...) Expand 10 before | Expand all | Expand 10 after
221 223
222 // For UMA and testing. If non-null, triggered when the app list is painted. 224 // For UMA and testing. If non-null, triggered when the app list is painted.
223 base::Closure next_paint_callback_; 225 base::Closure next_paint_callback_;
224 226
225 DISALLOW_COPY_AND_ASSIGN(AppListView); 227 DISALLOW_COPY_AND_ASSIGN(AppListView);
226 }; 228 };
227 229
228 } // namespace app_list 230 } // namespace app_list
229 231
230 #endif // UI_APP_LIST_VIEWS_APP_LIST_VIEW_H_ 232 #endif // UI_APP_LIST_VIEWS_APP_LIST_VIEW_H_
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698