Discard unused AppListPrefs code.

Discard unused AppListPrefs code.

This CL discard unused code that monitors changes in app list model
but does not expose any functionality.

BUG=641535
Test=git cl try

Committed: https://crrev.com/b8997b03c95df27570d0beab45cc42fe1df95c20
Cr-Commit-Position: refs/heads/master@{#426283}

[khmel, 2016-10-18 16:46:51]

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

[commit-bot: I haz the power, 2016-10-18 16:47:19]

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-18 17:14:10]

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

[commit-bot: I haz the power, 2016-10-18 17:14:11]

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_...)

[khmel, 2016-10-18 17:23:49]

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

[commit-bot: I haz the power, 2016-10-18 17:24:13]

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-18 18:48:11]

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

[commit-bot: I haz the power, 2016-10-18 18:48:12]

Dry run: This issue passed the CQ dry run.

[khmel, 2016-10-18 19:10:21]

khmel@chromium.org changed reviewers:
+ calamity@chromium.org, stevenjb@chromium.org

[khmel, 2016-10-18 19:10:22]

Hi Steven and Christopher,

PTAL

Please feel free to ban this CL if you think this is incorrect.

[stevenjb, 2016-10-18 20:20:29]

stevenjb@chromium.org changed reviewers:
+ benwells@chromium.org

[stevenjb, 2016-10-18 20:20:30]

lgtm if there are no objections from calamity@.

+benwells@ FYI since I'm not sure what Chris is working on. 

Ben, this is code that was written to store the app list locally but never
implemented. khmel@ is working on a slightly different approach
(https://codereview.chromium.org/2416133002/) so I suggested that we purge the
unimplemented code first.

[benwells, 2016-10-18 23:21:37]

On 2016/10/18 20:20:30, stevenjb wrote:
> lgtm if there are no objections from calamity@.
> 
> +benwells@ FYI since I'm not sure what Chris is working on. 
> 
> Ben, this is code that was written to store the app list locally but never
> implemented. khmel@ is working on a slightly different approach
> (https://codereview.chromium.org/2416133002/) so I suggested that we purge the
> unimplemented code first.

lgtm

Chris is travelling at the Polymer summit and probably hasn't seen this yet.

[calamity, 2016-10-19 04:52:20]

lgtm

[khmel, 2016-10-19 14:41:13]

The CQ bit was checked by khmel@chromium.org

[commit-bot: I haz the power, 2016-10-19 14:41: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, 2016-10-19 14:52:01]

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

[commit-bot: I haz the power, 2016-10-19 14:52:02]

Try jobs failed on following builders:
  chromium_presubmit on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/chromium_presub...)

[khmel, 2016-10-19 15:55:25]

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

[khmel, 2016-10-19 15:55:26]

Thank you all for review!

Hi sky@

Could you please review: 
c/b/prefs/browser_prefs.cc
c/b/sync/test/integration/two_client_app_list_sync_test.cc

PTAL

[sky, 2016-10-19 19:28:50]

LGTM

[khmel, 2016-10-19 19:34:18]

The CQ bit was checked by khmel@chromium.org

[commit-bot: I haz the power, 2016-10-19 19:35:03]

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-19 21:10:59]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-10-21 13:11:25]

Description was changed from

==========
Discard unused AppListPrefs code.

This CL discard unused code that monitors changes in app list model
but does not expose any functionality.

BUG=641535
Test=git cl try
==========

to

==========
Discard unused AppListPrefs code.

This CL discard unused code that monitors changes in app list model
but does not expose any functionality.

BUG=641535
Test=git cl try

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

[commit-bot: I haz the power, 2016-10-21 13:11:26]

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