Mojo: Generalize blink::URLRequest -> mojo::URLRequest conversion, and use it

Mojo: Generalize blink::URLRequest -> mojo::URLRequest conversion, and use it
to initialize requests in WebURLLoaderImpl.

BUG=394645
Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=289176

[darin (slow to review), 2014-08-12 16:23:12]

LGTM w/ nit:

https://codereview.chromium.org/453133002/diff/1/mojo/services/html_viewer/ht...
File mojo/services/html_viewer/html_document_view.cc (right):

https://codereview.chromium.org/453133002/diff/1/mojo/services/html_viewer/ht...
mojo/services/html_viewer/html_document_view.cc:169:
TypeConverter<URLRequestPtr, blink::WebURLRequest>::ConvertFrom(request);
You can just use URLRequest::From(request) here.

https://codereview.chromium.org/453133002/diff/1/mojo/services/html_viewer/we...
File mojo/services/html_viewer/weburlloader_impl.cc (right):

https://codereview.chromium.org/453133002/diff/1/mojo/services/html_viewer/we...
mojo/services/html_viewer/weburlloader_impl.cc:70: TypeConverter<URLRequestPtr,
blink::WebURLRequest>::ConvertFrom(request);
nit: URLRequest::From(request)

[Matt Perry, 2014-08-12 18:42:53]

https://codereview.chromium.org/453133002/diff/1/mojo/services/html_viewer/ht...
File mojo/services/html_viewer/html_document_view.cc (right):

https://codereview.chromium.org/453133002/diff/1/mojo/services/html_viewer/ht...
mojo/services/html_viewer/html_document_view.cc:169:
TypeConverter<URLRequestPtr, blink::WebURLRequest>::ConvertFrom(request);
On 2014/08/12 16:23:12, darin wrote:
> You can just use URLRequest::From(request) here.

Done.

https://codereview.chromium.org/453133002/diff/1/mojo/services/html_viewer/we...
File mojo/services/html_viewer/weburlloader_impl.cc (right):

https://codereview.chromium.org/453133002/diff/1/mojo/services/html_viewer/we...
mojo/services/html_viewer/weburlloader_impl.cc:70: TypeConverter<URLRequestPtr,
blink::WebURLRequest>::ConvertFrom(request);
On 2014/08/12 16:23:12, darin wrote:
> nit: URLRequest::From(request)

Done.

[Matt Perry, 2014-08-12 18:42:56]

The CQ bit was checked by mpcomplete@chromium.org

[commit-bot: I haz the power, 2014-08-12 18:44:09]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/mpcomplete@chromium.org/453133002/20001

[commit-bot: I haz the power, 2014-08-12 23:58:55]

FYI, CQ is re-trying this CL (attempt #1).
The failing builders are:
  android_aosp on tryserver.chromium.linux
(http://build.chromium.org/p/tryserver.chromium.linux/builders/android_aosp/bu...)
  win_chromium_x64_rel on tryserver.chromium.win
(http://build.chromium.org/p/tryserver.chromium.win/builders/win_chromium_x64_...)

[commit-bot: I haz the power, 2014-08-13 03:12:57]

Change committed as 289176