Avoiding use of NOTIFICATION_EXTENSION_READY_DEPRECATED

Remove deprecated NOTIFICATION_EXTENSION_READY_DEPRECATED from background_mode_manager.cc by use of ExtensionSystem::Get

BUG=354046
Committed: https://crrev.com/5dd6bb023fc593463c7b28214873495a88116218
Cr-Commit-Position: refs/heads/master@{#329141}

[Pranay, 2015-05-06 09:39:52]

pranay.kumar@samsung.com changed reviewers:
+ thestig@chromium.org

[Pranay, 2015-05-06 09:39:52]

PTAL

Thanks

[Lei Zhang, 2015-05-06 21:35:49]

thestig@chromium.org changed reviewers:
+ atwilson@chromium.org

[Lei Zhang, 2015-05-06 21:35:50]

Please ask a more local code owner from chrome/browser/background/OWNERS,
instead of from chrome/OWNERS.

https://codereview.chromium.org/1128033002/diff/1/chrome/browser/background/b...
File chrome/browser/background/background_mode_manager.cc (left):

https://codereview.chromium.org/1128033002/diff/1/chrome/browser/background/b...
chrome/browser/background/background_mode_manager.cc:389: // Extensions are
loaded, so we don't need to manually keep the browser
Can you preseve the comment in the new code?

https://codereview.chromium.org/1128033002/diff/1/chrome/browser/background/b...
File chrome/browser/background/background_mode_manager.h (left):

https://codereview.chromium.org/1128033002/diff/1/chrome/browser/background/b...
chrome/browser/background/background_mode_manager.h:116: 
Please leave this blank line in.

[Pranay, 2015-05-07 03:24:47]

Fixed Review Comments

https://codereview.chromium.org/1128033002/diff/1/chrome/browser/background/b...
File chrome/browser/background/background_mode_manager.cc (left):

https://codereview.chromium.org/1128033002/diff/1/chrome/browser/background/b...
chrome/browser/background/background_mode_manager.cc:389: // Extensions are
loaded, so we don't need to manually keep the browser
On 2015/05/06 21:35:49, Lei Zhang wrote:
> Can you preseve the comment in the new code?

Done.

https://codereview.chromium.org/1128033002/diff/1/chrome/browser/background/b...
File chrome/browser/background/background_mode_manager.h (left):

https://codereview.chromium.org/1128033002/diff/1/chrome/browser/background/b...
chrome/browser/background/background_mode_manager.h:116: 
On 2015/05/06 21:35:49, Lei Zhang wrote:
> Please leave this blank line in.

Done.

[Pranay, 2015-05-07 03:31:19]

Hi Lei,

Sure, will do the same about code owners review. Thanks for the review & for
adding Andrew as reviewer.
Updated the new patch set.

[Pranay, 2015-05-07 03:31:30]

PTAL

[Andrew T Wilson (Slow), 2015-05-11 10:07:53]

lgtm

[Pranay, 2015-05-11 10:08:39]

The CQ bit was checked by pranay.kumar@samsung.com

[Pranay, 2015-05-11 10:09:07]

Thanks!! :)

-Pranay

[commit-bot: I haz the power, 2015-05-11 10:09:36]

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

[commit-bot: I haz the power, 2015-05-11 11:02:51]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2015-05-11 11:03:46]

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