Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(143)

Unified Diff: chrome/browser/notifications/notification.h

Issue 574273003: Remove Notification::small_image_url_ as it has no users. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 6 years, 3 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « chrome/browser/notifications/message_center_notification_manager.cc ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/notifications/notification.h
diff --git a/chrome/browser/notifications/notification.h b/chrome/browser/notifications/notification.h
index 4c66aac5089474bb5b37a9c131f8eeb0b28de791..4140f707672592adf247f70a1ad739cef17cf7c3 100644
--- a/chrome/browser/notifications/notification.h
+++ b/chrome/browser/notifications/notification.h
@@ -74,11 +74,6 @@ class Notification : public message_center::Notification {
// Id of the delegate embedded inside this instance.
std::string delegate_id() const { return delegate()->id(); }
- content::WebContents* GetWebContents() const {
- return delegate()->GetWebContents();
- }
- void DoneRendering() { delegate()->ReleaseRenderViewHost(); }
-
NotificationDelegate* delegate() const { return delegate_.get(); }
private:
@@ -96,9 +91,6 @@ class Notification : public message_center::Notification {
// The URL of a large image to be displayed for a a rich notification.
GURL image_url_;
- // The URL of a small image to be displayed for a a rich notification.
- GURL small_image_url_;
-
// The user-supplied replace ID for the notification.
base::string16 replace_id_;
« no previous file with comments | « chrome/browser/notifications/message_center_notification_manager.cc ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698