| Commit message (Collapse) | Author | Age | Lines |
... | |
|
|
|
|
|
| |
There are situations where nsCSSClipPathinstance->CreateClipPath(dt)
returns null. We need to check for this before trying to use its
functions. If there is no clip path, then always return "no hit".
|
|
|
|
|
|
| |
Host name was purely being used for HPKP and since HPKP is killed,
this can also go. Currently it doesn't do anything other than
generating build warnings.
|
|
|
|
| |
Since the local variable is always initialized to false, we don't actually need to declare it and can just pass "false" directly as a parameter to the PrepareSheet function's bool. I was worried about code readability at first, but some well-placed comments took care of that.
|
|
|
|
| |
This clarifies the assumptions the code is making and the order in which the variables pass through the loading process. The new variable is set after the sheet is created and prepared, and is assumed to be false in the beginning.
|
|
|
|
| |
release build assertions for mismatching compartments.
|
| |
|
|
|
|
| |
This hooks up module scripts to the existing preload mechanism.
|
| |
|
| |
|
|
|
|
| |
This is just a clean port of 1322191 and follow-up 1325970. It really seems to add create a new way to access existing code relating to block formatting and floating elements rather than implementing new functionality, and it is mercifully straightforwards.
|
|
|
|
|
| |
Adds a self-hosted implementation of this map->object conversion.
This resolves #1639.
|
| |
|
| |
|
| |
|
| |
|
| |
|
| |
|
| |
|
| |
|
| |
|
|
|
|
|
| |
Basically, NSC_GetTokenInfo doesn't lock slot->slotLock before accessing slot
after obtaining it, even though slotLock is defined as its lock.
|
| |
|
| |
|
| |
|
|
|
|
|
|
|
| |
Unless a user is debugging media errors, this detail is unnecessary to report
and could include sensitive data which could be abused by third-party
requesters. This aligns it with the standard success/error paradigms in normal
browsing situations.
|
|
|
|
|
|
|
|
|
|
| |
This commit does several things:
- Moves the pref check from ScriptLoader to ns[I]Document so it can be called on
the document.
- Changes the atrribute freezing function to a better name that takes the
document as a parameter.
- Sets the proper async/defer attributes on HTML script elements based on
keywords and whether they are module scripts or not.
|
|
|
|
|
|
| |
This simplifies handling of combinations of async/defer by assigning one and
only one state to scripts.
If async then always async > if defer or module then defer > otherwise blocking.
|
|
|
|
|
|
| |
This makes sure we don't block body-referred sub-resources by head-referenced
defer and async scripts. This is important for all script loads, not just
modules, but is added here because it was run into while implementing modules.
|
| |
|
| |
|
|
|
|
| |
Because the spec says so.
|
| |
|
|
|
|
| |
module map after fetch errors.
|
| |
|
| |
|
|
|
|
| |
Because the spec says so.
|
| |
|
|
|
|
| |
Because the spec says so.
|
|
|
|
| |
Because the spec says so.
|
|
|
|
|
| |
This is an ahead-of time port to try and address #1624.
This is based on BZ 1461751 and Jon Coppeard's work in it.
|
| |
|
|
|
|
|
|
| |
a visited set
Ref: BZ 1365187
|
|
|
|
| |
Ref: BZ 1365187
|
|
|
|
|
|
|
|
| |
follow up
MODULE_STATUS_ERRORED is no more. Replacing with newer API.
Ref: BZ 1420420
|
|
|
|
|
|
| |
The added debug assertion does not work due to missing API.
They were added in BZ 1337491, 1395366, and others, but were primarily used for multi-threading.
This uses our existing non-multithreaded syntax instead, resolving a `no member named` build error.
|
|
|
|
|
|
| |
handling works correctly
Ref BZ 1395896
|
|
|
|
|
|
|
| |
Make it so they are not allocated on a background thread in a different zone to
the final module.
Ref: BZ 1372258
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
According to the spec this isn't created until the module is instantiated, but
we create it when we compile the module.
We stored this previously in InitialEnvironmentSlot and copied it to
EnvironmentSlot when it was supposed to be created, but we can just store it in
the latter slot straight away and check the module's status and return null if
it shouldn't exist yet.
This reduces the number of slots needed on a moduleObject to 17.
Re: BZ 1420412 Part 1
We can't implement the second part to further reduce our number of slots,
because it relies on SetProxyReservedSlot which in turn relies on
rearchitecturing JS proxies to make reserved slots dynamic. That's a rabbit hole
we really don't want to fall into.
So, we'll end up being a bit slower because it can't be in-line allocated with
having more than 16 slots, but so be it. I sincerely doubt it will make any
practical difference.
|
|
|
|
|
|
|
|
| |
This updates module implementation to match spec regarding handling of
instantiation errors, after it was changed yet again, this time to not remember
instantiation errors, but instead immediately rethrow applicable ones.
Ref: BZ 1420420
|
|
|
|
|
|
|
| |
When module dependencies fail, don't spam with errors for each import; only fire
the error event once.
Ref: BZ 1421259
|