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

Side by Side Diff: ash/shelf/shelf_tooltip_manager.cc

Issue 1957653003: mash: Don't close system tray bubble for clicks in menus/notifications (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: . 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 #include "ash/shelf/shelf_tooltip_manager.h" 5 #include "ash/shelf/shelf_tooltip_manager.h"
6 6
7 #include "ash/shelf/shelf.h" 7 #include "ash/shelf/shelf.h"
8 #include "ash/shelf/shelf_layout_manager.h" 8 #include "ash/shelf/shelf_layout_manager.h"
9 #include "ash/shelf/shelf_view.h" 9 #include "ash/shelf/shelf_view.h"
10 #include "ash/shell.h" 10 #include "ash/shell.h"
(...skipping 155 matching lines...) Expand 10 before | Expand all | Expand 10 after
166 } 166 }
167 167
168 void ShelfTooltipManager::ShowTooltipWithDelay(views::View* view) { 168 void ShelfTooltipManager::ShowTooltipWithDelay(views::View* view) {
169 if (!shelf_layout_manager_ || shelf_layout_manager_->IsVisible()) { 169 if (!shelf_layout_manager_ || shelf_layout_manager_->IsVisible()) {
170 timer_.Start(FROM_HERE, base::TimeDelta::FromMilliseconds(timer_delay_), 170 timer_.Start(FROM_HERE, base::TimeDelta::FromMilliseconds(timer_delay_),
171 base::Bind(&ShelfTooltipManager::ShowTooltip, 171 base::Bind(&ShelfTooltipManager::ShowTooltip,
172 weak_factory_.GetWeakPtr(), view)); 172 weak_factory_.GetWeakPtr(), view));
173 } 173 }
174 } 174 }
175 175
176 void ShelfTooltipManager::OnMousePressed(const ui::MouseEvent& event, 176 void ShelfTooltipManager::OnMousePressed(const gfx::Point& location_in_screen,
177 const gfx::Point& location_in_screen) { 177 views::Widget* target) {
178 // Close on any mouse press events inside or outside the tooltip. 178 // Close on any mouse press events inside or outside the tooltip.
179 Close(); 179 Close();
180 } 180 }
181 181
182 void ShelfTooltipManager::OnTouchPressed(const ui::TouchEvent& event, 182 void ShelfTooltipManager::OnTouchPressed(const gfx::Point& location_in_screen,
183 const gfx::Point& location_in_screen) { 183 views::Widget* target) {
184 // Close on any touch press events inside or outside the tooltip. 184 // Close on any touch press events inside or outside the tooltip.
185 Close(); 185 Close();
186 } 186 }
187 187
188 void ShelfTooltipManager::OnEvent(ui::Event* event) { 188 void ShelfTooltipManager::OnEvent(ui::Event* event) {
189 // Mouse and touch press events are handled via views::PointerWatcher. 189 // Mouse and touch press events are handled via views::PointerWatcher.
190 if (event->type() == ui::ET_MOUSE_PRESSED || 190 if (event->type() == ui::ET_MOUSE_PRESSED ||
191 event->type() == ui::ET_TOUCH_PRESSED) { 191 event->type() == ui::ET_TOUCH_PRESSED) {
192 return; 192 return;
193 } 193 }
(...skipping 49 matching lines...) Expand 10 before | Expand all | Expand 10 after
243 // AutoHide state change happens during an event filter, so immediate close 243 // AutoHide state change happens during an event filter, so immediate close
244 // may cause a crash in the HandleMouseEvent() after the filter. So we just 244 // may cause a crash in the HandleMouseEvent() after the filter. So we just
245 // schedule the Close here. 245 // schedule the Close here.
246 base::ThreadTaskRunnerHandle::Get()->PostTask( 246 base::ThreadTaskRunnerHandle::Get()->PostTask(
247 FROM_HERE, 247 FROM_HERE,
248 base::Bind(&ShelfTooltipManager::Close, weak_factory_.GetWeakPtr())); 248 base::Bind(&ShelfTooltipManager::Close, weak_factory_.GetWeakPtr()));
249 } 249 }
250 } 250 }
251 251
252 } // namespace ash 252 } // namespace ash
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698