Changes:
- modified the criteria for using 1 as the successful return code by using a combination of factors; is it Windows, is it 32bit, and is it Reftest.
Differential Revision: https://phabricator.services.mozilla.com/D21596
--HG--
extra : moz-landing-system : lando
Unfortunately, we can't do fuzzy annotations in WPT yet. When we can, we should
adjust this annotation to "1px allowed to differ by <=8 in any color channel".
--HG--
extra : amend_source : c47d4d64056b795f6ad09124a7a862c7249d20f2
Per spec these should just go directly to the expando object; we were ignoring them instead.
Differential Revision: https://phabricator.services.mozilla.com/D16930
--HG--
extra : moz-landing-system : lando
The main behavior changes are:
1) We no longer create a new Window when doing document.open(). We use the
same Window but remove all the event listeners on it and on the existing DOM
tree before removing the document's existing kids.
2) We no longer create a new session history entry. The existing one always
gets replaced instead.
3) We now support document.open on documents that are not in a Window.
The reasons for the various test changes are as follows:
The change to browser_modifiedclick_inherit_principal.js is because we no
longer set the docshell to a wyciwyg URL when document.open() happens and the
test was depending on that to terminate.
browser_wyciwyg_urlbarCopying.js is being removed because it's trying to test
wyciwyg URIs, which no longer exist.
The changes in docshell/test/navigation are because document.open() no longer
affects session history. One of the tests was testing the interactions there
and is being removed; another is being repurposed to just test that
document.open() does not affect history.length.
The change to test_x-frame-options.html is because document.open() now removes
event listeners on the window, which it didn't use to do (and in the specific
case in this test reused the existing inner too, so the listener was still
around in practice). The new behavior matches other browsers.
The removal of test_bug172261.html is because document.open() no longer affects
session history, so you can't go back across it or forward to the "opened"
state, so the situation that test is trying to test no longer exists.
The changes to test_bug255820.html are because reloading a document after
document.open() will now just load the URL of the document that was the entry
document for the open() call, not reload the written content. So there's not
much point testing reload behavior, and in this test it was just reloading the
toplevel test file inside the frames.
The change to test_bug346659.html is because now we no longer create a new
Window on document.open().
The change to test_bug1232829.html is because document.open() (implicit in this
test) no longer adds history entries, so the back() was just leaving the test
page instead of going back across the document.open(). The test is a
crashtest in practice, so might still be testing something useful about how
document.open() interacts with animations.
The change to test_bug715739.html is because the URL of the document after
document.open() is now the URL of the entry document, not a wyciwyg URL, so
reload() has different behavior than it used to.
The change to test_bug329869.html is because now when we go back we're
reloading the original document we had, not doing a wyciwyg load, and the
security info now doesn't include the untrusted script.
The changes to the wpt expectations are removing a bunch of expected failures
now that we pass those tests and disabling some tests that are fundamentally
racy and hence fail randomly. The latter all have github issues filed for the
test problem.
The change to testing/web-platform/tests/common/object-association.js is fixing
tests that were not matching the spec (and were failing in other browsers).
The change to parser-uses-registry-of-owner-document.html is fixing tests that
were not matching the spec (and were failing in other browsers).
The change to document-write.tentative.html is because the test was buggy: it
was using the same iframe element for all its tests and racing loads from some
tests against API calls from other tests, etc. It's a wonder it ever managed
to pass, independent of these patches (and in fact it doesn't pass according to
wpt.fyi data, even in Firefox).
The changes in html/browsers/history/the-history-interface are because
document.open() no longer adds history entries. The test was failing in all
other browsers for the same reason.
The changes in html/browsers/history/the-location-interface are because
reloading a document.open()-created thing now loads the URL of the page that
was the entry document for the open() call. The test was failing in all other
browsers.
The change to reload_document_open_write.html is because we now reload the url
of the document that entered the script that called open() when we reload, not
the written content. Other browsers were failing this test too; Gecko with
the old document.open implementation was the only one that passed.
The change to http-refresh.py is to fix a test bug: it was not returning a
Content-Type header, so we were putting up helper app dialogs, etc.
The change to test_ext_contentscript.js is because we no create a new global
for document.open() calls. Kris Maglione OKed this part.
Differential Revision: https://phabricator.services.mozilla.com/D17323
--HG--
extra : moz-landing-system : lando
This removes the need to go through the browser-to-tab mapping when discarding
a tab and simplifies the relevant code. Besides being renamed discardBrowser()
was also split so that one can check if a tab can be discarded prior to trying
it.
Differential Revision: https://phabricator.services.mozilla.com/D20475
--HG--
extra : moz-landing-system : lando
This adds support for specifying per-suite perfherder alert thresholds and
updates the base content JS measurement to specify a 0.25% threshold.
Differential Revision: https://phabricator.services.mozilla.com/D21292
--HG--
extra : moz-landing-system : lando
There are few things that are either Fennec-specific or don't work
currently under GeckoView w/ e10s under TestRunnerActivity. Disable
these so we can get some testing going in automation.
This also replaces 'isFennec' with the more correct 'is_fennec'.
Differential Revision: https://phabricator.services.mozilla.com/D19016
--HG--
extra : moz-landing-system : lando
Now that the JsTerm is a React component, we shouldn't
let external consumers have to get the instance in order
to perform some actions on the console input.
We start with the most used actions, setInputValue and getInputValue,
and might extend that in the future.
Differential Revision: https://phabricator.services.mozilla.com/D20685
--HG--
extra : moz-landing-system : lando
Optimizing jars without preloading/reordering data only moves the
jar central directory to the beginning of the file, which, without
preloading information, is not very useful. Let's just stop doing it if
there's not going to be preloading/reordering information at all.
Differential Revision: https://phabricator.services.mozilla.com/D21170
--HG--
extra : moz-landing-system : lando
The HTML formatter now references the correct link and does not produce a 404 error
Differential Revision: https://phabricator.services.mozilla.com/D21207
--HG--
extra : moz-landing-system : lando
The HTML formatter now references the correct link and does not produce a 404 error
Differential Revision: https://phabricator.services.mozilla.com/D21207
--HG--
extra : moz-landing-system : lando
Automatic update from web-platform-tests
Add .mailmap to map people to their current names
Note that this isn't used by `git log` by default (as of 2.20.1): that
depends on log.mailmap being set to true.
--
Add Tess to .mailmap
--
wpt-commits: 39f54a3dd48b560289e0f455308acf7d32fad043, f8a1bfbe5454352d3f5b58845829968ff212519b
wpt-pr: 15353
Automatic update from web-platform-tests
HTML: noreferer was never a property of hyperlinks
This should have tested the rel attribute value, but as it hasn't been doing that and it would be quite a bit of work, let's remove this instead.
--
wpt-commits: 25c665a93231027be5941838cecb3208e360d4b1
wpt-pr: 15357
Automatic update from web-platform-tests
Add screen orientation test: fire event before promise is resolved (#15267)
--
wpt-commits: 8ed64687560332642c7b34ebd16011ae5b01d99a
wpt-pr: 15267
Automatic update from web-platform-tests
XMLSerializer: Fix prefixed attribute serialization
... in a case where its owner element has xmlns:prefix of which prefix
is same as the attribute and namespace is not same as the attribute.
e.g.
el.setAttributeNS("uri1", "p:n", "v");
el.setAttributeNS(XMLNS_URI, "xmlns:p", "uri2");
ShouldAddNamespaceAttribute() checked only existence of xmlns:prefix in
the element, but we should check existence of (prefix, namespace URI)
pair in the scope according to the specification.
So, this CL adds 'recording the namespace information' step [1] defined
by the specification, and ShouldAddNamespaceAttribute() checks all
available prefixes in the scope.
[1] https://w3c.github.io/DOM-Parsing/#recording-the-namespace
Bug: 929035
Change-Id: I575e8f652ae45f7583202443cc72d5afe5faf59d
Reviewed-on: https://chromium-review.googlesource.com/c/1460643
Reviewed-by: Yoshifumi Inoue <yosin@chromium.org>
Commit-Queue: Kent Tamura <tkent@chromium.org>
Cr-Commit-Position: refs/heads/master@{#631058}
--
wpt-commits: 265330b7d26194cb60157a18ac399127d21f6b99
wpt-pr: 15340
Automatic update from web-platform-tests
decimal points in numbers
Tests that decimal points are allowed *before* digits (like `.1`) and *between* digits (like `0.1`), but not *after* digits (like `1.`).
Tests <https://github.com/w3c/csswg-drafts/issues/3599>
--
fix whitespace for lint
--
Name all tests uniquely
--
Merge pull request #15337 from web-platform-tests/tabatkins-patch-2
decimal points in numbers
--
wpt-commits: 5fdd036dbd18fa6d2e7cf235bd9acc625a6df22e, f17e215eaa37316f1489a7b62ab103d2a58a0d89, 31e1a1bb5fadb08749e4b8070ad0f19ec5ea1088, e2eb25aaa6dd3c512b172588f3400f9c25a410c3
wpt-pr: 15337
Automatic update from web-platform-tests
url whitespace consumption
Ensure that whitespace before or after the string in a url() results in a correct parse.
Tests https://github.com/w3c/csswg-drafts/issues/3600
--
Merge pull request #15339 from web-platform-tests/tabatkins-patch-3
url whitespace consumption
--
wpt-commits: e0b3c4cf2f69e816c6b83432968dc7fc83d700f6, 0de6fa5638c7e314a0391fae7f6bf496309fb65d
wpt-pr: 15339
Automatic update from web-platform-tests
[css-box] Inheritance, initial values (#15312)
Test that CSS Box Model properties do not inherit.
https://drafts.csswg.org/css-box-3/#property-index
--
wpt-commits: 35bd69423b10a361a969139ba0271896dced9691
wpt-pr: 15312
Automatic update from web-platform-tests
Set click count correctly to dispatch "dblclick" event
In order to dispatch the "dblclick" event to DOM by sending synthetic
mouse events, we have to check the most recent two mouse press events.
If their dispatched times and locations are close enough, we will set
the mouse event's click count to 2, which will generate the "dblclick"
event.
Bug: 606367
Change-Id: I049d6463c6bf1393c9e8b496662359eeeef7682b
Reviewed-on: https://chromium-review.googlesource.com/c/1413362
Commit-Queue: Lan Wei <lanwei@chromium.org>
Reviewed-by: Navid Zolghadr <nzolghadr@chromium.org>
Cr-Commit-Position: refs/heads/master@{#630933}
--
wpt-commits: 1ddffbeeb6faed345ce7f999544ed1d29c7a203f
wpt-pr: 14911
Automatic update from web-platform-tests
Verify that contentDocument is updated during nested browsing context document parsing.
--
wpt-commits: ae6d56308684b97f30a01b455ce3eb205cba79aa
wpt-pr: 15268
Automatic update from web-platform-tests
Fix canonical form of rotate property.
Expected behavior is outlined in the spec here:
https://drafts.csswg.org/css-transforms-2/#individual-transform-serialization.
Summary:
* 2d serializes to just an angle.
* Axis must be specified for 3-D rotations. If the axis is parallel to the x, y or z axis, the appropriate axis name must be used.
Bug: 905274
Change-Id: I5d16f3301baf9720099e4b5b42a1b87077b97998
Reviewed-on: https://chromium-review.googlesource.com/c/1447844
Commit-Queue: Kevin Ellis <kevers@chromium.org>
Reviewed-by: Anders Hartvoll Ruud <andruud@chromium.org>
Cr-Commit-Position: refs/heads/master@{#630796}
--
wpt-commits: a01ac89be3f48fa64bd96d36ac1a65ac9074eb4f
wpt-pr: 15205
Automatic update from web-platform-tests
Make stability checks provide an explicit message for duplicate test names
--
wpt-commits: 23008e948bff4c245148cfcda2ebaa72149a2bb1
wpt-pr: 15330
Automatic update from web-platform-tests
Update mozprocess from 0.26 to 1.0.0
--
Update mozprocess from 0.26 to 1.0.0
--
Update mozprocess from 0.26 to 1.0.0
--
Update mozprocess from 0.26 to 1.0.0
--
Update mozprocess from 0.26 to 1.0.0
--
Update mozprocess from 0.26 to 1.0.0
--
Update mozprocess from 0.26 to 1.0.0
--
Update mozprocess from 0.26 to 1.0.0
--
Update mozprocess from 0.26 to 1.0.0
--
wpt-commits: 9a874520787308eecb14d85399cb5184efaf99ec, 999c546d4958eb649a71f0339563218a22f6d281, e032140d4ba4954c0704356f4848cc8f1d2dd590, ad09e7414f098c7833a65d336f01e76ba8c1dc15, a54f2b6ed9445d74fded895c45b561b1e7fd9314, 47bb668028ce2b7f6e564b47aee7ed12ce720532, f55469ea2fe1ed49f6b3f2d9701b793fc051bb67, 78c2264a4d9b28e89070670d6a73df470ae45aaf, 725876f98dc3fe1dd86c131b73724ac0e0a41147
wpt-pr: 15294
Automatic update from web-platform-tests
[css-grid] Overflow should be computed with the actual logical bottom
In order to compute the 'auto' height of the grid container, we set
temporarily a value based on the tracks' size. We can see this value as
a kind of 'intrinsic height'. Then, we call to the UpdateLogicalHeight
function to finally compute the grid container's actual logical height.
In order to compute the overflow area in the grid container we should
use the result of the ClientLogicalBottom function. However, we were
retrieving this value just after setting the tracks-based temporary
height, and before computing the actual logical height.
The consequence of this wrong logic is that empty grid areas may
affect the result of the ClientLogicalBottom and, as it's described
in the bug, conclude that there is a content overflow in the grid
container, even if such container has no grid items at all.
Since the grid itself is not a box, it can't contribute to the grid
container's content size; instead, its grid items should.
This change ensures that we always use the actual grid container's
logical bottom to properly compute the overflow area.
Bug: 928885
Change-Id: I05f86fbce06a83c0dbc0d5389bf0416763f8588a
Reviewed-on: https://chromium-review.googlesource.com/c/1459620
Commit-Queue: Javier Fernandez <jfernandez@igalia.com>
Reviewed-by: Manuel Rego <rego@igalia.com>
Cr-Commit-Position: refs/heads/master@{#630735}
--
wpt-commits: aa6f52fc7b99559377c760a550e842438efa91a1
wpt-pr: 15284
Automatic update from web-platform-tests
Fix remaining preload tests flaking on wpt.fyi
This is a follow-up to https://github.com/web-platform-tests/wpt/pull/15075.
https://crrev.com/c/1436777 successfully fixed the wpt.fyi flakiness
for link-header-preload-srcset.tentative.html and
link-header-preload-nonce.html. This patch fixes that for the remaining
preload tests.
- Use step_timeout instead of dummy.js?pipe=trickle, because
dummy.js?pipe=trickle may be loaded from cache with no delay.
- Replace verifyNumberOfDownloads() with
- verifyNumberOfResourceTimingEntries() if possible
- A new helper function verifyLoadedAndNoDoubleDownload() that
doesn't fail if the resource was already cached before running
the test.
Bug: 922343
Change-Id: Ia9b7c4bd49dd76463df4607349fe4f935e5410d0
Reviewed-on: https://chromium-review.googlesource.com/c/1442020
Reviewed-by: Kinuko Yasuda <kinuko@chromium.org>
Commit-Queue: Kunihiko Sakamoto <ksakamoto@chromium.org>
Cr-Commit-Position: refs/heads/master@{#629029}
--
wpt-commits: eb5aa0038ee7a938beabbe4c732a685af2581dbc
wpt-pr: 15129
Automatic update from web-platform-tests
Initial support for WorkletAnimation.playbackRate
Changes for scroll-linked worklet animations will come in a separate pull request.
Bug: 852475
Change-Id: Ie6dd14242797cbc14a7e1e377f23c3dda174fe15
Reviewed-on: https://chromium-review.googlesource.com/c/1423537
Reviewed-by: Stephen McGruer <smcgruer@chromium.org>
Reviewed-by: Majid Valipour <majidvp@chromium.org>
Commit-Queue: Olga Gerchikov <gerchiko@microsoft.com>
Cr-Commit-Position: refs/heads/master@{#628800}
--
wpt-commits: 9d1de21ddfa4338cfac14a887622b98207efe247
wpt-pr: 14946
Automatic update from web-platform-tests
[animation-worklet] Basic pause implementation (reland)
Pausing worklet animation now holds the time. This works as expected for main
thread animations. Implementing this for composited worklet animations will be
done in a follow up patch.
Major changes:
- Add and expose pause() method pausing the animation.
- Introduce hold_time that is used when animation is paused.
- Rework how current time is computed, it is now closer to
regular animations i.e., we either compute it based on "start time and
timeline.currentTime" or use "hold time".
- Instead of setting start time we now set the current time
which then works backward to compute either the start time or the hold time
based on the animation state.
- When transitioning animation play state, we now always set
the current time. Previously this was adhoc and inconsistent.
- Introduce has_started_ to differentiate when playing an
animation for the first time vs playing it from pause.
- Update playback_rate related calculation to use new logic.
Relanding: Original CL was reverted here [1]. Took the following steps to
address this:
- Address flakiness in unit test by using more accurate error
value matching other tests.
- Fix flakiness in layout test (worklet-animation-pause.https.html)
by using startTime and waiting for time to actually advance working around
pre-existing animation clock issue [2]. Also got rid of an invalid state
transition from pause->idle.
[1] https://chromium-review.googlesource.com/c/chromium/src/+/1434815
[2] http://crbug.com/785940
TEST:
- wpt/animation-worklet/worklet-animation-pause.https.html: js test for basic
current time calculations
- wpt/animation-worklet/worklet-animation-pause-immediately.https.html:
reftest for basic pause
- wpt/animation-worklet/worklet-animation-pause-result.https.html: reftest for
pause/resume.
- WorkletAnimationTest.PausePlay: unit test for basic state
transition and time calc
Bug: 821910,
Change-Id: I11fd2960443081be81055904d6d56a2abc3282f5
Reviewed-on: https://chromium-review.googlesource.com/c/1456640
Reviewed-by: Majid Valipour <majidvp@chromium.org>
Reviewed-by: Yi Gu <yigu@chromium.org>
Commit-Queue: Majid Valipour <majidvp@chromium.org>
Cr-Commit-Position: refs/heads/master@{#630548}
--
wpt-commits: 345300fad3945a5f1441fb2b2001109ca48f36e8
wpt-pr: 15279
Automatic update from web-platform-tests
[Payment Request] Add Web Platform Tests for hasEnrolledInstrument().
The failing expectations are required because hasEnrolledInstrument()
is not yet turned on by default in Blink. These tests pass when running
Chrome with --enable-features=PaymentRequestHasEnrolledInstrument.
Bug: 915907
Change-Id: I2189bfedad813e7942c7ddbd44aac78d1b895a30
Reviewed-on: https://chromium-review.googlesource.com/c/1461313
Commit-Queue: Danyao Wang <danyao@chromium.org>
Reviewed-by: Rouslan Solomakhin <rouslan@chromium.org>
Cr-Commit-Position: refs/heads/master@{#630578}
--
wpt-commits: a7498ddcfeeca02e7b62ab9e9b359fcaa5764c62
wpt-pr: 15306
Automatic update from web-platform-tests
Deflake RTCPeerConnection-track-stats.https.html.
This CL does the following:
1. Add a test that verifies we have a succeeded candidate pair in
getStats() as soon as iceConnectionState is "connected". This is
currently flaky (https://crbug.com/926170) and marked as such.
2. Add workaround to RTCPeerConnection-track-stats.https.html so that
it does not flake due to 1), and mark it not flaky.
3. Move shared helper functions to RTCPeerConnection-helper.js.
4. Add test coverage for making sure we can become connected in the
recvonly use case. This requires changing the prefix to
".https.html" of "RTCPeerConnection-iceConnectionState".
// Already reviewed by jonasolsson@chromium.orgTBR=guidou@chromium.org
Bug: 922955, 926170
Change-Id: I828e46273a84447c817595a466a5e143bde30eca
Reviewed-on: https://chromium-review.googlesource.com/c/1442201
Reviewed-by: Henrik Boström <hbos@chromium.org>
Commit-Queue: Henrik Boström <hbos@chromium.org>
Cr-Commit-Position: refs/heads/master@{#627421}
--
wpt-commits: 7179024d5dfb231073c7319948499d7e30f671cb
wpt-pr: 15133
Automatic update from web-platform-tests
Update dynamic import tests for HTML event handler changes
Fixes https://github.com/web-platform-tests/wpt/issues/15183. Follows https://github.com/whatwg/html/pull/4302.
This updates paths/expected nonce behavior now some methods of loading are not based off the current script.
--
wpt-commits: 8ecb503bc748ded75e7f5c224b95344a2966eeb9
wpt-pr: 15251
Automatic update from web-platform-tests
Remove OriginalTimingAllowOrigin from ResourceTimingInfo
The OriginalTimingAllowOrigin attribute was added in
https://codereview.chromium.org/271083002 to ensure TAO headers are
preserved on cached resources (304 responses). This CL removes it
because it seems that it is no longer needed. It also moves the
corresponding test to WPT.
Bug: 929453
Change-Id: I41e0c84b04a94acc27c22e375f55398d9fc411e5
Reviewed-on: https://chromium-review.googlesource.com/c/1459333
Commit-Queue: Nicolás Peña Moreno <npm@chromium.org>
Reviewed-by: Yoav Weiss <yoavweiss@chromium.org>
Cr-Commit-Position: refs/heads/master@{#630441}
--
wpt-commits: f84a067164651cc9069c36d6098d6a6b03d63816
wpt-pr: 15300
Automatic update from web-platform-tests
[css-overscroll-behavior] Computed values (#15287)
The computed values of the overscroll-behavior-x and overscroll-behavior-y
properties are the keywords as specified.
https://drafts.csswg.org/css-overscroll-behavior/#property-index
--
wpt-commits: 374bcf2e5aa678ce021ea8e8d0872114b9fd1acf
wpt-pr: 15287