Closed Bug 1111013 Opened 9 years ago Closed 9 years ago

Add aOpenParams.ignoreQueryString to switchToTabHavingURI to ignore query params without replacing

Categories

(Firefox :: General, defect)

defect
Not set
normal
Points:
2

Tracking

()

RESOLVED FIXED
Firefox 37
Iteration:
37.2
Tracking Status
firefox35 --- fixed
firefox36 --- fixed
firefox37 --- fixed

People

(Reporter: MattN, Assigned: MattN)

References

Details

Attachments

(1 file, 1 obsolete file)

Sometimes you want to ignore query parameters for switchToTabHavingURI without replacing them.
Flags: qe-verify-
Flags: firefox-backlog+
Attached file MozReview Request: bz://1111013/MattN (obsolete) —
Attachment #8535887 - Flags: review?(dao)
/r/1433 - Bug 1111013 - Add aOpenParams.ignoreQueryString to switchToTabHavingURI to ignore query params without replacing. r=dao

Pull down this commit:

hg pull review -r b12d87070e7fe4a08735aa7491302f3737a6debc
https://reviewboard.mozilla.org/r/1431/#review877

::: browser/base/content/test/general/browser_bug1025195_switchToTabHavingURI_aOpenParams.js
(Diff revision 1)
> +/* This Source Code Form is subject to the terms of the Mozilla Public

Note that this file was just renamed but mozreview isn't showing it properly.
Attachment #8535887 - Flags: review+
Comment on attachment 8535887 [details]
MozReview Request: bz://1111013/MattN

(Yoinking review since there's schedule pressure to land this ASAP.)
Attachment #8535887 - Flags: review?(dao)
https://hg.mozilla.org/integration/fx-team/rev/e6fdd2e17262
Flags: in-testsuite+
Whiteboard: [fixed-in-fx-team]
Comment on attachment 8535887 [details]
MozReview Request: bz://1111013/MattN

[Triage Comment]

No API change and shouldn't alter existing functionality, needed for Hello UI tour in Fx35.
Attachment #8535887 - Flags: approval-mozilla-beta+
Attachment #8535887 - Flags: approval-mozilla-aurora+
https://hg.mozilla.org/mozilla-central/rev/e6fdd2e17262
Status: ASSIGNED → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
Whiteboard: [fixed-in-fx-team]
Target Milestone: --- → Firefox 37
Attachment #8535887 - Attachment is obsolete: true
Attachment #8618916 - Flags: review+
You need to log in before you can comment on or make changes to this bug.