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

Side by Side Diff: content/public/browser/web_contents_user_data.h

Issue 2842833003: Update SupportsUserData uses with unique_ptr. (Closed)
Patch Set: rebase Created 3 years, 7 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 unified diff | Download patch
OLDNEW
1 // Copyright (c) 2012 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2012 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 #ifndef CONTENT_PUBLIC_BROWSER_WEB_CONTENTS_USER_DATA_H_ 5 #ifndef CONTENT_PUBLIC_BROWSER_WEB_CONTENTS_USER_DATA_H_
6 #define CONTENT_PUBLIC_BROWSER_WEB_CONTENTS_USER_DATA_H_ 6 #define CONTENT_PUBLIC_BROWSER_WEB_CONTENTS_USER_DATA_H_
7 7
8 #include "base/logging.h" 8 #include "base/logging.h"
9 #include "base/memory/ptr_util.h"
9 #include "base/supports_user_data.h" 10 #include "base/supports_user_data.h"
10 #include "content/public/browser/web_contents.h" 11 #include "content/public/browser/web_contents.h"
11 12
12 namespace content { 13 namespace content {
13 14
14 // A base class for classes attached to, and scoped to, the lifetime of a 15 // A base class for classes attached to, and scoped to, the lifetime of a
15 // WebContents. For example: 16 // WebContents. For example:
16 // 17 //
17 // --- in foo_tab_helper.h --- 18 // --- in foo_tab_helper.h ---
18 // class FooTabHelper : public content::WebContentsUserData<FooTabHelper> { 19 // class FooTabHelper : public content::WebContentsUserData<FooTabHelper> {
19 // public: 20 // public:
20 // ~FooTabHelper() override; 21 // ~FooTabHelper() override;
21 // // ... more public stuff here ... 22 // // ... more public stuff here ...
22 // private: 23 // private:
23 // explicit FooTabHelper(content::WebContents* contents); 24 // explicit FooTabHelper(content::WebContents* contents);
24 // friend class content::WebContentsUserData<FooTabHelper>; 25 // friend class content::WebContentsUserData<FooTabHelper>;
25 // // ... more private stuff here ... 26 // // ... more private stuff here ...
26 // } 27 // }
27 // --- in foo_tab_helper.cc --- 28 // --- in foo_tab_helper.cc ---
28 // DEFINE_WEB_CONTENTS_USER_DATA_KEY(FooTabHelper); 29 // DEFINE_WEB_CONTENTS_USER_DATA_KEY(FooTabHelper);
29 // 30 //
30 template <typename T> 31 template <typename T>
31 class WebContentsUserData : public base::SupportsUserData::Data { 32 class WebContentsUserData : public base::SupportsUserData::Data {
32 public: 33 public:
33 // Creates an object of type T, and attaches it to the specified WebContents. 34 // Creates an object of type T, and attaches it to the specified WebContents.
34 // If an instance is already attached, does nothing. 35 // If an instance is already attached, does nothing.
35 static void CreateForWebContents(WebContents* contents) { 36 static void CreateForWebContents(WebContents* contents) {
36 DCHECK(contents); 37 DCHECK(contents);
37 if (!FromWebContents(contents)) 38 if (!FromWebContents(contents))
38 contents->SetUserData(UserDataKey(), new T(contents)); 39 contents->SetUserData(UserDataKey(), base::WrapUnique(new T(contents)));
39 } 40 }
40 41
41 // Retrieves the instance of type T that was attached to the specified 42 // Retrieves the instance of type T that was attached to the specified
42 // WebContents (via CreateForWebContents above) and returns it. If no instance 43 // WebContents (via CreateForWebContents above) and returns it. If no instance
43 // of the type was attached, returns nullptr. 44 // of the type was attached, returns nullptr.
44 static T* FromWebContents(WebContents* contents) { 45 static T* FromWebContents(WebContents* contents) {
45 DCHECK(contents); 46 DCHECK(contents);
46 return static_cast<T*>(contents->GetUserData(UserDataKey())); 47 return static_cast<T*>(contents->GetUserData(UserDataKey()));
47 } 48 }
48 static const T* FromWebContents(const WebContents* contents) { 49 static const T* FromWebContents(const WebContents* contents) {
(...skipping 19 matching lines...) Expand all
68 // a declaration. Without it, this would be merely a declaration of a template 69 // a declaration. Without it, this would be merely a declaration of a template
69 // specialization. (C++98: 14.7.3.15; C++11: 14.7.3.13) 70 // specialization. (C++98: 14.7.3.15; C++11: 14.7.3.13)
70 // 71 //
71 #define DEFINE_WEB_CONTENTS_USER_DATA_KEY(TYPE) \ 72 #define DEFINE_WEB_CONTENTS_USER_DATA_KEY(TYPE) \
72 template<> \ 73 template<> \
73 int content::WebContentsUserData<TYPE>::kLocatorKey = 0 74 int content::WebContentsUserData<TYPE>::kLocatorKey = 0
74 75
75 } // namespace content 76 } // namespace content
76 77
77 #endif // CONTENT_PUBLIC_BROWSER_WEB_CONTENTS_USER_DATA_H_ 78 #endif // CONTENT_PUBLIC_BROWSER_WEB_CONTENTS_USER_DATA_H_
OLDNEW
« no previous file with comments | « content/browser/websockets/websocket_manager.cc ('k') | content/renderer/internal_document_state_data.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698