summaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeLines
...
| * | MoonchildProductions#1251 - Part 10: ipc_channel_posix.cc should use IOV_MAX.athenian2002019-10-21-1/+10
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | https://bugzilla.mozilla.org/show_bug.cgi?id=1345102 I assess this change to be low-risk for the following reasons: 1. It has been in Firefox since version 55 without issues. 2. The current behavior is not POSIX compliant, and is retained in the one instance where the new functionality causes issues. 3. It makes safer assumptions about implementation details than what we have now.
| * | MoonchildProductions#1251 - Part 9: Look for hypot in the math library (libm).athenian2002019-10-21-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | https://bugzilla.mozilla.org/show_bug.cgi?id=1351309 https://bugzilla.mozilla.org/show_bug.cgi?id=1309157 I assess this change to be low-risk for the following reasons: 1. It has been in Firefox since version 55 without issues. 2. It's nearly identical to the fix for bug 1309157 which is already in our tree, so that one would be causing problems if this one were going to. 3. Linux, Windows, and BSD are known to have a hypot function in their math libraries. 4. Even if it does break something, it should only break a test and not critical functionality.
| * | MoonchildProductions#1251 - Part 8: Align pointer for char_16t.athenian2002019-10-21-2/+17
| | | | | | | | | | | | | | | | | | | | | | | | https://bugzilla.mozilla.org/show_bug.cgi?id=1352449 Mozilla patch that's been in the code since Firefox 55. Seems like there have been no ill effects from implementing it, and it would only increase the portability of the UXP code. All the Solaris Firefox repos I've seen implement some variation on the jsexn patch, and this seems to be the cleanest version of it. I can add ifdefs if needed or there are performance concerns associated with this patch, but I get the impression this alignment backlog issue might affect a few platforms other than Solaris, though none were named. Otherwise I think they wouldn't have used "platforms that need it" in plural form or failed to ifdef it.
| * | MoonchildProductions#1251 - Part 7: All the posix_m* memory-related stuff, ↵athenian2002019-10-21-6/+33
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | gathered together. https://bugzilla.mozilla.org/show_bug.cgi?id=1158445 https://bugzilla.mozilla.org/show_bug.cgi?id=963983 https://bugzilla.mozilla.org/show_bug.cgi?id=1542758 Solaris madvise and malign don't perfectly map to their POSIX counterparts, and some Linux versions (especially Android) don't define the POSIX counterparts at all, so options are limited. Ideally posix_madvise and posix_malign should be the safer and more portable options for all platforms.
| * | MoonchildProductions#1251 - Part 6: Solaris needs an audio implementation.athenian2002019-10-21-0/+520
| | | | | | | | | | | | Current versions of libcubeb already have a Sun audio implementation, but in Firefox 52 and earlier, this was all they had. I'm not completely happy with this implementation because it has issues like video freezing if the soundcard isn't working, but I think fixing this or pulling in a newer libcubeb would be going too far for too little gain.
| * | MoonchildProductions#1251 - Part 5: Fix POSIX compliance issue in ↵athenian2002019-10-21-1/+3
| | | | | | | | | | | | | | | | | | | | | | | | process_util.h. https://bugzilla.mozilla.org/show_bug.cgi?id=1364865 Solaris doesn't define NAME_MAX because if you read the current POSIX standard literally, no system that supports multiple file systems or networking should be defining it. It's a pedantic choice given that they USED to define NAME_MAX, but Solaris always did take POSIX compliance seriously, for better or worse.
| * | MoonchildProductions#1251 - Part 4: Core build system changes, lots of ↵athenian2002019-10-21-32/+682
| | | | | | | | | | | | | | | | | | | | | | | | libevent/IPC junk. This is mostly ifdefs, but as you can see, Solaris is actually a lot like Linux. They're both more SysV than BSD at core, and most of the differences have more to do with Solaris not using glibc than anything else. I still need to audit a lot of these changes and understand why they're needed and what the alternative approaches are. After this patch, most of the core functionality needed to build Solaris is here.
| * | MoonchildProductions#1251 - Part 3: Finally end the long tradition of ↵athenian2002019-10-21-1/+15
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | casting getpid() to int. https://bugzilla.mozilla.org/show_bug.cgi?id=535106 https://bugzilla.mozilla.org/show_bug.cgi?id=1359841 Like many parts of the busted Solaris support, this one has its origins in the pre-Firefox days. Bug 535106, another Mozilla suite bug. It keeps coming up because the core issue is never addressed, the fact that nsTSubstring doesn't know how to handle pid_t. I think the explicit cast to int is a band-aid they use because they know if they touch that substring header file to make it handle pid_t, they'll probably be asked to fix all the other problems with it. I honestly think it just works by accident on other platforms because it's implicitly cast to signed or unsigned int, even though the POSIX standard says pid_t can be either long or int, and work as either a signed or unsigned integer. Whatever the case may be, it's handled better on Solaris now than it was. Ironically enough, the main point of having pid_t rather than just having pids be int or something is to hide this little implementation detail so you can just use pid_t for the return type in portable code without having to worry about what it is on a specific platform. The unfortunate way Mozilla implemented string functions turns that on its head and makes the good things about pid_t into liabilities rather than assets.
| * | MoonchildProductions#1251 - Part 2: Make the mozconfig loader POSIX-compliant.athenian2002019-10-21-20/+18
| | | | | | | | | | | | | | | | | | https://bugzilla.mozilla.org/show_bug.cgi?id=1360571 Solaris uses ksh as the default shell, and furthermore bash doesn't seem to support the 'local' keyword feature when invoked as sh on OpenIndiana. We could just change the script to use bash (it is an option even on Solaris), but this fix is available and Mozilla has been using it since Firefox 55 with no issues on any other platforms. It was specfically done this way to avoid any need to change existing mozconfig files, so I feel confident saying the change is totally benign and if anything the way it is now is technically a POSIX compliance issue inherited from Mozilla that we'll hit if we ever try to compile this on any Unix platform where bash isn't sh.
| * | MoonchildProductions#1251 - Part 1: Restore initial Solaris support, fixed up.athenian2002019-10-21-21/+945
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Compared with what Pale Moon had for Solaris originally, this is mostly the same zero point I started patching from, but I've made the following changes here after reviewing all this initial code I never looked at closely before. 1. In package-manifest.in for both Basilisk and Pale Moon, I've made the SPARC code for libfreebl not interefere with the x86 code, use the proper build flags, and also updated it to allow a SPARC64 build which is more likely to be used than the 32-bit SPARC code we had there. 2. See Mozilla bug #832272 and the old rules.mk patch from around Firefox 30 in oracle/solaris-userland. I believe they screwed up NSINSTALL on Solaris when they were trying to streamline the NSS buildsystem, because they started having unexplained issues with it around that time after Firefox 22 that they never properly resolved until Mozilla began building NSS with gyp files. I'm actually not even sure how relevant the thing they broke actually is to Solaris at this point, bug 665509 is so old it predates Firefox itself and goes back to the Mozilla suite days. I believe $(INSTALL) -t was wrong, and they meant $(NSINSTALL) -t because that makes more sense and is closer to what was there originally. It's what they have for WINNT, and it's possible a fix more like that could serve for Solaris as well. Alternatively, we could get rid of all these half-broken Makefiles and start building NSS with gyp files like Mozilla did. 3. I've completely cut out support for the Sun compiler and taken into account the reality that everyone builds Firefox (and therefore its forks) with GCC now on Solaris. This alone helped clean up a lot of the uglier parts of the code. 4. I've updated all remaining SOLARIS build flags to the newer XP_SOLARIS, because the SOLARIS flag is no longer set when building Solaris. 5. I've confirmed the workaround in gtxFontconfigFonts.cpp is no longer necessary. The Solaris people got impatient about implementing a half-baked patch for a fontconfig feature that wasn't ready yet back in 2009, and somehow convinced Mozilla to patch their software to work around it when really they should have just fixed or removed their broken fontconfig patch. The feature they wanted has since been implemented properly, and no version of Solaris still uses the broken patch that required this fix. If anyone had ever properly audited this code, it would have been removed a long time ago.
* | | #1261 - Update status bar component for the removal of Object.(un)watchwolfbeast2019-11-01-15/+51
| |/ |/| | | | | | | | | | | This fixes the regression from #1257 by adding in an Object.prototype.watch() shim, based on Eli Grey's polyfill. This resolves #1261.
* | Issue #1257 - Part 3: Remove/update tests.wolfbeast2019-10-27-2940/+34
| | | | | | | | | | | | | | This removes a ton of tests that are no longer relevant with (un)watch removed (e.g. testing stability/bugs in the watchpoint system itself which has never been the most stable), and updates others that would previously rely on watch/unwatch, so that they don't unexpectedly fail.
* | Issue #1257 - Part 2: Remove watch/unwatch and JS watchpoint class.wolfbeast2019-10-27-677/+65
| |
* | Issue #1257 - Part1: Remove watch class-hook and proxy trap.wolfbeast2019-10-26-145/+5
| |
* | Merge branch 'nss-work'wolfbeast2019-10-24-24/+89
|\ \
| * | Force clobber.wolfbeast2019-10-24-1/+1
| | |
| * | Update NSS versionwolfbeast2019-10-24-6/+7
| | |
| * | Add length checks for cryptographic primitivesKevin Jacobs2019-10-24-9/+56
| | | | | | | | | | | | | | | This rollup patch adds additional length checks around cryptographic primitives.
| * | Support longer (up to RFC maximum) HKDF outputswolfbeast2019-10-24-8/+25
|/ / | | | | | | | | | | HKDF-Expand enforces a maximum output length much shorter than stated in the RFC. This patch aligns the implementation with the RFC by allocating more output space when necessary.
* | Add size checks to WebGLContext::BufferData()wolfbeast2019-10-24-0/+12
| | | | | | | | | | | | | | | | On MacOS, particularly large allocations within the platform limits (1.2G+) will fail and crash. This adds a specific size check for that when working around driver bugs (default). While there, added a generic size_t limited size check for the platform, and reporting OOM if too large.
* | Fix type barrier in IonBuilder::jsop_getimport.wolfbeast2019-10-24-3/+1
| |
* | Adjust tokenization of U+0000Henri Sivonen2019-10-24-13/+2
| |
* | Prevent nr_ice_component_insert_pair from leaking.Byron Campen2019-10-24-1/+6
| |
* | Leverage strings to get working dirs in nsUpdateDriver.wolfbeast2019-10-23-13/+18
| |
* | Ensure that file actors created after the database was closed are expired.Simon Giesecke2019-10-23-0/+3
| |
* | Avoid following the prototype chainwolfbeast2019-10-23-3/+20
| | | | | | | | | | No longer follow the value's prototype chain when creating index updates in IndexedDB.
* | Add null check in Http2Session::RecvAltSvcwolfbeast2019-10-23-0/+1
| |
* | Issue #1255 - Port upstream fix from libexpatwolfbeast2019-10-23-7/+13
| |
* | Avoid uint32_t overflow in js shell by checking size of file beforewolfbeast2019-10-22-0/+5
| | | | | | | | | | | | trying to stuff something insanely large into a Uint8Array. See also: BMO 1571911
* | Issue #1253 - Reset performance object on navigationwolfbeast2019-10-21-1/+13
|/ | | | | | | This also addresses clearing of document dependent JS slots which might get out of sync with innerWindow navigation; relevant comments added. This resolves #1253
* Issue #1229 - Remove fallback for $INSTDIRwolfbeast2019-10-21-15/+0
| | | | This resolves #1229
* Issue #1231 - Correct defines for Mac and keep universal prefs generic.wolfbeast2019-10-21-6/+6
|
* Merge pull request #1249 from g4jc/nss_exportsNew Tobin Paradigm2019-10-18-0/+2
|\ | | | | Issue #1243 - Update List of NSS Exported Symbols
| * Issue #1243 - Update List of NSS Exported SymbolsGaming4JC2019-10-17-0/+2
|/ | | | Add NSS_CMSSignedData_GetDigestAlgs and NSS_CMSSignedData_hasDigests which are required for security patches in mailnews applications. Ref: m-c bug 1526473
* Merge branch 'master' of https://github.com/MoonchildProductions/UXPMatt A. Tobin2019-10-16-3/+3
|\
| * Merge pull request #1246 from dimag0g/masterMoonchild2019-10-12-2/+2
| |\ | | | | | | Replace calls to undefined functions isMarkable() and toMarkablePointer()
| | * Replace calls to undefined functions isMarkable() and toMarkablePointer()Dmitry Grigoryev2019-10-10-2/+2
| |/
| * No Issue - Expand HWA over RDP to Windows 8.1 and 10.wolfbeast2019-10-08-1/+1
| | | | | | | | | | | | | | When Mozilla implemented this initially, only Windows 8 existed. Because of the strict equal check, 8.1 and 10 didn't get HWA over RDP while they are perfectly capable of doing so with RemoteFX. This change allows any version of Windows from 8.0 onwards to use HWA over RDP.
* | Don't treat format warnings as errors in xpconnectMatt A. Tobin2019-10-16-1/+1
| | | | | | | | GCC 9 compiler does not like the way we have it in XPCWrappedNative.cpp
* | Fix build errors with newer glibc versionsMatt A. Tobin2019-10-16-14/+4
|/
* Merge pull request #1244 from dimag0g/masterMoonchild2019-10-06-0/+3
|\ | | | | Added missing libwebp files to update.sh
| * Added missing libwebp files to update.shDmitry Grigoryev2019-10-04-0/+3
|/
* No issue - Update TZ data to 2019cwolfbeast2019-10-02-106/+103
|
* Merge pull request #1238 from dimag0g/masterMoonchild2019-09-30-0/+814
|\ | | | | Add libwebp NEON-specific files
| * Merge branch 'master' of https://github.com/dimag0g/UXPDmitry Grigoryev2019-09-28-1/+11
| |\
| * | Added missing libwebp NEON filesDmitry Grigoryev2019-09-28-0/+814
| | |
* | | Merge pull request #1237 from g4jc/css-workMoonchild2019-09-29-881/+4947
|\ \ \ | |_|/ |/| | CSS Grid Improvements
| * | Issue #1230 - Part 3: Update ReftestsGaming4JC2019-09-28-122/+143
| | | | | | | | | | | | | | | Ref: 1434478 part 7 - Update tests and enable some previously temporarily disabled Grid reftests from bug 1427608.
| * | Issue #1230 - Part 2: Align devtools to the changes in css-gridGaming4JC2019-09-28-2/+4
| | | | | | | | | | | | | | | Ref: 1398537 part 4 - [css-multicol] Implement percentages for 'column-gap' (automated update of devtools).
| * | Issue #1230 - Part 1: Fix Back-computing percentages for intrinsic sizing in ↵Gaming4JC2019-09-28-319/+240
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Layout CSS-Grid List of relevant patches applied: 1398537 part 2 - [css-multicol] Implement percentages for 'column-gap' (Gecko part). 1434478 part 1 - [css-grid] Stop back-computing percentage grid gaps when the percentage basis is indefinite. Treat them as zero sized instead. 1434478 part 2 - Stop back-computing percentage padding/margin when the percentage basis is indefinite. Treat them as zero sized instead. 1434478 part 3 - Remove IntrinsicISizeOffsetData::hPctPadding/hPctMargin members since they are now unused. 1434478 part 4 - Factor out constants like NS_UNCONSTRAINEDSIZE so they can be used in headers without needing nsIFrame.h (idempotent patch). 1434478 part 5 - Create nsLayoutUtils::ResolveToLength for resolving CSS <length-percentage> (idempotent patch). 1434478 part 6 - Propagate a percentage basis to nsIFrame::IntrinsicISizeOffsets for resolving padding/margin. This is needed only for CSS Grid since in other cases we're only using IntrinsicISizeOffsets in the inline-axis and the percentage basis is always indefinite for *intrinsic sizing*. When calculating the intrinsic size of grid items in the grid container's block axis however, we do have a definite size for the grid area in the inline-axis and it should be used per: https://drafts.csswg.org/css-grid/#algo-overview "2. Next, the track sizing algorithm resolves the sizes of the grid rows, using the grid column sizes calculated in the previous step." (Percentage padding/margin for grid items is always resolved against the grid area's inline-size nowadays.)