247222cae0e82a8e743a8de593f674144f0fb4ad: Bug 1397764 - Part 2. Generate BINDGEN_CFLAGS for Android. r=chmanchester
Makoto Kato <m_kato@ga2.so-net.ne.jp> - Fri, 29 Sep 2017 16:46:52 +0900 - rev 677198
Push 83722 by bmo:sledru@mozilla.com at Tue, 10 Oct 2017 10:01:59 +0000
Bug 1397764 - Part 2. Generate BINDGEN_CFLAGS for Android. r=chmanchester Building Fennec/Android uses cross compiler toolchain. So we have to generate clang options (include path for c++ headers and gcc headers, gcc-toolchain path and etc) from NDK path for bindgen. The following options are required for android build. (from stlport_cppflags) -I$topsrcdir/android-ndk/sources/cxx-stl/llvm-libc++/libcxx/include -I$topsrcdir/android-ndk/sources/android/support/include -I$topsrcdir/android-ndk/sources/cxx-stl/llvm-libc++abi/libcxxabi/include" (others for clang) -isystem $topsrcdir/android-ndk/platforms/android-9/arch-arm/usr/include -gcc-toolchain $topsrcdir/android-ndk/toolchains/arm-linux-androideabi-4.9/prebuilt/linux-x86_64 -I$topsrcdir/android-ndk/toolchains/arm-linux-androideabi-4.9/prebuilt/linux-x86_64/lib/gcc/arm-linux-androideabi/4.9/include -I$topsrcdir/android-ndk/toolchains/arm-linux-androideabi-4.9/prebuilt/linux-x86_64/lib/gcc/arm-linux-androideabi/4.9/include-fixed" Also, since bindgen_cflags_defaults uses as default, some funcions requires '--help'. MozReview-Commit-ID: 7zfhw3IxQ2W
e3242e1acec053b6c6cf2697ac92fa886dd92cfe: Bug 1397764 - Part 1. Move STLPORT_CPPFLAGS to moz.configure. r=chmanchester
Makoto Kato <m_kato@ga2.so-net.ne.jp> - Fri, 29 Sep 2017 16:45:44 +0900 - rev 677197
Push 83722 by bmo:sledru@mozilla.com at Tue, 10 Oct 2017 10:01:59 +0000
Bug 1397764 - Part 1. Move STLPORT_CPPFLAGS to moz.configure. r=chmanchester To build sytlo, we have to set compiler flags via BINDGEN_CFLAGS. Since we pass stlport flags to clang, I would like to move STLPORT_CPPFLAGS to moz.configure. MozReview-Commit-ID: 26jvUqUvwTY
34563d832147afe87b369f49f555e0ccc8268aa4: Bug 1405983 - Part 2: Modify tests for delay. r=pbro
Daisuke Akatsuka <dakatsuka@mozilla.com> - Tue, 10 Oct 2017 10:51:27 +0900 - rev 677196
Push 83722 by bmo:sledru@mozilla.com at Tue, 10 Oct 2017 10:01:59 +0000
Bug 1405983 - Part 2: Modify tests for delay. r=pbro MozReview-Commit-ID: 6ByuW2Q33Vf
c2d9bd9f5561914c84f584dab3d23ccf669f8674: Bug 1405983 - Part 1: Display delay area in summary graph as 0 if fill is none or forwards. r=pbro
Daisuke Akatsuka <dakatsuka@mozilla.com> - Tue, 10 Oct 2017 10:51:22 +0900 - rev 677195
Push 83722 by bmo:sledru@mozilla.com at Tue, 10 Oct 2017 10:01:59 +0000
Bug 1405983 - Part 1: Display delay area in summary graph as 0 if fill is none or forwards. r=pbro MozReview-Commit-ID: 6PRlPThxRw8
8ed041ceccbd1aec843b1b12ab0e895b1ee89f12: servo: Merge #18777 - Improved headless Servo performance (from mateon1:perf/headless-event-poll); r=jdm
Mateusz Naściszewski <matin1111@wp.pl> - Mon, 09 Oct 2017 16:45:41 -0500 - rev 677194
Push 83722 by bmo:sledru@mozilla.com at Tue, 10 Oct 2017 10:01:59 +0000
servo: Merge #18777 - Improved headless Servo performance (from mateon1:perf/headless-event-poll); r=jdm Now the main thread doesn't waste 100% CPU, polling for events. --- - [X] `./mach build -d` does not report any errors - [X] `./mach test-tidy` does not report any errors - [X] These changes fix #18770 (github issue number if applicable). <!-- Either: --> - [ ] There are tests for these changes OR - [X] These changes do not require tests because: I verified Servo works correctly in headless mode, and the linked issue is gone. Source-Repo: https://github.com/servo/servo Source-Revision: 54f6e87939be7adb08c39a1f7644bed9eeaae88e
5909ec7d5d53b840746b267c8668724bf8704906: Bug 1384691 - Unconditionally set mHasSelectionShadow when -moz-selection pseudo element is used. r=jfkthame
Xidorn Quan <me@upsuper.org> - Wed, 13 Sep 2017 14:30:58 +1000 - rev 677193
Push 83722 by bmo:sledru@mozilla.com at Tue, 10 Oct 2017 10:01:59 +0000
Bug 1384691 - Unconditionally set mHasSelectionShadow when -moz-selection pseudo element is used. r=jfkthame The HasAuthorSpecifiedRules check was from the initial implementation of text-shadow support on ::-moz-selection in bug721750. There doesn't seem to be anything mentioning why this check is necessary. Also given text-shadow is inherited by default, it doesn't seems this change would lead to any difference. MozReview-Commit-ID: 6Qbotdxykwi
db3f44db75c5dbd74c013bc9467587f6d1bec050: Bug 1302470 Part 6: Properly check for a frame's visibility, do not abuse isRangeVisible() for that purpose. r=mikedeboer
Mike de Boer <mdeboer@mozilla.com> - Fri, 28 Apr 2017 19:06:35 +0200 - rev 677192
Push 83722 by bmo:sledru@mozilla.com at Tue, 10 Oct 2017 10:01:59 +0000
Bug 1302470 Part 6: Properly check for a frame's visibility, do not abuse isRangeVisible() for that purpose. r=mikedeboer MozReview-Commit-ID: ErviFQrJR1u
20c8d9fcd818e96bb42d1eaf672b7ddce1c21936: Bug 1302470 Part 5: Connect up FinderHighlighter.jsm with the new isRangeVisible function. r=mikedeboer
Mike de Boer <mdeboer@mozilla.com> - Mon, 23 Jan 2017 17:39:07 +0100 - rev 677191
Push 83722 by bmo:sledru@mozilla.com at Tue, 10 Oct 2017 10:01:59 +0000
Bug 1302470 Part 5: Connect up FinderHighlighter.jsm with the new isRangeVisible function. r=mikedeboer MozReview-Commit-ID: ABfAoZ4CBjP
c366831a783731cc53fea78636b7318c274e09aa: Bug 1302470 Part 4: Change nsDisplayList::GetOpaqueRegion non-opaque lists to build up a region from its children. r=mattwoodrow
Brad Werth <bwerth@mozilla.com> - Thu, 20 Apr 2017 10:50:41 -0700 - rev 677190
Push 83722 by bmo:sledru@mozilla.com at Tue, 10 Oct 2017 10:01:59 +0000
Bug 1302470 Part 4: Change nsDisplayList::GetOpaqueRegion non-opaque lists to build up a region from its children. r=mattwoodrow MozReview-Commit-ID: LEuvazbz1X
97cf3bf307743a4baabb8f893eecbe0846e8879f: Bug 1302470 Part 3: Fix the case where HTML buttons need to generate display item children when doing opaque hit tests. r=mattwoodrow
Brad Werth <bwerth@mozilla.com> - Wed, 30 Nov 2016 14:35:37 -0800 - rev 677189
Push 83722 by bmo:sledru@mozilla.com at Tue, 10 Oct 2017 10:01:59 +0000
Bug 1302470 Part 3: Fix the case where HTML buttons need to generate display item children when doing opaque hit tests. r=mattwoodrow MozReview-Commit-ID: HwDYsnMJkM8
9dfcb2f9db7dcfe69ef348a9eb3cecaaf240c140: Bug 1302470 Part 2: Branch IsRangeVisible to delegate to IsRangeRendered when range is in viewport. r=masayuki
Brad Werth <bwerth@mozilla.com> - Thu, 20 Apr 2017 10:21:52 -0700 - rev 677188
Push 83722 by bmo:sledru@mozilla.com at Tue, 10 Oct 2017 10:01:59 +0000
Bug 1302470 Part 2: Branch IsRangeVisible to delegate to IsRangeRendered when range is in viewport. r=masayuki MozReview-Commit-ID: LZyvt08I9vz
33ce36cb798bb3c9ebf781558a9082dd0479ccf7: Bug 1302470 Part 1: Create a IsRangeRendered function to test range visibility in the display list. r=mstange,smaug
Brad Werth <bwerth@mozilla.com> - Fri, 24 Mar 2017 14:45:41 -0700 - rev 677187
Push 83722 by bmo:sledru@mozilla.com at Tue, 10 Oct 2017 10:01:59 +0000
Bug 1302470 Part 1: Create a IsRangeRendered function to test range visibility in the display list. r=mstange,smaug MozReview-Commit-ID: u0N73luIz7
024ce7dd2f65d6266fc86b2c330ae2e3347df415: Bug 1270140 - Add Intl.RelativeTimeFormat. r=Waldo
Zibi Braniecki <zbraniecki@mozilla.com> - Fri, 06 Oct 2017 07:00:08 -0700 - rev 677186
Push 83722 by bmo:sledru@mozilla.com at Tue, 10 Oct 2017 10:01:59 +0000
Bug 1270140 - Add Intl.RelativeTimeFormat. r=Waldo MozReview-Commit-ID: GqetnVVmXXL
e6510aeb4e94df029cc47d2d00cd2f19ba5cc9d3: Bug 1406980 - Fix the -Wunused-lambda-capture warning r=gerald
Sylvestre Ledru <sledru@mozilla.com> - Mon, 09 Oct 2017 18:36:10 +0200 - rev 677185
Push 83722 by bmo:sledru@mozilla.com at Tue, 10 Oct 2017 10:01:59 +0000
Bug 1406980 - Fix the -Wunused-lambda-capture warning r=gerald MozReview-Commit-ID: HtItBaoGenR
d57bcfffe5ba6daa51d76fb87ab64dfb44055291: Bug 1406303 - Don't heap-allocate the global chunk radix tree. r=njn
Mike Hommey <mh+mozilla@glandium.org> - Fri, 06 Oct 2017 16:18:01 +0900 - rev 677184
Push 83722 by bmo:sledru@mozilla.com at Tue, 10 Oct 2017 10:01:59 +0000
Bug 1406303 - Don't heap-allocate the global chunk radix tree. r=njn Now that the radix tree structure has a fixed size, we can just allocate the chunk radix tree object statically.
6ee5dedf4f6422ffb10cc7de612810233ddcdd49: Bug 1406303 - Make the number of significant bits used by the radix tree a template parameter. r=njn
Mike Hommey <mh+mozilla@glandium.org> - Fri, 06 Oct 2017 15:50:00 +0900 - rev 677183
Push 83722 by bmo:sledru@mozilla.com at Tue, 10 Oct 2017 10:01:59 +0000
Bug 1406303 - Make the number of significant bits used by the radix tree a template parameter. r=njn All the parameters of the radix tree (bits per level, height) are derived from the aBits argument to ::Create in a straightforward way. aBits itself is a constant at the call point, making them all constants, so we can turn all of them as constants at compile time instead of storing as data.
94b89dd64fc7e79f9407c82413f389353e86aef3: Bug 1406303 - Only store 2 levels of bit sizes for the radix tree. r=njn
Mike Hommey <mh+mozilla@glandium.org> - Fri, 06 Oct 2017 15:24:07 +0900 - rev 677182
Push 83722 by bmo:sledru@mozilla.com at Tue, 10 Oct 2017 10:01:59 +0000
Bug 1406303 - Only store 2 levels of bit sizes for the radix tree. r=njn All levels except the first are using the same size, and in some cases, even the first uses the same size. Only storing those two different sizes allows to fix the class size, while not making the code significantly more complex.
387202ecf01168dc58abca10a6a16026c181c171: Bug 1406303 - Simplify the calculation of AddressRadixTree's height. r=njn
Mike Hommey <mh+mozilla@glandium.org> - Fri, 06 Oct 2017 11:32:27 +0900 - rev 677181
Push 83722 by bmo:sledru@mozilla.com at Tue, 10 Oct 2017 10:01:59 +0000
Bug 1406303 - Simplify the calculation of AddressRadixTree's height. r=njn The tree height was defined as: height = aBits / bits_per_level; if (height * bits_per_level != aBits) { height++; } What's wanted here is a height that covers all the bits, where the first level might cover less than bits_per_level. So aBits / bits_per_level gets us the height covered by levels with exactly bits_per_level bits. The tree height is one more when there are remaining bits. Put differently, we can write aBits as: aBits = bits_per_level * x + y with y < bits_per_level. We have: aBits / bits_per_level = x. height = x when y = 0, and x + 1 when y > 0. We're looking for a number z such that height = (aBits + z) / bits_per_level. Or: height = (bits_per_level * x + y + z) / bits_per_level. = x + (y + z) / bits_per_level. So we're looking for a z such that (y + z) / bits_per_level = 0 when y = 0 = 1 when y > 0 The properties of the integer division are such that the above means: 0 <= y + z < bits_per_level when y = 0 bits_per_level <= y + z < 2 * bits_per_level when y > 0 Which gives us: 0 <= z < bits_per_level bits_per_level - y <= z < 2 * bits_per_level - y when y > 0 y being < bit_per_level per the constraint further above, 2 * bits_per_level - y > bits_per_level. So all in all, we want a z such that bits_per_level - y <= z < bits_per_level with 0 < y < bits_per_level The largest value where this is true is z = bits_per_level - 1. In summary, height = (aBits + bits_per_level - 1) / bits_per_level is the same as the height as originally defined. With that formula, it's self evident that height * bits_per_level is always >= aBits, so we remove the assertion.
cd3472504445b4e8a3e3b6a55c309c63f197fee6: Bug 1406303 - Simplify the calculation of AddressRadixTree's bits_per_level. r=njn
Mike Hommey <mh+mozilla@glandium.org> - Thu, 28 Sep 2017 15:35:21 +0900 - rev 677180
Push 83722 by bmo:sledru@mozilla.com at Tue, 10 Oct 2017 10:01:59 +0000
Bug 1406303 - Simplify the calculation of AddressRadixTree's bits_per_level. r=njn bits_per_level was defined as: ffs(pow2_ceil((kNodeSize / sizeof(void*)))) - 1 kNodeSize is (1U << 14) when SIZEOF_PTR is 4 (sizeof(void*) being the same). Otherwise, it's CACHELINE, which is (1U << 6). The most important part, though, is that it's always a power of 2. And it's divided by sizeof(void*) which is always a power or 2. The result of that division is thus always a power of 2, as long as kNodeSize is larger than the size of a pointer, which it is. The argument to pow2_ceil being a power of 2, pow2_ceil is a noop, so it can go away. And the argument to ffs being a power of 2, it returns one more than n that matches 1 << n == value. So overall the expression returns the number of shifts for kNodeSize / SIZEOF_PTR. Transforming kNodeSize to a number of shifts/power of 2, the expression can then be simplified as kNodeSize2Pow - SIZEOF_PTR_2POW.
7b2056d32ff5f93ff2f4b224be8eed32a7097f79: Bug 1406303 - Turn malloc_rtree_t into a C++ class. r=njn
Mike Hommey <mh+mozilla@glandium.org> - Fri, 06 Oct 2017 10:49:24 +0900 - rev 677179
Push 83722 by bmo:sledru@mozilla.com at Tue, 10 Oct 2017 10:01:59 +0000
Bug 1406303 - Turn malloc_rtree_t into a C++ class. r=njn The only semantic change is in the value returned by Set, which now returns whether the value could be set or not.
(0) -300000 -100000 -30000 -10000 -3000 -1000 -300 -100 -50 -20 +20 +50 +100 +300 +1000 +3000 +10000 +30000 +100000 tip