mac: Fix flickering of install extension window.

mac: Fix flickering of install extension window.

Changing properties of an NSWindow can synchronously affect the display. Hide
the window before it is added to the NSWindow hierarchy. Then animate it in when
it is already in the correct location.

BUG=595239
Committed: https://crrev.com/fce13c0eee832c68383fed90c9e01ee22fd893ad
Cr-Commit-Position: refs/heads/master@{#385425}

[erikchen, 2016-04-06 01:14:50]

Description was changed from

==========
mac: Fix flickering of install extension window.

The frame of ConstrainedWindowCustomSheet should be updated before adding it as
a sub window, as the latter operation can be synchronous.

BUG=595239
==========

to

==========
mac: Fix flickering of install extension window.

Changing properties of an NSWindow can synchronously affect the display. Hide
the window before it is added to the NSWindow hierarchy. Then animate it in when
it is already in the correct location.

BUG=595239
==========

[erikchen, 2016-04-06 01:15:27]

erikchen@chromium.org changed reviewers:
+ tapted@chromium.org

[erikchen, 2016-04-06 01:15:28]

tapted: Please review.

[erikchen, 2016-04-06 01:16:55]

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

[commit-bot: I haz the power, 2016-04-06 01:17:26]

Dry run: CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1862793003/20001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1862793003/20001

[tapted, 2016-04-06 01:20:29]

lgtm

[erikchen, 2016-04-06 01:20:50]

The CQ bit was unchecked by erikchen@chromium.org

[erikchen, 2016-04-06 01:20:54]

The CQ bit was checked by erikchen@chromium.org

[commit-bot: I haz the power, 2016-04-06 01:21:03]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1862793003/20001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1862793003/20001

[commit-bot: I haz the power, 2016-04-06 07:19:38]

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

[commit-bot: I haz the power, 2016-04-06 07:19:39]

Try jobs failed on following builders:
  linux_chromium_rel_ng on tryserver.chromium.linux (JOB_TIMED_OUT,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)

[tapted, 2016-04-06 09:06:44]

The CQ bit was checked by tapted@chromium.org

[commit-bot: I haz the power, 2016-04-06 09:06:59]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1862793003/20001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1862793003/20001

[commit-bot: I haz the power, 2016-04-06 09:51:54]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-04-06 09:53:04]

Description was changed from

==========
mac: Fix flickering of install extension window.

Changing properties of an NSWindow can synchronously affect the display. Hide
the window before it is added to the NSWindow hierarchy. Then animate it in when
it is already in the correct location.

BUG=595239
==========

to

==========
mac: Fix flickering of install extension window.

Changing properties of an NSWindow can synchronously affect the display. Hide
the window before it is added to the NSWindow hierarchy. Then animate it in when
it is already in the correct location.

BUG=595239

Committed: https://crrev.com/fce13c0eee832c68383fed90c9e01ee22fd893ad
Cr-Commit-Position: refs/heads/master@{#385425}
==========

[commit-bot: I haz the power, 2016-04-06 09:53:05]

Patchset 2 (id:??) landed as
https://crrev.com/fce13c0eee832c68383fed90c9e01ee22fd893ad
Cr-Commit-Position: refs/heads/master@{#385425}