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

Side by Side Diff: ui/events/blink/input_handler_proxy.cc

Issue 2773593005: Move logic of recording main thread scrolling reasons from cc to blink::ScrollManager (Closed)
Patch Set: Bug fix Created 3 years, 9 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 "ui/events/blink/input_handler_proxy.h" 5 #include "ui/events/blink/input_handler_proxy.h"
6 6
7 #include <stddef.h> 7 #include <stddef.h>
8 8
9 #include <algorithm> 9 #include <algorithm>
10 10
(...skipping 534 matching lines...) Expand 10 before | Expand all | Expand 10 after
545 "Renderer4.MainThreadWheelScrollReason"; 545 "Renderer4.MainThreadWheelScrollReason";
546 546
547 DCHECK(device == blink::WebGestureDeviceTouchpad || 547 DCHECK(device == blink::WebGestureDeviceTouchpad ||
548 device == blink::WebGestureDeviceTouchscreen); 548 device == blink::WebGestureDeviceTouchscreen);
549 549
550 if (device != blink::WebGestureDeviceTouchpad && 550 if (device != blink::WebGestureDeviceTouchpad &&
551 device != blink::WebGestureDeviceTouchscreen) { 551 device != blink::WebGestureDeviceTouchscreen) {
552 return; 552 return;
553 } 553 }
554 554
555 // NonCompositedScrollReasons should only be set on the main thread.
556 DCHECK(!cc::MainThreadScrollingReason::NonCompositedScrollReasons(reasons));
557
555 // UMA_HISTOGRAM_ENUMERATION requires that the enum_max must be strictly 558 // UMA_HISTOGRAM_ENUMERATION requires that the enum_max must be strictly
556 // greater than the sample value. kMainThreadScrollingReasonCount doesn't 559 // greater than the sample value. kMainThreadScrollingReasonCount doesn't
557 // include the NotScrollingOnMain enum but the histograms do so adding 560 // include the NotScrollingOnMain enum but the histograms do so adding
558 // the +1 is necessary. 561 // the +1 is necessary.
559 uint32_t mainThreadScrollingReasonEnumMax = 562 uint32_t mainThreadScrollingReasonEnumMax =
560 cc::MainThreadScrollingReason::kMainThreadScrollingReasonCount + 1; 563 cc::MainThreadScrollingReason::kMainThreadScrollingReasonCount + 1;
561 if (reasons == cc::MainThreadScrollingReason::kNotScrollingOnMain) { 564 if (reasons == cc::MainThreadScrollingReason::kNotScrollingOnMain) {
562 if (device == blink::WebGestureDeviceTouchscreen) { 565 if (device == blink::WebGestureDeviceTouchscreen) {
563 UMA_HISTOGRAM_ENUMERATION( 566 UMA_HISTOGRAM_ENUMERATION(
564 kGestureHistogramName, 567 kGestureHistogramName,
(...skipping 1051 matching lines...) Expand 10 before | Expand all | Expand 10 after
1616 scroll_elasticity_controller_->GetWeakPtr(), gesture_event, 1619 scroll_elasticity_controller_->GetWeakPtr(), gesture_event,
1617 scroll_result)); 1620 scroll_result));
1618 } 1621 }
1619 1622
1620 void InputHandlerProxy::SetTickClockForTesting( 1623 void InputHandlerProxy::SetTickClockForTesting(
1621 std::unique_ptr<base::TickClock> tick_clock) { 1624 std::unique_ptr<base::TickClock> tick_clock) {
1622 tick_clock_ = std::move(tick_clock); 1625 tick_clock_ = std::move(tick_clock);
1623 } 1626 }
1624 1627
1625 } // namespace ui 1628 } // namespace ui
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698