1322581 - [geckoview] Move prompt-related delegate function from ChromeDelegate into PromptDelegate

Status: RESOLVED FIXED

(Core Graveyard :: Embedding: APIs, defect)

Description

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

And hook them up to Gecko.

Comment 1

[(inactive) Jim Chen [:jchen] [:darchons]]

Attachment: 1. Remove GeckoView.ChromeDelegate (v1)

Remove the old ChromeDelegate interface, to be replaced by
PromptDelegate in a future patch.

Comment 2

[(inactive) Jim Chen [:jchen] [:darchons]]

Attachment: 2. Add PromptDelegate interface (v1)

Add a PromptDelegate interface that implements possible prompts shown by
a GeckoView application. All prompt methods include a callback parameter
for the implementation to call back to GeckoView with results from the
prompt.

Comment 3

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

Comment on attachment 8854548 [details] [diff] [review]
2. Add PromptDelegate interface (v1)

Review of attachment 8854548 [details] [diff] [review]:
-----------------------------------------------------------------

I can't really tell of this is a complete interface for everything we need to expose, but it's certainly a good start.

Comment 4

[(inactive) Jim Chen [:jchen] [:darchons]]

Attachment: 3. Add event listener for GeckoView:Prompt event (v1)

Add an event listener for the GeckoView:Prompt event, which JS code will
use to sent over prompt requests and to receive prompt results. Both
global and per-GeckoView listeners are required because we may not know
the origin GeckoView for certain prompts, so some prompts will not have
an associated GeckoView. This is also the reason for having a static
default PromptDelegate in additional to an instance per-GeckoView
PromptDelegate. All prompts without associated GeckoViews are sent
directly to the default PromptDelegate.

Comment 5

[(inactive) Jim Chen [:jchen] [:darchons]]

Attachment: 4. Add JS GeckoView prompt implementation (v1)

Add the necessary XPCOM components to handle prompts for GeckoView. The
JS code mostly package the prompts into GeckoView:Prompt events, and send
them to the Java side if in parent process, or to the parent process if
in child process.

Comment 6

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

Comment on attachment 8856579 [details] [diff] [review]
4. Add JS GeckoView prompt implementation (v1)

Review of attachment 8856579 [details] [diff] [review]:
-----------------------------------------------------------------

It may be a good idea to find a better reviewer for GeckoViewPrompt.js, but anything is better than nothing right now.

Comment 7

[(inactive) Jim Chen [:jchen] [:darchons]]

Attachment: 5. Add PromptDelegate implementation to GeckoView example (v1)

Add a sample implementation of PromptDelegate to GeckoView example.

Comment 8

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/mozilla-inbound/rev/b408d1666080
1. Remove GeckoView.ChromeDelegate; r=snorp
https://hg.mozilla.org/integration/mozilla-inbound/rev/1d518d531034
2. Add PromptDelegate interface; r=snorp
https://hg.mozilla.org/integration/mozilla-inbound/rev/cb346aa4c851
3. Add event listener for GeckoView:Prompt event; r=snorp
https://hg.mozilla.org/integration/mozilla-inbound/rev/a03fcccb6400
4. Add JS GeckoView prompt implementation; r=snorp
https://hg.mozilla.org/integration/mozilla-inbound/rev/13fd2a40e7d6
5. Add PromptDelegate implementation to GeckoView example; r=snorp

Comment 9

[Carsten Book [:Tomcat]]

https://hg.mozilla.org/mozilla-central/rev/b408d1666080
https://hg.mozilla.org/mozilla-central/rev/1d518d531034
https://hg.mozilla.org/mozilla-central/rev/cb346aa4c851
https://hg.mozilla.org/mozilla-central/rev/a03fcccb6400
https://hg.mozilla.org/mozilla-central/rev/13fd2a40e7d6