Remove class blink::WebURLResponsePrivate

Remove class blink::WebURLResponsePrivate

WebURLResponse can directly point to a ResourceResponse object instead.

BUG=625529
Committed: https://crrev.com/8036fd1c9bbd45f18ab78716e641c6d6cf974f79
Cr-Commit-Position: refs/heads/master@{#404611}

[Adam Rice, 2016-07-11 02:29:25]

ricea@chromium.org changed reviewers:
+ kinuko@chromium.org

[Adam Rice, 2016-07-11 02:29:26]

I'm not sure that this is better, but it is shorter. PTAL.

[kinuko, 2016-07-11 03:44:12]

Looks fine to me.  lgtm

https://codereview.chromium.org/2133693002/diff/20001/third_party/WebKit/publ...
File third_party/WebKit/public/platform/WebURLResponse.h (right):

https://codereview.chromium.org/2133693002/diff/20001/third_party/WebKit/publ...
third_party/WebKit/public/platform/WebURLResponse.h:298: BLINK_PLATFORM_EXPORT
explicit WebURLResponse(ResourceResponse*);
Put this inside #if INSIDE_BLINK ?

[Adam Rice, 2016-07-11 05:42:01]

The CQ bit was checked by ricea@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-07-11 05:42:15]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[Adam Rice, 2016-07-11 05:44:18]

https://codereview.chromium.org/2133693002/diff/20001/third_party/WebKit/publ...
File third_party/WebKit/public/platform/WebURLResponse.h (right):

https://codereview.chromium.org/2133693002/diff/20001/third_party/WebKit/publ...
third_party/WebKit/public/platform/WebURLResponse.h:298: BLINK_PLATFORM_EXPORT
explicit WebURLResponse(ResourceResponse*);
On 2016/07/11 03:44:12, kinuko wrote:
> Put this inside #if INSIDE_BLINK ?

Done.

[commit-bot: I haz the power, 2016-07-11 07:18:33]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-07-11 07:18:34]

Dry run: This issue passed the CQ dry run.

[Adam Rice, 2016-07-11 07:30:57]

The CQ bit was checked by ricea@chromium.org

[Adam Rice, 2016-07-11 07:30:58]

The patchset sent to the CQ was uploaded after l-g-t-m from kinuko@chromium.org
Link to the patchset: https://codereview.chromium.org/2133693002/#ps40001
(title: "Restrict the unowned constructor to WebURLResponse")

[commit-bot: I haz the power, 2016-07-11 07:31:05]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-07-11 07:34:48]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2016-07-11 07:36:30]

Description was changed from

==========
Remove class blink::WebURLResponsePrivate

WebURLResponse can directly point to a ResourceResponse object instead.

BUG=625529
==========

to

==========
Remove class blink::WebURLResponsePrivate

WebURLResponse can directly point to a ResourceResponse object instead.

BUG=625529

Committed: https://crrev.com/8036fd1c9bbd45f18ab78716e641c6d6cf974f79
Cr-Commit-Position: refs/heads/master@{#404611}
==========

[commit-bot: I haz the power, 2016-07-11 07:36:31]

Patchset 3 (id:??) landed as
https://crrev.com/8036fd1c9bbd45f18ab78716e641c6d6cf974f79
Cr-Commit-Position: refs/heads/master@{#404611}