Closed
Bug 356225
Opened 19 years ago
Closed 19 years ago
Duplicate if check in nsWebBrowserPersist.cpp
Categories
(Core Graveyard :: File Handling, defect)
Core Graveyard
File Handling
Tracking
(Not tracked)
RESOLVED
FIXED
mozilla1.9alpha1
People
(Reporter: sciguyryan, Assigned: sciguyryan)
References
()
Details
Attachments
(1 file)
1.43 KB,
patch
|
bzbarsky
:
review+
bzbarsky
:
superreview+
|
Details | Diff | Splinter Review |
See the url above - the same is statement is checked twice. Patch coming up that removes the duplicate if.
Assignee | ||
Comment 1•19 years ago
|
||
Patch v1 to remove the duplicate if statement.
Assignee: file-handling → sciguyryan+bugzilla
Status: NEW → ASSIGNED
Attachment #241889 -
Flags: superreview?(benjamin)
Attachment #241889 -
Flags: review?(bzbarsky)
![]() |
||
Comment 2•19 years ago
|
||
Comment on attachment 241889 [details] [diff] [review]
Patch v1
r+sr=bzbarsky
Attachment #241889 -
Flags: superreview?(benjamin)
Attachment #241889 -
Flags: superreview+
Attachment #241889 -
Flags: review?(bzbarsky)
Attachment #241889 -
Flags: review+
Updated•19 years ago
|
Whiteboard: [checkin needed]
Comment 3•19 years ago
|
||
mozilla/embedding/components/webbrowserpersist/src/nsWebBrowserPersist.cpp 1.122
Status: ASSIGNED → RESOLVED
Closed: 19 years ago
OS: Windows XP → All
Hardware: PC → All
Resolution: --- → FIXED
Whiteboard: [checkin needed]
Target Milestone: --- → mozilla1.9alpha
Updated•9 years ago
|
Product: Core → Core Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•