Set CaptureClient instance for aura_demo and aura_bench

Set CaptureClient instance for aura_demo and aura_bench


BUG=138400
TEST=None


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

[Hongbo Min, 2012-07-21 14:15:16]

Set the CaptureClient property for root window, otherwise, the
client::GetCaptureClient returns null and cause null reference. 

Pls have a review. Thanks.

[Ben Goodger (Google), 2012-07-23 17:19:49]

lgtm

http://codereview.chromium.org/10808071/diff/3001/ui/aura/demo/demo_main.cc
File ui/aura/demo/demo_main.cc (right):

http://codereview.chromium.org/10808071/diff/3001/ui/aura/demo/demo_main.cc#n...
ui/aura/demo/demo_main.cc:103: new
aura::shared::RootWindowCaptureClient(root_window_));
4-space indent

[commit-bot: I haz the power, 2012-07-24 03:15:13]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/hongbo.min@intel.com/10808071/2004

[Hongbo Min, 2012-07-24 03:15:13]

On 2012/07/23 17:19:49, Ben Goodger (Google) wrote:
> lgtm
> 
> http://codereview.chromium.org/10808071/diff/3001/ui/aura/demo/demo_main.cc
> File ui/aura/demo/demo_main.cc (right):
> 
>
http://codereview.chromium.org/10808071/diff/3001/ui/aura/demo/demo_main.cc#n...
> ui/aura/demo/demo_main.cc:103: new
> aura::shared::RootWindowCaptureClient(root_window_));
> 4-space indent

Thanks a lot.

[commit-bot: I haz the power, 2012-07-24 04:28:57]

Try job failure for 10808071-2004 (retry) on linux_rel for step "check_deps".
It's a second try, previously, step "check_deps" failed.
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=linux_rel&...

[Hongbo Min, 2012-07-24 05:12:50]

On 2012/07/24 04:28:57, I haz the power (commit-bot) wrote:
> Try job failure for 10808071-2004 (retry) on linux_rel for step "check_deps".
> It's a second try, previously, step "check_deps" failed.
>
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=linux_rel&...

Failed to check deps due to the ui/aura/shared is excluded for aura.

Do we need to add DEPS files for both ui/aura/bench and ui/aura/demo directory
or implement a CaptureClient interface for them?

[commit-bot: I haz the power, 2012-07-24 05:49:04]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/hongbo.min@intel.com/10808071/3002

[commit-bot: I haz the power, 2012-07-24 07:13:50]

Try job failure for 10808071-3002 (retry) on mac_rel for step "browser_tests".
It's a second try, previously, step "browser_tests" failed.
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=mac_rel&nu...

[Hongbo Min, 2012-07-24 07:25:07]

On 2012/07/24 07:13:50, I haz the power (commit-bot) wrote:
> Try job failure for 10808071-3002 (retry) on mac_rel for step "browser_tests".
> It's a second try, previously, step "browser_tests" failed.
>
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=mac_rel&nu...

Two failures in browser_tests:
1. ExtensionApiTest.Rlz
2. SavePageBrowserTest.SaveHTMLOnly

But this patch doesn't touch them at all but blocked to commit by them.

Ben, How to deal with such a case? Thanks.

[commit-bot: I haz the power, 2012-07-24 09:53:13]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/hongbo.min@intel.com/10808071/3002

[commit-bot: I haz the power, 2012-07-24 11:23:23]

Change committed as 148083