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

Side by Side Diff: content/common/input/web_input_event_traits.cc

Issue 2007413002: [POC; Not for Review] Don't use PostTask queueing between compositor and main thread. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Improve mouse move by making it non blocking 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 "content/common/input/web_input_event_traits.h" 5 #include "content/common/input/web_input_event_traits.h"
6 6
7 #include <bitset> 7 #include <bitset>
8 #include <limits> 8 #include <limits>
9 9
10 #include "base/logging.h" 10 #include "base/logging.h"
(...skipping 472 matching lines...) Expand 10 before | Expand all | Expand 10 after
483 void WebInputEventTraits::Coalesce(const WebInputEvent& event_to_coalesce, 483 void WebInputEventTraits::Coalesce(const WebInputEvent& event_to_coalesce,
484 WebInputEvent* event) { 484 WebInputEvent* event) {
485 DCHECK(event); 485 DCHECK(event);
486 Apply(WebInputEventCoalesce(), event->type, event_to_coalesce, event); 486 Apply(WebInputEventCoalesce(), event->type, event_to_coalesce, event);
487 } 487 }
488 488
489 bool WebInputEventTraits::ShouldBlockEventStream(const WebInputEvent& event) { 489 bool WebInputEventTraits::ShouldBlockEventStream(const WebInputEvent& event) {
490 switch (event.type) { 490 switch (event.type) {
491 case WebInputEvent::MouseDown: 491 case WebInputEvent::MouseDown:
492 case WebInputEvent::MouseUp: 492 case WebInputEvent::MouseUp:
493 case WebInputEvent::MouseMove:
tdresser 2016/05/30 14:05:32 Will we still coalesce these correctly now? Or wil
dtapuska 2016/06/01 18:22:53 Mouse moves will go to the compositor and will be
493 case WebInputEvent::MouseEnter: 494 case WebInputEvent::MouseEnter:
494 case WebInputEvent::MouseLeave: 495 case WebInputEvent::MouseLeave:
495 case WebInputEvent::ContextMenu: 496 case WebInputEvent::ContextMenu:
496 case WebInputEvent::GestureScrollBegin: 497 case WebInputEvent::GestureScrollBegin:
497 case WebInputEvent::GestureScrollEnd: 498 case WebInputEvent::GestureScrollEnd:
498 case WebInputEvent::GestureShowPress: 499 case WebInputEvent::GestureShowPress:
499 case WebInputEvent::GestureTapUnconfirmed: 500 case WebInputEvent::GestureTapUnconfirmed:
500 case WebInputEvent::GestureTapDown: 501 case WebInputEvent::GestureTapDown:
501 case WebInputEvent::GestureTapCancel: 502 case WebInputEvent::GestureTapCancel:
502 case WebInputEvent::GesturePinchBegin: 503 case WebInputEvent::GesturePinchBegin:
(...skipping 24 matching lines...) Expand all
527 528
528 uint32_t WebInputEventTraits::GetUniqueTouchEventId( 529 uint32_t WebInputEventTraits::GetUniqueTouchEventId(
529 const WebInputEvent& event) { 530 const WebInputEvent& event) {
530 if (WebInputEvent::isTouchEventType(event.type)) { 531 if (WebInputEvent::isTouchEventType(event.type)) {
531 return static_cast<const WebTouchEvent&>(event).uniqueTouchEventId; 532 return static_cast<const WebTouchEvent&>(event).uniqueTouchEventId;
532 } 533 }
533 return 0U; 534 return 0U;
534 } 535 }
535 536
536 } // namespace content 537 } // namespace content
OLDNEW
« no previous file with comments | « no previous file | content/renderer/input/input_event_filter.h » ('j') | content/renderer/input/input_event_filter.cc » ('J')

Powered by Google App Engine
This is Rietveld 408576698