ozone: Disable tests that create multiple RootWindowHosts

ozone: Disable tests that create multiple RootWindowHosts

BUG=319986
Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=235670

[kalyank, 2013-11-15 20:41:25]

https://codereview.chromium.org/70233017/diff/1/ui/aura/root_window_unittest.cc
File ui/aura/root_window_unittest.cc (right):

https://codereview.chromium.org/70233017/diff/1/ui/aura/root_window_unittest....
ui/aura/root_window_unittest.cc:1184: #if defined(USE_OZONE)
Shouldn't we do this only when views support is disabled or in "single window"
mode and not for Ozone in general ?

[spang, 2013-11-15 21:09:52]

On 2013/11/15 20:41:25, kalyank wrote:
>
https://codereview.chromium.org/70233017/diff/1/ui/aura/root_window_unittest.cc
> File ui/aura/root_window_unittest.cc (right):
> 
>
https://codereview.chromium.org/70233017/diff/1/ui/aura/root_window_unittest....
> ui/aura/root_window_unittest.cc:1184: #if defined(USE_OZONE)
> Shouldn't we do this only when views support is disabled or in "single window"
> mode and not for Ozone in general ?

This is inside aura; views is not relevant here.

[sadrul, 2013-11-15 22:36:36]

lgtm

https://codereview.chromium.org/70233017/diff/1/ui/aura/root_window_unittest.cc
File ui/aura/root_window_unittest.cc (right):

https://codereview.chromium.org/70233017/diff/1/ui/aura/root_window_unittest....
ui/aura/root_window_unittest.cc:1185: #define MAYBE_ValidRootDuringDestruction
DISABLED_ValidRootDuringDestruction
You should add a comment here that ozone currently doesn't support multiple root
windows correctly (and file a bug to fix that and reference that from here).

[spang, 2013-11-15 22:48:02]

https://codereview.chromium.org/70233017/diff/1/ui/aura/root_window_unittest.cc
File ui/aura/root_window_unittest.cc (right):

https://codereview.chromium.org/70233017/diff/1/ui/aura/root_window_unittest....
ui/aura/root_window_unittest.cc:1185: #define MAYBE_ValidRootDuringDestruction
DISABLED_ValidRootDuringDestruction
On 2013/11/15 22:36:37, sadrul wrote:
> You should add a comment here that ozone currently doesn't support multiple
root
> windows correctly (and file a bug to fix that and reference that from here).

Done.

[commit-bot: I haz the power, 2013-11-15 22:56:42]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/spang@chromium.org/70233017/70001

[commit-bot: I haz the power, 2013-11-16 00:56:53]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/spang@chromium.org/70233017/70001

[commit-bot: I haz the power, 2013-11-18 08:41:44]

Change committed as 235670