Summary: | [iOS] Calls to the device motion API should be done in the UI process | ||||||||||||||
---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Per Arne Vollan <pvollan> | ||||||||||||
Component: | WebKit Misc. | Assignee: | Per Arne Vollan <pvollan> | ||||||||||||
Status: | RESOLVED FIXED | ||||||||||||||
Severity: | Normal | CC: | achristensen, bfulgham, cdumez, commit-queue, dbates, esprehn+autocc, ews-watchlist, ggaren, kangil.han, webkit-bug-importer | ||||||||||||
Priority: | P2 | Keywords: | InRadar | ||||||||||||
Version: | WebKit Nightly Build | ||||||||||||||
Hardware: | Unspecified | ||||||||||||||
OS: | Unspecified | ||||||||||||||
Bug Depends on: | 209831 | ||||||||||||||
Bug Blocks: | 206085, 206387 | ||||||||||||||
Attachments: |
|
Description
Per Arne Vollan
2019-12-02 15:09:19 PST
Created attachment 384662 [details]
Patch
Created attachment 384670 [details]
Patch
Created attachment 384673 [details]
Patch
Created attachment 385207 [details]
Patch
Created attachment 385285 [details]
Patch
Comment on attachment 385285 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=385285&action=review r=me, but please consider changing those bare pointers (perhaps even in a follow-up patch). > Source/WebCore/platform/ios/DeviceMotionClientIOS.h:55 > + DeviceMotionController* m_controller { nullptr }; I'm a little suspicious of these bare pointers. Could they be WeakPtrs? (In reply to Brent Fulgham from comment #7) > Comment on attachment 385285 [details] > Patch > > View in context: > https://bugs.webkit.org/attachment.cgi?id=385285&action=review > > r=me, but please consider changing those bare pointers (perhaps even in a > follow-up patch). > > > Source/WebCore/platform/ios/DeviceMotionClientIOS.h:55 > > + DeviceMotionController* m_controller { nullptr }; > > I'm a little suspicious of these bare pointers. Could they be WeakPtrs? I think they can be made WeakPtrs, I will look into that :) Thanks for reviewing! (In reply to Brent Fulgham from comment #7) > Comment on attachment 385285 [details] > Patch > > View in context: > https://bugs.webkit.org/attachment.cgi?id=385285&action=review > > r=me, but please consider changing those bare pointers (perhaps even in a > follow-up patch). > > > Source/WebCore/platform/ios/DeviceMotionClientIOS.h:55 > > + DeviceMotionController* m_controller { nullptr }; > > I'm a little suspicious of these bare pointers. Could they be WeakPtrs? Filed https://bugs.webkit.org/show_bug.cgi?id=205093. Comment on attachment 385285 [details] Patch Clearing flags on attachment: 385285 Committed r253357: <https://trac.webkit.org/changeset/253357> All reviewed patches have been landed. Closing bug. Comment on attachment 385285 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=385285&action=review > Source/WebCore/ChangeLog:3 > + [iOS] Calls to the device motion API should be done in the UI process This change seems to have badly broken the device motion API :( see <rdar://problem/60720953>. |