Remove unnecessary image downloading handler of notifications.

chrome/browser/notifications/notification_object_proxy.h

 // 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.
 
 #ifndef CHROME_BROWSER_NOTIFICATIONS_NOTIFICATION_OBJECT_PROXY_H_
 #define CHROME_BROWSER_NOTIFICATIONS_NOTIFICATION_OBJECT_PROXY_H_
 
 #include <string>
 
 #include "base/memory/scoped_ptr.h"
@@ skipping 16 matching lines @@
   NotificationObjectProxy(
       content::RenderFrameHost* render_frame_host,
       scoped_ptr<content::DesktopNotificationDelegate> delegate);
 
   // NotificationDelegate implementation.
   virtual void Display() OVERRIDE;
   virtual void Error() OVERRIDE;
   virtual void Close(bool by_user) OVERRIDE;
   virtual void Click() OVERRIDE;
   virtual std::string id() const OVERRIDE;
-  virtual content::WebContents* GetWebContents() const OVERRIDE;
 
  protected:
   friend class base::RefCountedThreadSafe<NotificationObjectProxy>;
 
   virtual ~NotificationObjectProxy();
 
  private:
   // Callback information to find the JS Notification object where it lives.
   int render_process_id_;
   int render_frame_id_;

[Peter Beverloo, 2014/09/19 11:27:46]

The only reason we keep track of the render_process_id_ and the render_frame_id_
in NotificationObjectProxy (and require a RFH* in the constructor) is for
getting the WebContents in GetWebContents(), so all of that can go as well now.

Happy to clean this up after you're done, together with the ContentBrowserClient
plumbing that's also related to it.

[Jun Mukai, 2014/09/19 22:34:29]

Nice catch! Removed them.

   scoped_ptr<content::DesktopNotificationDelegate> delegate_;
   bool displayed_;
   std::string id_;
 };
 
 #endif  // CHROME_BROWSER_NOTIFICATIONS_NOTIFICATION_OBJECT_PROXY_H_