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

Unified Diff: third_party/WebKit/Source/core/css/resolver/StyleBuilderCustom.cpp

Issue 2399633003: reflow comments in core/css/resolver (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/css/resolver/StyleBuilderCustom.cpp
diff --git a/third_party/WebKit/Source/core/css/resolver/StyleBuilderCustom.cpp b/third_party/WebKit/Source/core/css/resolver/StyleBuilderCustom.cpp
index 65f290940c5f232da962a58f37bfcdf4f4eca1cf..d2286563aa2a67cfd756ae8c1c1dee878bde61cc 100644
--- a/third_party/WebKit/Source/core/css/resolver/StyleBuilderCustom.cpp
+++ b/third_party/WebKit/Source/core/css/resolver/StyleBuilderCustom.cpp
@@ -3,10 +3,12 @@
* Copyright (C) 1999 Lars Knoll (knoll@kde.org)
* (C) 2004-2005 Allan Sandfeld Jensen (kde@carewolf.com)
* Copyright (C) 2006, 2007 Nicholas Shanks (webkit@nickshanks.com)
- * Copyright (C) 2005, 2006, 2007, 2008, 2009, 2010, 2011, 2012, 2013 Apple Inc. All rights reserved.
+ * Copyright (C) 2005, 2006, 2007, 2008, 2009, 2010, 2011, 2012, 2013 Apple Inc.
+ * All rights reserved.
* Copyright (C) 2007 Alexey Proskuryakov <ap@webkit.org>
* Copyright (C) 2007, 2008 Eric Seidel <eric@webkit.org>
- * Copyright (C) 2008, 2009 Torch Mobile Inc. All rights reserved. (http://www.torchmobile.com/)
+ * Copyright (C) 2008, 2009 Torch Mobile Inc. All rights reserved.
+ * (http://www.torchmobile.com/)
* Copyright (c) 2011, Code Aurora Forum. All rights reserved.
* Copyright (C) Research In Motion Limited 2011. All rights reserved.
* Redistribution and use in source and binary forms, with or without
@@ -129,18 +131,16 @@ void StyleBuilder::applyProperty(CSSPropertyID id,
bool isInitial = value.isInitialValue() ||
(!state.parentNode() && value.isInheritedValue());
- DCHECK(!isInherit ||
- !isInitial); // isInherit -> !isInitial && isInitial -> !isInherit
- DCHECK(
- !isInherit ||
- (state.parentNode() &&
- state
- .parentStyle())); // isInherit -> (state.parentNode() && state.parentStyle())
+ // isInherit => !isInitial && isInitial => !isInherit
+ DCHECK(!isInherit || !isInitial);
+ // isInherit => (state.parentNode() && state.parentStyle())
+ DCHECK(!isInherit || (state.parentNode() && state.parentStyle()));
if (!state.applyPropertyToRegularStyle() &&
(!state.applyPropertyToVisitedLinkStyle() ||
!isValidVisitedLinkProperty(id))) {
- // Limit the properties that can be applied to only the ones honored by :visited.
+ // Limit the properties that can be applied to only the ones honored by
+ // :visited.
return;
}
@@ -228,7 +228,8 @@ void StyleBuilderFunctions::applyValueCSSPropertyCursor(
cursorElement->href()->currentValue()->value()))
image.clearImageResource();
- // Set the hot spot if it wasn't specified in the CSS but is specified in the SVG.
+ // Set the hot spot if it wasn't specified in the CSS but is specified
+ // in the SVG.
if (!hotSpotSpecified) {
hotSpotSpecified = true;
SVGLengthContext lengthContext(0);
@@ -497,7 +498,8 @@ void StyleBuilderFunctions::applyValueCSSPropertyTextAlign(
const CSSValue& value) {
if (value.isIdentifierValue() &&
toCSSIdentifierValue(value).getValueID() != CSSValueWebkitMatchParent) {
- // Special case for th elements - UA stylesheet text-align does not apply if parent's computed value for text-align is not its initial value
+ // Special case for th elements - UA stylesheet text-align does not apply if
+ // parent's computed value for text-align is not its initial value
// https://html.spec.whatwg.org/multipage/rendering.html#tables-2
const CSSIdentifierValue& identValue = toCSSIdentifierValue(value);
if (identValue.getValueID() == CSSValueInternalCenter &&
@@ -590,7 +592,8 @@ void StyleBuilderFunctions::applyValueCSSPropertyVerticalAlign(
}
static void resetEffectiveZoom(StyleResolverState& state) {
- // Reset the zoom in effect. This allows the setZoom method to accurately compute a new zoom in effect.
+ // Reset the zoom in effect. This allows the setZoom method to accurately
+ // compute a new zoom in effect.
state.setEffectiveZoom(state.parentStyle()
? state.parentStyle()->effectiveZoom()
: ComputedStyle::initialZoom());
@@ -770,8 +773,8 @@ void StyleBuilderFunctions::applyInitialCSSPropertyContent(
void StyleBuilderFunctions::applyInheritCSSPropertyContent(
StyleResolverState&) {
- // FIXME: In CSS3, it will be possible to inherit content. In CSS2 it is not. This
- // note is a reminder that eventually "inherit" needs to be supported.
+ // FIXME: In CSS3, it will be possible to inherit content. In CSS2 it is not.
+ // This note is a reminder that eventually "inherit" needs to be supported.
}
void StyleBuilderFunctions::applyValueCSSPropertyContent(

Powered by Google App Engine
This is Rietveld 408576698