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

Side by Side Diff: chrome/browser/blocked_popup_container.h

Issue 434109: Don't save popup blocker whitelist modifications performed while off the reco... (Closed) Base URL: svn://chrome-svn/chrome/trunk/src/
Patch Set: '' Created 11 years 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
OLDNEW
1 // Copyright (c) 2009 The Chromium Authors. All rights reserved. Use of this 1 // Copyright (c) 2009 The Chromium Authors. All rights reserved. Use of this
2 // source code is governed by a BSD-style license that can be found in the 2 // source code is governed by a BSD-style license that can be found in the
3 // LICENSE file. 3 // LICENSE file.
4 4
5 // Defines the public interface for the blocked popup notifications. This 5 // Defines the public interface for the blocked popup notifications. This
6 // interface should only be used by TabContents. Users and subclasses of 6 // interface should only be used by TabContents. Users and subclasses of
7 // TabContents should use the appropriate methods on TabContents to access 7 // TabContents should use the appropriate methods on TabContents to access
8 // information about blocked popups. 8 // information about blocked popups.
9 9
10 // TODO(idanan): Rename class to BlockedContentContainer. 10 // TODO(idanan): Rename class to BlockedContentContainer.
(...skipping 260 matching lines...) Expand 10 before | Expand all | Expand 10 after
271 void UpdateView(); 271 void UpdateView();
272 272
273 // Overridden from notificationObserver: 273 // Overridden from notificationObserver:
274 virtual void Observe(NotificationType type, 274 virtual void Observe(NotificationType type,
275 const NotificationSource& source, 275 const NotificationSource& source,
276 const NotificationDetails& details); 276 const NotificationDetails& details);
277 277
278 // The TabContents that owns and constrains this BlockedPopupContainer. 278 // The TabContents that owns and constrains this BlockedPopupContainer.
279 TabContents* owner_; 279 TabContents* owner_;
280 280
281 // The PrefService we can query to find out what's on the whitelist. 281 // The PrefService we can query to find out what's on the whitelist. If the
282 // profile is off the record, this will be NULL.
282 PrefService* prefs_; 283 PrefService* prefs_;
283 284
284 // Once the container is hidden, this is set to prevent it from reappearing. 285 // Once the container is hidden, this is set to prevent it from reappearing.
285 bool has_been_dismissed_; 286 bool has_been_dismissed_;
286 287
287 // Registrar to handle notifications we care about. 288 // Registrar to handle notifications we care about.
288 NotificationRegistrar registrar_; 289 NotificationRegistrar registrar_;
289 290
290 // The whitelisted hosts, which we allow to open popups directly. 291 // The whitelisted hosts, which we allow to open popups directly.
291 Whitelist whitelist_; 292 Whitelist whitelist_;
(...skipping 16 matching lines...) Expand all
308 // Our platform specific view. 309 // Our platform specific view.
309 BlockedPopupContainerView* view_; 310 BlockedPopupContainerView* view_;
310 311
311 // The profile for the browser associated with the container. 312 // The profile for the browser associated with the container.
312 Profile* profile_; 313 Profile* profile_;
313 314
314 DISALLOW_IMPLICIT_CONSTRUCTORS(BlockedPopupContainer); 315 DISALLOW_IMPLICIT_CONSTRUCTORS(BlockedPopupContainer);
315 }; 316 };
316 317
317 #endif // CHROME_BROWSER_BLOCKED_POPUP_CONTAINER_H_ 318 #endif // CHROME_BROWSER_BLOCKED_POPUP_CONTAINER_H_
OLDNEW
« no previous file with comments | « no previous file | chrome/browser/blocked_popup_container.cc » ('j') | chrome/browser/blocked_popup_container.cc » ('J')

Powered by Google App Engine
This is Rietveld 408576698