1321644 - Remove calls to AndroidBridge::PostTaskToUiThread from APZ

Status: RESOLVED FIXED

(GeckoView :: General, defect)

Description

[Randall Barker [:rbarker]]

Once GeckoView support nsThread and MessageLoop in the Android UI thread, it will no longer be necessary to have code that calls AndroidBridge::PostTaskToUiThread for dispatching to the UI/Controller thread.

Comment 1

[Randall Barker [:rbarker]]

Attachment: 0001-Bug-1321642-Add-missing-include-guards-to-nsDeviceContextAndroid.h-16120113-1ac30dc.patch

Comment 2

[Randall Barker [:rbarker]]

Comment on attachment 8816273 [details] [diff] [review]
0001-Bug-1321642-Add-missing-include-guards-to-nsDeviceContextAndroid.h-16120113-1ac30dc.patch

Attached to wrong bug

Comment 3

[Randall Barker [:rbarker]]

Attachment: 0001-Bug-1321644-part-1-Remove-android-specific-message-loop-code-from-APZThreadUtils.cpp-16120113-a515cc6.patch

Comment 4

[Randall Barker [:rbarker]]

Attachment: 0002-Bug-1321644-part-2-Remove-AndroidBridge-PostTaskToUiThread-from-RemoteContentController.cpp-16120113-1474d0a.patch

Comment 5

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/mozilla-inbound/rev/c25bce77b775
part 1, Remove android specific message loop code from APZThreadUtils.cpp r=botond
https://hg.mozilla.org/integration/mozilla-inbound/rev/60ba1893ecf3
part 2, Remove AndroidBridge::PostTaskToUiThread from RemoteContentController.cpp r=botond

Comment 6

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

I had to back this (and the other bug from your push) out for android xpcshell bustage like https://treeherder.mozilla.org/logviewer.html#?job_id=40966554&repo=mozilla-inbound

https://hg.mozilla.org/integration/mozilla-inbound/rev/f0840950a19b8d610f4974d8f2d908866bfa7493

Comment 7

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/mozilla-inbound/rev/98d6700b6e2c
part 1, Remove android specific message loop code from APZThreadUtils.cpp r=botond
https://hg.mozilla.org/integration/mozilla-inbound/rev/fb7b4a449847
part 2, Remove AndroidBridge::PostTaskToUiThread from RemoteContentController.cpp r=botond

Comment 8

[Phil Ringnalda (:philor)]

https://hg.mozilla.org/mozilla-central/rev/98d6700b6e2c
https://hg.mozilla.org/mozilla-central/rev/fb7b4a449847