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

Unified Diff: ui/web_dialogs/constrained_web_dialog_ui.h

Issue 10831407: Kill PropertyBag, switch WebContents to SupportsUserData. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: works Created 8 years, 4 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
Index: ui/web_dialogs/constrained_web_dialog_ui.h
diff --git a/ui/web_dialogs/constrained_web_dialog_ui.h b/ui/web_dialogs/constrained_web_dialog_ui.h
index 4837c5f9dd6aa98c6bed58123c764d250c2e711a..04e90844ca372bfa331fbb6488f4fc3f035c7538 100644
--- a/ui/web_dialogs/constrained_web_dialog_ui.h
+++ b/ui/web_dialogs/constrained_web_dialog_ui.h
@@ -13,12 +13,9 @@ class ConstrainedWindow;
class Profile;
class TabContents;
-namespace base {
-template<class T> class PropertyAccessor;
-}
-
namespace content {
class RenderViewHost;
+class WebContents;
}
namespace ui {
@@ -69,14 +66,13 @@ class WEB_DIALOGS_EXPORT ConstrainedWebDialogUI
virtual void RenderViewCreated(
content::RenderViewHost* render_view_host) OVERRIDE;
- // Returns a property accessor that can be used to set the
- // ConstrainedWebDialogDelegate property on a WebContents.
- static base::PropertyAccessor<ConstrainedWebDialogDelegate*>&
- GetPropertyAccessor();
+ // Sets the delegate on the WebContents.
+ static void SetConstrainedDelegate(content::WebContents* web_contents,
+ ConstrainedWebDialogDelegate* delegate);
protected:
- // Returns the WebContents' PropertyBag's ConstrainedWebDialogDelegate.
- // Returns NULL if that property is not set.
+ // Returns the ConstrainedWebDialogDelegate saved with the WebContents.
+ // Returns NULL if no such delegate is set.
ConstrainedWebDialogDelegate* GetConstrainedDelegate();
private:

Powered by Google App Engine
This is Rietveld 408576698