Fix TabRestoreServiceTest and TranslateManagerTest when using use_webkit_compositor=1

Fix TabRestoreServiceTest and TranslateManagerTest when using use_webkit_compositor=1

BUG=106157
TEST=browser_tests

Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=113288

[oshima, 2011-12-06 18:40:51]

http://codereview.chromium.org/8790012/diff/2001/chrome/browser/translate/tra...
File chrome/browser/translate/translate_manager_browsertest.cc (right):

http://codereview.chromium.org/8790012/diff/2001/chrome/browser/translate/tra...
chrome/browser/translate/translate_manager_browsertest.cc:41: #include
"third_party/WebKit/Source/WebKit/chromium/public/WebKit.h"
move new includes into "if defined(USE_WEBKIT_COMPOSITOR)

[oshima, 2011-12-06 18:41:19]

and can you check linux_aura bot failure?

On 2011/12/06 18:40:51, oshima wrote:
>
http://codereview.chromium.org/8790012/diff/2001/chrome/browser/translate/tra...
> File chrome/browser/translate/translate_manager_browsertest.cc (right):
> 
>
http://codereview.chromium.org/8790012/diff/2001/chrome/browser/translate/tra...
> chrome/browser/translate/translate_manager_browsertest.cc:41: #include
> "third_party/WebKit/Source/WebKit/chromium/public/WebKit.h"
> move new includes into "if defined(USE_WEBKIT_COMPOSITOR)

[piman, 2011-12-06 18:44:33]

On Tue, Dec 6, 2011 at 10:41 AM, <oshima@chromium.org> wrote:

> and can you check linux_aura bot failure?


It's unrelated.... I will send another try job before submitting.

Antoine


>
>
> On 2011/12/06 18:40:51, oshima wrote:
>
> http://codereview.chromium.**org/8790012/diff/2001/chrome/**
>
browser/translate/translate_**manager_browsertest.cc<http://codereview.chromium.org/8790012/diff/2001/chrome/browser/translate/translate_manager_browsertest.cc>
>
>> File chrome/browser/translate/**translate_manager_browsertest.**cc
>> (right):
>>
>
>
> http://codereview.chromium.**org/8790012/diff/2001/chrome/**
>
browser/translate/translate_**manager_browsertest.cc#**newcode41<http://codereview.chromium.org/8790012/diff/2001/chrome/browser/translate/translate_manager_browsertest.cc#newcode41>
>
>> chrome/browser/translate/**translate_manager_browsertest.**cc:41:
>> #include
>> "third_party/WebKit/Source/**WebKit/chromium/public/WebKit.**h"
>> move new includes into "if defined(USE_WEBKIT_COMPOSITOR)
>>
>
>
>
>
http://codereview.chromium.**org/8790012/<http://codereview.chromium.org/8790...
>

[piman, 2011-12-06 19:30:49]

http://codereview.chromium.org/8790012/diff/2001/chrome/browser/translate/tra...
File chrome/browser/translate/translate_manager_browsertest.cc (right):

http://codereview.chromium.org/8790012/diff/2001/chrome/browser/translate/tra...
chrome/browser/translate/translate_manager_browsertest.cc:41: #include
"third_party/WebKit/Source/WebKit/chromium/public/WebKit.h"
On 2011/12/06 18:40:51, oshima wrote:
> move new includes into "if defined(USE_WEBKIT_COMPOSITOR)

Done.

[oshima, 2011-12-06 19:31:02]

LGTM

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

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/piman@chromium.org/8790012/8001

[commit-bot: I haz the power, 2011-12-06 21:52:19]

Try job failure for 8790012-8001 (retry) on linux_clang for step "compile"
(clobber build).
It's a second try, previously, step "compile" failed.
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=linux_clan...

[commit-bot: I haz the power, 2011-12-06 21:58:41]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/piman@chromium.org/8790012/8001

[commit-bot: I haz the power, 2011-12-06 23:41:29]

Change committed as 113288