1328990 - Convert FormAssistPopup events to bundle events

Status: RESOLVED FIXED

(GeckoView :: General, defect)

Comment 1

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

Attachment: Convert FormAssistPopup events to bundle events (v1)

Convert "FormAssist:*" events in FormAssistPopup to
GeckoBundle/BundleEventListener events.  UI thread events are used
because the listener performs operations on the UI thread.

Also convert the "FormAssist:*" observers in browser.js to use
WindowEventDispatcher.

The "FormAssist:AutoComplete" event that goes from Gecko to Java is
renamed to "FormAssist:AutoCompleteResult" to prevent conflict with the
event with the same name that goes from Java to Gecko.

Comment 2

[Sebastian Kaspari (:sebastian; :pocmo)]

Comment on attachment 8826013 [details] [diff] [review]
Convert FormAssistPopup events to bundle events (v1)

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

::: mobile/android/base/java/org/mozilla/gecko/FormAssistPopup.java
@@ -266,5 @@
> -            mH = rect.getDouble("h");
> -        } catch (JSONException e) {
> -            // Bail if we can't get the correct dimensions for the popup.
> -            Log.e(LOGTAG, "Error getting FormAssistPopup dimensions", e);
> -            return false;

I wonder if we are going to see some new crashes because we almost always caught (not only here) and logged JSON errors. So we never really saw if there are issues here in the wild - but let's fix them as they come  up.

Comment 3

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/mozilla-inbound/rev/2823e4d46cf5
Convert FormAssistPopup events to bundle events; r=sebastian

Comment 4

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/mozilla-central/rev/2823e4d46cf5