Closed
Bug 102154
Opened 23 years ago
Closed 23 years ago
browser.xml refers to bogus property nsIWebNavigation.LOAD_FLAGS_BYPASS_NONE
Categories
(Core :: XUL, defect)
Tracking
()
VERIFIED
FIXED
People
(Reporter: bugzilla, Assigned: jag+mozilla)
Details
Attachments
(1 file)
1021 bytes,
patch
|
rginda
:
review+
|
Details | Diff | Splinter Review |
Rightclicking on a link and choosing open in a new tab produces: Warning: reference to undefined property nsIWebNavigation.LOAD_FLAGS_BYPASS_NONE Source File: chrome://global/content/bindings/browser.xml#browser.loadURI() Line: 2 You should turn on javascript strict warnings in the "Edit -> Prefs -> Debug" so we could avoid all the strict warning fixup...!
Comment 1•23 years ago
|
||
-> xpt
Assignee: asa → hyatt
Component: Browser-General → XP Toolkit/Widgets
QA Contact: doronr → jrgm
Comment 2•23 years ago
|
||
Hmm, looks like this one should definitely be cleaned up. There is indeed no such property 'LOAD_FLAGS_BYPASS_NONE' on the nsIWebNavigation interface, which means we are passing 'undefined' as the flags value in those calls to nsIWebNavigation. That's not good. http://lxr.mozilla.org/seamonkey/search?string=LOAD_FLAGS_BYPASS_NONE
Assignee: hyatt → jaggernaut
Updated•23 years ago
|
Summary: javascript strict warnings in browser.xml → browser.xml refers to bogus property nsIWebNavigation.LOAD_FLAGS_BYPASS_NONE
Assignee | ||
Comment 3•23 years ago
|
||
Comment 4•23 years ago
|
||
Comment on attachment 51436 [details] [diff] [review] Simple fix. r=rginda
Attachment #51436 -
Flags: review+
Comment 5•23 years ago
|
||
checked in, rev 1.7, marking fixed.
Status: NEW → RESOLVED
Closed: 23 years ago
Resolution: --- → FIXED
Assignee | ||
Comment 6•23 years ago
|
||
Thanks John.
You need to log in
before you can comment on or make changes to this bug.
Description
•