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

Side by Side 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: enumname: fixcasts 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 unified diff | Download patch
« no previous file with comments | « no previous file | cc/input/input_handler.h » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 // Copyright 2014 The Chromium Authors. All rights reserved. 1 // Copyright 2014 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 "cc/blink/web_layer_impl.h" 5 #include "cc/blink/web_layer_impl.h"
6 6
7 #include <stddef.h> 7 #include <stddef.h>
8 #include <stdint.h> 8 #include <stdint.h>
9 9
10 #include <utility> 10 #include <utility>
(...skipping 327 matching lines...) Expand 10 before | Expand all | Expand 10 after
338 } 338 }
339 339
340 void WebLayerImpl::setHaveScrollEventHandlers(bool have_scroll_event_handlers) { 340 void WebLayerImpl::setHaveScrollEventHandlers(bool have_scroll_event_handlers) {
341 layer_->SetHaveScrollEventHandlers(have_scroll_event_handlers); 341 layer_->SetHaveScrollEventHandlers(have_scroll_event_handlers);
342 } 342 }
343 343
344 bool WebLayerImpl::haveScrollEventHandlers() const { 344 bool WebLayerImpl::haveScrollEventHandlers() const {
345 return layer_->have_scroll_event_handlers(); 345 return layer_->have_scroll_event_handlers();
346 } 346 }
347 347
348 static_assert(static_cast<cc::InputHandler::MainThreadScrollingReason>( 348 static_assert(static_cast<uint32_t>(
349 blink::WebMainThreadScrollingReason::NotScrollingOnMain) == 349 blink::WebMainThreadScrollingReason::NotScrollingOnMain) ==
350 cc::InputHandler::NOT_SCROLLING_ON_MAIN, 350 cc::MainThreadScrollingReason::kNotScrollingOnMain,
351 "InputHandler::MainThreadScrollingReason and " 351 "MainThreadScrollingReason and "
352 "WebMainThreadScrollingReason enums must match");
353 static_assert(static_cast<cc::InputHandler::MainThreadScrollingReason>(
354 blink::WebMainThreadScrollingReason::
355 HasBackgroundAttachmentFixedObjects) ==
356 cc::InputHandler::HAS_BACKGROUND_ATTACHMENT_FIXED_OBJECTS,
357 "InputHandler::MainThreadScrollingReason and "
358 "WebMainThreadScrollingReason enums must match");
359 static_assert(static_cast<cc::InputHandler::MainThreadScrollingReason>(
360 blink::WebMainThreadScrollingReason::
361 HasNonLayerViewportConstrainedObjects) ==
362 cc::InputHandler::HAS_NON_LAYER_VIEWPORT_CONSTRAINED_OBJECTS,
363 "InputHandler::MainThreadScrollingReason and "
364 "WebMainThreadScrollingReason enums must match"); 352 "WebMainThreadScrollingReason enums must match");
365 static_assert( 353 static_assert(
366 static_cast<cc::InputHandler::MainThreadScrollingReason>( 354 static_cast<uint32_t>(blink::WebMainThreadScrollingReason::
355 HasBackgroundAttachmentFixedObjects) ==
356 cc::MainThreadScrollingReason::kHasBackgroundAttachmentFixedObjects,
357 "MainThreadScrollingReason and "
358 "WebMainThreadScrollingReason enums must match");
359 static_assert(
360 static_cast<uint32_t>(blink::WebMainThreadScrollingReason::
361 HasNonLayerViewportConstrainedObjects) ==
362 cc::MainThreadScrollingReason::kHasNonLayerViewportConstrainedObjects,
363 "MainThreadScrollingReason and "
364 "WebMainThreadScrollingReason enums must match");
365 static_assert(
366 static_cast<uint32_t>(
367 blink::WebMainThreadScrollingReason::ThreadedScrollingDisabled) == 367 blink::WebMainThreadScrollingReason::ThreadedScrollingDisabled) ==
368 cc::InputHandler::THREADED_SCROLLING_DISABLED, 368 cc::MainThreadScrollingReason::kThreadedScrollingDisabled,
369 "InputHandler::MainThreadScrollingReason and " 369 "MainThreadScrollingReason and "
370 "WebMainThreadScrollingReason enums must match"); 370 "WebMainThreadScrollingReason enums must match");
371 static_assert(static_cast<cc::InputHandler::MainThreadScrollingReason>( 371 static_assert(static_cast<uint32_t>(
372 blink::WebMainThreadScrollingReason::ScrollBarScrolling) == 372 blink::WebMainThreadScrollingReason::ScrollBarScrolling) ==
373 cc::InputHandler::SCROLL_BAR_SCROLLING, 373 cc::MainThreadScrollingReason::kScrollbarScrolling,
374 "InputHandler::MainThreadScrollingReason and " 374 "MainThreadScrollingReason and "
375 "WebMainThreadScrollingReason enums must match"); 375 "WebMainThreadScrollingReason enums must match");
376 static_assert(static_cast<cc::InputHandler::MainThreadScrollingReason>( 376 static_assert(
377 blink::WebMainThreadScrollingReason::PageOverlay) == 377 static_cast<uint32_t>(blink::WebMainThreadScrollingReason::PageOverlay) ==
378 cc::InputHandler::PAGE_OVERLAY, 378 cc::MainThreadScrollingReason::kPageOverlay,
379 "InputHandler::MainThreadScrollingReason and " 379 "MainThreadScrollingReason and "
380 "WebMainThreadScrollingReason enums must match"); 380 "WebMainThreadScrollingReason enums must match");
381 381
382 void WebLayerImpl::addMainThreadScrollingReasons( 382 void WebLayerImpl::addMainThreadScrollingReasons(
383 blink::WebMainThreadScrollingReason::WebMainThreadScrollingReason 383 blink::WebMainThreadScrollingReason::WebMainThreadScrollingReason
384 main_thread_scrolling_reasons) { 384 main_thread_scrolling_reasons) {
385 DCHECK(main_thread_scrolling_reasons); 385 DCHECK(main_thread_scrolling_reasons);
386 // WebLayerImpl should only know about non-transient scrolling 386 // WebLayerImpl should only know about non-transient scrolling
387 // reasons. Transient scrolling reasons are computed per hit test. 387 // reasons. Transient scrolling reasons are computed per hit test.
388 DCHECK_LE( 388 DCHECK_LE(
389 main_thread_scrolling_reasons, 389 main_thread_scrolling_reasons,
390 static_cast< 390 static_cast<
391 blink::WebMainThreadScrollingReason::WebMainThreadScrollingReason>( 391 blink::WebMainThreadScrollingReason::WebMainThreadScrollingReason>(
392 cc::InputHandler::MaxNonTransientScrollingReason)); 392 cc::MainThreadScrollingReason::kMaxNonTransientScrollingReason));
393 layer_->AddMainThreadScrollingReasons( 393 layer_->AddMainThreadScrollingReasons(main_thread_scrolling_reasons);
394 static_cast<cc::InputHandler::MainThreadScrollingReason>(
395 main_thread_scrolling_reasons));
396 } 394 }
397 395
398 void WebLayerImpl::clearMainThreadScrollingReasons() { 396 void WebLayerImpl::clearMainThreadScrollingReasons() {
399 layer_->ClearMainThreadScrollingReasons(); 397 layer_->ClearMainThreadScrollingReasons();
400 } 398 }
401 399
402 bool WebLayerImpl::shouldScrollOnMainThread() const { 400 bool WebLayerImpl::shouldScrollOnMainThread() const {
403 return layer_->should_scroll_on_main_thread(); 401 return layer_->should_scroll_on_main_thread();
404 } 402 }
405 403
(...skipping 212 matching lines...) Expand 10 before | Expand all | Expand 10 after
618 616
619 Layer* WebLayerImpl::layer() const { 617 Layer* WebLayerImpl::layer() const {
620 return layer_.get(); 618 return layer_.get();
621 } 619 }
622 620
623 void WebLayerImpl::SetContentsOpaqueIsFixed(bool fixed) { 621 void WebLayerImpl::SetContentsOpaqueIsFixed(bool fixed) {
624 contents_opaque_is_fixed_ = fixed; 622 contents_opaque_is_fixed_ = fixed;
625 } 623 }
626 624
627 } // namespace cc_blink 625 } // namespace cc_blink
OLDNEW
« no previous file with comments | « no previous file | cc/input/input_handler.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698