1346923 - Change PVRManager::GetDisplays to async

Status: RESOLVED FIXED

(Core :: WebVR, enhancement)

Comment 1

[:kip (Kearwood Gilbert)]

Attachment: Bug 1346923 - Remove PVRManager::GetDisplays sync IPC

- PVRManager::GetDisplays was a sync IPC that was part of an optimization
  for a use case that has been eliminated by changes to the WebVR spec.
- This was an optimization for Navigator.activeVRDisplays that would allow
  enumeration of displays active in any content process without powering
  on any additional VR hardware.  This will no longer be necessary as the
  activeVRDisplays has been restricted to returning only the displays
  active in the current javascript context.

Review commit: https://reviewboard.mozilla.org/r/120756/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/120756/

Comment 2

[Daosheng Mu[:daoshengmu]]

Comment on attachment 8847842 [details]
Bug 1346923 - Remove PVRManager::GetDisplays sync IPC

https://reviewboard.mozilla.org/r/120756/#review122754

LGTM

Comment 3

[Kan-Ru Chen [:kanru] (UTC+9)]

Comment on attachment 8847842 [details]
Bug 1346923 - Remove PVRManager::GetDisplays sync IPC

https://reviewboard.mozilla.org/r/120756/#review123718

Comment 4

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/65c5873eef04
Remove PVRManager::GetDisplays sync IPC r=daoshengmu,kanru

Comment 5

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

https://hg.mozilla.org/mozilla-central/rev/65c5873eef04