[Webkit-unassigned] [Bug 24747] preloading logic caused the same resource was loaded and reloaded.(sent two requests for same resource)

[bugzilla-daemon at webkit.org]

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


ap at webkit.org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
  Attachment #30712|review?(ap at webkit.org)      |review+
               Flag|                            |




------- Comment #22 from ap at webkit.org  2009-05-27 12:24 PDT -------
(From update of attachment 30712)
> +        Reviewed by NOBODY (OOPS!).
> +        https://bugs.webkit.org/show_bug.cgi?id=24747
> +
> +        Add a test case for multiple requests for same sub-resource due to preload.

We usually put an empty line after "Reviewed by" line, not between bug
reference and title.

> +
> +        * http\tests\loading\preload-img-test-expected.txt: Added.
> +        * http\tests\loading\preload-img-test.html: Added.
> +        * http\tests\loading\resources\preload-test.jpg: Added.
> +        * http\tests\resources\network-simulator.php:

One day we should fix prepare-ChangeLog to not emit Windows paths.

> +main frame - didStartProvisionalLoadForFrame
> +main frame - didCommitLoadForFrame
> +main frame - didFinishDocumentLoadForFrame
> +main frame - didHandleOnloadEventsForFrame
> +main frame - didFinishLoadForFrame

These lines are useless - but I know that they are generated because the test
in in a loading/ directory. Maybe we just need a new directory for preloading
tests.

> +       document.getElementById("outputPanel").innerHTML = "FALIED";

Typo: "FAILED".

+  <span id="outputPanel">PASSED</span>

It's not good to have "PASSED" output from the beginning - if the test doesn't
run to completion, that could mask the failure.

These are all extremely minor nitpicks, just wanted to mention them for the
future. No need to fix them to land this patch.

It is really great to have a framework for preloading tests, and thanks for
refactoring network-simulator!

r=me


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