Resubmit: Remove deprecated extension notifications from ProcessManager

Remove deprecated extension notifications from ProcessManager and use ExtensionRegistry instead.

ProcessManagerTest inherts ExtensionsTest for the test settings. So now unit test is passed.

R=rdevlin.cronin@chromium.org
BUG=354046
TEST=extensions_unittests

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

[limasdf, 2014-07-31 15:34:11]

PTAL. I hope no reverting!

[Devlin, 2014-08-04 18:45:20]

lgtm with nit. :)

https://codereview.chromium.org/434593002/diff/40001/extensions/browser/proce...
File extensions/browser/process_manager.cc (right):

https://codereview.chromium.org/434593002/diff/40001/extensions/browser/proce...
extensions/browser/process_manager.cc:731: if (system->ready().is_signaled()) {
Let's go ahead and inline system:
if (ExtensionSystem::Get(browser_context)->ready().is_signaled())

[limasdf, 2014-08-05 06:26:52]

https://codereview.chromium.org/434593002/diff/40001/extensions/browser/proce...
File extensions/browser/process_manager.cc (right):

https://codereview.chromium.org/434593002/diff/40001/extensions/browser/proce...
extensions/browser/process_manager.cc:731: if (system->ready().is_signaled()) {
On 2014/08/04 18:45:20, Devlin wrote:
> Let's go ahead and inline system:
> if (ExtensionSystem::Get(browser_context)->ready().is_signaled())

Done.

[limasdf, 2014-08-05 06:26:56]

The CQ bit was checked by limasdf@gmail.com

[commit-bot: I haz the power, 2014-08-05 06:28:54]

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

[commit-bot: I haz the power, 2014-08-05 10:54:08]

FYI, CQ is re-trying this CL (attempt #1).
The failing builders are:
  android_aosp on tryserver.chromium.linux
(http://build.chromium.org/p/tryserver.chromium.linux/builders/android_aosp/bu...)
  linux_chromium_rel_swarming on tryserver.chromium.linux
(http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)

[commit-bot: I haz the power, 2014-08-05 11:18:59]

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

[commit-bot: I haz the power, 2014-08-05 11:18:59]

Try jobs failed on following builders:
  android_aosp on tryserver.chromium.linux
(http://build.chromium.org/p/tryserver.chromium.linux/builders/android_aosp/bu...)

[limasdf, 2014-08-05 11:34:14]

The CQ bit was checked by limasdf@gmail.com

[commit-bot: I haz the power, 2014-08-05 11:38:40]

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

[commit-bot: I haz the power, 2014-08-05 12:23:19]

Change committed as 287521

[Adrian Kuegel, 2014-08-05 13:27:26]

A revert of this CL has been created in
https://codereview.chromium.org/443623002/ by akuegel@chromium.org.

The reason for reverting is: Speculative revert, see
http://build.chromium.org/p/chromium.memory/builders/Linux%20ASan%20LSan%20Te...
The stack trace mentions extensions::ExtensionRegistry::RemoveObserver.

[limasdf, 2014-08-06 18:00:08]

Maybe I should debug with ASAN?