Bug 527433 - nsIWebContentConverterRegistrar::getContentHandlers count out param should be optional. r=gavin

This commit is contained in:
Dão Gottwald 2009-11-09 16:45:42 +01:00
parent d18cfbef98
commit 16ecfbb336
3 changed files with 4 additions and 3 deletions

View File

@ -134,7 +134,8 @@ interface nsIWebContentConverterService : nsIWebContentHandlerRegistrar
* The content type to get handlers for
* @returns An array of nsIWebContentHandlerInfo objects
*/
void getContentHandlers(in AString contentType, out unsigned long count,
void getContentHandlers(in AString contentType,
[optional] out unsigned long count,
[retval,array,size_is(count)] out nsIWebContentHandlerInfo handlers);
/**

View File

@ -1041,7 +1041,7 @@ FeedWriter.prototype = {
// List of web handlers
var wccr = Cc["@mozilla.org/embeddor.implemented/web-content-handler-registrar;1"].
getService(Ci.nsIWebContentConverterService);
var handlers = wccr.getContentHandlers(this._getMimeTypeForFeedType(feedType), {});
var handlers = wccr.getContentHandlers(this._getMimeTypeForFeedType(feedType));
if (handlers.length != 0) {
for (var i = 0; i < handlers.length; ++i) {
menuItem = this._document.createElementNS(XUL_NS, "menuitem");

View File

@ -661,7 +661,7 @@ FeedHandlerInfo.prototype = {
}
// Add the registered web handlers. There can be any number of these.
var webHandlers = this._converterSvc.getContentHandlers(this.type, {});
var webHandlers = this._converterSvc.getContentHandlers(this.type);
for each (let webHandler in webHandlers)
this._possibleApplicationHandlers.appendElement(webHandler, false);