1325155 - Convert text selection events to bundle events

Status: RESOLVED FIXED

(GeckoView :: General, defect)

Comment 1

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

Attachment: 1. Add Object overrides for GeckoBundle (v1)

Add overrides for Object methods in GeckoBundle, and redirected them to
the corresponding methods in SimpleArrayMap. For equals, a custom
implementation is used because the equals implementation in
SimpleArrayMap had bugs across different support library versions.

Comment 2

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

Attachment: 2. Convert text selection events to bundle events (v1)

Convert the "TextSelection:*" events to use BundleEventListener /
GeckoBundle, in both ActionBarTextSelection.java and
FloatingToolbarTextSelection.java. UI events are used because the
listeners require the UI thread. The "TextSelection:Update" listeners
are removed because the event is no longer sent by ActionBarHandler.

Comment 3

[Sebastian Kaspari (:sebastian; :pocmo)]

Comment on attachment 8820828 [details] [diff] [review]
1. Add Object overrides for GeckoBundle (v1)

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

I wonder if it makes sense to add a junit4 test for GeckoBundle? Should be pretty straight forward.

::: mobile/android/geckoview/src/main/java/org/mozilla/gecko/util/GeckoBundle.java
@@ +663,5 @@
> +    }
> +
> +    @Override // Object
> +    public String toString() {
> +        return mMap.toString();

Ah, that's a good thing to have. I recently wrote this locally too, for debugging.

> public String toString() {
>     return "GeckoBundle[" + mMap.toString() + "]";
> }

Comment 4

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

(In reply to Sebastian Kaspari (:sebastian) from comment #3)
> Comment on attachment 8820828 [details] [diff] [review]
> 1. Add Object overrides for GeckoBundle (v1)
> 
> Review of attachment 8820828 [details] [diff] [review]:
> -----------------------------------------------------------------
> 
> I wonder if it makes sense to add a junit4 test for GeckoBundle? Should be
> pretty straight forward.

Filed bug 1326101; would be a nice intern project.

Comment 5

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/mozilla-inbound/rev/58e189739172
1. Add Object overrides for GeckoBundle; r=sebastian
https://hg.mozilla.org/integration/mozilla-inbound/rev/2a43ca16d1c1
2. Convert text selection events to bundle events; r=sebastian

Comment 6

[Wes Kocher (:KWierso) (Not reading bugmail; email directly if needed)]

https://hg.mozilla.org/mozilla-central/rev/58e189739172
https://hg.mozilla.org/mozilla-central/rev/2a43ca16d1c1