Comment #1 on issue 19931 by bugdro...@chromium.org: The  
ResourceDispatcherHost class does not honor the full contract of the  
deferred flag
http://code.google.com/p/chromium/issues/detail?id=19931

The following revision refers to this bug:
     http://src.chromium.org/viewvc/chrome?view=rev&revision=24069

------------------------------------------------------------------------
r24069 | ana...@chromium.org | 2009-08-21 23:03:42 -0700 (Fri, 21 Aug 2009)  
| 15 lines
Changed paths:
    M  
http://src.chromium.org/viewvc/chrome/trunk/src/chrome/common/resource_dispatcher.cc?r1=24069&r2=24068
    M  
http://src.chromium.org/viewvc/chrome/trunk/src/chrome/common/resource_dispatcher_unittest.cc?r1=24069&r2=24068

The ResourceDispatcher object does not honor the deferred load flag for a  
request correctly. If this flag is set it correctly queues up any  
subsequent responses.

When the flag is reset it starts dispatching these responses. If the  
deferred flag is set in the context
of these responses it continues to dispatch further responses, which is not  
correct.

Fix is to check if the deferred flag for a request is set in the context of  
a response and stop dispatching if yes.

This fixes bug http://code.google.com/p/chromium/issues/detail?id=19931 and  
could potentially explain
this http://code.google.com/p/chromium/issues/detail?id=19393

Bug=19931
Test= Covered by unit test.


Review URL: http://codereview.chromium.org/173187
------------------------------------------------------------------------


--
You received this message because you are listed in the owner
or CC fields of this issue, or because you starred this issue.
You may adjust your issue notification preferences at:
http://code.google.com/hosting/settings

--~--~---------~--~----~------------~-------~--~----~
Automated mail from issue updates at http://crbug.com/
Subscription options: http://groups.google.com/group/chromium-bugs
-~----------~----~----~----~------~----~------~--~---

Reply via email to