Remove PerAppSettingsService

Remove PerAppSettingsService

Was only required for different |HostDesktopType|s which are being
removed.

R=benwells@chromium.org
BUG=558054
Committed: https://crrev.com/9017a43db41052ac7f4104ff8bbd216fc35849a7
Cr-Commit-Position: refs/heads/master@{#374789}

[scottmg, 2016-02-09 21:12:41]

Description was changed from

==========
Remove PerAppSettingsService

Was only required for different |HostDesktopType|s which are being
removed.

BUG=558054
==========

to

==========
Remove PerAppSettingsService

Was only required for different |HostDesktopType|s which are being
removed.

R=benwells@chromium.org
BUG=558054
==========

[scottmg, 2016-02-09 21:12:41]

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

[scottmg, 2016-02-10 19:24:29]

scottmg@chromium.org changed reviewers:
+ asargent@chromium.org
- benwells@chromium.org

[scottmg, 2016-02-10 19:24:30]

->asargent for timezone-y-ness

[asargent_no_longer_on_chrome, 2016-02-10 22:06:22]

lgtm, however I'm not in chrome/browser/ui/views/apps/OWNERS so you still want
to either wait for benwells/tapted or perhaps TBR.

[scottmg, 2016-02-10 22:21:03]

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

[scottmg, 2016-02-10 22:21:03]

Oh right, thanks. +tapted for
	chrome/browser/ui/views/apps/chrome_native_app_window_views_win.cc.

[scottmg, 2016-02-10 22:37:34]

Actually, the code in there was already unused, so I'm just going to tbr. But
please let me know if you see anything amiss and I'll do it in a followup.

[scottmg, 2016-02-10 22:37:43]

Description was changed from

==========
Remove PerAppSettingsService

Was only required for different |HostDesktopType|s which are being
removed.

R=benwells@chromium.org
BUG=558054
==========

to

==========
Remove PerAppSettingsService

Was only required for different |HostDesktopType|s which are being
removed.

TBR=benwells@chromium.org
BUG=558054
==========

[scottmg, 2016-02-10 22:37:48]

The CQ bit was checked by scottmg@chromium.org

[commit-bot: I haz the power, 2016-02-10 22:39:27]

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

[tapted, 2016-02-10 22:52:38]

nice :) lgtm

(although I guess you still need benwells for application_launch.cc)

[scottmg, 2016-02-10 23:03:41]

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

[scottmg, 2016-02-10 23:03:41]

Oh, geez, oops. Ok, I guess you get to take a look after all Ben.

[benwells, 2016-02-10 23:05:38]

lgtm

sorry about delay, yesterday was a write off...

[scottmg, 2016-02-10 23:21:59]

On 2016/02/10 23:05:38, benwells wrote:
> lgtm
> 
> sorry about delay, yesterday was a write off...

np, thanks!

[scottmg, 2016-02-10 23:37:19]

Description was changed from

==========
Remove PerAppSettingsService

Was only required for different |HostDesktopType|s which are being
removed.

TBR=benwells@chromium.org
BUG=558054
==========

to

==========
Remove PerAppSettingsService

Was only required for different |HostDesktopType|s which are being
removed.

R=benwells@chromium.org
BUG=558054
==========

[commit-bot: I haz the power, 2016-02-10 23:48:09]

Description was changed from

==========
Remove PerAppSettingsService

Was only required for different |HostDesktopType|s which are being
removed.

R=benwells@chromium.org
BUG=558054
==========

to

==========
Remove PerAppSettingsService

Was only required for different |HostDesktopType|s which are being
removed.

R=benwells@chromium.org
BUG=558054
==========

[commit-bot: I haz the power, 2016-02-10 23:48:10]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-02-16 22:32:47]

Description was changed from

==========
Remove PerAppSettingsService

Was only required for different |HostDesktopType|s which are being
removed.

R=benwells@chromium.org
BUG=558054
==========

to

==========
Remove PerAppSettingsService

Was only required for different |HostDesktopType|s which are being
removed.

R=benwells@chromium.org
BUG=558054

Committed: https://crrev.com/9017a43db41052ac7f4104ff8bbd216fc35849a7
Cr-Commit-Position: refs/heads/master@{#374789}
==========

[commit-bot: I haz the power, 2016-02-16 22:32:48]

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