Closed Bug 356225 Opened 19 years ago Closed 19 years ago

Duplicate if check in nsWebBrowserPersist.cpp

Categories

(Core Graveyard :: File Handling, defect)

defect
Not set
trivial

Tracking

(Not tracked)

RESOLVED FIXED
mozilla1.9alpha1

People

(Reporter: sciguyryan, Assigned: sciguyryan)

References

()

Details

Attachments

(1 file)

See the url above - the same is statement is checked twice. Patch coming up that removes the duplicate if.
Attached patch Patch v1Splinter Review
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 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+
Whiteboard: [checkin needed]
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
Product: Core → Core Graveyard
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: