[Webkit-unassigned] [Bug 41592] ProgressTracker looks not smart

[bugzilla-daemon at webkit.org]

https://bugs.webkit.org/show_bug.cgi?id=41592





--- Comment #4 from Ryuan Choi <ryuan.choi at gmail.com>  2010-07-07 19:27:24 PST ---
Thank you for your interesting

(In reply to comment #3)
> (From update of attachment 60705 [details])
> Thanks for the patch, but a number of things are unclear to me:
> 
> WebCore/loader/ProgressTracker.cpp:184
>  +      if (frame->loader()->client())
> Do we need to null-check the client here?  Is the client disappearing somehow?
> 
It's my mistake. I'll remove it.

> WebCore/loader/ProgressTracker.cpp: 
>  +      // FIXME: Can this ever happen?
> Did we resolve this FIXME?  It might be worth explaining when item might be NULL.
> 
this patch make a case which item was null.
so I moved this check logic like below
-    item->bytesReceived += bytesReceived;
-    if (item->bytesReceived > item->estimatedLength) {
-        m_totalPageAndResourceBytesToLoad += ((item->bytesReceived * 2) - item->estimatedLength);
-        item->estimatedLength = item->bytesReceived * 2;
+
+    if (item) {
+        item->bytesReceived += bytesReceived;
+        if (item->bytesReceived > item->estimatedLength) {
+            m_totalPageAndResourceBytesToLoad += ((item->bytesReceived * 2) - item->estimatedLength);
+            item->estimatedLength = item->bytesReceived * 2;
+        }
     }

> In general, we would like to see a test with each patch.  Is there a way to test this code change?  I'm not 100% sure this is the right direction.  It's difficult to tell from this bug report and from the ChangeLog exactly what problem we're solving here.  It's also unclear to me how this change effects all the different clients of resource load notifier.

you are right, but I don't know how to make proper test case.
In almost other network situation, this wasn't happened.
especially I am testing wifi through slow proxy server.

anyway I'll prepare some video file and more log for you.
If I am possible, I can show you this issues using android browser (based on proyo) in my test environment.

-- 
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.