3e9ceef9b2802d2d2353b8389e6f09c77f991577: Bug 1360068 - Overhaul WebAppActivity to use GeckoView rather than GeckoApp. r=jchen,daleharvey
Dylan Roeh <droeh@mozilla.com> - Wed, 31 May 2017 16:59:50 -0500 - rev 361672
Push 31935 by ryanvm@gmail.com at Thu, 01 Jun 2017 00:25:58 +0000
Bug 1360068 - Overhaul WebAppActivity to use GeckoView rather than GeckoApp. r=jchen,daleharvey
5eb1cfd59d0017e714ac7eafc78561e346d5ed5f: Bug 1356346 - Overhaul CustomTabsActivity.java to use GeckoView rather than GeckoApp. r=jchen, walkingice
Dylan Roeh <droeh@mozilla.com> - Tue, 25 Apr 2017 11:48:05 -0500 - rev 361671
Push 31935 by ryanvm@gmail.com at Thu, 01 Jun 2017 00:25:58 +0000
Bug 1356346 - Overhaul CustomTabsActivity.java to use GeckoView rather than GeckoApp. r=jchen, walkingice
b94150e0a2991fe40a3504355fad38969b413da0: Bug 1365599 - Make Tabs use the window event dispatcher rather than global in some instances. r=jchen
Dylan Roeh <droeh@mozilla.com> - Wed, 31 May 2017 16:58:54 -0500 - rev 361670
Push 31935 by ryanvm@gmail.com at Thu, 01 Jun 2017 00:25:58 +0000
Bug 1365599 - Make Tabs use the window event dispatcher rather than global in some instances. r=jchen
ef0ddbc20a37650c964cd6d6e0547b9339868f16: Bug 1358798 - add a test preventing us from loading scripts unintentionally during startup, r=mconley,mccr8.
Florian Quèze <florian@queze.net> - Wed, 31 May 2017 23:00:43 +0200 - rev 361669
Push 31935 by ryanvm@gmail.com at Thu, 01 Jun 2017 00:25:58 +0000
Bug 1358798 - add a test preventing us from loading scripts unintentionally during startup, r=mconley,mccr8.
65251b0ed973675e2d490458fedfc5cb75753abb: Bug 1352889 - Ensure that PLDHashTable's second hash doesn't have padding with 0 bits for tables with capacity larger than 2^16. r=njn
L. David Baron <dbaron@dbaron.org> - Wed, 31 May 2017 13:44:02 -0700 - rev 361668
Push 31935 by ryanvm@gmail.com at Thu, 01 Jun 2017 00:25:58 +0000
Bug 1352889 - Ensure that PLDHashTable's second hash doesn't have padding with 0 bits for tables with capacity larger than 2^16. r=njn PLDHashTable takes the result of the hash function and multiplies it by kGoldenRatio to ensure that it has a good distribution of bits across the 32-bit hash value, and then zeroes out the low bit so that it can be used for the collision flag. This result is called hash0. From hash0 it computes two different numbers used to find entries in the table storage: hash1 is used to find an initial position in the table to begin searching for an entry; hash2 is then used to repeatedly offset that position (mod the size of the table) to build a chain of positions to search. In a table with capacity 2^c entries, hash1 is simply the upper c bits of hash0. This patch does not change this. Prior to this patch, hash2 was the c bits below hash1, padded at the low end with zeroes when c > 16. (Note that bug 927705, changeset 1a02bec165e16f370cace3da21bb2b377a0a7242, increased the maximum capacity from 2^23 to 2^26 since 2^23 was sometimes insufficient!) This manner of computing hash2 is problematic because it increases the risk of long chains for very large tables, since there is less variation in the hash2 result due to the zero padding. So this patch changes the hash2 computation by using the low bits of hash0 instead of shifting it around, thus avoiding 0 bits in parts of the hash2 value that are significant. Note that this changes what hash2 is in all cases except when the table capacity is exactly 2^16, so it does change our hashing characteristics. For tables with capacity less than 2^16, it should be using a different second hash, but with the same amount of random-ish data. For tables with capacity greater than 2^16, it should be using more random-ish data. Note that this patch depends on the patch for bug 1353458 in order to avoid causing test failures. MozReview-Commit-ID: JvnxAMBY711
d8c7a5c7cb77f8fc3527a4b020291b920a7afda2: Bug 1352888 - Don't set the collision flag when adding to PLDHashTable if we've already found the entry we're going to add. r=njn
L. David Baron <dbaron@dbaron.org> - Wed, 31 May 2017 13:44:02 -0700 - rev 361667
Push 31935 by ryanvm@gmail.com at Thu, 01 Jun 2017 00:25:58 +0000
Bug 1352888 - Don't set the collision flag when adding to PLDHashTable if we've already found the entry we're going to add. r=njn PLDHashTable's entry store has two types of unoccupied entries: free entries and removed entries. The search of a chain of entries (determined by the hash value) in the entry store to search for an entry can stop at free entries, but it continues across removed entries, because removed entries are entries that may have been skipped over when we were adding the value we're searching for to the hash, but have since been removed. For live entries, we also maintain this distinction by using one bit of storage for a collision flag, which notes that if the hashtable entry is removed, its place in the entry store must become a removed entry rather than a free entry. When we add a new entry to the table, Add's semantics require that we return an existing entry if there is one, and only create a new entry if no existing entry exists. (Bug 1352198 suggests the possibility of a faster alternative Add API where the caller guarantees that the key is not already in the hashtable.) When we search for the existing entry, we must thus continue the search across removed entries, even though we record the first removed entry found to return if the search for an existing entry fails. The existing code adds the collision flag through the entire table search during an Add. This patch changes that behavior so that we only add the collision flag prior to finding the first removed entry. Adding it after we find the first removed entry is unnecessary, since we are not making that entry part of a path to a new entry. If it is part of a path to an existing entry, it will already have the collision flag set. This patch effectively puts an if (!firstRemoved) around the else branch of the if (MOZ_UNLIKELY(EntryIsRemoved(entry))), and then refactors that condition outwards since it is now around the contents of both the if and else branches. MozReview-Commit-ID: CsXnMYttHVy
62c2ff7599a301ec3e3cfee0e16325ab2afa0eea: Bug 1369037 - Make the assertions about NS_FRAME_PART_OF_IBSPLIT stricter and update the associated frame properties in nsContainerFrame::DestroyFrom instead of nsFrame::DestroyFrom. r=jfkthame
Mats Palmgren <mats@mozilla.com> - Wed, 31 May 2017 21:29:49 +0200 - rev 361666
Push 31935 by ryanvm@gmail.com at Thu, 01 Jun 2017 00:25:58 +0000
Bug 1369037 - Make the assertions about NS_FRAME_PART_OF_IBSPLIT stricter and update the associated frame properties in nsContainerFrame::DestroyFrom instead of nsFrame::DestroyFrom. r=jfkthame MozReview-Commit-ID: G8NQ70xzkQU
bd52d785f0d482d6a106b5d5603c3c334a521b2e: bug 1369015 - Remove a useless do_QueryFrame() call. r=jfkthame
Mats Palmgren <mats@mozilla.com> - Wed, 31 May 2017 21:29:49 +0200 - rev 361665
Push 31935 by ryanvm@gmail.com at Thu, 01 Jun 2017 00:25:58 +0000
bug 1369015 - Remove a useless do_QueryFrame() call. r=jfkthame GetParent() already returns a nsContainerFrame* so there's no need to do_QueryFrame that to a nsContainerFrame* ! MozReview-Commit-ID: B8CwGkUWIoT
45409ba9a985de2d6572055f5d6f552dff74f4f6: bug 1368547 part 2 - Remove nsFrameManager/PresShell methods dealing with placeholders and introduce a nsIFrame::GetPlaceholderFrame() convenience method. r=jfkthame
Mats Palmgren <mats@mozilla.com> - Wed, 31 May 2017 21:29:49 +0200 - rev 361664
Push 31935 by ryanvm@gmail.com at Thu, 01 Jun 2017 00:25:58 +0000
bug 1368547 part 2 - Remove nsFrameManager/PresShell methods dealing with placeholders and introduce a nsIFrame::GetPlaceholderFrame() convenience method. r=jfkthame MozReview-Commit-ID: GBUiUBqJxhQ
ddcdb354143a9018a9bc94cef7a395ef46873f8b: bug 1368547 part 1 - Remove nsFrameManagerBase::mPlaceholderMap and store the placeholder on a frame property on the out-of-flow instead. r=jfkthame
Mats Palmgren <mats@mozilla.com> - Wed, 31 May 2017 21:29:49 +0200 - rev 361663
Push 31935 by ryanvm@gmail.com at Thu, 01 Jun 2017 00:25:58 +0000
bug 1368547 part 1 - Remove nsFrameManagerBase::mPlaceholderMap and store the placeholder on a frame property on the out-of-flow instead. r=jfkthame MozReview-Commit-ID: 6AOxc2XgtO1
422e3c006cf2b56d94f969d4b8e29b0982103437: Bug 1368802 - nsFrameIterator::GetPlaceholderFrame should only try to get the placeholder for out-of-flow frames, because in-flow frames never have a placeholder. r=jfkthame
Mats Palmgren <mats@mozilla.com> - Wed, 31 May 2017 21:29:49 +0200 - rev 361662
Push 31935 by ryanvm@gmail.com at Thu, 01 Jun 2017 00:25:58 +0000
Bug 1368802 - nsFrameIterator::GetPlaceholderFrame should only try to get the placeholder for out-of-flow frames, because in-flow frames never have a placeholder. r=jfkthame MozReview-Commit-ID: CcAlCWMHQQX
fc6012cad03e870b46ce4ddc2077762c31f8f9b7: Bug 1349835 - Create accessibles for <input type=checkbox/radio> with -moz-appearance:none. r=surkov
Mats Palmgren <mats@mozilla.com> - Wed, 31 May 2017 21:29:49 +0200 - rev 361661
Push 31935 by ryanvm@gmail.com at Thu, 01 Jun 2017 00:25:58 +0000
Bug 1349835 - Create accessibles for <input type=checkbox/radio> with -moz-appearance:none. r=surkov MozReview-Commit-ID: Cot3jyZBdpU
b0ebe9dd56aeae5db543e72156886bf2404d90c1: Bug 1368941 - Remove gonk specific code from ProcessPriorityManager, r=gsvelto
Andrea Marchesini <amarchesini@mozilla.com> - Wed, 31 May 2017 21:23:52 +0200 - rev 361660
Push 31935 by ryanvm@gmail.com at Thu, 01 Jun 2017 00:25:58 +0000
Bug 1368941 - Remove gonk specific code from ProcessPriorityManager, r=gsvelto
7d9c81795eeda760c11c4c8eb217cfecb7699d01: Bug 1368955 - Split URL.cpp in URLMainThread.{cpp,h} and URLWorker.{cpp,h}, r=qdot
Andrea Marchesini <amarchesini@mozilla.com> - Wed, 31 May 2017 21:18:56 +0200 - rev 361659
Push 31935 by ryanvm@gmail.com at Thu, 01 Jun 2017 00:25:58 +0000
Bug 1368955 - Split URL.cpp in URLMainThread.{cpp,h} and URLWorker.{cpp,h}, r=qdot
cadb789ebd3972c8e5f65bff0fc12ae61e162e69: Bug 1368950 - Remove URL Constructor with a URL base argument, r=qdot
Andrea Marchesini <amarchesini@mozilla.com> - Wed, 31 May 2017 21:18:19 +0200 - rev 361658
Push 31935 by ryanvm@gmail.com at Thu, 01 Jun 2017 00:25:58 +0000
Bug 1368950 - Remove URL Constructor with a URL base argument, r=qdot
0f4474710f24dbce9a92ebc75610fe8ab34bd638: Bug 1369010 - Remove dead code in ContentParent.{cpp,h}, r=qdot
Andrea Marchesini <amarchesini@mozilla.com> - Wed, 31 May 2017 21:17:24 +0200 - rev 361657
Push 31935 by ryanvm@gmail.com at Thu, 01 Jun 2017 00:25:58 +0000
Bug 1369010 - Remove dead code in ContentParent.{cpp,h}, r=qdot
7bd0e6b5a99a7401f4e17abeff03b366f07955e6: Bug 1368065 - Add a test for WebSocket and non-ascii char, r=qdot
Andrea Marchesini <amarchesini@mozilla.com> - Wed, 31 May 2017 21:16:42 +0200 - rev 361656
Push 31935 by ryanvm@gmail.com at Thu, 01 Jun 2017 00:25:58 +0000
Bug 1368065 - Add a test for WebSocket and non-ascii char, r=qdot
313abe9e730ed46ac008d9867cc1e9619eb881c8: Bug 1369073 - Expose MessagePort/MessageChannel to xpcshell tests, r=bholley
Andrea Marchesini <amarchesini@mozilla.com> - Wed, 31 May 2017 21:16:03 +0200 - rev 361655
Push 31935 by ryanvm@gmail.com at Thu, 01 Jun 2017 00:25:58 +0000
Bug 1369073 - Expose MessagePort/MessageChannel to xpcshell tests, r=bholley
78a9d7baaf8bf0073e06d8369ca33bd5134b92f9: Bug 1368712 - Get rid of nsIBrowserElementAPI.{set,get}Visible, r=kanru
Andrea Marchesini <amarchesini@mozilla.com> - Wed, 31 May 2017 21:15:23 +0200 - rev 361654
Push 31935 by ryanvm@gmail.com at Thu, 01 Jun 2017 00:25:58 +0000
Bug 1368712 - Get rid of nsIBrowserElementAPI.{set,get}Visible, r=kanru
16af57faa3e09019e7b69786e55c7c941e099836: Bug 1363975 - Have loadOneTab() provide the correct triggeringPrincipal. r=gijs,smaug
Christoph Kerschbaumer <ckerschb@christophkerschbaumer.com> - Wed, 31 May 2017 20:59:02 +0200 - rev 361653
Push 31935 by ryanvm@gmail.com at Thu, 01 Jun 2017 00:25:58 +0000
Bug 1363975 - Have loadOneTab() provide the correct triggeringPrincipal. r=gijs,smaug
(0) -300000 -100000 -30000 -10000 -3000 -1000 -300 -100 -50 -20 +20 +50 +100 +300 +1000 +3000 +10000 +30000 +100000 tip