Add ExtensionService comments that various methods are deprecated.

Add ExtensionService comments that various methods are deprecated.

R=yoz@chromium.org

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

[Yoyo Zhou, 2014-07-28 22:00:13]

https://codereview.chromium.org/426553010/diff/1/chrome/browser/extensions/ex...
File chrome/browser/extensions/extension_service.h (right):

https://codereview.chromium.org/426553010/diff/1/chrome/browser/extensions/ex...
chrome/browser/extensions/extension_service.h:88: // DEPRECATED. Use
ExtensionRegistry instead.
It's really weird to say DEPRECATED 3 times for one function, isn't it?

https://codereview.chromium.org/426553010/diff/1/chrome/browser/extensions/ex...
chrome/browser/extensions/extension_service.h:112: // DEPRECATED: Use
ExtensionRegistry instead.
Isn't this kind of redundant with 117?

[not at google - send to devlin, 2014-07-28 22:03:11]

https://codereview.chromium.org/426553010/diff/1/chrome/browser/extensions/ex...
File chrome/browser/extensions/extension_service.h (right):

https://codereview.chromium.org/426553010/diff/1/chrome/browser/extensions/ex...
chrome/browser/extensions/extension_service.h:88: // DEPRECATED. Use
ExtensionRegistry instead.
On 2014/07/28 22:00:13, Yoyo Zhou wrote:
> It's really weird to say DEPRECATED 3 times for one function, isn't it?

maybe :) it's seriously deprecated. I still see CLs that use these methods, but
I can remove the second two.

https://codereview.chromium.org/426553010/diff/1/chrome/browser/extensions/ex...
chrome/browser/extensions/extension_service.h:112: // DEPRECATED: Use
ExtensionRegistry instead.
On 2014/07/28 22:00:13, Yoyo Zhou wrote:
> Isn't this kind of redundant with 117?

ditto

[not at google - send to devlin, 2014-07-28 22:03:20]

that is to say, ptal.

[Yoyo Zhou, 2014-07-28 22:21:04]

LGTM

https://codereview.chromium.org/426553010/diff/20001/chrome/browser/extension...
File chrome/browser/extensions/extension_service.h (right):

https://codereview.chromium.org/426553010/diff/20001/chrome/browser/extension...
chrome/browser/extensions/extension_service.h:93: // extensions. Equivalent to:
Say "Use instead" instead of "equivalent to"

[not at google - send to devlin, 2014-07-28 22:23:04]

https://codereview.chromium.org/426553010/diff/20001/chrome/browser/extension...
File chrome/browser/extensions/extension_service.h (right):

https://codereview.chromium.org/426553010/diff/20001/chrome/browser/extension...
chrome/browser/extensions/extension_service.h:93: // extensions. Equivalent to:
On 2014/07/28 22:21:04, Yoyo Zhou wrote:
> Say "Use instead" instead of "equivalent to"

Done, though I had "equivalent to" because the repetition of "use instead"
looked weird to me.

[not at google - send to devlin, 2014-07-28 22:23:53]

Committed patchset #3 manually as r286002 (presubmit successful).