{"push_id":2193116906,"size":5,"distinct_size":1,"ref":"refs/heads/chromium-export-cl-807604","head":"76a59f35bc30c8197df0d27148e76d61ce48ad5d","before":"3f8b478faca8b9f9f818c9f4fcb1694d0bc5dab2","commits":[{"sha":"e71027171a0883b4dbcfc9682e7b421c1ed5de5f","author":{"name":"Hiroki Nakagawa","email":"b2d90856c2d161182e6b29522395145231d42aad@chromium.org"},"message":"Worklet: Fix credentials tests for worklets\n\nBefore this CL, credentials tests for worklets are completely broken as follows:\n\n- In credentials.py, |credentials_mode| is compared by \"is\" that conducts the\n  object equality check. As a result, all conditions unexpectedly pass through\n  and the script always returns a valid response.\n- In credentials.py, |is_cross_origin| value is always |False| because requests\n  don't have the GET param.\n- fetch() needs { credentials: 'include' } option for configuring the document's\n  cookie with the Set-Cookie header, but it's not specified.\n- Response headers are not specified for a 404 response. This results in test\n  timeout instead of a network error.\n\nThis CL fixes these bugs and cleans up tests.\n\nBug: 738769\nChange-Id: I9aeb11fca23dec3fd3057a765fa60eb6e59d8258\nReviewed-on: https://chromium-review.googlesource.com/799730\nCommit-Queue: Hiroki Nakagawa <nhiroki@chromium.org>\nReviewed-by: Matt Falkenhagen <falken@chromium.org>\nCr-Commit-Position: refs/heads/master@{#523713}","distinct":false,"url":"https://api.github.com/repos/w3c/web-platform-tests/commits/e71027171a0883b4dbcfc9682e7b421c1ed5de5f"},{"sha":"09c54454864a73f680039359c633931f60e1fd10","author":{"name":"Rob Buis","email":"2e8eeb0c5b311e5ebe51c9c1cef4668481688e8c@samsung.com"},"message":"Sync CSSStyleDeclaration.setProperty with the spec\n\nSync CSSStyleDeclaration.setProperty with the spec [1], specifically\nmake value and property not nullable, treat null as empty string\nand for value change default from null to empty string.\n\nTEST: LayoutTests/fast/css/cssom-setproperty-undefined-value.html\n\n[1] https://drafts.csswg.org/cssom/#the-cssstyledeclaration-interface\n\nChange-Id: I9ca067626c65aed5cce7f9bcb23af1133a0822b6\nReviewed-on: https://chromium-review.googlesource.com/806751\nCommit-Queue: Rob Buis <rob.buis@samsung.com>\nReviewed-by: Philip Jägenstedt <foolip@chromium.org>\nCr-Commit-Position: refs/heads/master@{#523726}","distinct":false,"url":"https://api.github.com/repos/w3c/web-platform-tests/commits/09c54454864a73f680039359c633931f60e1fd10"},{"sha":"dc224ebe92b876c9b3629f00e704024c4903cf89","author":{"name":"Becca Hughes","email":"359d353ca86e4ceb6d464321fad5f2e429d103f1@chromium.org"},"message":"Feature Policy: Add WPT tests for autoplay\n\nAdd web platform tests for the new autoplay feature.\n\nBUG=788390\n\nChange-Id: I621ddd4f70b6882d837397b3f2f464c99253af13\nReviewed-on: https://chromium-review.googlesource.com/806535\nReviewed-by: Jochen Eisinger <jochen@chromium.org>\nReviewed-by: Mounir Lamouri <mlamouri@chromium.org>\nReviewed-by: Ian Clelland <iclelland@chromium.org>\nCommit-Queue: Becca Hughes <beccahughes@chromium.org>\nCr-Commit-Position: refs/heads/master@{#523746}","distinct":false,"url":"https://api.github.com/repos/w3c/web-platform-tests/commits/dc224ebe92b876c9b3629f00e704024c4903cf89"},{"sha":"9e4cb250b95d689d4aa7192a4ffe7980a4de73a7","author":{"name":"Harald Alvestrand","email":"86a86adb986ca1cb3ab2bad9a5f713d1f7588ae8@chromium.org"},"message":"Added test for generating stats for a track added using addTrack\n\nAlso fixes a misspelling of \"timestamp\" as \"timeStamp\".\n\nBug: webrtc:8616\nChange-Id: I76304d9235bbb1e4fca1e821a2e10dff0777bb2d\nReviewed-on: https://chromium-review.googlesource.com/824269\nCommit-Queue: Harald Alvestrand <hta@chromium.org>\nReviewed-by: Henrik Boström <hbos@chromium.org>\nCr-Commit-Position: refs/heads/master@{#523806}","distinct":false,"url":"https://api.github.com/repos/w3c/web-platform-tests/commits/9e4cb250b95d689d4aa7192a4ffe7980a4de73a7"},{"sha":"76a59f35bc30c8197df0d27148e76d61ce48ad5d","author":{"name":"Chris Cunningham","email":"486871187251eb825f109ec43f563cab418c65c1@chromium.org"},"message":"Stop using findMediaFile(...) in LayoutTests\n\nHistorically, findMediaFile() helped choose the right extension based\non what the browser could play. Chromium cannot play some proprietary\nformats (h264, aac, etc), so this check would choose the .ogv format\nin that case.\n\nBut things have gotten crufty.\n1) A long time ago we started running layout tests on builds that *do*\n   include proprietary codecs\n2) But we lie about that to the tests via\n     media::RemoveProprietaryMediaTypesAndCodecsForTests()\n   This makes debugging tests a pain because manual execution doesn't\n   call this method and its easy to forget that the test wrapper does.\n3) Lots of tests never bothered to call findMediaFile anway and just\n   hardcoded the use of an mp4 with proprietary codecs, which worked\n   because the method in #2 above only changes canPlayType responses,\n   it doesn't remove the actual support for proprietary codecs.\n4) findMediaFile is about to be busted anyway because it only queries the\n   file mime type without supplying codec info. eg\n      canPlayType(\"video/mp4\") == \"maybe\" -> lets use the mp4!\n   and even chromium will now \"maybe\" for this now that we no longer\n   consider mp4 proprietary (though codecs like h264 still are!)\n\nSo this patch does the following:\n\n1) Deletes findMediaFile() and instead hardcode use of the ogg file. This\n   maintains the existing behavior and avoids a massive rebaseline.\n2) Delete media::RemoveProprietaryMediaTypesAndCodecsForTests(). Tests\n   and manual runs now behave the same.\n3) Delete ancient media-can-play-* LayoutTests that just call canPlayType\n   with various codecs and are duplicated by content and chrome browser\n   tests (which is a better place for these checks)\n4) Updates mediasource-config-change-mp4-* expectations to expect that\n   they should run and pass on *all* platforms (previously just android)\n\nThere are still many tests that hard code the use of mp4 files. If we\nlater desire to see LayoutTests run without proprietary codecs, someone\ncan transition those tests. No one seems to mind at the moment.\n\nIt is also not a goal to make LayoutTests try all the supported codecs.\nThat is covered by unit/integration tests.\n\nDue to #2 above, a handful of tests in external/wpt/media-source now fail\nbecause they use mp4. Fixing these failures is tracked in Issue 794338.\n\nChange-Id: Ie357ae075c880b78d5ee9e95c1b7cc69d9d8a328\nBUG: 327115,746579,787575,568704,794338\nReviewed-on: https://chromium-review.googlesource.com/807604\nReviewed-by: Dale Curtis <dalecurtis@chromium.org>\nReviewed-by: Peter Beverloo <peter@chromium.org>\nCommit-Queue: Chrome Cunningham <chcunningham@chromium.org>\nCr-Commit-Position: refs/heads/master@{#523821}","distinct":true,"url":"https://api.github.com/repos/w3c/web-platform-tests/commits/76a59f35bc30c8197df0d27148e76d61ce48ad5d"}]}
{"actor":{"display_login":"chromium-wpt-export-bot"}}