Closed Bug 1939878 Opened 2 months ago Closed 1 month ago

[wpt-sync] Sync PR 49904 - [WebDriver BiDi] Fix issue with cached JS

Categories

(Remote Protocol :: WebDriver BiDi, task, P4)

Product:
Remote Protocol
Component:
WebDriver BiDi
Type:
task

Tracking

(firefox136 fixed)

Status:
RESOLVED FIXED
Milestone:
136 Branch
Tracking Flags:
Tracking Status
firefox136 --- fixed

People

(Reporter: wpt-sync, Unassigned)

References

(Blocks 3 open bugs,
URL
)

Details

(Whiteboard: [wptsync downstream])

Sync web-platform-tests PR 49904 into mozilla-central (this bug is closed when the sync is complete).

PR: https://github.com/web-platform-tests/wpt/pull/49904
Details from upstream follow.

Nikolay Vitkov <nvitkov@chromium.org> wrote:

[WebDriver BiDi] Fix issue with cached JS

It seem that Chrome sends only 1 request when the there are multiple scripts with the same URL, this look like optimization on our end.

Tested locally with HTML:

<!doctype html>
<html lang="en">
  <head>
    <meta charset="UTF-8" />
    <meta name="viewport" content="width=device-width, initial-scale=1.0" />
    <title>Document</title>
    <script src="./console.js"></script>
    <script src="./console.js"></script>
  </head>
  <body></body>
</html>

And JS:

console.log(2);
Component: web-platform-tests → WebDriver BiDi
Product: Testing → Remote Protocol
Whiteboard: [wptsync downstream] → [wptsync downstream error]
Whiteboard: [wptsync downstream error] → [wptsync downstream]
Whiteboard: [wptsync downstream] → [wptsync downstream error]
Whiteboard: [wptsync downstream error] → [wptsync downstream]

CI Results

Ran 12 Firefox configurations based on mozilla-central, and Firefox, Chrome, and Safari on GitHub CI

Total 3 tests and 11 subtests

Status Summary

Firefox

OK : 3
PASS : 26

Chrome

OK : 3
PASS : 26

Safari

OK : 3
ERROR: 26

Links

Gecko CI (Treeherder)
GitHub PR Head
GitHub PR Base

Pushed by wptsync@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/525d00f23c3b [wpt PR 49904] - [WebDriver BiDi] Fix issue with memory cache in Chrome, a=testonly

https://hg.mozilla.org/mozilla-central/rev/525d00f23c3b

Status: NEW → RESOLVED
Closed: 1 month ago
status-firefox136: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → 136 Branch
You need to log in before you can comment on or make changes to this bug.