Closed Bug 1906100 Opened 3 months ago Closed 3 months ago

Rename "network.setCacheBypass" command to "network.setCacheBehavior" and adjust "cacheBehavior" argument

Categories

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

task
Points:
3

Tracking

(firefox129 fixed)

RESOLVED FIXED
129 Branch
Tracking Status
firefox129 --- fixed

People

(Reporter: whimboo, Assigned: whimboo)

References

()

Details

(Whiteboard: [webdriver:m12][wptsync upstream][webdriver:relnote])

Attachments

(3 files)

The initial version landed via bug 1901032 as network.setCacheBypass. However, further work on PR 721 has led us to rename the command. Additionally, the argument has been adjusted and is now called cacheBehavior, which is an enum with the following values: "default" and "bypass".

Summary: Rename "network.setCacheBypass" command to "network.setCacheBehavior" → Rename "network.setCacheBypass" command to "network.setCacheBehavior" and adjust "cacheBehavior" argument
Assignee: nobody → hskupin
Status: NEW → ASSIGNED
Points: --- → 3
Pushed by hskupin@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/79af45d71dd3 [webdriver-bidi] Rename "network.setCacheBypass" command to "network.setCacheBehavior" and adjust "cacheBehavior" argument. r=webdriver-reviewers,jdescottes https://hg.mozilla.org/integration/autoland/rev/d35419eadeb1 [webdriver-client] Update command "network.setCacheBypass" to "network.setCacheBehavior". r=webdriver-reviewers,jdescottes https://hg.mozilla.org/integration/autoland/rev/cfbed0d187da [wdspec] Update tests for renamed "network.setCacheBypass" command to "network.setCacheBehavior". r=webdriver-reviewers,jdescottes
Failed to create upstream wpt PR due to merge conflicts. This requires fixup from a wpt sync admin.
Flags: needinfo?(james)
Flags: needinfo?(aborovova)
Whiteboard: [webdriver:m12] → [webdriver:m12], [wptsync upstream error]

(In reply to Web Platform Test Sync Bot [:wpt-sync] (Matrix: #interop:mozilla.org) from comment #5)

Failed to create upstream wpt PR due to merge conflicts. This requires fixup
from a wpt sync admin.

FYI this fails because https://github.com/web-platform-tests/wpt/pull/46935 isn't merged yet.

Status: ASSIGNED → RESOLVED
Closed: 3 months ago
Resolution: --- → FIXED
Target Milestone: --- → 129 Branch
Created web-platform-tests PR https://github.com/web-platform-tests/wpt/pull/47030 for changes under testing/web-platform/tests
Whiteboard: [webdriver:m12], [wptsync upstream error] → [webdriver:m12], [wptsync upstream]
Flags: needinfo?(james)
Flags: needinfo?(aborovova)
Upstream PR was closed without merging
Upstream PR merged by moz-wptsync-bot
Upstream PR merged by moz-wptsync-bot
Whiteboard: [webdriver:m12], [wptsync upstream] → [webdriver:m12][wptsync upstream][webdriver:relnote]
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: