6d14932fb843455f20e536849b73c0f92ba1871e: Bug 1401922 - P2. Only set error when non handled. r=alwu
Jean-Yves Avenard <jyavenard@mozilla.com> - Thu, 21 Sep 2017 13:29:54 +0200 - rev 382353
Push 95312 by philringnalda@gmail.com at Fri, 22 Sep 2017 03:50:01 +0000
Bug 1401922 - P2. Only set error when non handled. r=alwu MozReview-Commit-ID: CF2Iw0i0nD8
907df99df1bba77704558b8ff712b4b75b897439: Bug 1401922 - P1. Ensure an error is returned when no decoder could be created. r=alwu
Jean-Yves Avenard <jyavenard@mozilla.com> - Thu, 21 Sep 2017 13:25:05 +0200 - rev 382352
Push 95312 by philringnalda@gmail.com at Fri, 22 Sep 2017 03:50:01 +0000
Bug 1401922 - P1. Ensure an error is returned when no decoder could be created. r=alwu PDMFactory::CreateDecoder may not always modify CreateDecoderParams::mError as not all PDM handle this optional return value. MozReview-Commit-ID: K8WFA0o778U
2dbb8579cec09771587e3dfbb035059130786051: Bug 1401743: Disable importance for accessibility of pocket MORE arrow. r=liuche
Michael Comella <michael.l.comella@gmail.com> - Thu, 21 Sep 2017 13:20:18 -0700 - rev 382351
Push 95312 by philringnalda@gmail.com at Fri, 22 Sep 2017 03:50:01 +0000
Bug 1401743: Disable importance for accessibility of pocket MORE arrow. r=liuche The proper solution here is to put MORE and the arrow in a single focusable container but this needs to get uplifted so I went with the simpler change. For non-visual users, there is no difference. For visual users, the only "more" is focused, rather than "more" and the arrow in one container. The code as it is slightly more complex because we have to hide and add click listeners to multiple objects. MozReview-Commit-ID: JZFLc8jvXII
bd2f45d07990febaa1020e3f413c932943f24de9: Bug 1382388 - Make the Payments Dialog unprivileged & add "abort" support. r=MattN
Jonathan Guillotte-Blouin <jguillotteblouin@mozilla.com> - Thu, 21 Sep 2017 13:49:41 -0700 - rev 382350
Push 95312 by philringnalda@gmail.com at Fri, 22 Sep 2017 03:50:01 +0000
Bug 1382388 - Make the Payments Dialog unprivileged & add "abort" support. r=MattN MozReview-Commit-ID: IXexL4ju2Fj
87f57d513b3723d6ff023d4f6d7319b95e25013a: servo: Merge #18579 - Remove unused feature gates (from mbrubeck:features); r=KiChjang
Matt Brubeck <mbrubeck@limpet.net> - Thu, 21 Sep 2017 15:08:04 -0500 - rev 382349
Push 95312 by philringnalda@gmail.com at Fri, 22 Sep 2017 03:50:01 +0000
servo: Merge #18579 - Remove unused feature gates (from mbrubeck:features); r=KiChjang CC #5286. --- - [x] `./mach build -d` does not report any errors - [x] `./mach test-tidy` does not report any errors - [x] These changes do not require tests because no code changes Source-Repo: https://github.com/servo/servo Source-Revision: 29517d553e6c2fddc7e3cf0c0abef58c6d0f34aa
0a367a71ca4381367346f9d600472d80205e72e3: Bug 1396951 - 6. Remove setLayerView/getLayerView; r=snorp
Jim Chen <nchen@mozilla.com> - Thu, 21 Sep 2017 17:36:07 -0400 - rev 382348
Push 95312 by philringnalda@gmail.com at Fri, 22 Sep 2017 03:50:01 +0000
Bug 1396951 - 6. Remove setLayerView/getLayerView; r=snorp Remove `GeckoAppShell.setLayerView()/getLayerView()` now that it's no longer used anywhere. MozReview-Commit-ID: 6URNFhSs01P
b5a88815b3a6316615c424a8fbb3e292e159e6a0: Bug 1396951 - 5. Remove getLayerView usage from Robocop code; r=gbrown
Jim Chen <nchen@mozilla.com> - Thu, 21 Sep 2017 17:36:07 -0400 - rev 382347
Push 95312 by philringnalda@gmail.com at Fri, 22 Sep 2017 03:50:01 +0000
Bug 1396951 - 5. Remove getLayerView usage from Robocop code; r=gbrown Find the Fennec LayerView through `Solo.getView()` and the View id instead of going through `GeckoAppShell.getLayerView()`. MozReview-Commit-ID: FVcPM0fYorf
e9bc6e08091bc04a3d91db3555bf7ab715542912: Bug 1396951 - 4. Pass in GeckoView instance when sending a11y event; r=snorp
Jim Chen <nchen@mozilla.com> - Thu, 21 Sep 2017 17:36:07 -0400 - rev 382346
Push 95312 by philringnalda@gmail.com at Fri, 22 Sep 2017 03:50:01 +0000
Bug 1396951 - 4. Pass in GeckoView instance when sending a11y event; r=snorp Pass in a `GeckoView` instance when sending a11y events so we're not dependent on `GeckoAppShell.getLayerView()`. However, there's likely more work to be done to make a11y work for any GeckoView. MozReview-Commit-ID: DBeDOX5c3qY
155c1005c388da8a8a9b5f45d148905eba542a95: Bug 1396951 - 3. Move GeckoAppShell.viewSizeChanged() to GeckoLayerClient; r=rbarker
Jim Chen <nchen@mozilla.com> - Thu, 21 Sep 2017 17:36:07 -0400 - rev 382345
Push 95312 by philringnalda@gmail.com at Fri, 22 Sep 2017 03:50:01 +0000
Bug 1396951 - 3. Move GeckoAppShell.viewSizeChanged() to GeckoLayerClient; r=rbarker Let `GeckoLayerClient` directly deal with scroll-to-input on resize, instead of going through `GeckoAppShell` and relying on `getLayerView()`. This is a necessary fix to let us remove `getLayerView()`, and in a follow-up bug we should actually fix scroll-to-input to work on any GeckoView. MozReview-Commit-ID: 1xsHh2vg08M
71134284dc36a17d0a6293b596e5374d1482bcf9: Bug 1396951 - 2. Don't use getLayerView() in GeckoInputConnection; r=esawin
Jim Chen <nchen@mozilla.com> - Thu, 21 Sep 2017 17:36:02 -0400 - rev 382344
Push 95312 by philringnalda@gmail.com at Fri, 22 Sep 2017 03:50:01 +0000
Bug 1396951 - 2. Don't use getLayerView() in GeckoInputConnection; r=esawin In GeckoInputConnection, use the current view available through `getView()`, instead of using `GeckoAppShell.getLayerView()`. MozReview-Commit-ID: Hc9AUz5SNEs
5e5ce56330175c77e0470e2f5b7c2c6776be8f53: Bug 1396951 - 1. Add and use HapticFeedbackDelegate; r=snorp
Jim Chen <nchen@mozilla.com> - Thu, 21 Sep 2017 17:36:02 -0400 - rev 382343
Push 95312 by philringnalda@gmail.com at Fri, 22 Sep 2017 03:50:01 +0000
Bug 1396951 - 1. Add and use HapticFeedbackDelegate; r=snorp Instead of using `getLayerView()` to perform haptic feedback, this patch adds a `HapticFeedbackDelegate`, which `GeckoApplication` implements to call `performHapticFeedback()` on the active view. Also, use HapticFeedbackDelegate elsewhere in the Fennec codebase where we want to perform haptic feedback. MozReview-Commit-ID: GAArA6yJFNF
9f5bf3e0af8270ab31092655b4962848cf633df9: Bug 1397975 - Show opt-in dialog for is_default with non built-in engines. r=aswan
Michael Kaply <mozilla@kaply.com> - Thu, 07 Sep 2017 19:38:55 -0500 - rev 382342
Push 95312 by philringnalda@gmail.com at Fri, 22 Sep 2017 03:50:01 +0000
Bug 1397975 - Show opt-in dialog for is_default with non built-in engines. r=aswan MozReview-Commit-ID: 67SvzDcM7kK
cd91b8be78f6026f0fabd0ea2566aef82543b1ec: Bug 1351673 - Use a single-threaded work queue to process batching downloader work items r=rnewman
Grigory Kruglov <gkruglov@mozilla.com> - Thu, 21 Sep 2017 16:53:03 -0400 - rev 382341
Push 95312 by philringnalda@gmail.com at Fri, 22 Sep 2017 03:50:01 +0000
Bug 1351673 - Use a single-threaded work queue to process batching downloader work items r=rnewman Before we'd recurse instead while fetching multiple batches, overflowing the stack on older devices. MozReview-Commit-ID: 37BG6zGBdn0
c6d94140d11b651cae7bdf8b44ddc028ebea94eb: bug 1382005, update compare-locales to 2.1, r=glandium
Axel Hecht <axel@pike.org> - Mon, 18 Sep 2017 18:31:24 +0200 - rev 382340
Push 95312 by philringnalda@gmail.com at Fri, 22 Sep 2017 03:50:01 +0000
bug 1382005, update compare-locales to 2.1, r=glandium Vendor new dependency for compare-locales, python-fluent. This is the 0.4.2 release of python-fluent. Also, make mach command a bare minimum wrapper. The compare-locales mach command used to own a couple of defaults and opinions. Now that those opinions and defaults are in the Makefiles, this command can be much simpler. As a side effect, this should make the thunderbird port easier, where none of the mach defaults worked. Update l10n.mk for compare-locales 2.x: The command line interface is different now, with positional arguments. Also, the l10n config file isn't a default anymore (that never worked for thunderbird). And the merge dir is now the parent of the locale dir, much like we already anticipated in the code. MozReview-Commit-ID: DxVVKyVSt5y
40023b79401d49e45e1eb53acb2b96976f24a939: Bug 1401737 - 1. Add a diagnostic crash; r=me
Jim Chen <nchen@mozilla.com> - Thu, 21 Sep 2017 23:41:22 -0400 - rev 382339
Push 95311 by nchen@mozilla.com at Fri, 22 Sep 2017 03:42:17 +0000
Bug 1401737 - 1. Add a diagnostic crash; r=me Add a diagnostic crash for the unexpected case where GeckoService.register() is called a second time. We know the stack for the second call, but we want to know the stack for the first call, so we introduce this intentional crash. r=me for trivial diagnostic patch which will be backed out ASAP. MozReview-Commit-ID: HobtPiVSSTR
14cf84cd05639c0bd7db54118102b1742e90cadf: Bug 1047098 - "Clear Recent History" must clean up all the ServiceWorkers, r=bkelly
Andrea Marchesini <amarchesini@mozilla.com> - Thu, 21 Sep 2017 21:32:00 +0200 - rev 382338
Push 95310 by bugmail@asutherland.org at Fri, 22 Sep 2017 03:35:11 +0000
Bug 1047098 - "Clear Recent History" must clean up all the ServiceWorkers, r=bkelly This includes minor shutdown fixes by :asuth as discussed on https://bugzilla.mozilla.org/show_bug.cgi?id=1047098#c56 and c57.
9f05ef54c84d9b63fb3ad2dd2916df34cb35df8d: Bug 1394146 - Avoid overflow on backwards iteration of IC entries. r=jandem
Sean Stangl <sstangl@mozilla.com> - Thu, 21 Sep 2017 15:37:00 -0400 - rev 382337
Push 95309 by ryanvm@gmail.com at Fri, 22 Sep 2017 02:57:13 +0000
Bug 1394146 - Avoid overflow on backwards iteration of IC entries. r=jandem
2fac6141f39e2ef741884165f56a675561924f39: Bug 1401450 - Fix incorrect guid and input block id when sending coalesced mouse move event. r=smaug.
Stone Shih <sshih@mozilla.com> - Wed, 20 Sep 2017 12:59:08 +0800 - rev 382336
Push 95308 by sshih@mozilla.com at Fri, 22 Sep 2017 01:33:00 +0000
Bug 1401450 - Fix incorrect guid and input block id when sending coalesced mouse move event. r=smaug.
4dce29de45e3c5ecf8476bd77195c11a4ebe64ad: Bug 1400193 (part 2) - Shrink PLDHashTable. r=froydnj.
Nicholas Nethercote <nnethercote@mozilla.com> - Fri, 15 Sep 2017 20:04:29 +1000 - rev 382335
Push 95307 by nnethercote@mozilla.com at Fri, 22 Sep 2017 00:50:56 +0000
Bug 1400193 (part 2) - Shrink PLDHashTable. r=froydnj. This patch reduces sizeof(PLDHashTable) as follows. - 64-bit: from 40 bytes to 32 - 32-bit: from 28 bytes to 20 It does this by doing the following. - It moves mGeneration from EntryStore to PLDHashTable, to avoid unnecessary padding on 64-bit. This requires tweaking EntryStore::Set() as explained in a comment. - It also shrinks mGeneration from uint32_t to uint16_t, saving 2 bytes of data. - It shrinks mEntrySize from uint32_t to uint8_t, to cut 3 bytes of data. - It shrinks mHashShift from int16_t to uint8_t, trimming another byte of data, and moves it, saving another 2 bytes of padding. And it reorders the fields so the word-sized ones are at the start, which makes it easier to imagine the memory layout. The patch also adds a test, and fixes some misordered function arguments in existing tests.
bd2852c72caef1f1ac622ff8862d8dc72fc072bd: Bug 1400193 (part 1) - Fix subtle bug in PLDHashTable's move constructor. r=froydnj.
Nicholas Nethercote <nnethercote@mozilla.com> - Fri, 15 Sep 2017 20:02:33 +1000 - rev 382334
Push 95307 by nnethercote@mozilla.com at Fri, 22 Sep 2017 00:50:56 +0000
Bug 1400193 (part 1) - Fix subtle bug in PLDHashTable's move constructor. r=froydnj. The current code replaces one PLDHashTable's mGeneration with another. If the two generation values happen to be equal, it will look as though the storage hasn't changed when really it has. The fix is to use EntryStore::Set() to update mEntryStore, which increments mGeneration appropriately.
(0) -300000 -100000 -30000 -10000 -3000 -1000 -300 -100 -50 -20 +20 +50 +100 +300 +1000 +3000 +10000 +30000 +100000 tip