Unregister from GCM when the only GCM app is removed

Unregister from GCM when the only GCM app is removed

The bug occurred because we remove app handler on extension/app unload.
We should not do it at this time since uninstall will trigger unload
first.

BUG=457433
TEST=tests updated
TBR=kalman@chromium.org

Committed: https://crrev.com/514b6b0475dd0b15a7a17f020247eaba72841544
Cr-Commit-Position: refs/heads/master@{#315846}

[jianli, 2015-02-10 22:52:38]

jianli@chromium.org changed reviewers:
+ fgorski@chromium.org

[fgorski, 2015-02-10 23:10:01]

lgtm

https://codereview.chromium.org/913943002/diff/1/chrome/browser/extensions/ex...
File chrome/browser/extensions/extension_gcm_app_handler_unittest.cc (right):

https://codereview.chromium.org/913943002/diff/1/chrome/browser/extensions/ex...
chrome/browser/extensions/extension_gcm_app_handler_unittest.cc:151:
ExtensionGCMAppHandler::OnUnregisterCompleted(app_id,result);
nit: space after a comma. (run git cl format)

[jianli, 2015-02-10 23:22:46]

New patchsets have been uploaded after l-g-t-m from fgorski@chromium.org

[jianli, 2015-02-10 23:23:06]

The CQ bit was checked by jianli@chromium.org

[commit-bot: I haz the power, 2015-02-10 23:24:10]

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

[commit-bot: I haz the power, 2015-02-10 23:30:55]

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

[commit-bot: I haz the power, 2015-02-10 23:30:55]

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

[jianli, 2015-02-11 00:07:50]

The CQ bit was checked by jianli@chromium.org

[commit-bot: I haz the power, 2015-02-11 00:09:04]

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

[commit-bot: I haz the power, 2015-02-11 01:26:09]

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

[commit-bot: I haz the power, 2015-02-11 01:26:10]

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

[jianli, 2015-02-11 18:59:32]

The CQ bit was checked by jianli@chromium.org

[commit-bot: I haz the power, 2015-02-11 19:00:35]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/913943002/40001

[commit-bot: I haz the power, 2015-02-11 20:25:10]

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

[commit-bot: I haz the power, 2015-02-11 20:25:11]

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

[jianli, 2015-02-11 21:10:49]

Patchset #3 (id:40001) has been deleted

[jianli, 2015-02-11 21:59:37]

The CQ bit was checked by jianli@chromium.org

[commit-bot: I haz the power, 2015-02-11 22:00:35]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/913943002/60001

[commit-bot: I haz the power, 2015-02-11 22:07:15]

Committed patchset #3 (id:60001)

[commit-bot: I haz the power, 2015-02-11 22:08:53]

Patchset 3 (id:??) landed as
https://crrev.com/514b6b0475dd0b15a7a17f020247eaba72841544
Cr-Commit-Position: refs/heads/master@{#315846}