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

Issues Search
    My Issues | Starred     Open | Closed | All

Issue 541383003: Clean up forward declarations in Source/core/css (Closed)

Created:
6 years, 3 months ago by rwlbuis
Modified:
6 years, 3 months ago
CC:
blink-reviews, kenneth.christiansen, Yoav Weiss, krit, blink-reviews-css, kouhei+svg_chromium.org, ed+blinkwatch_opera.com, fs, dglazkov+blink, apavlov+blink_chromium.org, gyuyoung.kim_webkit.org, darktears, f(malita), Stephen Chennney, rune+blink, pdr., rwlbuis
Base URL:
https://chromium.googlesource.com/chromium/blink.git@master
Project:
blink
Visibility:
Public.

Description

Clean up forward declarations in Source/core/css These forward references are not needed anymore. Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=181505

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+5 lines, -72 lines) Patch
M Source/core/css/CSSBorderImageSliceValue.h View 1 chunk +0 lines, -2 lines 0 comments Download
M Source/core/css/CSSCalculationValue.h View 1 chunk +0 lines, -3 lines 0 comments Download
M Source/core/css/CSSCharsetRule.h View 1 chunk +0 lines, -2 lines 0 comments Download
M Source/core/css/CSSComputedStyleDeclaration.h View 1 chunk +0 lines, -1 line 0 comments Download
M Source/core/css/CSSCrossfadeValue.h View 1 chunk +0 lines, -1 line 0 comments Download
M Source/core/css/CSSFontFace.h View 1 chunk +0 lines, -3 lines 0 comments Download
M Source/core/css/CSSFontSelector.h View 1 chunk +0 lines, -4 lines 0 comments Download
M Source/core/css/CSSImageGeneratorValue.h View 1 chunk +0 lines, -1 line 0 comments Download
M Source/core/css/CSSImageValue.h View 1 chunk +0 lines, -1 line 0 comments Download
M Source/core/css/CSSImportRule.h View 1 chunk +0 lines, -2 lines 0 comments Download
M Source/core/css/CSSLineBoxContainValue.h View 1 chunk +0 lines, -2 lines 0 comments Download
M Source/core/css/CSSPropertySourceData.h View 1 chunk +0 lines, -2 lines 0 comments Download
M Source/core/css/CSSStyleDeclaration.h View 1 chunk +0 lines, -1 line 0 comments Download
M Source/core/css/CSSStyleSheet.h View 1 chunk +0 lines, -4 lines 0 comments Download
M Source/core/css/CSSSupportsRule.h View 1 chunk +0 lines, -1 line 0 comments Download
M Source/core/css/CSSValue.h View 1 chunk +0 lines, -1 line 0 comments Download
M Source/core/css/ElementRuleCollector.h View 1 chunk +0 lines, -1 line 0 comments Download
M Source/core/css/FontFace.h View 1 chunk +0 lines, -1 line 0 comments Download
M Source/core/css/MediaQueryEvaluator.h View 1 chunk +0 lines, -1 line 0 comments Download
M Source/core/css/MediaQueryList.h View 1 chunk +0 lines, -2 lines 0 comments Download
M Source/core/css/MediaQueryListListener.h View 1 chunk +0 lines, -2 lines 0 comments Download
M Source/core/css/PageRuleCollector.h View 1 chunk +0 lines, -1 line 0 comments Download
M Source/core/css/PropertySetCSSStyleDeclaration.h View 1 chunk +0 lines, -1 line 0 comments Download
M Source/core/css/RuleFeature.h View 1 chunk +0 lines, -3 lines 0 comments Download
M Source/core/css/StylePropertySerializer.h View 1 chunk +1 line, -0 lines 0 comments Download
M Source/core/css/StylePropertySet.h View 1 chunk +0 lines, -4 lines 0 comments Download
M Source/core/css/StyleRule.h View 1 chunk +0 lines, -1 line 0 comments Download
M Source/core/css/parser/BisonCSSParser.h View 2 chunks +0 lines, -7 lines 0 comments Download
M Source/core/css/parser/CSSParserValues.h View 1 chunk +0 lines, -1 line 0 comments Download
M Source/core/css/parser/CSSPropertyParser.h View 1 chunk +0 lines, -2 lines 0 comments Download
M Source/core/css/resolver/SharedStyleFinder.h View 1 chunk +0 lines, -1 line 0 comments Download
M Source/core/css/resolver/StyleBuilder.h View 1 chunk +0 lines, -2 lines 0 comments Download
M Source/core/css/resolver/StyleResolver.h View 2 chunks +0 lines, -8 lines 0 comments Download
M Source/core/css/resolver/StyleResolverState.h View 1 chunk +0 lines, -1 line 0 comments Download
M Source/core/svg/SVGDocumentExtensions.h View 1 chunk +0 lines, -1 line 0 comments Download
M Source/core/svg/SVGEnumeration.h View 1 chunk +2 lines, -0 lines 0 comments Download
M Source/core/svg/SVGString.h View 1 chunk +2 lines, -0 lines 0 comments Download
M Source/core/svg/properties/SVGProperty.h View 1 chunk +0 lines, -1 line 0 comments Download

Messages

Total messages: 15 (7 generated)
rwlbuis
PTAL
6 years, 3 months ago (2014-09-05 22:33:17 UTC) #2
kouhei (in TOK)
lgtm
6 years, 3 months ago (2014-09-05 22:51:22 UTC) #4
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/rob.buis@samsung.com/541383003/1
6 years, 3 months ago (2014-09-05 22:54:48 UTC) #6
commit-bot: I haz the power
Exceeded time limit waiting for builds to trigger.
6 years, 3 months ago (2014-09-06 04:55:49 UTC) #8
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/rob.buis@samsung.com/541383003/1
6 years, 3 months ago (2014-09-06 10:22:24 UTC) #10
commit-bot: I haz the power
Exceeded time limit waiting for builds to trigger.
6 years, 3 months ago (2014-09-06 12:24:01 UTC) #12
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/rob.buis@samsung.com/541383003/1
6 years, 3 months ago (2014-09-06 19:08:43 UTC) #14
commit-bot: I haz the power
6 years, 3 months ago (2014-09-07 08:15:54 UTC) #15
Message was sent while issue was closed. 
Committed patchset #1 (id:1) as 181505

Powered by Google App Engine
This is Rietveld 408576698