Add HasClickedListener to the notification delegate in background_content_service.

Add HasClickedListener to the notification delegate in background_content_service.

This method was added at r201847 to indicate a notification can be
clickable or not (by changing mouse cursor, for now).

BUG=241266
TEST=none
R=atwilson@chromium.org

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

[Andrew T Wilson (Slow), 2013-05-27 12:36:50]

LGTM with one nit - I'd like a comment in the code describing why this is
required (not just in the CL description).

https://codereview.chromium.org/16023005/diff/1/chrome/browser/background/bac...
File chrome/browser/background/background_contents_service.cc (right):

https://codereview.chromium.org/16023005/diff/1/chrome/browser/background/bac...
chrome/browser/background/background_contents_service.cc:107: virtual bool
HasClickedListener() OVERRIDE { return true; }
Please add a comment describing why this is required.

[Jun Mukai, 2013-05-27 19:10:28]

https://codereview.chromium.org/16023005/diff/1/chrome/browser/background/bac...
File chrome/browser/background/background_contents_service.cc (right):

https://codereview.chromium.org/16023005/diff/1/chrome/browser/background/bac...
chrome/browser/background/background_contents_service.cc:107: virtual bool
HasClickedListener() OVERRIDE { return true; }
On 2013/05/27 12:36:50, Andrew T Wilson wrote:
> Please add a comment describing why this is required.

I may miss your point. Those methods are inherited from NotificationDelegate,
and I think such comments should be written in NotificationDelegate, not here.

[Andrew T Wilson (Slow), 2013-05-28 07:27:57]

On 2013/05/27 19:10:28, Jun Mukai wrote:
>
https://codereview.chromium.org/16023005/diff/1/chrome/browser/background/bac...
> File chrome/browser/background/background_contents_service.cc (right):
> 
>
https://codereview.chromium.org/16023005/diff/1/chrome/browser/background/bac...
> chrome/browser/background/background_contents_service.cc:107: virtual bool
> HasClickedListener() OVERRIDE { return true; }
> On 2013/05/27 12:36:50, Andrew T Wilson wrote:
> > Please add a comment describing why this is required.
> 
> I may miss your point. Those methods are inherited from NotificationDelegate,
> and I think such comments should be written in NotificationDelegate, not here.

Makes sense. LGTM.

[commit-bot: I haz the power, 2013-05-28 16:24:39]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/mukai@chromium.org/16023005/1

[commit-bot: I haz the power, 2013-05-28 18:47:38]

Change committed as 202613