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

Unified Diff: cc/blink/web_layer_impl.cc

Issue 1604053002: cc: Fix enum style conformance in InputHandler. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 4 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
« no previous file with comments | « no previous file | cc/input/input_handler.h » ('j') | cc/input/main_thread_scrolling_reason.h » ('J')
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: cc/blink/web_layer_impl.cc
diff --git a/cc/blink/web_layer_impl.cc b/cc/blink/web_layer_impl.cc
index 56c2af9c51513b1f35850bfb71213a7e0d1fe0d2..16a441f2360ad63bb7e4f6ae31d59d94c61af50d 100644
--- a/cc/blink/web_layer_impl.cc
+++ b/cc/blink/web_layer_impl.cc
@@ -345,38 +345,34 @@ bool WebLayerImpl::haveScrollEventHandlers() const {
return layer_->have_scroll_event_handlers();
}
-static_assert(static_cast<cc::InputHandler::MainThreadScrollingReason>(
- blink::WebMainThreadScrollingReason::NotScrollingOnMain) ==
- cc::InputHandler::NOT_SCROLLING_ON_MAIN,
- "InputHandler::MainThreadScrollingReason and "
- "WebMainThreadScrollingReason enums must match");
-static_assert(static_cast<cc::InputHandler::MainThreadScrollingReason>(
- blink::WebMainThreadScrollingReason::
- HasBackgroundAttachmentFixedObjects) ==
- cc::InputHandler::HAS_BACKGROUND_ATTACHMENT_FIXED_OBJECTS,
- "InputHandler::MainThreadScrollingReason and "
- "WebMainThreadScrollingReason enums must match");
-static_assert(static_cast<cc::InputHandler::MainThreadScrollingReason>(
- blink::WebMainThreadScrollingReason::
- HasNonLayerViewportConstrainedObjects) ==
- cc::InputHandler::HAS_NON_LAYER_VIEWPORT_CONSTRAINED_OBJECTS,
- "InputHandler::MainThreadScrollingReason and "
+static_assert(blink::WebMainThreadScrollingReason::NotScrollingOnMain ==
+ cc::MainThreadScrollingReason::kNotScrollingOnMain,
+ "MainThreadScrollingReason and "
"WebMainThreadScrollingReason enums must match");
static_assert(
- static_cast<cc::InputHandler::MainThreadScrollingReason>(
- blink::WebMainThreadScrollingReason::ThreadedScrollingDisabled) ==
- cc::InputHandler::THREADED_SCROLLING_DISABLED,
- "InputHandler::MainThreadScrollingReason and "
+ blink::WebMainThreadScrollingReason::HasBackgroundAttachmentFixedObjects ==
+ cc::MainThreadScrollingReason::kHasBackgroundAttachmentFixedObjects,
+ "MainThreadScrollingReason and "
+ "WebMainThreadScrollingReason enums must match");
+static_assert(
+ blink::WebMainThreadScrollingReason::
+ HasNonLayerViewportConstrainedObjects ==
+ cc::MainThreadScrollingReason::kHasNonLayerViewportConstrainedObjects,
+ "MainThreadScrollingReason and "
+ "WebMainThreadScrollingReason enums must match");
+static_assert(
+
+ blink::WebMainThreadScrollingReason::ThreadedScrollingDisabled ==
+ cc::MainThreadScrollingReason::kThreadedScrollingDisabled,
+ "MainThreadScrollingReason and "
"WebMainThreadScrollingReason enums must match");
-static_assert(static_cast<cc::InputHandler::MainThreadScrollingReason>(
- blink::WebMainThreadScrollingReason::ScrollBarScrolling) ==
- cc::InputHandler::SCROLL_BAR_SCROLLING,
- "InputHandler::MainThreadScrollingReason and "
+static_assert(blink::WebMainThreadScrollingReason::ScrollBarScrolling ==
+ cc::MainThreadScrollingReason::kScrollbarScrolling,
+ "MainThreadScrollingReason and "
"WebMainThreadScrollingReason enums must match");
-static_assert(static_cast<cc::InputHandler::MainThreadScrollingReason>(
- blink::WebMainThreadScrollingReason::PageOverlay) ==
- cc::InputHandler::PAGE_OVERLAY,
- "InputHandler::MainThreadScrollingReason and "
+static_assert(blink::WebMainThreadScrollingReason::PageOverlay ==
+ cc::MainThreadScrollingReason::kPageOverlay,
+ "MainThreadScrollingReason and "
"WebMainThreadScrollingReason enums must match");
void WebLayerImpl::addMainThreadScrollingReasons(
@@ -389,10 +385,8 @@ void WebLayerImpl::addMainThreadScrollingReasons(
main_thread_scrolling_reasons,
static_cast<
blink::WebMainThreadScrollingReason::WebMainThreadScrollingReason>(
- cc::InputHandler::MaxNonTransientScrollingReason));
- layer_->AddMainThreadScrollingReasons(
- static_cast<cc::InputHandler::MainThreadScrollingReason>(
- main_thread_scrolling_reasons));
+ cc::MainThreadScrollingReason::kMaxNonTransientScrollingReason));
+ layer_->AddMainThreadScrollingReasons(main_thread_scrolling_reasons);
}
void WebLayerImpl::clearMainThreadScrollingReasons() {
« no previous file with comments | « no previous file | cc/input/input_handler.h » ('j') | cc/input/main_thread_scrolling_reason.h » ('J')

Powered by Google App Engine
This is Rietveld 408576698