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

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

Issue 2569273002: Add constructors to WebInputEvents and setters so we can work at cleaning up these public structs. (Closed)
Patch Set: Fix mouse up event sender not modifying modifiers Created 4 years 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/fullscreen_controller_test.cc
diff --git a/chrome/browser/ui/exclusive_access/fullscreen_controller_test.cc b/chrome/browser/ui/exclusive_access/fullscreen_controller_test.cc
index c348742bd57da0de98bc55a9cc2135b344da9c22..acbf893fa8c748e521dfc2d00eb1bad98a9a69fd 100644
--- a/chrome/browser/ui/exclusive_access/fullscreen_controller_test.cc
+++ b/chrome/browser/ui/exclusive_access/fullscreen_controller_test.cc
@@ -14,6 +14,7 @@
#include "content/public/browser/native_web_keyboard_event.h"
#include "content/public/browser/web_contents.h"
#include "content/public/test/test_navigation_observer.h"
+#include "ui/events/base_event_utils.h"
#include "ui/events/keycodes/keyboard_codes.h"
using content::WebContents;
@@ -46,7 +47,9 @@ void FullscreenControllerTest::LostMouseLock() {
}
bool FullscreenControllerTest::SendEscapeToFullscreenController() {
- content::NativeWebKeyboardEvent event;
+ content::NativeWebKeyboardEvent event(blink::WebInputEvent::KeyDown,
+ blink::WebInputEvent::NoModifiers,
+ ui::EventTimeForNow());
majidvp 2016/12/19 20:09:52 nit: can this be TimeStampForTesting
dtapuska 2016/12/20 19:49:21 Done.
event.windowsKeyCode = ui::VKEY_ESCAPE;
return GetExclusiveAccessManager()->HandleUserKeyPress(event);
}

Powered by Google App Engine
This is Rietveld 408576698