Closed
Bug 1283844
Opened 8 years ago
Closed 8 years ago
Convert sensor event to native method
Categories
(Core Graveyard :: Widget: Android, defect)
Tracking
(firefox50 fixed)
RESOLVED
FIXED
mozilla50
Tracking | Status | |
---|---|---|
firefox50 | --- | fixed |
People
(Reporter: jchen, Assigned: jchen)
References
Details
Attachments
(3 files)
15.08 KB,
patch
|
snorp
:
review+
|
Details | Diff | Splinter Review |
18.15 KB,
patch
|
snorp
:
review+
|
Details | Diff | Splinter Review |
6.40 KB,
patch
|
jchen
:
review+
|
Details | Diff | Splinter Review |
Convert SENSOR_EVENT in GeckoEvent to a native method call in GeckoAppShell.
Assignee | ||
Comment 1•8 years ago
|
||
Implement SensorEventListener directly in GeckoAppShell instead of indirectly through GeckoInterface and GeckoApp, because the SensorEventListener consumer is in GeckoAppShell.
Attachment #8768130 -
Flags: review?(snorp)
Assignee | ||
Comment 2•8 years ago
|
||
Remove SENSOR_EVENT from GeckoEvent and implement it as a native method in GeckoAppShell that is invoked by the sensor event listener in GeckoAppShell.
Attachment #8768131 -
Flags: review?(snorp)
Assignee | ||
Comment 3•8 years ago
|
||
Attachment #8768132 -
Flags: review+
Attachment #8768130 -
Flags: review?(snorp) → review+
Attachment #8768131 -
Flags: review?(snorp) → review+
Pushed by nchen@mozilla.com: https://hg.mozilla.org/integration/mozilla-inbound/rev/a77fc8548621 Implement sensor event listener in GeckoAppShell; r=snorp https://hg.mozilla.org/integration/mozilla-inbound/rev/f39dea845b18 Implement SENSOR_EVENT as native call; r=snorp https://hg.mozilla.org/integration/mozilla-inbound/rev/ecc3a30bda03 Update auto-generated bindings; r=me
Comment 5•8 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/a77fc8548621 https://hg.mozilla.org/mozilla-central/rev/f39dea845b18 https://hg.mozilla.org/mozilla-central/rev/ecc3a30bda03
Status: ASSIGNED → RESOLVED
Closed: 8 years ago
status-firefox50:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla50
Updated•3 years ago
|
Product: Core → Core Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•