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

Unified Diff: third_party/WebKit/public/platform/WebURLResponse.h

Issue 2133693002: Remove class blink::WebURLResponsePrivate (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Remove WebURLResponsePrivate.h from gypi file Created 4 years, 5 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 | « third_party/WebKit/Source/platform/exported/WrappedResourceResponse.h ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: third_party/WebKit/public/platform/WebURLResponse.h
diff --git a/third_party/WebKit/public/platform/WebURLResponse.h b/third_party/WebKit/public/platform/WebURLResponse.h
index 243be1e18a55e4c4a4d2d8d819b6456a59e3df53..27c42118049df9567aaeeb2c0852e03ea756faf8 100644
--- a/third_party/WebKit/public/platform/WebURLResponse.h
+++ b/third_party/WebKit/public/platform/WebURLResponse.h
@@ -46,7 +46,6 @@ class WebHTTPHeaderVisitor;
class WebHTTPLoadInfo;
class WebURL;
class WebURLLoadTiming;
-class WebURLResponsePrivate;
class WebURLResponse {
public:
@@ -293,18 +292,21 @@ public:
BLINK_PLATFORM_EXPORT void setExtraData(ExtraData*);
protected:
- // Permit subclasses to set arbitrary WebURLResponsePrivate pointer as
- // |m_private|. Parameter must be non-null. |m_owningPrivate| is not set
- // in this case.
- BLINK_PLATFORM_EXPORT explicit WebURLResponse(WebURLResponsePrivate*);
+ // Permit subclasses to set arbitrary ResourceResponse pointer as
+ // |m_resourceResponse|. Parameter must be non-null.
+ // |m_ownedResourceResponse| is not set in this case.
+ BLINK_PLATFORM_EXPORT explicit WebURLResponse(ResourceResponse*);
kinuko 2016/07/11 03:44:12 Put this inside #if INSIDE_BLINK ?
Adam Rice 2016/07/11 05:44:18 Done.
private:
- // If this instance owns WebURLResponsePrivate |m_owningPrivate| is
- // non-null and is pointed by |m_private|.
- std::unique_ptr<WebURLResponsePrivate> m_owningPrivate;
+ struct ResourceResponseContainer;
+
+ // If this instance owns a ResourceResponse then |m_ownedResourceResponse|
+ // is non-null and |m_resourceResponse| points to the ResourceResponse
+ // instance it contains.
+ std::unique_ptr<ResourceResponseContainer> m_ownedResourceResponse;
// Should never be null.
- WebURLResponsePrivate* m_private;
+ ResourceResponse* m_resourceResponse;
};
} // namespace blink
« no previous file with comments | « third_party/WebKit/Source/platform/exported/WrappedResourceResponse.h ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698