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

Unified Diff: cc/input/main_thread_scrolling_reason.h

Issue 2638013002: Record box shadow as main thread scrolling reasons (Closed)
Patch Set: Record box shadow from where it gets detected && remove translucent border 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: cc/input/main_thread_scrolling_reason.h
diff --git a/cc/input/main_thread_scrolling_reason.h b/cc/input/main_thread_scrolling_reason.h
index 899aad868407c165368e50b0303279291b9918e6..f0c5f7a8a7fae6c17939b800270a27af80e95c1e 100644
--- a/cc/input/main_thread_scrolling_reason.h
+++ b/cc/input/main_thread_scrolling_reason.h
@@ -41,6 +41,7 @@ struct MainThreadScrollingReason {
kBackgroundNotOpaqueInRectAndLCDText = 1 << 18,
kHasBorderRadius = 1 << 19,
kHasClipRelatedProperty = 1 << 20,
+ kHasBoxShadowAndLCDText = 1 << 21,
// Transient scrolling reasons. These are computed for each scroll begin.
kNonFastScrollableRegion = 1 << 5,
@@ -55,7 +56,7 @@ struct MainThreadScrollingReason {
// New flags should increment this number but it should never be decremented
// because the values are used in UMA histograms. It should also be noted
// that it excludes the kNotScrollingOnMain value.
- kMainThreadScrollingReasonCount = 21,
+ kMainThreadScrollingReasonCount = 22,
};
// Returns true if the given MainThreadScrollingReason can be set by the main
@@ -67,7 +68,7 @@ struct MainThreadScrollingReason {
kScrollbarScrolling | kPageOverlay | kHandlingScrollFromMainThread |
kCustomScrollbarScrolling | kHasOpacityAndLCDText |
kHasTransformAndLCDText | kBackgroundNotOpaqueInRectAndLCDText |
- kHasBorderRadius | kHasClipRelatedProperty;
+ kHasBorderRadius | kHasClipRelatedProperty | kHasBoxShadowAndLCDText;
return (reasons & reasons_set_by_main_thread) == reasons;
}
@@ -125,6 +126,8 @@ struct MainThreadScrollingReason {
tracedValue->AppendString("Has border radius");
if (reasons & MainThreadScrollingReason::kHasClipRelatedProperty)
tracedValue->AppendString("Has clip related property");
+ if (reasons & MainThreadScrollingReason::kHasBoxShadowAndLCDText)
+ tracedValue->AppendString("Has box shadow and LCD text");
// Transient scrolling reasons.
if (reasons & MainThreadScrollingReason::kNonFastScrollableRegion)

Powered by Google App Engine
This is Rietveld 408576698