Fix aura_unittests for Ozone X11.

Fix aura_unittests and view_unittests for Ozone X11.

There are assumptions in the code that PlatformWindow::SetBounds() will
change PlatformWindow bounds immediately. This isn't quite true with
X11, as the operation is async, but we can just assume SetBounds() was
successful. This is what WindowTreeHostX11 does.

The PlatformWindow will get a ConfigureNotify message with the actual
bounds later and if they don't match the requested bounds it will notify
the delegate of the new bounds.

Also, change Ozone CrOS bot to run aura_unittests and view_unittest with
Ozone X11 now that they work.

BUG=664570
Review-Url: https://codereview.chromium.org/2665193002
Cr-Commit-Position: refs/heads/master@{#447437}
Committed: https://chromium.googlesource.com/chromium/src/+/ff673af0e1005f36546e4503557aceadae0cd093

[kylechar, 2017-01-31 17:44:11]

Description was changed from

==========
Fix aura_unittests for Ozone X11.

There are assumptions in the code that PlatformWindow::SetBounds() will
change PlatformWindow bounds immediately. This isn't quite true with
X11, as the operation is async, but we can just assume SetBounds() was
successful. This is what WindowTreeHostX11 does.

The PlatformWindow will get a ConfigureNotify message with the actual
bounds later and if they don't match the requested bounds it will notify
the delegate of the new bounds.

BUG=664570
==========

to

==========
Fix aura_unittests for Ozone X11.

There are assumptions in the code that PlatformWindow::SetBounds() will
change PlatformWindow bounds immediately. This isn't quite true with
X11, as the operation is async, but we can just assume SetBounds() was
successful. This is what WindowTreeHostX11 does.

The PlatformWindow will get a ConfigureNotify message with the actual
bounds later and if they don't match the requested bounds it will notify
the delegate of the new bounds.

Also, change Ozone CrOS bot to run aura_unittests with Ozone X11 now
that they work.

BUG=664570
==========

[kylechar, 2017-01-31 17:49:17]

kylechar@chromium.org changed reviewers:
+ sadrul@chromium.org

[sadrul, 2017-01-31 17:51:42]

lgtm

[kylechar, 2017-01-31 17:59:56]

kylechar@chromium.org changed reviewers:
+ jam@chromium.org

[kylechar, 2017-01-31 17:59:56]

+jam for testing/buildbot/chromium.chromiumos.json

[kylechar, 2017-01-31 19:22:49]

Description was changed from

==========
Fix aura_unittests for Ozone X11.

There are assumptions in the code that PlatformWindow::SetBounds() will
change PlatformWindow bounds immediately. This isn't quite true with
X11, as the operation is async, but we can just assume SetBounds() was
successful. This is what WindowTreeHostX11 does.

The PlatformWindow will get a ConfigureNotify message with the actual
bounds later and if they don't match the requested bounds it will notify
the delegate of the new bounds.

Also, change Ozone CrOS bot to run aura_unittests with Ozone X11 now
that they work.

BUG=664570
==========

to

==========
Fix aura_unittests and view_unittests for Ozone X11.

There are assumptions in the code that PlatformWindow::SetBounds() will
change PlatformWindow bounds immediately. This isn't quite true with
X11, as the operation is async, but we can just assume SetBounds() was
successful. This is what WindowTreeHostX11 does.

The PlatformWindow will get a ConfigureNotify message with the actual
bounds later and if they don't match the requested bounds it will notify
the delegate of the new bounds.

Also, change Ozone CrOS bot to run aura_unittests and view_unittest with
Ozone X11 now that they work.

BUG=664570
==========

[kylechar, 2017-01-31 19:24:06]

One slight change sadrul, even if the XWindow isn't mapped aura still expects to
get notified if the bounds changed. This fixes the last of views_unittests too.

[kylechar, 2017-01-31 20:09:26]

On 2017/01/31 19:24:06, kylechar wrote:
> One slight change sadrul, even if the XWindow isn't mapped aura still expects
to
> get notified if the bounds changed. This fixes the last of views_unittests
too.

Now ash_content_unittests are failing because setup isn't done when
PlatformWindowDelegate::OnBoundsChanged() get called with the initial window
size. ScreenAsh has no Display yet and it crashes. 

What we really want is to specify an initial PlatformWindow size that doesn't
trigger PlatformWindowDelegate::OnBoundsChanged(). Any later resizes should
trigger it though. That's what Ozone headless is doing. I'll see what the best
way to do this is...

[kylechar, 2017-01-31 21:18:13]

kylechar@chromium.org changed reviewers:
- jam@chromium.org

[kylechar, 2017-01-31 21:18:13]

Okay, all fixed. Please take another look sadrul as I've had to touch a couple
more files.

-jam (broader approvals needed now)
+sky for both services/ui/* and testing/buildbot/chromium.chromiumos.json

[sadrul, 2017-01-31 21:20:36]

still lgtm

[kylechar, 2017-01-31 21:28:58]

kylechar@chromium.org changed reviewers:
+ sky@chromium.org

[kylechar, 2017-01-31 21:28:58]

+sky for testing/* and services/ui/* (for real this time)

[kylechar, 2017-01-31 21:29:28]

The CQ bit was checked by kylechar@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2017-01-31 21:30:01]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-01-31 22:29:39]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2017-01-31 22:29:40]

Dry run: Try jobs failed on following builders:
  win_chromium_rel_ng on master.tryserver.chromium.win (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.win/builders/win_chromium_rel_...)

[sky, 2017-02-01 00:32:28]

LGTM

[kylechar, 2017-02-01 02:14:56]

The CQ bit was checked by kylechar@chromium.org

[commit-bot: I haz the power, 2017-02-01 02:15:38]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-02-01 03:28:57]

CQ is committing da patch.

Bot data: {"patchset_id": 40001, "attempt_start_ts": 1485915296990920,
"parent_rev": "524226b50ce8c27dcb2d868d449c7f65aa94f50c", "commit_rev":
"ff673af0e1005f36546e4503557aceadae0cd093"}

[commit-bot: I haz the power, 2017-02-01 03:33:41]

Description was changed from

==========
Fix aura_unittests and view_unittests for Ozone X11.

There are assumptions in the code that PlatformWindow::SetBounds() will
change PlatformWindow bounds immediately. This isn't quite true with
X11, as the operation is async, but we can just assume SetBounds() was
successful. This is what WindowTreeHostX11 does.

The PlatformWindow will get a ConfigureNotify message with the actual
bounds later and if they don't match the requested bounds it will notify
the delegate of the new bounds.

Also, change Ozone CrOS bot to run aura_unittests and view_unittest with
Ozone X11 now that they work.

BUG=664570
==========

to

==========
Fix aura_unittests and view_unittests for Ozone X11.

There are assumptions in the code that PlatformWindow::SetBounds() will
change PlatformWindow bounds immediately. This isn't quite true with
X11, as the operation is async, but we can just assume SetBounds() was
successful. This is what WindowTreeHostX11 does.

The PlatformWindow will get a ConfigureNotify message with the actual
bounds later and if they don't match the requested bounds it will notify
the delegate of the new bounds.

Also, change Ozone CrOS bot to run aura_unittests and view_unittest with
Ozone X11 now that they work.

BUG=664570

Review-Url: https://codereview.chromium.org/2665193002
Cr-Commit-Position: refs/heads/master@{#447437}
Committed:
https://chromium.googlesource.com/chromium/src/+/ff673af0e1005f36546e4503557a...
==========

[commit-bot: I haz the power, 2017-02-01 03:33:42]

Committed patchset #3 (id:40001) as
https://chromium.googlesource.com/chromium/src/+/ff673af0e1005f36546e4503557a...