catapult-project / catapult

Deprecated Catapult GitHub. Please instead use http://crbug.com "Speed>Benchmarks" component for bugs and https://chromium.googlesource.com/catapult for downloading and editing source code..
https://chromium.googlesource.com/catapult
BSD 3-Clause "New" or "Revised" License
1.91k stars 562 forks source link

[wpr-go] Incorrect order of responses for requests with the same url #4645

Open ulan opened 4 years ago

ulan commented 4 years ago

WPR seems to mix up the order of responses in findBestMatchInArchivedRequestSet.

Steps to reproduce:

Expected: The replay runs without errors. Actual: The web page shows "Error: Class$obf_1001: removeSent called when not waiting for ack"

Note that the live run may fail in future if the web page changes. Let me know if that happens and I'll adjust the JS code in the CL.

CalebRouleau commented 4 years ago

Thanks for filing this!

Moved to https://bugs.chromium.org/p/chromium/issues/detail?id=1001658

We don't use the github issue tracker anymore since it is easier to stay aligned with chromium development if we use the Monorail issue tracker.