Closed Bug 1763863 Opened 3 years ago Closed 3 years ago

[wpt-sync] Sync PR 33575 - Remove Chrom(e|ium)'s version string suffix in MojoJS install

Categories

(Testing :: web-platform-tests, task, P4)

Product:
Testing
Component:
web-platform-tests
Type:
task

Tracking

(firefox102 fixed)

Status:
RESOLVED FIXED
Milestone:
102 Branch
Tracking Flags:
Tracking Status
firefox102 --- fixed

People

(Reporter: wpt-sync, Unassigned)

References

(Depends on 1 open bug,
URL
)

Details

(Whiteboard: [wptsync downstream])

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

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

Daniel Smith <56164590+DanielRyanSmith@users.noreply.github.com> wrote:

Remove Chrom(e|ium)'s version string suffix in MojoJS install

MojoJS will be downloaded based on matching Chrome/Chromium's version string. However, channel suffixes are sometimes present in the version string which need to be removed (e.g. 97.0.4692.8 dev). Without removing this suffix, the install will fail and display an error message like this:
ERROR A valid MojoJS version cannot be found for browser binary version 97.0.4692.8 dev.
This change handles this case and moves this process to a helper method as it is frequently used.

Whiteboard: [wptsync downstream] → [wptsync downstream error]
Whiteboard: [wptsync downstream error] → [wptsync downstream]
Pushed by wptsync@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/f474419afb8c [wpt PR 33575] - Remove Chrom(e|ium)'s version string suffix in MojoJS install, a=testonly

https://hg.mozilla.org/mozilla-central/rev/f474419afb8c

Status: NEW → RESOLVED
Closed: 3 years ago
status-firefox102: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → 102 Branch
You need to log in before you can comment on or make changes to this bug.