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

Unified Diff: chrome/browser/ui/exclusive_access/exclusive_access_manager.cc

Issue 2573073003: Collapse the API surface on WebInputEvent via accessor functions. (Closed)
Patch Set: Fix nits Created 3 years, 11 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 side-by-side diff with in-line comments
Download patch
Index: chrome/browser/ui/exclusive_access/exclusive_access_manager.cc
diff --git a/chrome/browser/ui/exclusive_access/exclusive_access_manager.cc b/chrome/browser/ui/exclusive_access/exclusive_access_manager.cc
index f1fee46eca51d2528d3f1d15c9e03e583cdafc99..a977e56ff2c05313be62997890dd213317bcfc30 100644
--- a/chrome/browser/ui/exclusive_access/exclusive_access_manager.cc
+++ b/chrome/browser/ui/exclusive_access/exclusive_access_manager.cc
@@ -134,14 +134,14 @@ bool ExclusiveAccessManager::HandleUserKeyPress(
}
if (IsExperimentalKeyboardLockUIEnabled()) {
- if (event.type == content::NativeWebKeyboardEvent::KeyUp &&
+ if (event.type() == content::NativeWebKeyboardEvent::KeyUp &&
hold_timer_.IsRunning()) {
// Seeing a key up event on Esc with the hold timer running cancels the
// timer and doesn't exit. This means the user pressed Esc, but not long
// enough to trigger an exit
hold_timer_.Stop();
- } else if (event.type == content::NativeWebKeyboardEvent::RawKeyDown &&
- !hold_timer_.IsRunning()) {
+ } else if (event.type() == content::NativeWebKeyboardEvent::RawKeyDown &&
+ !hold_timer_.IsRunning()) {
// Seeing a key down event on Esc when the hold timer is stopped starts
// the timer. When the timer reaches 0, the callback will trigger an exit
// from fullscreen/mouselock.
« no previous file with comments | « chrome/browser/ui/cocoa/extensions/extension_view_mac.mm ('k') | chrome/browser/ui/views/frame/browser_view.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698