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

Unified Diff: third_party/WebKit/Source/core/layout/LayoutScrollbar.cpp

Issue 2400863005: Reformat comments in core/layout up until LayoutTableRow (Closed)
Patch Set: Created 4 years, 2 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: third_party/WebKit/Source/core/layout/LayoutScrollbar.cpp
diff --git a/third_party/WebKit/Source/core/layout/LayoutScrollbar.cpp b/third_party/WebKit/Source/core/layout/LayoutScrollbar.cpp
index 86d44ee7a83c5b1abd29f4e9d5f074c7e08fb47c..55aea6c35361e90caed1a074e3833e5f72c7f17e 100644
--- a/third_party/WebKit/Source/core/layout/LayoutScrollbar.cpp
+++ b/third_party/WebKit/Source/core/layout/LayoutScrollbar.cpp
@@ -61,7 +61,8 @@ LayoutScrollbar::LayoutScrollbar(ScrollableArea* scrollableArea,
m_owningFrame(owningFrame) {
ASSERT(ownerNode || owningFrame);
- // FIXME: We need to do this because LayoutScrollbar::styleChanged is called as soon as the scrollbar is created.
+ // FIXME: We need to do this because LayoutScrollbar::styleChanged is called
+ // as soon as the scrollbar is created.
// Update the scrollbar size.
IntRect rect(0, 0, 0, 0);
@@ -82,9 +83,10 @@ LayoutScrollbar::~LayoutScrollbar() {
if (m_parts.isEmpty())
return;
- // When a scrollbar is detached from its parent (causing all parts removal) and
- // ready to be destroyed, its destruction can be delayed because of RefPtr
- // maintained in other classes such as EventHandler (m_lastScrollbarUnderMouse).
+ // When a scrollbar is detached from its parent (causing all parts removal)
+ // and ready to be destroyed, its destruction can be delayed because of
+ // RefPtr maintained in other classes such as EventHandler
+ // (m_lastScrollbarUnderMouse).
// Meanwhile, we can have a call to updateScrollbarPart which recreates the
// scrollbar part. So, we need to destroy these parts since we don't want them
// to call on a destroyed scrollbar. See webkit bug 68009.
@@ -167,8 +169,8 @@ PassRefPtr<ComputedStyle> LayoutScrollbar::getScrollbarPseudoStyle(
owningLayoutObject()->style());
// Scrollbars for root frames should always have background color
// unless explicitly specified as transparent. So we force it.
- // This is because WebKit assumes scrollbar to be always painted and missing background
- // causes visual artifact like non-paint invalidated dirty region.
+ // This is because WebKit assumes scrollbar to be always painted and missing
+ // background causes visual artifact like non-paint invalidated dirty region.
if (result && m_owningFrame && m_owningFrame->view() &&
!m_owningFrame->view()->isTransparent() && !result->hasBackground())
result->setBackgroundColor(StyleColor(Color::white));
@@ -190,7 +192,8 @@ void LayoutScrollbar::updateScrollbarParts(bool destroy) {
if (destroy)
return;
- // See if the scrollbar's thickness changed. If so, we need to mark our owning object as needing a layout.
+ // See if the scrollbar's thickness changed. If so, we need to mark our
+ // owning object as needing a layout.
bool isHorizontal = orientation() == HorizontalScrollbar;
int oldThickness = isHorizontal ? height() : width();
int newThickness = 0;
@@ -253,7 +256,8 @@ void LayoutScrollbar::updateScrollbarPart(ScrollbarPart partType,
!destroy && partStyle && partStyle->display() != EDisplay::None;
if (needLayoutObject && partStyle->display() != EDisplay::Block) {
- // See if we are a button that should not be visible according to OS settings.
+ // See if we are a button that should not be visible according to OS
+ // settings.
WebScrollbarButtonsPlacement buttonsPlacement = theme().buttonsPlacement();
switch (partType) {
case BackButtonStartPart:

Powered by Google App Engine
This is Rietveld 408576698