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

Issue 24430002: Rename attach and detach to createRenderTree/destroyRenderTree (Closed)

Created:
7 years, 3 months ago by esprehn
Modified:
4 years, 10 months ago
CC:
adamk+blink_chromium.org, blink-reviews, chromiumbugtracker_adobe.com, dglazkov+blink, eae+blinkwatch, feature-media-reviews_chromium.org, f(malita), pdr, Stephen Chennney, nessy, vcarbune.chromium, webcomponents-bugzilla_chromium.org
Visibility:
Public.

Description

Rename attach and detach to createRenderTree/destroyRenderTree This patch renames attach and detach to something more sensible. It also renames lazyAttach to scheduleRenderTreeCreation. It leaves the attached() bit since I'm not sure of a good name for that yet.

Patch Set 1 #

Total comments: 3
Unified diffs Side-by-side diffs Delta from patch set Stats (+228 lines, -236 lines) Patch
M Source/core/dom/ContainerNode.h View 2 chunks +2 lines, -23 lines 0 comments Download
M Source/core/dom/ContainerNode.cpp View 3 chunks +21 lines, -8 lines 0 comments Download
M Source/core/dom/ContainerNodeAlgorithms.h View 1 chunk +5 lines, -5 lines 0 comments Download
M Source/core/dom/Document.h View 1 chunk +2 lines, -2 lines 0 comments Download
M Source/core/dom/Document.cpp View 4 chunks +5 lines, -5 lines 0 comments Download
M Source/core/dom/Element.h View 1 chunk +2 lines, -2 lines 0 comments Download
M Source/core/dom/Element.cpp View 8 chunks +13 lines, -13 lines 0 comments Download
M Source/core/dom/ElementRareData.h View 1 chunk +1 line, -1 line 0 comments Download
M Source/core/dom/Node.h View 5 chunks +18 lines, -18 lines 2 comments Download
M Source/core/dom/Node.cpp View 4 chunks +15 lines, -15 lines 0 comments Download
M Source/core/dom/NodeRenderingTraversal.cpp View 1 chunk +3 lines, -3 lines 0 comments Download
M Source/core/dom/PseudoElement.h View 1 chunk +1 line, -1 line 0 comments Download
M Source/core/dom/PseudoElement.cpp View 1 chunk +2 lines, -2 lines 0 comments Download
M Source/core/dom/Text.h View 1 chunk +1 line, -1 line 0 comments Download
M Source/core/dom/Text.cpp View 3 chunks +4 lines, -4 lines 0 comments Download
M Source/core/dom/shadow/ElementShadow.h View 1 chunk +2 lines, -2 lines 0 comments Download
M Source/core/dom/shadow/ElementShadow.cpp View 4 chunks +7 lines, -7 lines 0 comments Download
M Source/core/dom/shadow/InsertionPoint.h View 1 chunk +2 lines, -2 lines 0 comments Download
M Source/core/dom/shadow/InsertionPoint.cpp View 2 chunks +13 lines, -13 lines 0 comments Download
M Source/core/dom/shadow/ShadowRoot.h View 1 chunk +1 line, -1 line 0 comments Download
M Source/core/dom/shadow/ShadowRoot.cpp View 3 chunks +4 lines, -4 lines 0 comments Download
M Source/core/html/HTMLDetailsElement.cpp View 1 chunk +1 line, -1 line 0 comments Download
M Source/core/html/HTMLFormControlElement.h View 1 chunk +1 line, -1 line 0 comments Download
M Source/core/html/HTMLFormControlElement.cpp View 1 chunk +2 lines, -2 lines 0 comments Download
M Source/core/html/HTMLFrameElement.h View 1 chunk +1 line, -1 line 0 comments Download
M Source/core/html/HTMLFrameElement.cpp View 1 chunk +2 lines, -2 lines 0 comments Download
M Source/core/html/HTMLFrameElementBase.h View 1 chunk +1 line, -1 line 0 comments Download
M Source/core/html/HTMLFrameElementBase.cpp View 1 chunk +2 lines, -2 lines 0 comments Download
M Source/core/html/HTMLFrameSetElement.h View 1 chunk +1 line, -1 line 0 comments Download
M Source/core/html/HTMLFrameSetElement.cpp View 2 chunks +2 lines, -2 lines 0 comments Download
M Source/core/html/HTMLImageElement.h View 1 chunk +1 line, -1 line 0 comments Download
M Source/core/html/HTMLImageElement.cpp View 1 chunk +2 lines, -2 lines 0 comments Download
M Source/core/html/HTMLInputElement.h View 2 chunks +2 lines, -2 lines 0 comments Download
M Source/core/html/HTMLInputElement.cpp View 6 chunks +8 lines, -8 lines 0 comments Download
M Source/core/html/HTMLLIElement.h View 1 chunk +1 line, -1 line 0 comments Download
M Source/core/html/HTMLLIElement.cpp View 1 chunk +2 lines, -2 lines 0 comments Download
M Source/core/html/HTMLMediaElement.h View 1 chunk +1 line, -1 line 0 comments Download
M Source/core/html/HTMLMediaElement.cpp View 1 chunk +2 lines, -2 lines 0 comments Download
M Source/core/html/HTMLObjectElement.cpp View 1 chunk +2 lines, -2 lines 1 comment Download
M Source/core/html/HTMLOptGroupElement.h View 1 chunk +2 lines, -2 lines 0 comments Download
M Source/core/html/HTMLOptGroupElement.cpp View 2 chunks +4 lines, -4 lines 0 comments Download
M Source/core/html/HTMLOptionElement.h View 1 chunk +2 lines, -2 lines 0 comments Download
M Source/core/html/HTMLOptionElement.cpp View 2 chunks +4 lines, -4 lines 0 comments Download
M Source/core/html/HTMLPlugInElement.h View 1 chunk +1 line, -1 line 0 comments Download
M Source/core/html/HTMLPlugInElement.cpp View 2 chunks +2 lines, -2 lines 0 comments Download
M Source/core/html/HTMLPlugInImageElement.h View 1 chunk +2 lines, -2 lines 0 comments Download
M Source/core/html/HTMLPlugInImageElement.cpp View 3 chunks +5 lines, -5 lines 0 comments Download
M Source/core/html/HTMLProgressElement.h View 1 chunk +1 line, -1 line 0 comments Download
M Source/core/html/HTMLProgressElement.cpp View 1 chunk +2 lines, -2 lines 0 comments Download
M Source/core/html/HTMLSelectElement.cpp View 2 chunks +2 lines, -2 lines 0 comments Download
M Source/core/html/HTMLVideoElement.h View 1 chunk +1 line, -1 line 0 comments Download
M Source/core/html/HTMLVideoElement.cpp View 1 chunk +2 lines, -2 lines 0 comments Download
M Source/core/html/HTMLViewSourceDocument.cpp View 2 chunks +2 lines, -2 lines 0 comments Download
M Source/core/html/PluginDocument.h View 1 chunk +1 line, -1 line 0 comments Download
M Source/core/html/PluginDocument.cpp View 1 chunk +2 lines, -2 lines 0 comments Download
M Source/core/html/shadow/ClearButtonElement.h View 1 chunk +1 line, -1 line 0 comments Download
M Source/core/html/shadow/ClearButtonElement.cpp View 1 chunk +2 lines, -2 lines 0 comments Download
M Source/core/html/shadow/PasswordGeneratorButtonElement.h View 1 chunk +1 line, -1 line 0 comments Download
M Source/core/html/shadow/PasswordGeneratorButtonElement.cpp View 1 chunk +2 lines, -2 lines 0 comments Download
M Source/core/html/shadow/PickerIndicatorElement.h View 1 chunk +1 line, -1 line 0 comments Download
M Source/core/html/shadow/PickerIndicatorElement.cpp View 1 chunk +2 lines, -2 lines 0 comments Download
M Source/core/html/shadow/SliderThumbElement.h View 1 chunk +1 line, -1 line 0 comments Download
M Source/core/html/shadow/SliderThumbElement.cpp View 1 chunk +2 lines, -2 lines 0 comments Download
M Source/core/html/shadow/SpinButtonElement.h View 1 chunk +1 line, -1 line 0 comments Download
M Source/core/html/shadow/SpinButtonElement.cpp View 1 chunk +2 lines, -2 lines 0 comments Download
M Source/core/html/shadow/TextControlInnerElements.h View 3 chunks +3 lines, -3 lines 0 comments Download
M Source/core/html/shadow/TextControlInnerElements.cpp View 3 chunks +6 lines, -6 lines 0 comments Download
M Source/core/page/DOMWindow.cpp View 2 chunks +2 lines, -2 lines 0 comments Download
M Source/core/page/Frame.cpp View 1 chunk +1 line, -1 line 0 comments Download
M Source/core/svg/SVGImageElement.h View 1 chunk +1 line, -1 line 0 comments Download
M Source/core/svg/SVGImageElement.cpp View 1 chunk +2 lines, -2 lines 0 comments Download
M Source/core/svg/SVGTests.cpp View 1 chunk +2 lines, -2 lines 0 comments Download
M Source/core/svg/SVGUseElement.h View 1 chunk +1 line, -1 line 0 comments Download
M Source/core/svg/SVGUseElement.cpp View 1 chunk +2 lines, -2 lines 0 comments Download

Messages

Total messages: 7 (0 generated)
esprehn
Do you think attachRenderTree would be better?
7 years, 3 months ago (2013-09-24 21:17:41 UTC) #1
leviw_travelin_and_unemployed
On 2013/09/24 21:17:41, esprehn wrote: > Do you think attachRenderTree would be better? I prefer ...
7 years, 3 months ago (2013-09-24 21:23:00 UTC) #2
ojan
lgtm Please make the naming consistent (s/reconstruct/recreate). https://codereview.chromium.org/24430002/diff/1/Source/core/dom/Node.h File Source/core/dom/Node.h (right): https://codereview.chromium.org/24430002/diff/1/Source/core/dom/Node.h#newcode411 Source/core/dom/Node.h:411: void scheduleRenderTreeReconstruction(); ...
7 years, 3 months ago (2013-09-24 22:41:24 UTC) #3
esprehn
On 2013/09/24 22:41:24, ojan wrote: > lgtm > > Please make the naming consistent (s/reconstruct/recreate). ...
7 years, 3 months ago (2013-09-24 22:45:27 UTC) #4
eseidel
Did you give up? :)
6 years, 10 months ago (2014-02-14 03:33:43 UTC) #5
esprehn
On 2014/02/14 03:33:43, eseidel wrote: > Did you give up? :) Yeah so many merge ...
6 years, 10 months ago (2014-02-15 01:22:57 UTC) #6
esprehn
4 years, 10 months ago (2016-02-10 00:12:40 UTC) #7
We should rename these things someday...

Powered by Google App Engine
This is Rietveld 408576698