Closed Bug 1321644 Opened 3 years ago Closed 3 years ago

Remove calls to AndroidBridge::PostTaskToUiThread from APZ

Categories

(GeckoView :: General, defect)

defect
Not set

Tracking

(firefox53 fixed)

RESOLVED FIXED
mozilla53
Tracking Status
firefox53 --- fixed

People

(Reporter: rbarker, Assigned: rbarker)

References

Details

Attachments

(2 files, 1 obsolete file)

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.
Assignee: nobody → rbarker
Depends on: 1319850
Attachment #8816273 - Flags: review?(nchen)
Attachment #8816273 - Flags: review?(nchen)
Comment on attachment 8816273 [details] [diff] [review]
0001-Bug-1321642-Add-missing-include-guards-to-nsDeviceContextAndroid.h-16120113-1ac30dc.patch

Attached to wrong bug
Attachment #8816273 - Attachment is obsolete: true
Attachment #8816275 - Flags: review?(botond)
Attachment #8816276 - Flags: review?(botond)
Attachment #8816275 - Flags: review?(botond) → review+
Attachment #8816276 - Flags: review?(botond) → review+
Pushed by rbarker@mozilla.com:
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
Flags: needinfo?(rbarker)
Pushed by rbarker@mozilla.com:
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
https://hg.mozilla.org/mozilla-central/rev/98d6700b6e2c
https://hg.mozilla.org/mozilla-central/rev/fb7b4a449847
Status: NEW → RESOLVED
Closed: 3 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 53
Depends on: 1326091
Product: Firefox for Android → GeckoView
Target Milestone: Firefox 53 → mozilla53
You need to log in before you can comment on or make changes to this bug.