dbfef31316376758111683c6763ee85f6dbaac1b: Bug 1429019 - Remove QT support code's reference. r=glandium
MatthieuBaratte <dev.mbar@gmail.com> - Tue, 13 Mar 2018 23:06:26 +0100 - rev 411455
Push 62054 by sledru@mozilla.com at Tue, 03 Apr 2018 08:36:33 +0000
Bug 1429019 - Remove QT support code's reference. r=glandium MozReview-Commit-ID: 2W0Fg0P76Ln
59615d0d8365ac00301617d8c15246e2781c77c1: Bug 1448732 - Part 3: Adjust requestLongerTimeout. r=pbro
Daisuke Akatsuka <dakatsuka@mozilla.com> - Tue, 03 Apr 2018 12:26:57 +0900 - rev 411454
Push 62053 by dakatsuka@mozilla.com at Tue, 03 Apr 2018 08:24:05 +0000
Bug 1448732 - Part 3: Adjust requestLongerTimeout. r=pbro MozReview-Commit-ID: Ia4mjyHyBci
d51d434ff1a9a4e4e857caad671fba9c98d485c0: Bug 1448732 - Part 2: Open detail pane by clicking on summary graph instead of selecting the node. r=pbro
Daisuke Akatsuka <dakatsuka@mozilla.com> - Tue, 03 Apr 2018 12:26:36 +0900 - rev 411453
Push 62053 by dakatsuka@mozilla.com at Tue, 03 Apr 2018 08:24:05 +0000
Bug 1448732 - Part 2: Open detail pane by clicking on summary graph instead of selecting the node. r=pbro MozReview-Commit-ID: CeMj2VJwRR5
64c98c77e0f930c5bca2786670d66b0e7d6cf9bf: Bug 1448732 - Part 1: Remove extra animated elements before open animation inspector. r=pbro
Daisuke Akatsuka <dakatsuka@mozilla.com> - Tue, 03 Apr 2018 12:26:10 +0900 - rev 411452
Push 62053 by dakatsuka@mozilla.com at Tue, 03 Apr 2018 08:24:05 +0000
Bug 1448732 - Part 1: Remove extra animated elements before open animation inspector. r=pbro MozReview-Commit-ID: Ds7fNJKDYc7
6ef7e2cd46f5e5d033f823ad4d604112a5aaf832: Bug 1448732 - Part 0: Modify variables and code format for consistency. r=pbro
Daisuke Akatsuka <dakatsuka@mozilla.com> - Tue, 03 Apr 2018 12:25:44 +0900 - rev 411451
Push 62053 by dakatsuka@mozilla.com at Tue, 03 Apr 2018 08:24:05 +0000
Bug 1448732 - Part 0: Modify variables and code format for consistency. r=pbro In this patch, modify the following things: * Modify variable name of test data list to "TEST_DATA" * Modify variable name of testing animation target class name to "targetClass" * Some comments format MozReview-Commit-ID: J33RRcm4chO
5506727613da421f524877e817c1fd1529160ab8: Bug 1281907 - Part 2 - Add reftest for scrollbars with pinch-zooming. r=botond
Jan Henning <jh+bugzilla@buttercookie.de> - Sun, 01 Apr 2018 17:40:04 +0200 - rev 411450
Push 62052 by mozilla@buttercookie.de at Tue, 03 Apr 2018 07:18:41 +0000
Bug 1281907 - Part 2 - Add reftest for scrollbars with pinch-zooming. r=botond MozReview-Commit-ID: K7YoEPQLJzU
a2e0f2e299bcc1cad4ea74e55d4d7b9c0f6c1a49: Bug 1281907 - Part 1 - Include zoom level compensation in clip transform passed to caller. r=botond
Jan Henning <jh+bugzilla@buttercookie.de> - Sat, 24 Mar 2018 21:27:06 +0100 - rev 411449
Push 62052 by mozilla@buttercookie.de at Tue, 03 Apr 2018 07:18:41 +0000
Bug 1281907 - Part 1 - Include zoom level compensation in clip transform passed to caller. r=botond If we don't do that, in the case of a root scrollbar the clip transform passed to our caller will incorrectly be scaled with the current content resolution. This means that that the position of the clip rect won't match the actual position of the scrollbar if the resolution isn't ~1.0, so the scrollbars will be clipped out of existence when the content is (pinch-) zoomed in or out. MozReview-Commit-ID: 5yXa9EpTJ2g
edbbd00906426ed221f277f44bd7b8fac70e450d: Bug 1450654: Remove ResolveTag usage in cocoa code. r=mstange
Emilio Cobos Álvarez <emilio@crisal.io> - Mon, 02 Apr 2018 13:29:58 +0200 - rev 411448
Push 62051 by ecoal95@gmail.com at Tue, 03 Apr 2018 07:17:16 +0000
Bug 1450654: Remove ResolveTag usage in cocoa code. r=mstange We don't have any binding in the tree that extends="xul:menupopup", which is the only thing that would change the result. Also, I constrained it to the XUL namespace instead of just checking the node name. But again, we don't have non-XUL menupopups. MozReview-Commit-ID: 7M0beoJI86Y
2ee160335e150f20df3471b60b5e58ce873e2ff6: Bug 1448225 - Convert StylePrefs to StaticPrefs. r=emilio
Nicholas Nethercote <nnethercote@mozilla.com> - Tue, 20 Mar 2018 10:56:37 +1100 - rev 411447
Push 62050 by ecoal95@gmail.com at Tue, 03 Apr 2018 07:07:13 +0000
Bug 1448225 - Convert StylePrefs to StaticPrefs. r=emilio The new StaticPrefs machinery means that StylePrefs can be removed. Note that this approach mirrors all static prefs into Rust, but I have only updated structs.rs for the prefs that Stylo uses. On a CLOSED TREE, since a sheriff closed the tree while I was about to land this via autoland. MozReview-Commit-ID: G1SY0987WJ7
3fe0b81bd9de17b369e6f4eb189babf4b65a5203: servo: Merge #20508 - Convert StylePrefs to StaticPrefs (from nnethercote:bug-1448225); r=emilio
Nicholas Nethercote <nnethercote@mozilla.com> - Tue, 03 Apr 2018 01:54:24 -0400 - rev 411446
Push 62049 by servo-vcs-sync@mozilla.com at Tue, 03 Apr 2018 06:45:45 +0000
servo: Merge #20508 - Convert StylePrefs to StaticPrefs (from nnethercote:bug-1448225); r=emilio <!-- Please describe your changes on the following line: --> This is the Servo part of https://bugzilla.mozilla.org/show_bug.cgi?id=1448225. --- <!-- Thank you for contributing to Servo! Please replace each `[ ]` by `[X]` when the step is complete, and replace `__` with appropriate data: --> - [x] `./mach build -d` does not report any errors - [x] `./mach test-tidy` does not report any errors - [x] These changes fix https://bugzilla.mozilla.org/show_bug.cgi?id=1448225 <!-- Either: --> - [ ] There are tests for these changes OR - [X] These changes do not require tests because tested on the Gecko side. <!-- Also, please make sure that "Allow edits from maintainers" checkbox is checked, so that we can help you if you get stuck somewhere along the way.--> <!-- Pull requests that do not address these steps are welcome, but they will require additional verification as part of the review process. --> r? emilio Source-Repo: https://github.com/servo/servo Source-Revision: 0a3ffc0c6045eb6983273010d91a35267d8ec3fa
20614b63d378254fc8f3caecaec9b02cd7522e2c: Bug 1450859 - Remove layout.css.expensive-style-struct-assertions.enabled pref. r=emilio
Cameron McCormack <cam@mcc.id.au> - Tue, 03 Apr 2018 15:58:32 +1000 - rev 411445
Push 62048 by cam@mcc.id.au at Tue, 03 Apr 2018 06:18:01 +0000
Bug 1450859 - Remove layout.css.expensive-style-struct-assertions.enabled pref. r=emilio With the old style system removed, this pref has no effect.
fc4a2fff3c89b8691a81a8c101ca94251115f000: Bug 1450195 - Remove old-event-emitter usage from gcli; r=miker.
Nicolas Chevobbe <nchevobbe@mozilla.com> - Fri, 30 Mar 2018 09:20:11 +0200 - rev 411444
Push 62047 by nchevobbe@mozilla.com at Tue, 03 Apr 2018 05:40:41 +0000
Bug 1450195 - Remove old-event-emitter usage from gcli; r=miker. MozReview-Commit-ID: IYAQJ4KqEet
c3b12adc84b141d795fc1c86d9c0754c3b4e4241: Bug 1441971 - Restart aborted script load for waiting XULDocument loads; r=bz
Jim Chen <nchen@mozilla.com> - Wed, 28 Feb 2018 14:46:06 -0500 - rev 411443
Push 62046 by nchen@mozilla.com at Tue, 03 Apr 2018 04:35:26 +0000
Bug 1441971 - Restart aborted script load for waiting XULDocument loads; r=bz When a particular XULDocument load is aborted, its pending script loads are aborted as well. However, there may be additional XULDocuments waiting for the same scripts to load. If we continue with the abort, all the waiting docs will end up with aborted script loads as well. This patch makes us load the script again for the waiting docs, so that one aborted doc load does not affect other docs that just happen to be waiting on the same scripts to load. MozReview-Commit-ID: FFGBjOLXCDt
5c9c6f5a4ca35835b9614a8cdb3eefda44cb585d: Bug 1448599 - fix WebRequest events for redirected channels when onBeforeRedirect is not used, r=kmag
Shane Caraveo <scaraveo@mozilla.com> - Mon, 02 Apr 2018 21:12:07 -0500 - rev 411442
Push 62045 by mixedpuppy@gmail.com at Tue, 03 Apr 2018 04:26:49 +0000
Bug 1448599 - fix WebRequest events for redirected channels when onBeforeRedirect is not used, r=kmag If any WebRequest listener *other* than onBeforeRedirect was used, and a channel was redirected, the ChannelWrapper was not updated with the redirected channel. This broke most listeners for redirected channels and specifically an extension would never see events for the new channel. MozReview-Commit-ID: BwXToTD5LEu
102872860c7ab710800251aac7814bb5bb1168f9: Bug 1438688, part 6 - Compile XPT information to C++ at build time. r=glandium,njn
Andrew McCreight <continuation@gmail.com> - Mon, 12 Mar 2018 10:30:35 -0700 - rev 411441
Push 62044 by amccreight@mozilla.com at Tue, 03 Apr 2018 04:24:55 +0000
Bug 1438688, part 6 - Compile XPT information to C++ at build time. r=glandium,njn This patch handles the actual generation of the static data structures used to represent XPT information. XPT files are generated in the same way as they are now, but they are used only as an intermediate representation to speed up incremental compilation rather than something used by Firefox itself. Instead of linking XPTs into a single big XPT file at packaging time, they are linked into a single big C++ file at build time, that defines the various static consts in XPTHeader. In xpt.py, every data structure that can get written to disk gets an additional code_gen() method that returns a representation of that data structure as C++ source code. CodeGenData aggregates this information together, handling deduplication and the final source code generation. The ctors are needed for XPTConstValue to statically initialize the different union cases without resorting to designated initializers, which are part of C99, not C++. Designated initializers appear to be supported in C++ code by Clang and GCC, but not MSVC. The ctors must be constexpr to ensure they are actually statically initialized so they can be shared between Firefox processes. I also removed an unnecessary "union" in XPTConstDescriptor. Together, these patches reduce the amount of memory reported by xpti-working-set from about 860,000 bytes to about 200,000 bytes. The remaining memory is used for xptiInterface and xptiTypelibGuts (which are thin wrappers around the XPT interfaces and header) and hash tables to speed up looking up interfaces by name or IID. That could potentially be eliminated from dynamic allocations in follow up work. These patches did not affect memory reporting because XPT arenas are still used by the remaining XPTI data structures. MozReview-Commit-ID: Jvi9ByCPa6H
f1dd107a1ad42d8bb46574e5f23965316991c836: Bug 1438688, part 5 - Merge XPTInterfaceDirectoryEntry and XPTInterfaceDescriptor. r=njn
Andrew McCreight <continuation@gmail.com> - Mon, 12 Mar 2018 13:36:15 -0700 - rev 411440
Push 62044 by amccreight@mozilla.com at Tue, 03 Apr 2018 04:24:55 +0000
Bug 1438688, part 5 - Merge XPTInterfaceDirectoryEntry and XPTInterfaceDescriptor. r=njn With fully linked XPT data, there is exactly one directory entry per descriptor, plus one per unresolved interface. There are 1200 of the former and 40 of the latter. By merging them, we save a 32 bit int per directory entry, at the cost of 11 bytes per unresolved interface. This will make VerifyAndAddEntryIfNew slightly slower because it has to do an nsID equality check rather than a null check, but I can't imagine that will matter. My main goal for this patch is to reduce the size of the executable, to avoid a regression with my static XPT information patches, but it should reduce memory a little bit, too. MozReview-Commit-ID: L35YdOuAyF4
fe00ed08356dd3dab2f7dac4b7a000bcc7df953e: Bug 1438688, part 4 - Hoist arrays to XPTHeader. r=njn
Andrew McCreight <continuation@gmail.com> - Wed, 28 Feb 2018 15:12:07 -0800 - rev 411439
Push 62044 by amccreight@mozilla.com at Tue, 03 Apr 2018 04:24:55 +0000
Bug 1438688, part 4 - Hoist arrays to XPTHeader. r=njn To allow XPT information to be shared between processes, it needs to not contain any pointers, because they cause relocations. I've eliminated pointers by hoisting all of the variable length data structures to XPTHeader, into a single array for each type of data. These data structures now use an index into these arrays to find their first element. Strings are similar, but they are mashed into a single giant string, including embedded null terminators. Modifying the accessor methods to support this is easy, because there is only a single global instance of each of these arrays in XPTHeader. MozReview-Commit-ID: 5rgwaEBvDYl
af6c0673ec6ffc38d30d95ea42caa7001cd19ef4: Bug 1438688, part 3 - Remove XPT files from the packaging process. r=glandium
Andrew McCreight <continuation@gmail.com> - Mon, 05 Mar 2018 14:27:29 -0800 - rev 411438
Push 62044 by amccreight@mozilla.com at Tue, 03 Apr 2018 04:24:55 +0000
Bug 1438688, part 3 - Remove XPT files from the packaging process. r=glandium Now that XPT files are not loaded from files at runtime, code for packaging XPT files can be removed. This means that a couple of test XPIDL interfaces will get shipped in builds to users that weren't before, but I don't think that matters much. This also puts XPT files into the local objdir for the XPIDL makefile, instead of dist/bin, because they are no longer part of the distribution. MozReview-Commit-ID: 7gWj8KWUun3
1f3b60346b7750df21c3a93a2eef0d801d77e421: Bug 1438688, part 2b - Eliminate XPTHeader data structure. r=njn
Andrew McCreight <continuation@gmail.com> - Tue, 27 Mar 2018 14:04:01 -0700 - rev 411437
Push 62044 by amccreight@mozilla.com at Tue, 03 Apr 2018 04:24:55 +0000
Bug 1438688, part 2b - Eliminate XPTHeader data structure. r=njn Now that there is only one XPTHeader, we can devolve the fields in it to avoid some indirection. The biggest part here is getting rid of the mHeader field on xptiTypelibGuts. The array is [] instead of * to avoid a relocation, by ensuring that XPTHeader::kInterfaceDirectory as well as the data it points to cannot be changed. MozReview-Commit-ID: AzvNTNZKkfi
e3b0f068f61e4b352072c1246609ad5c50930587: Bug 1438688, part 2 - Load XPT information from a static variable instead of a file. r=njn
Andrew McCreight <continuation@gmail.com> - Wed, 28 Feb 2018 12:51:39 -0800 - rev 411436
Push 62044 by amccreight@mozilla.com at Tue, 03 Apr 2018 04:24:55 +0000
Bug 1438688, part 2 - Load XPT information from a static variable instead of a file. r=njn This patch removes C++ code related to reading in XPT information from files. (Code related to packaging XPT files will be removed in the next patch.) This includes code in the manifest parser, in addition to the actual code for parsing files. XPT information is now loaded directly from a single static data structure, XPTHeader::kHeader, which will be automatically generated at compile time from .idl files (via .xpt files). Note that the script to do that is not added until part 6 of this patch series, so linking will fail on parts 2 through 5. I inlined XPTInterfaceInfoManager::RegisterXPTHeader into the ctor, because that is the only caller. It feels like the lock there should not be needed any more, but I left it alone for now. The forward declaration of XPTArena in xptiprivate.h is needed because it was being bootlegged via xpt_xdr.h. Some of the data structures in reflect/xptinfo/ (which wrap the xpt_struct.h data structures) are still allocated using XPTArena. Hopefully we can get rid of that in followup work. I also deleted a lot of comments in xpt_struct.h that talk about the on-disk format. I also deleted checking of the major version number, because that should not matter when the XPT information is baked into the executable. MozReview-Commit-ID: 6NJdaCWRBhU
(0) -300000 -100000 -30000 -10000 -3000 -1000 -300 -100 -50 -20 +20 +50 +100 +300 +1000 +3000 +10000 +30000 tip