1405081 - [geckoview] Add high-level fetch API

Status: RESOLVED FIXED

(GeckoView :: General, enhancement, P2)

Description

[James Willcox (:snorp) ([e-mail]) (he/him)]

There are some cases where we would like to fetch a resource through Gecko from Java. AFAIK we have never had a great way to do this in Fennec, but such an interface makes sense alongside GeckoView. We probably want this in order to re-fetch web app manifest or launcher icons, for instance.

Comment 1

[James Willcox (:snorp) ([e-mail]) (he/him)]

Attachment: Bug 1405081 - Web executor wip

Comment 2

[James Willcox (:snorp) ([e-mail]) (he/him)]

Nick, attached is a rough first pass at this. The patch has a lot of problems, but does work for the small smoke test. The interesting parts are WebRequest, WebResponse, WebExecutor, and GeckoWebExecutor.

NI just to make sure it doesn't get lost in your bugmail.

Comment 3

[James Willcox (:snorp) ([e-mail]) (he/him)]

Attachment: Bug 1405081 - Move GeckoRuntime creation out of GeckoSesionTestRule r?jchen

This allows us to easily share a GeckoRuntime with other tests that
don't use GeckoSessionTestRule.

Comment 4

[James Willcox (:snorp) ([e-mail]) (he/him)]

Attachment: Bug 1405081 - Add WebRequestError and use it in NavigationDelegate.onLoadError r?esawin,jchen

Depends on D7796

Comment 5

[James Willcox (:snorp) ([e-mail]) (he/him)]

Attachment: Bug 1405081 - Move GeckoSessionTestRule.waitForResult() into UiThreadUtils r?jchen

Depends on D7797

Comment 6

[James Willcox (:snorp) ([e-mail]) (he/him)]

Attachment: Bug 1405081 - Add WebRequest, WebResponse, and WebExecutor to GeckoView r?jchen,esawin

This provides a Fetch-like API for GeckoView apps.

Depends on D7798

Comment 7

[James Willcox (:snorp) ([e-mail]) (he/him)]

Attachment: Bug 1405081 - Move default timeout log into Environment utility class r?jchen

Depends on D7798

Comment 8

[James Willcox (:snorp) ([e-mail]) (he/him)]

Attachment: Bug 1405081 - Add partial HttpBin implementation for local HTTP tests r?esawin

This was derived from work done by Andrew Gaul <[e-mail]> for
the Chaos HTTP proxy.

Comment 9

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/mozilla-inbound/rev/017c51559d4a
Move GeckoRuntime creation out of GeckoSesionTestRule r=jchen
https://hg.mozilla.org/integration/mozilla-inbound/rev/5fda5eb5fd01
Add WebRequestError and use it in NavigationDelegate.onLoadError r=esawin,jchen
https://hg.mozilla.org/integration/mozilla-inbound/rev/778ef7a62d5d
Move GeckoSessionTestRule.waitForResult() into UiThreadUtils r=jchen
https://hg.mozilla.org/integration/mozilla-inbound/rev/e24c0262f83e
Move default timeout logic into Environment utility class r=jchen
https://hg.mozilla.org/integration/mozilla-inbound/rev/a768da5d9f82
Add partial HttpBin implementation for local HTTP tests r=esawin
https://hg.mozilla.org/integration/mozilla-inbound/rev/91f7f603954c
Add WebRequest, WebResponse, and GeckoWebExecutor to GeckoView r=jchen,esawin

Comment 10

[Daniel Varga [:dvarga]]

https://hg.mozilla.org/mozilla-central/rev/017c51559d4a
https://hg.mozilla.org/mozilla-central/rev/5fda5eb5fd01
https://hg.mozilla.org/mozilla-central/rev/778ef7a62d5d
https://hg.mozilla.org/mozilla-central/rev/e24c0262f83e
https://hg.mozilla.org/mozilla-central/rev/a768da5d9f82
https://hg.mozilla.org/mozilla-central/rev/91f7f603954c

Comment 11

[Chris Peterson [:cpeterson]]

64=wontfix because we don't need to uplift this fix for Focus 8.0's GV 64 beta test

Comment 12

[Chris Peterson [:cpeterson]]

status-geckoview64=wontfix

Comment 13

[Nick Alexander :nalexander [he/him]]

I'm embarrassed to say that I didn't provide feedback on this in a timely manner.  Sorry, all.