Remove class blink::WebURLResponsePrivate

third_party/WebKit/public/platform/WebURLResponse.h

 /*
  * Copyright (C) 2009 Google Inc. All rights reserved.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions are
  * met:
  *
  *     * Redistributions of source code must retain the above copyright
  * notice, this list of conditions and the following disclaimer.
  *     * Redistributions in binary form must reproduce the above
@@ skipping 28 matching lines @@
 #include "public/platform/modules/serviceworker/WebServiceWorkerResponseType.h"
 #include <memory>
 
 namespace blink {
 
 class ResourceResponse;
 class WebHTTPHeaderVisitor;
 class WebHTTPLoadInfo;
 class WebURL;
 class WebURLLoadTiming;
-class WebURLResponsePrivate;
 
 class WebURLResponse {
 public:
     enum HTTPVersion { HTTPVersionUnknown,
         HTTPVersion_0_9,
         HTTPVersion_1_0,
         HTTPVersion_1_1,
         HTTPVersion_2_0 };
     enum SecurityStyle {
         SecurityStyleUnknown,
@@ skipping 226 matching lines @@
     // Extra data associated with the underlying resource response. Resource
     // responses can be copied. If non-null, each copy of a resource response
     // holds a pointer to the extra data, and the extra data pointer will be
     // deleted when the last resource response is destroyed. Setting the extra
     // data pointer will cause the underlying resource response to be
     // dissociated from any existing non-null extra data pointer.
     BLINK_PLATFORM_EXPORT ExtraData* getExtraData() const;
     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
 
 #endif