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

Side by Side Diff: chrome/browser/chromeos/notifications/balloon_view.h

Issue 8585011: Make html_contents_ in BallonViewImpl scoped_ptr. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: will land suppression separately Created 9 years, 1 month 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 unified diff | Download patch | Annotate | Revision Log
« no previous file with comments | « no previous file | chrome/browser/chromeos/notifications/balloon_view.cc » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 // Copyright (c) 2011 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2011 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 // Draws the view for the balloons. 5 // Draws the view for the balloons.
6 6
7 #ifndef CHROME_BROWSER_CHROMEOS_NOTIFICATIONS_BALLOON_VIEW_H_ 7 #ifndef CHROME_BROWSER_CHROMEOS_NOTIFICATIONS_BALLOON_VIEW_H_
8 #define CHROME_BROWSER_CHROMEOS_NOTIFICATIONS_BALLOON_VIEW_H_ 8 #define CHROME_BROWSER_CHROMEOS_NOTIFICATIONS_BALLOON_VIEW_H_
9 #pragma once 9 #pragma once
10 10
(...skipping 81 matching lines...) Expand 10 before | Expand all | Expand 10 after
92 92
93 // Denies the permission to show the ballooon from its source. 93 // Denies the permission to show the ballooon from its source.
94 void DenyPermission(); 94 void DenyPermission();
95 95
96 // Returns the renderer's native view. 96 // Returns the renderer's native view.
97 gfx::NativeView GetParentNativeView(); 97 gfx::NativeView GetParentNativeView();
98 98
99 // Non-owned pointer to the balloon which owns this object. 99 // Non-owned pointer to the balloon which owns this object.
100 Balloon* balloon_; 100 Balloon* balloon_;
101 101
102 // The renderer of the HTML contents. Pointer owned by the views hierarchy. 102 // The renderer of the HTML contents.
103 BalloonViewHost* html_contents_; 103 scoped_ptr<BalloonViewHost> html_contents_;
104 104
105 // A widget for ControlView. 105 // A widget for ControlView.
106 scoped_ptr<views::Widget> control_view_host_; 106 scoped_ptr<views::Widget> control_view_host_;
107 107
108 bool stale_; 108 bool stale_;
109 content::NotificationRegistrar notification_registrar_; 109 content::NotificationRegistrar notification_registrar_;
110 // A sticky flag. A sticky notification cannot be dismissed by a user. 110 // A sticky flag. A sticky notification cannot be dismissed by a user.
111 bool sticky_; 111 bool sticky_;
112 // True if a notification should have info/option/dismiss label/buttons. 112 // True if a notification should have info/option/dismiss label/buttons.
113 bool controls_; 113 bool controls_;
114 // True if the notification is being closed. 114 // True if the notification is being closed.
115 bool closed_; 115 bool closed_;
116 // True to enable WebUI in the notification. 116 // True to enable WebUI in the notification.
117 bool web_ui_; 117 bool web_ui_;
118 118
119 DISALLOW_COPY_AND_ASSIGN(BalloonViewImpl); 119 DISALLOW_COPY_AND_ASSIGN(BalloonViewImpl);
120 }; 120 };
121 121
122 } // namespace chromeos 122 } // namespace chromeos
123 123
124 #endif // CHROME_BROWSER_CHROMEOS_NOTIFICATIONS_BALLOON_VIEW_H_ 124 #endif // CHROME_BROWSER_CHROMEOS_NOTIFICATIONS_BALLOON_VIEW_H_
OLDNEW
« no previous file with comments | « no previous file | chrome/browser/chromeos/notifications/balloon_view.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698