[MD Settings][People] Fixes the 'arrow icon' appearing momentarily on page refresh.

[MD Settings][People] Fixes the 'arrow icon' appearing momentarily on page refresh.

BUG=657338
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation

Committed: https://crrev.com/177bf7169b7de6de71d449a4592fe7032b6dd260
Cr-Commit-Position: refs/heads/master@{#428153}

[Moe, 2016-10-27 03:48:46]

Description was changed from

==========
[MD Settings][People] Fixes the 'arrow icon' appearing momentarily on page
refresh.

BUG=657338
==========

to

==========
[MD Settings][People] Fixes the 'arrow icon' appearing momentarily on page
refresh.

BUG=657338
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation
==========

[Moe, 2016-10-27 03:49:14]

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

[commit-bot: I haz the power, 2016-10-27 03:49:29]

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

[Moe, 2016-10-27 03:54:56]

mahmadi@chromium.org changed reviewers:
+ tommycli@chromium.org

[Moe, 2016-10-27 03:54:57]

Hi Tommy, Please review this CL.

It seems like while syncStatus is undefined, the bindings for the dom-if
templates enclosing the 'disconnect button' and the 'signin button' are skipped
allowing the middle section (containing the arrow icon) to expand until the
syncStatus is returned and one of the buttons is displayed.
hiding the entire row until syncStatus is returned solves the problem. Although
technically the row still flickers as a whole.

[commit-bot: I haz the power, 2016-10-27 05:13:58]

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

[commit-bot: I haz the power, 2016-10-27 05:13:59]

Dry run: Try jobs failed on following builders:
  android_n5x_swarming_rel on master.tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/android_n5x_...)

[Moe, 2016-10-27 17:21:36]

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

[commit-bot: I haz the power, 2016-10-27 17:22:06]

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, 2016-10-27 18:05:53]

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

[commit-bot: I haz the power, 2016-10-27 18:05:54]

Dry run: Try jobs failed on following builders:
  linux_chromium_chromeos_rel_ng on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)

[Moe, 2016-10-27 20:29:03]

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

[Moe, 2016-10-27 20:29:16]

Patchset #1 (id:1) has been deleted

[commit-bot: I haz the power, 2016-10-27 20:29:39]

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

[Moe, 2016-10-27 20:29:44]

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

[commit-bot: I haz the power, 2016-10-27 20:30:17]

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

[tommycli, 2016-10-27 21:10:36]

lgtm seems like an improvement, even if it's not perfect...

Since the sync status is fetched asynchronously, it may never be perfect...

[commit-bot: I haz the power, 2016-10-27 21:46:08]

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

[commit-bot: I haz the power, 2016-10-27 21:46:09]

Dry run: This issue passed the CQ dry run.

[Moe, 2016-10-27 21:59:25]

The CQ bit was checked by mahmadi@chromium.org

[Moe, 2016-10-27 21:59:34]

On 2016/10/27 21:10:36, tommycli wrote:
> lgtm seems like an improvement, even if it's not perfect...
> 
> Since the sync status is fetched asynchronously, it may never be perfect...

thanks!

[commit-bot: I haz the power, 2016-10-27 22:00:07]

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, 2016-10-27 22:08:14]

Committed patchset #1 (id:20001)

[commit-bot: I haz the power, 2016-10-27 22:11:12]

Description was changed from

==========
[MD Settings][People] Fixes the 'arrow icon' appearing momentarily on page
refresh.

BUG=657338
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation
==========

to

==========
[MD Settings][People] Fixes the 'arrow icon' appearing momentarily on page
refresh.

BUG=657338
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation

Committed: https://crrev.com/177bf7169b7de6de71d449a4592fe7032b6dd260
Cr-Commit-Position: refs/heads/master@{#428153}
==========

[commit-bot: I haz the power, 2016-10-27 22:11:13]

Patchset 1 (id:??) landed as
https://crrev.com/177bf7169b7de6de71d449a4592fe7032b6dd260
Cr-Commit-Position: refs/heads/master@{#428153}