Closed
Bug 102413
Opened 24 years ago
Closed 24 years ago
nsIWebShell's GetURL using const PRUnichar**
Categories
(Core :: DOM: Navigation, defect)
Tracking
()
RESOLVED
FIXED
People
(Reporter: aaronlev, Assigned: adamlock)
References
Details
Attachments
(2 files)
|
2.64 KB,
patch
|
jag+mozilla
:
review+
|
Details | Diff | Splinter Review |
|
2.77 KB,
patch
|
jag+mozilla
:
review+
rpotts
:
superreview+
|
Details | Diff | Splinter Review |
Should not be const
Needs to be fixed so we can use getter_Copies with an XPIDLString
|
||
Comment 2•24 years ago
|
||
Looks good, except that in mozilla/webshell/tests/viewer/nsBrowserWindow.cpp
near where you made the change there you're leaking the PRUnichar* you're
getting. Not necessarily part of this patch, but you may want to fix that while
you're there (using nsXPIDLString and getter_Copies).
r=jag
|
|
||
Updated•24 years ago
|
Attachment #52499 -
Flags: review+
*** Bug 102410 has been marked as a duplicate of this bug. ***
|
|
||
Comment 5•24 years ago
|
||
Comment on attachment 52507 [details] [diff] [review]
Second patch with getter_Copies
r=jag
I like this :-)
Attachment #52507 -
Flags: review+
|
|
||
Comment 6•24 years ago
|
||
(Of course, I could point out that one could save that string copy by making
ViewSource take const nsAString& instead of nsString, but that is really a
different bug altogether, so I won't ;-) ).
|
||
Comment 7•24 years ago
|
||
Comment on attachment 52507 [details] [diff] [review]
Second patch with getter_Copies
sr=rpotts@netscape.com
Attachment #52507 -
Flags: superreview+
Fix is checked in.
Status: NEW → RESOLVED
Closed: 24 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•