Closed
Bug 1625792
Opened 6 years ago
Closed 6 years ago
Remove nsIURIContentListener.onStartURIOpen.
Categories
(Core :: DOM: Navigation, defect)
Core
DOM: Navigation
Tracking
()
RESOLVED
FIXED
mozilla76
| Tracking | Status | |
|---|---|---|
| firefox76 | --- | fixed |
People
(Reporter: emilio, Assigned: emilio)
Details
Attachments
(2 files)
|
47 bytes,
text/x-phabricator-request
|
Details | Review | |
|
6.47 KB,
patch
|
Details | Diff | Splinter Review |
No description provided.
|
Assignee | |
Comment 1•6 years ago
|
||
|
|
Assignee | |
Comment 2•6 years ago
|
||
It's only used to check whether the docshell is gone already, and we need to
deal with that in other places anyway. No other useful implementation.
In comm-central there are a couple, but the only interesting one is under
suite/, which is not maintained code and broken already.
Pushed by ealvarez@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/d583227c1647
Remove nsIURIContentListener.onStartURIOpen. r=mattwoodrow
|
|
Assignee | |
Comment 4•6 years ago
|
||
PSA: This will bust thunderbird, but you just need to remove the method implementations, I did check them before writing the patch.
Flags: needinfo?(mkmelin+mozilla)
|
|
Assignee | |
Comment 6•6 years ago
|
||
Yup, looks great!
|
||
Comment 7•6 years ago
|
||
| bugherder | ||
Status: NEW → RESOLVED
Closed: 6 years ago
status-firefox76:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla76
Pushed by mkmelin@iki.fi:
https://hg.mozilla.org/comm-central/rev/6401b977f4cb
Remove nsIURIContentListener.onStartURIOpen (comm-central parts) r=emilio DONTBUILD
You need to log in
before you can comment on or make changes to this bug.
Description
•