SeaMonkey part for Add newChannel2 to nsIProtocolHandler (Bug 1067471) and Extend NewChannel() with loadinfo argument in nsIAboutModule (Bug 1067468)

RESOLVED FIXED in seamonkey2.33

Status

SeaMonkey
General
RESOLVED FIXED
3 years ago
3 years ago

People

(Reporter: Philip Chee, Assigned: Philip Chee)

Tracking

Trunk
seamonkey2.33

SeaMonkey Tracking Flags

(seamonkey2.33 fixed)

Details

Attachments

(1 attachment)

(Assignee)

Description

3 years ago
Relevant Firefox/Gecko bugs:
Bug 1067471 - Add newChannel2 to nsIProtocolHandler that takes a loadinfo as an argument
Bug 1067468 - Extend NewChannel() with loadinfo argument in nsIAboutModule
(Assignee)

Comment 1

3 years ago
Created attachment 8511134 [details] [diff] [review]
Patch v1.0 Proposed fix.

More fallout from mozilla-central. This patch may not be necessary.
Attachment #8511134 - Flags: review?(neil)

Comment 2

3 years ago
Comment on attachment 8511134 [details] [diff] [review]
Patch v1.0 Proposed fix.

I suppose at some point we'll have to do something with that new parameter...
Attachment #8511134 - Flags: review?(neil) → review+
(Assignee)

Comment 3

3 years ago
(In reply to neil@parkwaycc.co.uk from comment #2)
> Comment on attachment 8511134 [details] [diff] [review]
> Patch v1.0 Proposed fix.
> 
> I suppose at some point we'll have to do something with that new parameter...

That appears to be the plan over in Mozilla-central.
(Assignee)

Comment 4

3 years ago
comm-central changeset e8f0a5702a90
Pushed to comm-central http://hg.mozilla.org/comm-central/rev/e8f0a5702a90
Status: ASSIGNED → RESOLVED
Last Resolved: 3 years ago
status-seamonkey2.33: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → seamonkey2.33
Comment on attachment 8511134 [details] [diff] [review]
Patch v1.0 Proposed fix.

Review of attachment 8511134 [details] [diff] [review]:
-----------------------------------------------------------------

::: suite/feeds/src/FeedConverter.js
@@ +509,5 @@
>      return uri;
>    },
>  
>    newChannel: function newChannel(aUri) {
> +    return this.newChannel2(inputURI, null);

Too much copy&paste. Now selecting a feed from the feeds icon to the right of the location bar throws an error:

Error: ReferenceError: inputURI is not defined
Source File: resource://gre/components/FeedConverter.js
Line: 513
(Assignee)

Comment 6

3 years ago
(In reply to Jens Hatlak (:InvisibleSmiley) from comment #5)
> Too much copy&paste. Now selecting a feed from the feeds icon to the right
> of the location bar throws an error:

I pushed a bustage fix to comm-central and comm-aurora
http://hg.mozilla.org/comm-central/rev/941971dc9aec
http://hg.mozilla.org/releases/comm-aurora/rev/a9f68b06d1b4
You need to log in before you can comment on or make changes to this bug.