Closed Bug 76034 Opened 23 years ago Closed 23 years ago

saveURI api busted

Categories

(Core Graveyard :: Embedding: APIs, defect)

x86
Linux
defect
Not set
major

Tracking

(Not tracked)

VERIFIED FIXED
mozilla0.9

People

(Reporter: mpgritti, Assigned: adamlock)

Details

Attachments

(1 file)

Saving images with this api is no more working in mozilla cvs:

/mozilla/embedding/browser/webBrowser/nsIWebBrowserPersist.idl
void saveURI(in nsIURI aURI, in nsIInputStream aPostData, in string aFileName);

I think it can be caused by this change: (the date is near when I noticed the
bug, and it seems related)

http://bonsai.mozilla.org/cvsview2.cgi?diff_mode=context&whitespace_mode=show&subdir=mozilla/embedding/browser/webBrowser&command=DIFF_FRAMESET&file=nsWebBrowserPersist.cpp&rev1=1.11&rev2=1.12&root=/cvsroot

I can reproduce on this url for example:

http://www.gnome.org/images/banner-gnomeis
Status: UNCONFIRMED → NEW
Ever confirmed: true
Hi, do you have the Mozilla source? Can you try deleting the line 
"persist->Release();" from the method nsWebBrowser::SaveURI in 
mozilla/embedding/browser/webBrowser/nsWebBrowser.cpp and see if that fixes your 
problems?

I recently changed how the refcounting behaviour works and it's possible that 
the line is the cause of your problems. If so let me know and I will submit a 
patch.
Target Milestone: --- → mozilla0.9.1
Hi,
Removing persist->Release () fix it.
It would be really good if this could be checked in before 0.9 :)
Attached patch One line patchSplinter Review
The 0.9 freeze is tonight but I will try and sneak it in since I have another 
patch on that file to checkin as well. It's very low risk.
Target Milestone: mozilla0.9.1 → mozilla0.9
Fix is checked in.
Status: NEW → RESOLVED
Closed: 23 years ago
Resolution: --- → FIXED
Correction: Changing QA contact for the Embed API bugs to David Epstein.
QA Contact: mdunn → depstein
changed qa contact to dsirnapalli. He's working with nsIWebBrowserPersist.
QA Contact: depstein → dsirnapalli
checked nsWebBrowser.cpp. line "persist->Release();" no longer exists in  method 
SaveURI. marking it  to verified.
Status: RESOLVED → VERIFIED
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: