Closed
Bug 1343758
Opened 7 years ago
Closed 7 years ago
gfx-labeling Label runnables in gfx vr.
Categories
(Core :: Graphics, enhancement)
Core
Graphics
Tracking
()
RESOLVED
FIXED
mozilla55
Tracking | Status | |
---|---|---|
firefox55 | --- | fixed |
People
(Reporter: vliu, Assigned: kechen)
References
Details
(Whiteboard: [gfx-noted][QDL][TDC-MVP][GFX])
Attachments
(1 file)
See https://wiki.mozilla.org/Quantum/DOM#Labeling for the story. Currently runnable categories in gfx vr include sending ipc message between content and parent side [1]. ./gfx/vr/ipc/PVRLayer.ipdl ./gfx/vr/ipc/PVRManager.ipdl [1]; https://hg.mozilla.org/mozilla-central/annotate/e150eaff1f83e4e4a97d1e30c57d233859efe9cb/ipc/chromium/src/base/message_loop.cc#l286
Reporter | ||
Updated•7 years ago
|
Blocks: gfx-labeling
Updated•7 years ago
|
Whiteboard: [gfx-noted]
Assignee | ||
Updated•7 years ago
|
Assignee: nobody → kechen
Comment hidden (mozreview-request) |
Comment 2•7 years ago
|
||
mozreview-review |
Comment on attachment 8850356 [details] Bug 1343758 - Label the tasks in vr module. https://reviewboard.mozilla.org/r/122986/#review125238 ::: gfx/vr/VRDisplayPresentation.cpp:78 (Diff revision 1) > > - RefPtr<VRLayerChild> vrLayer = static_cast<VRLayerChild*>(manager->CreateVRLayer(mDisplayClient->GetDisplayInfo().GetDisplayID(), leftBounds, rightBounds)); > + nsCOMPtr<nsIEventTarget> target; > + nsIDocument* doc; > + doc = canvasElement->OwnerDoc(); > + if (doc) { > + target = doc->GetDocGroup()->EventTargetFor(TaskCategory::Other); nit: target = doc->EventTargetFor(TaskCategory::Other);
Attachment #8850356 -
Flags: review?(btseng) → review+
Comment hidden (mozreview-request) |
Assignee | ||
Comment 4•7 years ago
|
||
VRManagerChild doesn't belong to any tab or document, it is bound with process like CompositorBridgeChild. The current mechanism can not support this kind of ipc so I won't label it in this bug.
Comment 5•7 years ago
|
||
mozreview-review |
Comment on attachment 8850356 [details] Bug 1343758 - Label the tasks in vr module. https://reviewboard.mozilla.org/r/122986/#review126026 r=me with nit from bevistseng
Attachment #8850356 -
Flags: review?(kgilbert) → review+
Pushed by jacheng@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/ccf27d7cdcdc Label the tasks in vr module. r=bevistseng,kip
Comment 7•7 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/ccf27d7cdcdc
Status: NEW → RESOLVED
Closed: 7 years ago
status-firefox55:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla55
Updated•7 years ago
|
Whiteboard: [gfx-noted] → [gfx-noted][QDL][TDC-MVP][GFX]
You need to log in
before you can comment on or make changes to this bug.
Description
•