Rename ExtensionWarningBadgeService to WarningBadgeService
As the comment in extension_warning_badge_service.h by battre@,
rename ExtensionWarningBadgeService to WarningBadgeService and related class.
And rename the files too.
BUG=458829R=rockot@chromium.org
Committed: https://crrev.com/6fcce6acc4e79e19f0910d1b3e9eb2e3327e0856
Cr-Commit-Position: refs/heads/master@{#317992}
looks ok but also +battre since he wrote the TODO https://codereview.chromium.org/921423003/diff/1/chrome/browser/extensions/warning_badge_service.h File chrome/browser/extensions/warning_badge_service.h (right): https://codereview.chromium.org/921423003/diff/1/chrome/browser/extensions/warning_badge_service.h#newcode21 ...
5 years, 10 months ago
(2015-02-17 19:13:29 UTC)
#3
On 2015/02/17 at 19:13:29, rockot wrote: > looks ok but also +battre since he wrote ...
5 years, 10 months ago
(2015-02-19 14:26:38 UTC)
#4
On 2015/02/17 at 19:13:29, rockot wrote:
> looks ok but also +battre since he wrote the TODO
>
>
https://codereview.chromium.org/921423003/diff/1/chrome/browser/extensions/wa...
> File chrome/browser/extensions/warning_badge_service.h (right):
>
>
https://codereview.chromium.org/921423003/diff/1/chrome/browser/extensions/wa...
> chrome/browser/extensions/warning_badge_service.h:21: public
WarningService::Observer,
> Please align the base class specifiers.
>
> You can run 'git cl format' from your repo to automatically apply standard
formatting and catch things like this for you.
+1 to running git cl format. There are several locations that need to be updated
for formatting.
Thanks for taking care of this!
zhuoyu.qian
I have update the format as your comment, please have a review. Thank you.
5 years, 10 months ago
(2015-02-25 01:01:48 UTC)
#5
I have update the format as your comment, please have a review. Thank you.
Ken Rockot(use gerrit already)
Thanks! lgtm
5 years, 10 months ago
(2015-02-25 01:44:39 UTC)
#6
Issue 921423003: Rename ExtensionWarningBadgeService to WarningBadgeService
(Closed)
Created 5 years, 10 months ago by zhuoyu.qian
Modified 5 years, 10 months ago
Reviewers: Ken Rockot(use gerrit already), battre, Nico
Base URL: https://chromium.googlesource.com/chromium/src.git@master
Comments: 1