Closed
Bug 1579052
Opened 5 years ago
Closed 5 years ago
Markup view uses currentTarget walker, instead of nodeFront walker
Categories
(DevTools :: Inspector, task, P1)
DevTools
Inspector
Tracking
(firefox71 fixed)
RESOLVED
FIXED
Firefox 71
| Tracking | Status | |
|---|---|---|
| firefox71 | --- | fixed |
People
(Reporter: yulia, Assigned: yulia)
References
(Blocks 1 open bug)
Details
(Whiteboard: dt-fission-m1)
Attachments
(1 file)
|
(deleted),
text/x-phabricator-request
|
Details |
At the moment, in the markup view we use the inspector.walker walker, which is the currentTarget walker. However we use this for accessing information about nodes which might be remote. We should instead use the nodeFront's walkerFront
|
Assignee | |
Updated•5 years ago
|
Blocks: dt-fission-inspector
Priority: -- → P1
|
|
Assignee | |
Updated•5 years ago
|
Summary: Use relative walker for markup view → Markup view uses currentTarget walker, instead of nodeFront walker
|
|
Assignee | |
Comment 1•5 years ago
|
||
|
||
Updated•5 years ago
|
Attachment #9090694 -
Attachment description: Bug 1579052 - use relative nodeFront walker in markup view; r=pbro → Bug 1579052 - use relative nodeFront walker in markup view; r=rcaliman
|
||
Updated•5 years ago
|
Assignee: nobody → ystartsev
Status: NEW → ASSIGNED
Whiteboard: dt-fission
|
|
||
Updated•5 years ago
|
Attachment #9090694 -
Attachment description: Bug 1579052 - use relative nodeFront walker in markup view; r=rcaliman → Bug 1579052 - use relative nodeFront walker in markup view; r=pbro
Pushed by ystartsev@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/0eb8f05cf5b6
use relative nodeFront walker in markup view; r=rcaliman
|
||
Comment 3•5 years ago
|
||
| bugherder | ||
Status: ASSIGNED → RESOLVED
Closed: 5 years ago
status-firefox71:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → Firefox 71
|
||
Updated•5 years ago
|
|
||
Updated•5 years ago
|
Whiteboard: dt-fission → dt-fission dt-fission-m1
|
|
||
Updated•3 years ago
|
Whiteboard: dt-fission dt-fission-m1 → dt-fission-m1
You need to log in
before you can comment on or make changes to this bug.
Description
•