Partially revert r168100

Partially revert r168100

In r168100, I mis-committed an unnecessary change to wtf/ to fix compile failure.
(The wtf/ change became unnecessary because the compile failure was fixed
in https://codereview.chromium.org/182733004/.)
This CL reverts the wtf/ part of r168100.

TBR=tkent
NOTRY=true

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

[haraken, 2014-02-28 09:17:46]

PTAL

[sof, 2014-02-28 09:43:20]

Thanks! Can confirm that this addresses the build problems.

[haraken, 2014-02-28 09:56:37]

On 2014/02/28 09:43:20, sof wrote:
> Thanks! Can confirm that this addresses the build problems.

hmm, I don't think the compile failure is not related the getPtr CLs...

> ../../third_party/WebKit/Source/modules/speech/SpeechSynthesis.cpp:228:12:
error: cannot convert 'std::nullptr_t' to 'WebCore::SpeechSynthesisUtterance*'
in return
> ../../third_party/WebKit/Source/modules/speech/SpeechSynthesis.cpp:229:1:
error: control reaches end of non-void function [-Werror=return-type]


Either way I updated the CL to fix the nullptr issue.

[sof, 2014-02-28 09:58:43]

On 2014/02/28 09:56:37, haraken wrote:
> On 2014/02/28 09:43:20, sof wrote:
> > Thanks! Can confirm that this addresses the build problems.
> 
> hmm, I don't think the compile failure is not related the getPtr CLs...

No, that failure has been reverted already in http://crrev.com/184263003

[haraken, 2014-02-28 09:59:45]

On 2014/02/28 09:58:43, sof wrote:
> On 2014/02/28 09:56:37, haraken wrote:
> > On 2014/02/28 09:43:20, sof wrote:
> > > Thanks! Can confirm that this addresses the build problems.
> > 
> > hmm, I don't think the compile failure is not related the getPtr CLs...
> 
> No, that failure has been reverted already in http://crrev.com/184263003

ah, ok. So do you think that the patch set 1 will work?

[sof, 2014-02-28 10:01:59]

On 2014/02/28 09:59:45, haraken wrote:
> On 2014/02/28 09:58:43, sof wrote:
> > On 2014/02/28 09:56:37, haraken wrote:
> > > On 2014/02/28 09:43:20, sof wrote:
> > > > Thanks! Can confirm that this addresses the build problems.
> > > 
> > > hmm, I don't think the compile failure is not related the getPtr CLs...
> > 
> > No, that failure has been reverted already in http://crrev.com/184263003
> 
> ah, ok. So do you think that the patch set 1 will work?

Yes, compiles fine with current HEAD :)

[haraken, 2014-02-28 10:03:43]

On 2014/02/28 10:01:59, sof wrote:
> On 2014/02/28 09:59:45, haraken wrote:
> > On 2014/02/28 09:58:43, sof wrote:
> > > On 2014/02/28 09:56:37, haraken wrote:
> > > > On 2014/02/28 09:43:20, sof wrote:
> > > > > Thanks! Can confirm that this addresses the build problems.
> > > > 
> > > > hmm, I don't think the compile failure is not related the getPtr CLs...
> > > 
> > > No, that failure has been reverted already in http://crrev.com/184263003
> > 
> > ah, ok. So do you think that the patch set 1 will work?
> 
> Yes, compiles fine with current HEAD :)

Thanks. PTAL for the patch set 3.

[sof, 2014-02-28 10:06:25]

Doesn't count, but enable_oilpan=1 lgtm.

[haraken, 2014-02-28 11:00:51]

Let me land this CL with TBR=tkent, to fix build failures in oilpan bots.

[haraken, 2014-02-28 11:01:18]

The CQ bit was checked by haraken@chromium.org

[commit-bot: I haz the power, 2014-02-28 11:01:30]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/haraken@chromium.org/176883010/40001

[Erik Corry, 2014-02-28 11:13:58]

LGTM

[haraken, 2014-02-28 12:14:34]

The CQ bit was unchecked by haraken@chromium.org

[haraken, 2014-02-28 12:14:45]

The CQ bit was checked by haraken@chromium.org

[commit-bot: I haz the power, 2014-02-28 12:14:55]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/haraken@chromium.org/176883010/40001

[commit-bot: I haz the power, 2014-02-28 12:15:11]

Change committed as 168121

[tkent, 2014-03-03 00:53:29]

lgtm