Closed Bug 1515646 Opened 5 years ago Closed 5 years ago

Implement named targetting using BrowsingContext

Categories

(Core :: DOM: Navigation, enhancement, P2)

enhancement

Tracking

()

RESOLVED FIXED
mozilla67
Fission Milestone M1
Tracking Status
firefox67 --- fixed

People

(Reporter: farre, Assigned: farre)

References

Details

Attachments

(2 files)

      No description provided.
Assignee: nobody → afarre
Status: NEW → ASSIGNED
Priority: -- → P2
Try: https://treeherder.mozilla.org/#/jobs?repo=try&revision=74ff46ae21342d9813ffc3fea7c6d7b169314d16

Try won't really test that much, that's why I'm only running mochitests, where I've added new tests.
This implements the step of choosing a browsing context with
FindWithName, which should be equivalent to calling
nsIDocShellTreeItem.findItemWithName passing null for 'aRequestor' and
'aOriginalRequestor' and false for 'aSkipTabGroup'.
Test that BrowsingContext::{FindWithName, FindChildWithName} finds the
expected BrowsingContexts and also that BrowsingContext::FindWithName
finds the same browsing contexts as if
nsIDocShellTreeItem::FindItemWithName had been used.

Depends on D15190
Attachment #9032909 - Attachment description: Bug 1515646 - Add FindWithName and FindChildWithName to BrowsingContext. r=peterv → Bug 1515646 - Add FindWithName and FindChildWithName to BrowsingContext.
Attachment #9032910 - Attachment description: Bug 1515646 - Test BrowsingContext::{FindWithName, FindChildWithName}. r=peterv → Bug 1515646 - Test BrowsingContext::FindWithName.
Fission Milestone: --- → M1
Whiteboard: [1/30] waiting for review
Whiteboard: [1/30] waiting for review → [2/14] revised patches waiting for review
Whiteboard: [2/14] revised patches waiting for review → [2/14] patches r+
Pushed by afarre@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/9243ddacadcc
Add FindWithName and FindChildWithName to BrowsingContext. r=peterv
Pushed by afarre@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/996e8765c85a
Test BrowsingContext::FindWithName. r=peterv
Status: ASSIGNED → RESOLVED
Closed: 5 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla67
Whiteboard: [2/14] patches r+
Regressions: 1554715
Regressions: 1555287
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: