Closed
Bug 340179
Opened 19 years ago
Closed 19 years ago
Implement Web Applications 1.0 "registerContentHandler"
Categories
(Core :: DOM: Core & HTML, defect, P1)
Core
DOM: Core & HTML
Tracking
()
RESOLVED
FIXED
mozilla1.8.1beta1
People
(Reporter: bugs, Assigned: bugs)
References
()
Details
(Keywords: fixed1.8.1, Whiteboard: [swag:2d])
Attachments
(3 files)
12.93 KB,
patch
|
sicking
:
review+
darin.moz
:
superreview+
|
Details | Diff | Splinter Review |
23.36 KB,
patch
|
darin.moz
:
approval1.8.1+
|
Details | Diff | Splinter Review |
11.97 KB,
patch
|
darin.moz
:
review+
|
Details | Diff | Splinter Review |
To allow other web based feed providers to register to show up in the set of choices, we need to implement registerContentHandler. See URI for details.
Assignee | ||
Comment 1•19 years ago
|
||
This adds a nsIDOMClientInformation interface... also a nsIWebContentHandlerRegistrar interface & contract id that a component elsewhere has to implement... and makes nsNavigator implement the ClientInformation interface.
I'm not so hot on the location of nsIWebContentHandlerRegistrar in the source repository, but it was the best I could think of.
Attachment #224260 -
Flags: review?
Assignee | ||
Updated•19 years ago
|
Attachment #224260 -
Flags: review? → review?(bugmail)
Assignee | ||
Updated•19 years ago
|
Status: NEW → ASSIGNED
Flags: blocking1.8.1+
Priority: -- → P1
Comment on attachment 224260 [details] [diff] [review]
patch
This is missing the actual nsIWebContentHandlerRegistrar implementation. Is that being written in another bug or did you diff the wrong directories?
Or is the idea that that part will be handled by an extension?
r=me on the parts that are here though.
Attachment #224260 -
Flags: review?(bugmail) → review+
Assignee | ||
Comment 3•19 years ago
|
||
It is being implemented in another directory, right now browser/components/feeds, but I'll move it to toolkit once it begins to handle more than just feed types (probably in Firefox 3.0)
Need to circle around with Hixie once mroe and then I'll land this.
Thanks for the review!
Assignee | ||
Updated•19 years ago
|
Whiteboard: [swag:2d]
Assignee | ||
Updated•19 years ago
|
Attachment #224260 -
Flags: superreview?(darin)
Comment 4•19 years ago
|
||
Comment on attachment 224260 [details] [diff] [review]
patch
sr=darin
Attachment #224260 -
Flags: superreview?(darin) → superreview+
Comment 5•19 years ago
|
||
Please land this on trunk, let bake, and request approval-1.8.1 once it's cooled.
Whiteboard: [swag:2d] → [swag:2d] [checkin needed]
Comment 6•19 years ago
|
||
So I have a few questions:
1) Could we add comments to the new interfaces either explaining what the
methods do or pointing to the spec that explains it?
2) Is there a reason we don't implement the MUST about URIs not containing
'%s' from the spec?
3) This contract is expected to be implemented by embeddors, right? Do we
want to use the "embeddor.implemented" thing we were going to use as a
convention for such things at some point (e.g. "@mozilla.org/embeddor.implemented/bookmark-charset-resolver;1" is an example.
We should probably consider creating a CID file for content things like
this too...
Assignee | ||
Comment 7•19 years ago
|
||
bz-
How about:
"@mozilla.org/embeddor.implemented/web-content-handler-registrar;1" ?
I'll make a follow on patch.
As far as processing of the parameters themselves, I had delegated that to the implementation (in browser/). Do you think that should be brought in here? Otherwise I can add a note to the nsIWebContentHandlerRegistrar interface stipulating that the implementor must do this.
I thought the idea was that the interface was implenented by content in browser/? I.e. that no extensions or embeddors were involved in this?
Assignee | ||
Comment 9•19 years ago
|
||
Jonas, Camino may wish to implement this too, depending on how they want to handle feeds.
Assignee | ||
Comment 10•19 years ago
|
||
Attachment #228244 -
Flags: approval1.8.1?
Assignee | ||
Comment 11•19 years ago
|
||
Attachment #228245 -
Flags: review?(darin)
Comment 12•19 years ago
|
||
Comment on attachment 228244 [details] [diff] [review]
patch for branch, incorporating contract id and comments
a=darin on behalf of drivers
Attachment #228244 -
Flags: approval1.8.1? → approval1.8.1+
Updated•19 years ago
|
Attachment #228245 -
Flags: review?(darin) → review+
Assignee | ||
Comment 13•19 years ago
|
||
fixed-on-trunk, fixed-1.8-branch
Status: ASSIGNED → RESOLVED
Closed: 19 years ago
Resolution: --- → FIXED
Updated•19 years ago
|
Keywords: fixed1.8.1
Updated•19 years ago
|
Whiteboard: [swag:2d] [checkin needed] → [swag:2d]
Updated•12 years ago
|
Component: DOM: Mozilla Extensions → DOM
Updated•6 years ago
|
Component: DOM → DOM: Core & HTML
You need to log in
before you can comment on or make changes to this bug.
Description
•