Add HasClickedListener to the notification delegate in background_content_service.

chrome/browser/background/background_contents_service.cc

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "chrome/browser/background/background_contents_service.h"
 
 #include "base/basictypes.h"
 #include "base/bind.h"
 #include "base/command_line.h"
 #include "base/message_loop.h"
@@ skipping 86 matching lines @@
     } else {
       extensions::ExtensionSystem::Get(profile_)->extension_service()->
           ReloadExtension(extension_id_);
     }
 
     // Closing the balloon here should be OK, but it causes a crash on Mac
     // http://crbug.com/78167
     ScheduleCloseBalloon(extension_id_);
   }
 
+  virtual bool HasClickedListener() OVERRIDE { return true; }

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

Please add a comment describing why this is required.

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

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

+
   virtual std::string id() const OVERRIDE {
     return kNotificationPrefix + extension_id_;
   }
 
   virtual content::RenderViewHost* GetRenderViewHost() const OVERRIDE {
     return NULL;
   }
 
  private:
   virtual ~CrashNotificationDelegate() {}
@@ skipping 563 matching lines @@
     bool user_gesture,
     bool* was_blocked) {
   Browser* browser = chrome::FindLastActiveWithProfile(
       Profile::FromBrowserContext(new_contents->GetBrowserContext()),
       chrome::GetActiveDesktop());
   if (browser) {
     chrome::AddWebContents(browser, NULL, new_contents, disposition,
                            initial_pos, user_gesture, was_blocked);
   }
 }