In practice, Android never enforced restrictions on the tag length, and
in newer versions, the restriction is removed, so we shouldn't limit the
tag length at all.
MozReview-Commit-ID: JQF9FBdB5Fj
--HG--
extra : rebase_source : 71aa09210d694b68a72043f7588fbd799f385c23
Use the new "debug" and "warn" functions with template literals in
existing code.
MozReview-Commit-ID: 4ob6mom6pQF
--HG--
extra : rebase_source : 564f23c8de1256f73c085845fe030d8bbf45b19c
Inject new logging functions, "debug" and "warn", into each GeckoView JS
module that geckoview.js loads. Also do the same thing for frame script
classes that extend from GeckoViewContentModule.
The new logging functions are used with template literals (debug `hello
${foo} world`;), which are lazily evaluated, so disabled logs don't use
as many CPU cycles. They can also be easily enabled/disabled.
MozReview-Commit-ID: 7ZfYAMrcCyU
--HG--
extra : rebase_source : 8a830f29ea1cabcdc5055fc86c9880a5216aa456
Make Log.jsm functions support tagged template literals. For example,
instead of |logger.debug("foo " + bar)| or |logger.debug(`foo ${bar}`)|,
you can now use |logger.debug `foo ${bar}`| (without parentheses).
Using tagged template literals has the benefit of less verbosity
compared to regular string concatenation, with the added benefit of
lazily-stringified parameters -- the parameters are only stringified
when logging is enabled, possibly saving from an expensive stringify
operation.
This patch also fixes a bug in BasicFormatter where consecutive tokens
are not formatted correctly (e.g. "${a}${b}").
MozReview-Commit-ID: 9kjLvpZF5ch
--HG--
extra : rebase_source : ccf4e9fae9fa9ea7581de82296035fcc736ca58e
Add an AndroidAppender that lets Log.jsm output to the Android logs,
using AndroidLog.jsm. Because the Android logging system keeps track of
the log metadata (time/level/name) separately from the log message, the
patch also adds a separate AndroidFormatter that does not prepend the
metadata to the log message itself.
MozReview-Commit-ID: C9oBbgVQOEc
--HG--
extra : rebase_source : eb1e8622b059ee45b574830426194ea35643b37c
If the content process crashes, marionette can return None rather
than a valid result. In this case we want the test status to end up
as crash, which happens if we just propogate the None upwards.
Automatic update from web-platform-testsAdd tests for Event.srcElement
See https://github.com/whatwg/dom/issues/625 for details.
--
Add tests for Event.returnValue
See https://github.com/whatwg/dom/issues/625 for details.
wpt-commits: 13597c4af7ac923309e740920cd42bed88113e5f, 24f49ff15e22f7d81dbb87908efa0b5970b7add6
wpt-pr: 10258
wpt-commits: 13597c4af7ac923309e740920cd42bed88113e5f, 24f49ff15e22f7d81dbb87908efa0b5970b7add6
wpt-pr: 10258
Automatic update from web-platform-testsAvoid async/await in /webstorage/idlharness.html
Am sorry, Servo is dumb, we are actively trying to fix that on our side.
wpt-commits: 95992cd324d80acc68707b203be8c6cc2aebc6d5
wpt-pr: 10357
wpt-commits: 95992cd324d80acc68707b203be8c6cc2aebc6d5
wpt-pr: 10357
Automatic update from web-platform-testsservice worker: Add tests for inteception of workers after redirects.
This tests behavior discussed here:
https://github.com/w3c/ServiceWorker/issues/1289
Namely it tests when a request for a worker goes through a redirect
chain:
1) On redirect from A -> B, whether the service worker at B
sees the request.
2) After the final redirect, which service worker controls the
resulting client.
The tests are written as specified today. Therefore, Firefox
passes this test (verified in Nightly) and Chrome does not.
(Actually a small change is required to the test to make Firefox
pass it, see: https://bugzilla.mozilla.org/show_bug.cgi?id=1452528)
Currently it only tests shared worker but dedicated worker can
be added in a follow-up patch.
Bug: 829720
Change-Id: Id3b1ea8b952760be0ef9917f2c6a3afe60ca1fb5
Reviewed-on: https://chromium-review.googlesource.com/999241
Commit-Queue: Matt Falkenhagen <falken@chromium.org>
Reviewed-by: Hiroki Nakagawa <nhiroki@chromium.org>
Cr-Commit-Position: refs/heads/master@{#549125}
wpt-commits: 6fe36d79072d5261ea504435b0dfedaf39f5805a
wpt-pr: 10340
wpt-commits: 6fe36d79072d5261ea504435b0dfedaf39f5805a
wpt-pr: 10340
Automatic update from web-platform-testsCorrect serialization of box-shadow and text-shadow
Following https://github.com/w3c/csswg-drafts/issues/2305, the canonical serialization for box-shadow was changed to the color then lengths. https://drafts.csswg.org/css-text-decor-3/#text-shadow-property also defines the canonical serialization for text-shadow as the color and then lengths. This caused the test to fail in Firefox, Chrome, and Safari. Update the test to match the spec in both instances.
wpt-commits: a835486e59a94236a55107fe34925079b33ef247
wpt-pr: 9800
wpt-commits: a835486e59a94236a55107fe34925079b33ef247
wpt-pr: 9800
Automatic update from web-platform-tests[css-typed-om] Implement support for text-* properties.
There's a failing test due to Blink computing keyword values for
text-size-adjust to percentages, even though the spec says the computed
value is as specified.
Bug: 820299
Change-Id: I53c4177f1ce2205a4f2b6af00b1c61da4f5e774f
Reviewed-on: https://chromium-review.googlesource.com/994594
Commit-Queue: Darren Shen <shend@chromium.org>
Reviewed-by: Rune Lillesveen <futhark@chromium.org>
Cr-Commit-Position: refs/heads/master@{#549095}
wpt-commits: 8a4cdaf6df416d33c0b2cb591fb078d7448591db
wpt-pr: 10301
wpt-commits: 8a4cdaf6df416d33c0b2cb591fb078d7448591db
wpt-pr: 10301
Automatic update from web-platform-testsUpdate expected cssText for custom property serialization
See https://github.com/w3c/csswg-drafts/issues/2509#issuecomment-379152590 for the change to target9. Also remove the extra whitespace in target8 and target1 which causes these tests to fail in Chrome and Firefox.
Also remove testcase.propertyName from each test's name as this doesn't exist and just outputs undefined.
wpt-commits: d0d62244432d329aa22e9278d3e83184301c3e7f
wpt-pr: 10353
wpt-commits: d0d62244432d329aa22e9278d3e83184301c3e7f
wpt-pr: 10353
Automatic update from web-platform-testsImprove idlharness error message for exposure test (#10336)
When `obj` is not created successfully, there's an error like
> Cannot use 'in' operator to search for [member.name] in undefined
NB: I'm not a fan of having each sub-test re-test the same thing, but this is at least _consistent_ with the other code..
wpt-commits: 6e68c444ed00317ee7035fd2e1e144eb4a6a6091
wpt-pr: 10336
wpt-commits: 6e68c444ed00317ee7035fd2e1e144eb4a6a6091
wpt-pr: 10336
Automatic update from web-platform-testsCookie Store: Tentatively deflake special names test
The "expires" cases are flaky on all platforms on the bots, although
I can't reproduce locally. The test case sets a cookie that expires
"now" then expects it can't read it. It may be flaky due to "now"
being slightly different between processes, threads, and libraries,
allowing the read to succeed.
Set the expiry date to be an hour before "now", which should be
sufficient.
Bug: 829761
Change-Id: Icda1891310dd0a56769877f32405226390e729de
Reviewed-on: https://chromium-review.googlesource.com/999743
Commit-Queue: Joshua Bell <jsbell@chromium.org>
Reviewed-by: Victor Costan <pwnall@chromium.org>
Cr-Commit-Position: refs/heads/master@{#548903}
wpt-commits: a4c17db5364ca97ffa9bb76f9c8e6e155ecd3ddb
wpt-pr: 10349
wpt-commits: a4c17db5364ca97ffa9bb76f9c8e6e155ecd3ddb
wpt-pr: 10349
Automatic update from web-platform-testsMove AudioNode tests to WPT
Move the AudioNode tests to WPT.
Can't move tail-processing.html because it's specific to Chrome's tail
processing.
Firefox passes all tests except:
* audionode.html (because new AudioContext(1,44100,44100) succeeds but
shouldn't
* tests that use oac.suspend
* audionode-connect-method-chaining.html with an IndexSizeError
connecting
Bug: 745778
Change-Id: I0e21e2117a75246367ea2642759ed917ee3e028f
Reviewed-on: https://chromium-review.googlesource.com/994136
Reviewed-by: Hongchan Choi <hongchan@chromium.org>
Reviewed-by: Dirk Pranke <dpranke@chromium.org>
Commit-Queue: Raymond Toy <rtoy@chromium.org>
Cr-Commit-Position: refs/heads/master@{#548843}
wpt-commits: 68b07988d40f8a0250675fd0de3c6ae96921608e
wpt-pr: 10320
wpt-commits: 68b07988d40f8a0250675fd0de3c6ae96921608e
wpt-pr: 10320
Automatic update from web-platform-testsCookie Store: rework tests for 'expires' option
Simplify/rework the tests for the 'expires' options. Don't run the
tests with the special secure-only prefixes, as that's orthogonal to
the behvior.
In Chromium, this also reveals that support for HTTP date strings as
'expires' values, as described in the explainer[1] is not yet
supported.
[1] https://github.com/WICG/cookie-store/blob/gh-pages/explainer.md
Bug: 729800
Change-Id: I015fd643f18b59c8294f9dc62d576682132795fd
Reviewed-on: https://chromium-review.googlesource.com/994284
Commit-Queue: Joshua Bell <jsbell@chromium.org>
Reviewed-by: Victor Costan <pwnall@chromium.org>
Cr-Commit-Position: refs/heads/master@{#548818}
wpt-commits: 8b9501b721963f2b3886f25e20f17a2afe6bc381
wpt-pr: 10298
wpt-commits: 8b9501b721963f2b3886f25e20f17a2afe6bc381
wpt-pr: 10298
Automatic update from web-platform-testsUpdate ReadableStream to latest standard version
Update Blink's implementation of ReadableStream to
https://streams.spec.whatwg.org/commit-snapshots/37ace3d5f16cbea7aec4a0c80532c95059994d51/
This also updates TransformStream to use CreateReadableStream() instead
of calling the ReadableStream constructor.
Several common operations have become unused and have been removed.
The EXTERNALLY_CONTROLLED flag has been renamed to the slightly more
specific name "BLINK_LOCK_NOTIFICATIONS".
All external/wpt/streams/readable-streams tests now pass, and failing
expectations have been removed.
Most changes were already covered by the web-platform-tests. A new
readable-streams/patched-global.js test verifies that
ReadableStreamTee() does not use the global ReadableStream constructor
or otherwise touch the global object.
BUG=710728
Change-Id: I2b799045f47376adef305f5f88ad106161425b46
Reviewed-on: https://chromium-review.googlesource.com/992077
Commit-Queue: Adam Rice <ricea@chromium.org>
Reviewed-by: Yutaka Hirano <yhirano@chromium.org>
Cr-Commit-Position: refs/heads/master@{#548784}
wpt-commits: 8314b7b5ae86ebf5bc435558ae5b0af32b1d7756
wpt-pr: 10277
wpt-commits: 8314b7b5ae86ebf5bc435558ae5b0af32b1d7756
wpt-pr: 10277
Automatic update from web-platform-testsFix the |name| of PerformanceNavigationTiming
Previously, the |name| of a PerformanceNavigationTiming entry was the initial
URL (the request URL). After this CL, it is the response URL, so for example
a url of the form 'redirect?location=newLoc' will have 'newLoc' as the |name|.
Bug: 797465
Change-Id: Icab53ad8027d066422562c82bcf0354c264fea40
Reviewed-on: https://chromium-review.googlesource.com/996579
Reviewed-by: Yoav Weiss <yoav@yoav.ws>
Commit-Queue: Nicolás Peña Moreno <npm@chromium.org>
Cr-Commit-Position: refs/heads/master@{#548773}
wpt-commits: a0091fbff26091db1fd4b6ca07d9512edf7e6a95
wpt-pr: 10339
wpt-commits: a0091fbff26091db1fd4b6ca07d9512edf7e6a95
wpt-pr: 10339
Automatic update from web-platform-testsMake BlueTooth/USB requestDevice non-consuming.
The specs for both WebBlueTooth & WebUSB only require a user
activation during a requestDevice call. So consuming the activation
(vs just checking) is not a strong requirement. There was a concern
that "not consuming" could possibly mean multiple choosers for a single
user activation but our browser code already handles the situation
gracefully. Multiple requests to browser is possible from different
tabs anyway, even with (isolated) consumptions in individual tabs.
Therefore it makes sense to remove the consumption behavior here,
which is not spec-ed anyway. This would fit our goal of a simple
user activation model ("UserActivationV2" project) which should be
simple enough for cross-browser implementation and for a complete spec.
Bug: 786407
Change-Id: I3c9dbf43df34d4cb4a2a33214a6c44847fa7435b
Reviewed-on: https://chromium-review.googlesource.com/775866
Reviewed-by: Reilly Grant <reillyg@chromium.org>
Reviewed-by: Giovanni Ortuño Urquidi <ortuno@chromium.org>
Commit-Queue: Mustaq Ahmed <mustaq@chromium.org>
Cr-Commit-Position: refs/heads/master@{#548768}
wpt-commits: 6ba01ea2ffbafbb329352628b244815654b38156
wpt-pr: 10130
wpt-commits: 6ba01ea2ffbafbb329352628b244815654b38156
wpt-pr: 10130
Automatic update from web-platform-testsImprove error message for exposure_set test
`memberName` is undefined; error message is (was) `ReferenceError: memberName is not defined`.
Clearly supposed to be a check for duplication - `array.length < 0` was a really weird thing to see, though. Spec says it's always non-negative, so deleted that.
Error I see now is
"Multiple 'Exposed' extended attributes on Navigator"
wpt-commits: 6dd31359159f88202ef73da68c4ad2eef2a3bc99
wpt-pr: 10337
wpt-commits: 6dd31359159f88202ef73da68c4ad2eef2a3bc99
wpt-pr: 10337