Bug 50634 - [GTK] clears the subresources prematurely
Summary: [GTK] clears the subresources prematurely
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: WebKitGTK (show other bugs)
Version: 528+ (Nightly build)
Hardware: PC OS X 10.5
: P2 Normal
Assignee: Nobody
URL:
Keywords: Gtk
Depends on:
Blocks:
 
Reported: 2010-12-07 09:16 PST by Gustavo Noronha (kov)
Modified: 2010-12-07 21:41 PST (History)
1 user (show)

See Also:


Attachments
proposed fix (9.39 KB, patch)
2010-12-07 09:25 PST, Gustavo Noronha (kov)
mrobinson: review+
Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Gustavo Noronha (kov) 2010-12-07 09:16:07 PST
We currently clear subresources when a provisional load is started. If that load is stopped, we end up with a WebDataSource with 0 subresources in the page that is already loaded. A simple way to test this: in Epiphany go to debian.org, wait for it to load, try to save the page. You will notice an HTML file is saved, and a directory is created with the subresources. Now, enter gnome.org, hit enter and quickly hit stop so it doesn't actually load. Try saving the page again, and you'll see only the HTML file is saved - no subresources.
Comment 1 Gustavo Noronha (kov) 2010-12-07 09:25:56 PST
Created attachment 75823 [details]
proposed fix

This fixes the manual test case I mentioned, and adds an API test.
Comment 2 Martin Robinson 2010-12-07 09:36:11 PST
Comment on attachment 75823 [details]
proposed fix

Great. Thanks.
Comment 3 WebKit Review Bot 2010-12-07 10:17:37 PST
Attachment 75823 [details] did not pass style-queue:

Failed to run "[u'git', u'reset', u'--hard', u'refs/remotes/trunk']" exit_code: 128
error: Could not write new index file.
fatal: Could not reset index file to revision 'refs/remotes/trunk'.


If any of these errors are false positives, please file a bug against check-webkit-style.
Comment 4 WebKit Review Bot 2010-12-07 11:18:43 PST
Attachment 75823 [details] did not pass style-queue:

Failed to run "[u'git', u'reset', u'--hard', u'refs/remotes/trunk']" exit_code: 128
error: Could not write new index file.
fatal: Could not reset index file to revision 'refs/remotes/trunk'.


If any of these errors are false positives, please file a bug against check-webkit-style.
Comment 5 WebKit Review Bot 2010-12-07 12:20:07 PST
Attachment 75823 [details] did not pass style-queue:

Failed to run "[u'git', u'reset', u'--hard', u'refs/remotes/trunk']" exit_code: 128
error: Could not write new index file.
fatal: Could not reset index file to revision 'refs/remotes/trunk'.


If any of these errors are false positives, please file a bug against check-webkit-style.
Comment 6 Gustavo Noronha (kov) 2010-12-07 13:00:48 PST
Committed r73455: <http://trac.webkit.org/changeset/73455>
Comment 7 WebKit Review Bot 2010-12-07 21:41:40 PST
Attachment 75823 [details] did not pass style-queue:

Failed to run "['WebKitTools/Scripts/update-webkit']" exit_code: 2
Updating OpenSource
Incomplete data: Delta source ended unexpectedly at /usr/lib/git-core/git-svn line 5061

Died at WebKitTools/Scripts/update-webkit line 132.


If any of these errors are false positives, please file a bug against check-webkit-style.