Closed
Bug 1461510
Opened 6 years ago
Closed 6 years ago
[wpt-sync] Sync PR 11001 - Change mojo manifest interface syntax from foo::mojom::Bar to foo.mojom.Bar.
Categories
(Testing :: web-platform-tests, enhancement, P4)
Testing
web-platform-tests
Tracking
(firefox62 fixed)
RESOLVED
FIXED
mozilla62
Tracking | Status | |
---|---|---|
firefox62 | --- | fixed |
People
(Reporter: mozilla.org, Unassigned)
References
()
Details
(Whiteboard: [wptsync downstream])
Sync web-platform-tests PR 11001 into mozilla-central (this bug is closed when the sync is complete). PR: https://github.com/w3c/web-platform-tests/pull/11001 Details from upstream follow. Robert Sesek <rsesek@chromium.org> wrote: > Change mojo manifest interface syntax from foo::mojom::Bar to foo.mojom.Bar. > > This updates the bindings generators and hard-coded interface names in > tests and the service_manager. > > The remainder of the CL was generated by: > > for f in $(find . -name '*.json' -not \( -path './out/*' -o -path './third_party/*' -o -path '*.xcassets/*' \)); do > grep interface_provider_spec "./$f" >/dev/null && gsed -i s/::/./g "$f"; > done > > Bug: 835446 > Cq-Include-Trybots: luci.chromium.try:android_optional_gpu_tests_rel;luci.chromium.try:mac_optional_gpu_tests_rel;master.tryserver.chromium.linux:linux_mojo;master.tryserver.chromium.mac:ios-simulator-cronet;master.tryserver.chromium.mac:ios-simulator-full-configs > Change-Id: I0bb22615b9c66824c6f95da66e3d7016809d63c7 > > Reviewed-on: https://chromium-review.googlesource.com/1026419 > WPT-Export-Revision: 319b4b9d619b15dd9dc09e0529c5d059800e7184
Assignee | ||
Comment 1•6 years ago
|
||
PR 11001 applied with additional changes from upstream: b55a93ab1fa441d8998fcfb1173d00dab4b733c0
Assignee | ||
Comment 2•6 years ago
|
||
Pushed to try https://treeherder.mozilla.org/#/jobs?repo=try&revision=3b4eeb3ad39e397b82fc68a2d47fa988be54ba82
Assignee | ||
Comment 3•6 years ago
|
||
Pushed to try https://treeherder.mozilla.org/#/jobs?repo=try&revision=509f340511c8b68f1ca21e07f6f48bf954fa1597
Assignee | ||
Comment 4•6 years ago
|
||
Try push failed: decision task returned error
Assignee | ||
Comment 5•6 years ago
|
||
Pushed to try https://treeherder.mozilla.org/#/jobs?repo=try&revision=c1c3b5b99b5a136b42352990b00a08c3fd20ebc3
Assignee | ||
Comment 6•6 years ago
|
||
The PR was not expected to affect any tests, but the try push wasn't a success. Check the try results for infrastructure issues
Pushed by wptsync@mozilla.com: https://hg.mozilla.org/integration/mozilla-inbound/rev/6ea57382bd99 [wpt PR 11001] - Change mojo manifest interface syntax from foo::mojom::Bar to foo.mojom.Bar., a=testonly
Comment 8•6 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/6ea57382bd99
Status: NEW → RESOLVED
Closed: 6 years ago
status-firefox62:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla62
You need to log in
before you can comment on or make changes to this bug.
Description
•