Add TriggerOnUninstalled to ExtensionRegistry

Add TriggerOnUninstalled to ExtensionRegistry.
And change ExtensionToolbarModel to use it instead of NOTIFICATION_EXTENSION_UNINSTALLED.

R=kalman@chromium.org
BUG=354459
TEST=unit_test

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

[limasdf, 2014-05-23 20:17:03]

please take a look, after submit.
I'll rename NOTIFICATION_EXTENSION_UNINSTALLED to ~_DEPRECATED.

[not at google - send to devlin, 2014-05-23 20:24:05]

lgtm

https://codereview.chromium.org/289073008/diff/80001/chrome/browser/extension...
File chrome/browser/extensions/extension_toolbar_model.cc (right):

https://codereview.chromium.org/289073008/diff/80001/chrome/browser/extension...
chrome/browser/extensions/extension_toolbar_model.cc:197:
UninstalledExtension(extension);
given the only caller of UninstalledExtension is here, you might as well inline
it.

[limasdf, 2014-05-23 20:47:21]

Thanks. I'll commit then...

https://codereview.chromium.org/289073008/diff/80001/chrome/browser/extension...
File chrome/browser/extensions/extension_toolbar_model.cc (right):

https://codereview.chromium.org/289073008/diff/80001/chrome/browser/extension...
chrome/browser/extensions/extension_toolbar_model.cc:197:
UninstalledExtension(extension);
On 2014/05/23 20:24:05, kalman wrote:
> given the only caller of UninstalledExtension is here, you might as well
inline
> it.

Done.

[limasdf, 2014-05-23 20:47:33]

The CQ bit was checked by limasdf@gmail.com

[commit-bot: I haz the power, 2014-05-23 20:49:03]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/limasdf@gmail.com/289073008/100001

[commit-bot: I haz the power, 2014-05-24 00:11:55]

Change committed as 272649