Oilpan: fix build after r183460.

Oilpan: fix build after r183460.

R=haraken
BUG=420527
NOTRY=true

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

[sof, 2014-10-09 15:01:43]

Please take a look.

[haraken, 2014-10-09 15:05:35]

LGTM

https://codereview.chromium.org/642843002/diff/20001/Source/core/css/CSSGradi...
File Source/core/css/CSSGradientValue.cpp (right):

https://codereview.chromium.org/642843002/diff/20001/Source/core/css/CSSGradi...
Source/core/css/CSSGradientValue.cpp:124: struct
VectorTraits<blink::GradientStop> : SimpleClassVectorTraits<blink::GradientStop>
{

Can we use WTF_ALLOW_INIT_WITH_MEM_FUNCTIONS instead?

[sof, 2014-10-09 15:10:36]

https://codereview.chromium.org/642843002/diff/20001/Source/core/css/CSSGradi...
File Source/core/css/CSSGradientValue.cpp (right):

https://codereview.chromium.org/642843002/diff/20001/Source/core/css/CSSGradi...
Source/core/css/CSSGradientValue.cpp:105: ALLOW_ONLY_INLINE_ALLOCATION();
We could leave this out, not sure it is worth it.

The value object is used as a vector element, in a local array and as single
stack allocated value.

https://codereview.chromium.org/642843002/diff/20001/Source/core/css/CSSGradi...
Source/core/css/CSSGradientValue.cpp:124: struct
VectorTraits<blink::GradientStop> : SimpleClassVectorTraits<blink::GradientStop>
{
On 2014/10/09 15:05:35, haraken wrote:
> 
> Can we use WTF_ALLOW_INIT_WITH_MEM_FUNCTIONS instead?

The needsDestruction override is needed, unfortunately.

[sof, 2014-10-09 15:32:20]

https://codereview.chromium.org/642843002/diff/20001/Source/core/css/CSSGradi...
File Source/core/css/CSSGradientValue.cpp (right):

https://codereview.chromium.org/642843002/diff/20001/Source/core/css/CSSGradi...
Source/core/css/CSSGradientValue.cpp:105: ALLOW_ONLY_INLINE_ALLOCATION();
On 2014/10/09 15:10:36, sof wrote:
> We could leave this out, not sure it is worth it.
> 
> The value object is used as a vector element, in a local array and as single
> stack allocated value.

Landing now to heal the tree. If preferable without, will follow up.

[sof, 2014-10-09 15:32:27]

The CQ bit was checked by sigbjornf@opera.com

[commit-bot: I haz the power, 2014-10-09 15:33:43]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/642843002/20001

[commit-bot: I haz the power, 2014-10-09 15:34:44]

Committed patchset #2 (id:20001) as 183472