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

Issue 15864004: Move create() methods from StylePropertySet to MutableStylePropertySet. (Closed)

Created:
7 years, 7 months ago by darktears
Modified:
7 years, 7 months ago
CC:
blink-reviews, caseq+blink_chromium.org, loislo+blink_chromium.org, eustas+blink_chromium.org, alph+blink_chromium.org, eae+blinkwatch, yurys+blink_chromium.org, lushnikov+blink_chromium.org, vsevik+blink_chromium.org, pfeldman+blink_chromium.org, dglazkov+blink, Rik, apavlov+blink_chromium.org, adamk+blink_chromium.org, pdr, f(malita), Stephen Chennney, aandrey+blink_chromium.org
Visibility:
Public.

Description

Move create() methods from StylePropertySet to MutableStylePropertySet. Move the two create() methods that create MutableStylePropertySets from StylePropertySet to MutableStylePropertySet, and make them actually return MutableStylePropertySets. Switch to using {Pass,}RefPtr<MutableStylePropertySet> where appropriate to keep things building. This is a backport from WebKit r148395. Patch by Andreas Kling and reviewed by Antti Koivisto. R=apavlov@chromium.org Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=151009

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+52 lines, -50 lines) Patch
M Source/core/css/CSSComputedStyleDeclaration.cpp View 1 chunk +1 line, -1 line 0 comments Download
M Source/core/css/CSSParser.cpp View 1 chunk +1 line, -1 line 0 comments Download
M Source/core/css/DOMWindowCSS.cpp View 1 chunk +1 line, -1 line 0 comments Download
M Source/core/css/FontLoader.cpp View 1 chunk +1 line, -1 line 0 comments Download
M Source/core/css/StylePropertySet.h View 2 chunks +8 lines, -5 lines 0 comments Download
M Source/core/css/StylePropertySet.cpp View 2 chunks +3 lines, -3 lines 0 comments Download
M Source/core/css/WebKitCSSMatrix.cpp View 1 chunk +1 line, -1 line 0 comments Download
M Source/core/css/resolver/StyleResolver.cpp View 3 chunks +3 lines, -3 lines 0 comments Download
M Source/core/dom/Attr.cpp View 1 chunk +1 line, -1 line 0 comments Download
M Source/core/dom/Document.cpp View 1 chunk +1 line, -1 line 0 comments Download
M Source/core/dom/StyledElement.cpp View 2 chunks +2 lines, -2 lines 0 comments Download
M Source/core/editing/ApplyStyleCommand.cpp View 1 chunk +2 lines, -2 lines 0 comments Download
M Source/core/editing/EditingStyle.cpp View 8 chunks +9 lines, -9 lines 0 comments Download
M Source/core/editing/Editor.cpp View 1 chunk +1 line, -1 line 0 comments Download
M Source/core/editing/EditorCommand.cpp View 6 chunks +7 lines, -7 lines 0 comments Download
M Source/core/html/HTMLTableElement.cpp View 3 chunks +3 lines, -3 lines 0 comments Download
M Source/core/html/canvas/CanvasRenderingContext2D.cpp View 1 chunk +1 line, -1 line 0 comments Download
M Source/core/inspector/InspectorStyleSheet.cpp View 3 chunks +3 lines, -3 lines 0 comments Download
M Source/core/page/DragController.cpp View 1 chunk +0 lines, -1 line 0 comments Download
M Source/core/svg/SVGElementRareData.h View 1 chunk +1 line, -1 line 0 comments Download
M Source/core/svg/SVGFontFaceElement.cpp View 1 chunk +1 line, -1 line 0 comments Download
M Source/core/svg/SVGStyledElement.cpp View 1 chunk +1 line, -1 line 0 comments Download

Messages

Total messages: 9 (0 generated)
darktears
7 years, 7 months ago (2013-05-23 12:28:31 UTC) #1
apavlov
Awesome. LGTM
7 years, 7 months ago (2013-05-23 12:42:30 UTC) #2
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/alexis.menard@intel.com/15864004/1
7 years, 7 months ago (2013-05-23 12:59:24 UTC) #3
commit-bot: I haz the power
Retried try job too often on linux_layout_rel for step(s) webkit_tests http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=linux_layout_rel&number=9587
7 years, 7 months ago (2013-05-23 13:57:53 UTC) #4
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/alexis.menard@intel.com/15864004/1
7 years, 7 months ago (2013-05-23 14:06:06 UTC) #5
commit-bot: I haz the power
Retried try job too often on linux_layout_rel for step(s) webkit_tests http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=linux_layout_rel&number=9605
7 years, 7 months ago (2013-05-23 14:52:51 UTC) #6
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/alexis.menard@intel.com/15864004/1
7 years, 7 months ago (2013-05-23 15:14:20 UTC) #7
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/alexis.menard@intel.com/15864004/1
7 years, 7 months ago (2013-05-23 16:52:52 UTC) #8
darktears
7 years, 7 months ago (2013-05-23 18:06:53 UTC) #9
Message was sent while issue was closed.
Committed patchset #1 manually as r151009 (presubmit successful).

Powered by Google App Engine
This is Rietveld 408576698