summaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeLines
* Issue #618 - (async) Implement async attribute for inline module scripts.Moonchild2020-08-25-41/+73
| | | | | | | | | | 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.
* Issue #618 - (async) Keep track of script modes in a single mode state.Moonchild2020-08-25-17/+61
| | | | | | 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.
* Issue #618 - (async, preload) Correctly pass info about async/defer to parser.Moonchild2020-08-24-15/+53
| | | | | | 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.
* Issue #618 - (async) Split out function to add async request.Moonchild2020-08-24-23/+31
|
* Issue #618 - Rename some script load request flags to be more descriptive.Moonchild2020-08-24-11/+11
|
* Issue #618 - Make document.currentScript null in modules.Moonchild2020-08-23-4/+7
| | | | Because the spec says so.
* Issue #618 - Fix processing of non-parser-generated module scripts.Jon Coppeard2020-08-22-23/+32
|
* Issue #618 - Handle errors for inline module scripts and ensure we update theJon Coppeard2020-08-22-12/+18
| | | | module map after fetch errors.
* Issue #618 - Split SRI verification out from OnStreamComplete.Moonchild2020-08-22-19/+35
|
* Issue #618 - Split handling of load errors out from OnStreamComplete.Moonchild2020-08-22-59/+67
|
* Issue #618 - Integrity for descendant module scripts should be the empty stringMoonchild2020-08-22-4/+4
| | | | Because the spec says so.
* Issue #618 - Don't mute errors for module scripts because they always use CORSJon Coppeard2020-08-22-4/+4
|
* Merge branch 'master' into es-modules-workMoonchild2020-08-22-16407/+22265
|\
| * Follow up to eb28b1f32 - Correct inputmethod build file by putting ↵Matt A. Tobin2020-08-19-1/+2
| | | | | | | | | | | | Keyboard.jsm back in EXTRA_JS_MODULES JSMs are NOT Components.
| * [js] Reinstate precise floating point model for all js sources.Moonchild2020-08-19-1/+1
| |
| * Issue #457 - Silence SQLite compiler warningMoonchild2020-08-18-0/+4
| | | | | | | | | | | | | | | | | | | | | | | | | | Our use of SQLite explicitly avoids using MSVC's _msize which the lib would notrmally use. This was set up this way in BZ bug 719579 for legitimate reasons (because _msize would end up using the system symbol, which would crash trying to get the size information from jemalloc allocations). Since we build the SQLite lib separately there is no easy way to avoid this warning as it's not directly linked against mozglue.dll which has the jemalloc functions it wants for _msize when not using the system allocator. It is implicitly assuming the function is an extern returning an int (with the call specifically casting to int) which is correct (well, correct _enough_, anyway) so the warning can safely be ignored for our specific use as it all lines up at link time.
| * [db] Update SQLite lib to 3.33.0Moonchild2020-08-18-16361/+22133
| |
| * Merge pull request #1632 from athenian200/link_element_disabledMoonchild2020-08-18-44/+123
| |\ | | | | | | Respond to disabled attribute set on <link> elements from HTML
| | * Fix nitsathenian2002020-08-14-3/+3
| | |
| | * Issue #1629 - Part 3: Implement behind preference.athenian2002020-08-14-7/+26
| | | | | | | | | | | | | | | | | | This is not very "clean," and is mostly done in the same sloppy way as Emilio did it because that's basically the only way you can do it. Note well that this does NOT actually turn off everything I've done in a clean fashion like ifdefs would. For instance, the Explicitly Enabled flag is still present, but is now always false because the only condition that can set it true is behind the pref and therefore inert when this pref is off. Also, because the arguments of SetDisabled have changed, my modifications to SetMozDisabled must be present regardless of whether the pref is on or off. What I have done is turn off the actual reflection of the disabled attribute in Disabled and SetDisabled, as well as in AfterSetAttr. However, turning the pref off seems to restore more or less our old behavior, though there may be subtle differences unlike with an ifdef since this is, unfortunately, not an exact science and I can only turn off changes that happen within individual functions and not changes in how functions interact with each other.
| | * Issue #1629 - Part 2: Implement the Explicitly Enabled flag.athenian2002020-08-13-31/+79
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This part of the bug was significantly complicated by the following major refactors: https://bugzilla.mozilla.org/show_bug.cgi?id=1456435 https://bugzilla.mozilla.org/show_bug.cgi?id=1459498 As best as I can tell, we just need to implement the explicitly enabled flag on every instance of GetStyleSheetInfo, make sure aIsExplicitlyEnabled is false in every situation except the one where the disabled content attribute is removed from a link element, and enable alternate stylesheets if this flag is set on them. So we take the explicitly enabled flag as an input to PrepareSheet, and also add it to LoadStyleLink and LoadInlineStyle. I also decided not to defer loading of alternate stylesheets that have been explicitly enabled.
| | * Issue #1629 - Part 1: Implement basic logic in HTMLLinkElement.athenian2002020-08-13-12/+24
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | So basically, I'm trying to adapt this to UXP: https://bugzilla.mozilla.org/show_bug.cgi?id=1281135 The earliest source of difficulty while adapting Bug 1281135 to our codebase was simply getting the new ErrorResult flag added to the SetDisabled function to play nice with the SetMozDisabled function. At this point, the implementation can actually have a stylesheet be disabled by default but there are supposedly issues with alternate stylesheets. At first I played around with the return type of SetMozDisabled to no avail, but I found another solution fairly quickly. https://bugzilla.mozilla.org/show_bug.cgi?id=846972 https://bugzilla.mozilla.org/show_bug.cgi?id=1157898 Essentially, the way around the problem of the number of return arguments not matching up is to declare a local variable within SetMozDisabled called ErrorResult rv, and using that to store the return value of the ErrorResult argument from SetDisabled. After that, because ErrorCode was removed, you would return rv.StealNSResult() in order to report success or failure to any consumer that calls on SetMozDisabled.
| * | Merge pull request #1635 from g4jc/1620Moonchild2020-08-17-0/+2
| |\ \ | | | | | | | | Intrinsic Aspect Ratio: Debug Follow up.
| | * | Issue #1620 - Intrinsic Aspect Ratio: Debug Follow up.Gaming4JC2020-08-16-0/+2
| |/ / | | | | | | | | | | | | | | | | | | Newly introduced aspect-ratio property did not have CSS_PROP_LIST_EXCLUDE_INTERNAL defines, resulting in the following assertion: \!nsCSSProps::PropHasFlags(p, (1<<28)) (properties defined outside of #ifndef CSS_PROP_LIST_EXCLUDE_INTERNAL sections must not have the CSS_PROPERTY_INTERNAL flag), at ...layout/style/nsCSSProps.cpp:289 This patch resolves the assertion by adding #ifndef around the aspect-ratio property.
* | | Issue #618: Pass down referrer and referrer policy when fetching modules.Moonchild2020-08-13-25/+42
| | | | | | | | | | | | Because the spec says so.
* | | Issue #618: Ignore 'event' and 'for' attributes for module scripts.Moonchild2020-08-13-23/+30
| | | | | | | | | | | | Because the spec says so.
* | | Merge branch 'master' into es-modules-workMoonchild2020-08-13-2981/+4289
|\| |
| * | [CSS] Alias -webkit-appearance for compatibility reasonsMoonchild2020-08-10-0/+5
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Since this is supported as an alias by Firefox and Edge for the same reasons and we have websites using this to (attempt to) override the system-provided styling with their own, leaving out the only supported keyword we'd otherwise have (with -moz- prefix) but still stating -webkit-. TODO: unprefix this completely and make the vendor prefixes aliases.
| * | Merge pull request #1613 from RealityRipple/AspectRatioMoonchild2020-08-08-155/+379
| |\ \ | | | | | | | | [Image/CSS] Intrinsic Aspect Ratio
| | * | Issue #1620 - Enable Intrinsic Ratio by DefaultAndy2020-08-07-2/+2
| | | | | | | | | | | | | | | | A simpler name feels so much cleaner.
| | * | Issue #1620 - Remove Development CommentsAndy2020-08-07-18/+4
| | | |
| | * | Issue #1620 - Use Intrinsic Aspect Ratio for ImagesAndy2020-08-04-143/+381
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | https://bugzilla.mozilla.org/show_bug.cgi?id=1547231 https://bugzilla.mozilla.org/show_bug.cgi?id=1559094 https://bugzilla.mozilla.org/show_bug.cgi?id=1633434 https://bugzilla.mozilla.org/show_bug.cgi?id=1565690 https://bugzilla.mozilla.org/show_bug.cgi?id=1602047 Make use of Aspect Ratios in Image frames before Images are loaded. - Check for width and height HTML properties and create a ratio with them. - Overwrite HTML size values with actual image dimensions on load. - Collapse any frames with srcless images. Comments: dom/html/nsGenericHTMLElement.cpp:1483 layout/generic/nsImageFrame.cpp:289
| * | | Pref and disable getRootNode()Moonchild2020-08-07-1/+3
| | | | | | | | | | | | | | | | | | | | | | | | | | | | This is apparently used for fallback selection and if available it is "assumed" Shadow DOM is also available, while this is a utility function. Webcompat is a nightmare sometimes.
| * | | Issue #1628 - Remove redundant PREF_EM_UPDATE_BACKGROUND_URLMatt A. Tobin2020-08-07-8/+1
| |/ /
| * | Merge pull request #1623 from g4jc/libeditor_patchMoonchild2020-08-04-340/+411
| |\ \ | | | | | | | | Improve Performance of libeditor
| | * | Issue #1621 - Part 4: Check whether node can be splited.Gaming4JC2020-07-31-7/+12
| | | | | | | | | | | | | | | | | | | | | | | | At first, HTMLEditor::GetActiveEditingHost might return null in this situation, we should check whether nullptr is returned. At second, SplitNodeDeep returns error since curent is design mode and selection node has no parent. So we should check error. Ref: Bug 1350772
| | * | Issue #1621 - Part 3: Use nsIAtom to change attirbute if possible.Gaming4JC2020-07-31-64/+51
| | | | | | | | | | | | | | | | | | | | | | | | We can replace old nsIEditor API with nsIAtom version. Ref: Bug 1324996
| | * | Issue #1621 - Part 2: Implement nsIAtom version of ↵Gaming4JC2020-07-31-100/+143
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | SetAttribute/RemoveAttribute/CloneAttirubte. Add nsIAtom version of the following. - CloneAttribute - RemoveAttribute - RemoveAttributeOrEquivalent - SetAttribute - SetAttributeOrEquivalent Ref: Bug 1324996
| | * | Issue #1621 - Part 1: CSSEditUtils should use atom for CSS property if possible.Gaming4JC2020-07-31-188/+224
| | | | | | | | | | | | | | | | | | | | | | | | There is a lot of string compare when using CSS property name. We should use nsGkAtoms instead. Ref: Bug 1323138
| * | | Issue #1625 - Allow MailNews Oauth2 support to be configured in confvars.shMatt A. Tobin2020-08-03-1/+0
| | | | | | | | | | | | | | | | Baby has his bottle, you ok now?
| * | | [js] Try to catch bad pointers for GC and bail if not valid.Moonchild2020-08-03-0/+42
| | | |
| * | | Merge pull request #1622 from RealityRipple/masterMoonchild2020-08-03-229/+361
| |\ \ \ | | | | | | | | | | Convert Intrinsic Ratio from nsSize to Float
| | * | | Issue #1619 - Nits PickedAndy2020-08-02-8/+8
| | | | |
| | * | | Issue #1619 - Add Vertical Writing TestcaseAndy2020-08-01-20/+57
| | | | | | | | | | | | | | | | | | | | | | | | | Ensures aspect ratio numerator and denominator aren't swapped in vertical writing modes. https://bugzilla.mozilla.org/show_bug.cgi?id=1548768
| | * | | Issue #1619 - Missing Dimension ComputationAndy2020-07-31-0/+10
| | | | | | | | | | | | | | | | | | | | | | | | | This existed in Firefox before this bug. I don't know if it came from a previous bug or was removed post-fork.
| | * | | Issue #1619 - Convert Intrinsic Ratio to FloatAndy2020-07-31-229/+314
| | |/ / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | https://bugzilla.mozilla.org/show_bug.cgi?id=1547792 Aspect Ratio handling simplified by using floating point integers: - Multiplication of value (or inverse value) to a known side for Scaling - No unequal equal values such as "4/3" vs "8/6" vs "20/15" - Truly "Empty" aspect ratios, even if one dimension is not 0
| * | | Add license header to media/webrtc/trunk/build/mac/find_sdk_uxp.pyMatt A. Tobin2020-07-31-0/+3
| | | |
| * | | Use an alt script to properly determine the OSX SDK versionMatt A. Tobin2020-07-31-1/+48
| |/ /
| * | Merge pull request #1618 from g4jc/libeditor_patchMoonchild2020-07-31-20/+14
| |\ \ | | | | | | | | Clean up GetTextNode into TextEditRules.
| | * | Bug 1318570 - Clean up GetTextNode into TextEditRules.Gaming4JC2020-07-30-20/+14
| |/ / | | | | | | | | | | | | | | | EditorBase parameter of GetTextNode is unnecessary because it uses static method only. Also, we should return nsINode to reduce QI. Resolves #1617