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

Side by Side Diff: third_party/WebKit/public/platform/WebInputEvent.h

Issue 2481943002: Fix backspace double occuring on PDF files. (Closed)
Patch Set: Created 4 years, 1 month 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
« no previous file with comments | « no previous file | no next file » | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 /* 1 /*
2 * Copyright (C) 2009 Google Inc. All rights reserved. 2 * Copyright (C) 2009 Google Inc. All rights reserved.
3 * 3 *
4 * Redistribution and use in source and binary forms, with or without 4 * Redistribution and use in source and binary forms, with or without
5 * modification, are permitted provided that the following conditions are 5 * modification, are permitted provided that the following conditions are
6 * met: 6 * met:
7 * 7 *
8 * * Redistributions of source code must retain the above copyright 8 * * Redistributions of source code must retain the above copyright
9 * notice, this list of conditions and the following disclaimer. 9 * notice, this list of conditions and the following disclaimer.
10 * * Redistributions in binary form must reproduce the above 10 * * Redistributions in binary form must reproduce the above
(...skipping 323 matching lines...) Expand 10 before | Expand all | Expand 10 after
334 isSystemKey(false), 334 isSystemKey(false),
335 isBrowserShortcut(false) { 335 isBrowserShortcut(false) {
336 memset(&text, 0, sizeof(text)); 336 memset(&text, 0, sizeof(text));
337 memset(&unmodifiedText, 0, sizeof(unmodifiedText)); 337 memset(&unmodifiedText, 0, sizeof(unmodifiedText));
338 } 338 }
339 339
340 // Please refer to bug http://b/issue?id=961192, which talks about Webkit 340 // Please refer to bug http://b/issue?id=961192, which talks about Webkit
341 // keyboard event handling changes. It also mentions the list of keys 341 // keyboard event handling changes. It also mentions the list of keys
342 // which don't have associated character events. 342 // which don't have associated character events.
343 bool isCharacterKey() const { 343 bool isCharacterKey() const {
344 #if 0 344 // TODO(dtapuska): Determine if we can remove this method and just
345 // not actually generate events for these instead of filtering them out.
345 switch (windowsKeyCode) { 346 switch (windowsKeyCode) {
346 case VKEY_BACK: 347 case 0x08: // VK_BACK
347 case VKEY_ESCAPE: 348 case 0x1b: // VK_ESCAPE
348 return false; 349 return false;
349 } 350 }
350 #endif
351 return true; 351 return true;
352 } 352 }
353 }; 353 };
354 354
355 // WebMouseEvent -------------------------------------------------------------- 355 // WebMouseEvent --------------------------------------------------------------
356 356
357 class WebMouseEvent : public WebInputEvent, public WebPointerProperties { 357 class WebMouseEvent : public WebInputEvent, public WebPointerProperties {
358 public: 358 public:
359 // Renderer coordinates. Similar to viewport coordinates but without 359 // Renderer coordinates. Similar to viewport coordinates but without
360 // DevTools emulation transform or overscroll applied. i.e. the coordinates 360 // DevTools emulation transform or overscroll applied. i.e. the coordinates
(...skipping 294 matching lines...) Expand 10 before | Expand all | Expand 10 after
655 dispatchedDuringFling(false), 655 dispatchedDuringFling(false),
656 touchStartOrFirstTouchMove(false), 656 touchStartOrFirstTouchMove(false),
657 uniqueTouchEventId(0) {} 657 uniqueTouchEventId(0) {}
658 }; 658 };
659 659
660 #pragma pack(pop) 660 #pragma pack(pop)
661 661
662 } // namespace blink 662 } // namespace blink
663 663
664 #endif 664 #endif
OLDNEW
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698