Bug 53228 - [Gtk] Resource size is incorrectly reported to WebCore
Summary: [Gtk] Resource size is incorrectly reported to WebCore
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: New Bugs (show other bugs)
Version: 528+ (Nightly build)
Hardware: Other OS X 10.5
: P2 Normal
Assignee: Sergio Villar Senin
URL:
Keywords:
: 55363 (view as bug list)
Depends on:
Blocks:
 
Reported: 2011-01-27 01:04 PST by Sergio Villar Senin
Modified: 2011-02-28 11:44 PST (History)
1 user (show)

See Also:

Attachments
Patch (4.94 KB, patch)
2011-01-27 01:10 PST, Sergio Villar Senin 		mrobinson: review-
Details | Formatted Diff | Diff
Patch (5.49 KB, patch)
2011-02-28 11:12 PST, Sergio Villar Senin 		mrobinson: review+
Details | Formatted Diff | Diff
Show Obsolete (1) View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Sergio Villar Senin 2011-01-27 01:04:06 PST 
[Gtk] Resource size is incorrectly reported to WebCore
Comment 1 Sergio Villar Senin 2011-01-27 01:10:30 PST 
Created attachment 80303 [details]
Patch
Comment 2 Sergio Villar Senin 2011-01-27 01:13:55 PST 
It's easy to see the problem in web inspector. Just open the resources tab when visiting http://www.csszengarden.com for example and check the actual and xfer sizes of some resources when using epiphany and chrome.
Comment 3 Martin Robinson 2011-01-27 07:44:34 PST 
Comment on attachment 80303 [details]
Patch

This seems like a really good change, but we must be failing a test because of this. If not we should add a test somewhere.
Comment 4 Sergio Villar Senin 2011-01-27 09:43:33 PST 
most likely another version of bug #40049
Comment 5 Sergio Villar Senin 2011-02-28 10:23:38 PST 
Looks like the new test http/tests/inspector/network/network-size.html is the test we were looking for. It fails with current trunk and works with my patch.
Comment 6 Sergio Villar Senin 2011-02-28 11:12:24 PST 
Created attachment 84085 [details]
Patch

New rebased version.

Updated the skipped list
Comment 7 Sergio Villar Senin 2011-02-28 11:15:40 PST 
*** Bug 55363 has been marked as a duplicate of this bug. ***
Comment 8 Martin Robinson 2011-02-28 11:24:56 PST 
Comment on attachment 84085 [details]
Patch

Great work.
Comment 9 Sergio Villar Senin 2011-02-28 11:44:36 PST 
Committed r79889: <http://trac.webkit.org/changeset/79889>