Oilpan: Replace RefPtrs to Node and its subclasses in core/html with Oilpan transitin types.

Oilpan: Replace RefPtrs to Node and its subclasses in core/html with Oilpan transitin types.

This CL also declares VTTTreeBuilder as STACK_ALLOCATED and makes m_currentNode
and m_document traceable.

BUG=357163
Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=175467

[tkent, 2014-06-04 07:04:25]

Please review this.

[haraken, 2014-06-04 07:06:16]

LGTM

https://codereview.chromium.org/315843004/diff/1/Source/core/html/HTMLTableEl...
File Source/core/html/HTMLTableElement.cpp (right):

https://codereview.chromium.org/315843004/diff/1/Source/core/html/HTMLTableEl...
Source/core/html/HTMLTableElement.cpp:217: RefPtrWillBeRawPtr<ContainerNode>
parent;

= nullptr

https://codereview.chromium.org/315843004/diff/1/Source/core/html/track/vtt/V...
File Source/core/html/track/vtt/VTTCue.cpp (right):

https://codereview.chromium.org/315843004/diff/1/Source/core/html/track/vtt/V...
Source/core/html/track/vtt/VTTCue.cpp:427: RefPtrWillBeRawPtr<Node> clonedNode;

= nullptr

https://codereview.chromium.org/315843004/diff/1/Source/core/html/track/vtt/V...
File Source/core/html/track/vtt/VTTElement.cpp (right):

https://codereview.chromium.org/315843004/diff/1/Source/core/html/track/vtt/V...
Source/core/html/track/vtt/VTTElement.cpp:88: RefPtrWillBeRawPtr<HTMLElement>
htmlElement;

= nullptr

https://codereview.chromium.org/315843004/diff/1/Source/core/html/track/vtt/V...
File Source/core/html/track/vtt/VTTParser.cpp (right):

https://codereview.chromium.org/315843004/diff/1/Source/core/html/track/vtt/V...
Source/core/html/track/vtt/VTTParser.cpp:322: VTTTreeBuilder(Document& document)

Add explicit.

[Mads Ager (chromium), 2014-06-04 07:07:30]

lgtm2

[zerny-chromium, 2014-06-04 07:41:50]

lgtm

[tkent, 2014-06-04 08:03:25]

https://codereview.chromium.org/315843004/diff/1/Source/core/html/HTMLTableEl...
File Source/core/html/HTMLTableElement.cpp (right):

https://codereview.chromium.org/315843004/diff/1/Source/core/html/HTMLTableEl...
Source/core/html/HTMLTableElement.cpp:217: RefPtrWillBeRawPtr<ContainerNode>
parent;
On 2014/06/04 07:06:16, haraken wrote:
> 
> = nullptr

I don't think it's necessary.  It's very clear that |parent| is always
initialized.

https://codereview.chromium.org/315843004/diff/1/Source/core/html/track/vtt/V...
File Source/core/html/track/vtt/VTTCue.cpp (right):

https://codereview.chromium.org/315843004/diff/1/Source/core/html/track/vtt/V...
Source/core/html/track/vtt/VTTCue.cpp:427: RefPtrWillBeRawPtr<Node> clonedNode;
On 2014/06/04 07:06:16, haraken wrote:
> 
> = nullptr

Ditto.

https://codereview.chromium.org/315843004/diff/1/Source/core/html/track/vtt/V...
File Source/core/html/track/vtt/VTTElement.cpp (right):

https://codereview.chromium.org/315843004/diff/1/Source/core/html/track/vtt/V...
Source/core/html/track/vtt/VTTElement.cpp:88: RefPtrWillBeRawPtr<HTMLElement>
htmlElement;
On 2014/06/04 07:06:16, haraken wrote:
> 
> = nullptr

Done.

https://codereview.chromium.org/315843004/diff/1/Source/core/html/track/vtt/V...
File Source/core/html/track/vtt/VTTParser.cpp (right):

https://codereview.chromium.org/315843004/diff/1/Source/core/html/track/vtt/V...
Source/core/html/track/vtt/VTTParser.cpp:322: VTTTreeBuilder(Document& document)
On 2014/06/04 07:06:16, haraken wrote:
> 
> Add explicit.

Done though it's unrelated to the subject of this CL.

[tkent, 2014-06-04 08:03:28]

The CQ bit was checked by tkent@chromium.org

[commit-bot: I haz the power, 2014-06-04 08:04:21]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/tkent@chromium.org/315843004/20001

[commit-bot: I haz the power, 2014-06-04 09:37:09]

FYI, CQ is re-trying this CL (attempt #1).
The failing builders are:
  linux_blink_rel on tryserver.blink
(http://build.chromium.org/p/tryserver.blink/builders/linux_blink_rel/builds/1...)

[commit-bot: I haz the power, 2014-06-04 10:03:11]

Change committed as 175467