Disable AppList sync by default on non Chrome OS

Disable AppList sync by default on non Chrome OS

BUG=359138
For c/b/sync tests:

R=calamity@chromium.org, jennyz@chromium.org, xiyuan@chromium.org
TBR=zea@chromium.org

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

[stevenjb, 2014-04-02 18:16:03]

About to test on Windows, but please take a look when you can.

[jennyz, 2014-04-02 18:31:34]

lgtm

[stevenjb, 2014-04-02 22:33:44]

Had to fix a few tests, PTAL

[jennyz, 2014-04-02 23:15:08]

lgtm

[xiyuan, 2014-04-02 23:21:12]

lgtm

[stevenjb, 2014-04-02 23:23:52]

+calamity@

[stevenjb, 2014-04-03 00:36:02]

OK, now with better about_flags and test fixes from
https://codereview.chromium.org/212223012/ (thanks calamaty@)
PTAL

[stevenjb, 2014-04-03 00:47:41]

I realized that maintaining --disable-app-list-folder-ui is confusing and
problematic at this point so I eliminated it. Disabling just the folder UI is
potentially bugging and will definitely be confusing.
PTAL

[xiyuan, 2014-04-03 00:56:01]

On 2014/04/03 00:47:41, stevenjb wrote:
> I realized that maintaining --disable-app-list-folder-ui is confusing and
> problematic at this point so I eliminated it. Disabling just the folder UI is
> potentially bugging and will definitely be confusing.
> PTAL

Makes sense. SLGTM

[calamity, 2014-04-03 01:07:29]

lgtm.

https://codereview.chromium.org/222553003/diff/80001/chrome/app/generated_res...
File chrome/app/generated_resources.grd (right):

https://codereview.chromium.org/222553003/diff/80001/chrome/app/generated_res...
chrome/app/generated_resources.grd:6613: +        Enable App Launcher sync.
nit: This description should mention that this will also enable folders on
non-Mac.

[stevenjb, 2014-04-03 01:17:33]

https://codereview.chromium.org/222553003/diff/80001/chrome/app/generated_res...
File chrome/app/generated_resources.grd (right):

https://codereview.chromium.org/222553003/diff/80001/chrome/app/generated_res...
chrome/app/generated_resources.grd:6613: +        Enable App Launcher sync.
On 2014/04/03 01:07:30, calamity wrote:
> nit: This description should mention that this will also enable folders on
> non-Mac.

Done.

[stevenjb, 2014-04-03 06:33:10]

The CQ bit was checked by stevenjb@chromium.org

[commit-bot: I haz the power, 2014-04-03 06:33:47]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/stevenjb@chromium.org/222553003/100001

[commit-bot: I haz the power, 2014-04-03 10:12:50]

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

[commit-bot: I haz the power, 2014-04-03 10:12:51]

Commit queue rejected this change because the description was changed
between the time the change entered the commit queue and the time it
was ready to commit. You can safely check the commit box again.

[stevenjb, 2014-04-03 16:42:14]

Committed patchset #7 manually as r261445 (presubmit successful).