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

Side by Side Diff: chrome/browser/ui/views/location_bar/chrome_to_mobile_view.cc

Issue 10832282: Replace views::MouseEvent with ui::MouseEvent (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src/
Patch Set: Created 8 years, 4 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 | Annotate | Revision Log
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 #include "chrome/browser/ui/views/location_bar/chrome_to_mobile_view.h" 5 #include "chrome/browser/ui/views/location_bar/chrome_to_mobile_view.h"
6 6
7 #include "base/utf_string_conversions.h" 7 #include "base/utf_string_conversions.h"
8 #include "chrome/app/chrome_command_ids.h" 8 #include "chrome/app/chrome_command_ids.h"
9 #include "chrome/browser/command_updater.h" 9 #include "chrome/browser/command_updater.h"
10 #include "chrome/browser/ui/view_ids.h" 10 #include "chrome/browser/ui/view_ids.h"
(...skipping 45 matching lines...) Expand 10 before | Expand all | Expand 10 after
56 56
57 bool ChromeToMobileView::GetTooltipText(const gfx::Point& p, 57 bool ChromeToMobileView::GetTooltipText(const gfx::Point& p,
58 string16* tooltip) const { 58 string16* tooltip) const {
59 // Don't show tooltip to distract user if ChromeToMobileBubbleView is showing. 59 // Don't show tooltip to distract user if ChromeToMobileBubbleView is showing.
60 if (chrome::IsChromeToMobileBubbleViewShowing()) 60 if (chrome::IsChromeToMobileBubbleViewShowing())
61 return false; 61 return false;
62 62
63 return views::ImageView::GetTooltipText(p, tooltip); 63 return views::ImageView::GetTooltipText(p, tooltip);
64 } 64 }
65 65
66 bool ChromeToMobileView::OnMousePressed(const views::MouseEvent& event) { 66 bool ChromeToMobileView::OnMousePressed(const ui::MouseEvent& event) {
67 // Show the bubble on mouse release; that is standard button behavior. 67 // Show the bubble on mouse release; that is standard button behavior.
68 return true; 68 return true;
69 } 69 }
70 70
71 void ChromeToMobileView::OnMouseReleased(const views::MouseEvent& event) { 71 void ChromeToMobileView::OnMouseReleased(const ui::MouseEvent& event) {
72 if (event.IsOnlyLeftMouseButton() && HitTestPoint(event.location())) 72 if (event.IsOnlyLeftMouseButton() && HitTestPoint(event.location()))
73 command_updater_->ExecuteCommand(IDC_CHROME_TO_MOBILE_PAGE); 73 command_updater_->ExecuteCommand(IDC_CHROME_TO_MOBILE_PAGE);
74 } 74 }
75 75
76 bool ChromeToMobileView::OnKeyPressed(const ui::KeyEvent& event) { 76 bool ChromeToMobileView::OnKeyPressed(const ui::KeyEvent& event) {
77 if (event.key_code() == ui::VKEY_SPACE || 77 if (event.key_code() == ui::VKEY_SPACE ||
78 event.key_code() == ui::VKEY_RETURN) { 78 event.key_code() == ui::VKEY_RETURN) {
79 command_updater_->ExecuteCommand(IDC_CHROME_TO_MOBILE_PAGE); 79 command_updater_->ExecuteCommand(IDC_CHROME_TO_MOBILE_PAGE);
80 return true; 80 return true;
81 } 81 }
82 return false; 82 return false;
83 } 83 }
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698