Closed Bug 635630 Opened 14 years ago Closed 14 years ago

typo in nsWebHandlerApp.js: OPEN_DEFAULT_WINDOW instead of OPEN_DEFAULTWINDOW

Categories

(Core Graveyard :: File Handling, defect)

x86_64
Linux
defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED
mozilla8

People

(Reporter: arno, Assigned: arno)

Details

Attachments

(1 file)

Hi, in uriloader/exthandler/nsWebHandlerApp.js, we have // openURI browserDOMWin.openURI(uriToSend, null, // no window.opener Ci.nsIBrowserDOMWindow.OPEN_DEFAULT_WINDOW, Ci.nsIBrowserDOMWindow.OPEN_NEW); But nsIBrowserDOMWindow.idl says it should be OPEN_DEFAULTWINDOW (without the second underscore). This is problaby harmless because nsIBrowserDOMWindow.OPEN_DEFAULTWINDOW because is 0
Component: General → File Handling
QA Contact: general → file-handling
Attached patch fix typoSplinter Review
Assignee: nobody → arno
Attachment #546306 - Flags: review?(gavin.sharp)
Attachment #546306 - Flags: review?(gavin.sharp) → review+
Keywords: checkin-needed
Status: NEW → RESOLVED
Closed: 14 years ago
Keywords: checkin-needed
Resolution: --- → FIXED
Target Milestone: --- → mozilla8
Product: Core → Core Graveyard
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: