Open Bug 1607795 Opened 5 years ago Updated 2 years ago

Remove workaround from node actor introduced from bug 1598022

Categories

(DevTools :: Inspector, task, P2)

Product:
DevTools
Component:
Inspector
Type:
task

Tracking

(Fission Milestone:Future)

Project Flags:
Fission Milestone Future

People

(Reporter: ochameau, Unassigned)

References

Details

(Whiteboard: dt-fission-future)

Bug 1598022, patch https://phabricator.services.mozilla.com/D53985#change-vqg7QtjswpNC introduced the following check:

    if (Cu.getObjectPrincipal(global) == Cu.getObjectPrincipal(dbg)) {
      dump("Ignored system principal in getEmbedderElement!\n");
      return undefined;
    }

which should be removed once we have tab descriptors (bug 1579042) and we can have pure BrowsingContextTargetFront for tabs instead a special one just for local tabs.

Priority: -- → P2
Whiteboard: dt-fission-m2-mvp

Tracking Fission DevTools bugs for Fission Nightly (M6) milestone

Fission Milestone: --- → M6

Tracking dt-fission-m2-mvp bugs for Fission Nightly milestone (M6c)

Fission Milestone: M6 → M6c
Depends on: 1618693

This warning should be removed when we get a chance to fix bug 1618693, which is clearly a code debt cleanup.

Whiteboard: dt-fission-m2-mvp → dt-fission-m2-reserve

Bulk move of all dt-fission-m2-reserve bugs to Fission MVP milestone.

Fission Milestone: M6c → MVP

Moving old "dt-fission-m2-reserve" bugs to "dt-fission-future" because they don't block Fission MVP.

Fission Milestone: MVP → Future
Whiteboard: dt-fission-m2-reserve → dt-fission-future
Severity: normal → S3
You need to log in before you can comment on or make changes to this bug.