Bug 1358976 - Stop automatic triggers of nightly builds on mozilla-aurora r=dustin DONTBUILD a=test-only CLOSED TREE

Bug 1358976 - Stop automatic triggers of nightly builds on mozilla-aurora r=dustin DONTBUILD a=test-only CLOSED TREE

Bug 1357409 - [push-apk] Remove push-apk tasks on mozilla-aurora r=Callek a=release CLOSED TREE MozReview-Commit-ID: J0rShgZsZkt

Bug 1352477 - taskgraph ignores run-on-project for fennec-nightlies r=aki a=release CLOSED TREE MozReview-Commit-ID: B4vCAoRye7F

No bug - Tagging mozilla-aurora 09e97b55acb30be1f8e13b0f637f9cf152658799 with FIREFOX_BETA_54_BASE a=release DONTBUILD CLOSED TREE

Backed out changeset 076f4a79219b (bug 1355898) for bustage on a CLOSED TREE

Backed out changeset 3bab94261ee0 (bug 1355898)

Bug 1353459 - handle queueInputBuffer exceptions. r=esawin. a=gchang MozReview-Commit-ID: 1IgLfpmGnN3

Bug 1355898. Use the first frame of animated images for texImage2d. r=jgilbert. a=gchang The spec for canvas drawImage says to use the first frame. I can't find anything spec related at all referencing the issue for webgl (except bug 666855). So do the same as drawImage.

Bug 1355898. Rename flag SFE_WANT_FIRST_FRAME to SFE_WANT_FIRST_FRAME_IF_IMAGE to represent what it does better. r=jgilbert. a=gchang We only pay attention to the flag for images.

Bug 1354810, CycleCollectedJSContext::Get() needs to be null-checked, r=baku. a=gchang

Backed out changeset b7cdc8cfc61f (bug 1353459) for android test failures

Bug 1346012 - Handle dead object wrappers in more places in Promise code. r=shu a=gchang MozReview-Commit-ID: HlmKwoMub9D

Bug 1353459 - handle queueInputBuffer exceptions. r=esawin. a=gchang MozReview-Commit-ID: 1Tm0vcl3Uv7

Bug 1354973 - Remove view before add to new ViewGroup, r=sebastian. a=gchang MozReview-Commit-ID: 1Xdlbss6SVO

Bug 1356718 - Query gtk settings schemas to avoid crashing when schema is missing. r=jfkthame. a=gchang MozReview-Commit-ID: 2rqO4XpLbfO

Bug 1351482 - Use NSIS 3.01 for buildbot jobs. r=mshal. a=gchang MozReview-Commit-ID: ElKlo6QwKmR

Bug 1347191 - Add support for NSIS 3.01 to moz.configure. r=mhowell. a=gchang

Bug 1350330 - Make sure IPC channel still Open before sending IPC message. r=dvander. a=gchang

Bug 1349828 Smooth scroller of <scrollbox> should not try to scroll to outside of the range. r=mstange. a=gchang The scroll destination of the smooth scroller of <scrollbox> can be outside of the actual scrollable range. Therefore, it doesn't make scroll slower even when the end appears. This patch makes the destination always in the scrollable range. MozReview-Commit-ID: CfEGzhG7Jh7

Bug 1313977 - Don't round fractional mouse coordinates up, because doing so might push them outside the window. r=kats. a=gchang On my machine, if I have my mouse cursor positioned flush against the right edge of my screen (which is 1440x900@2x), locationInWindow has an x coordinate of 1439.99609375. This value was rounded up to an integer screen coordinate of 2880, and for that coordinate we don't find a target APZC, and consequently refuse to scroll. MozReview-Commit-ID: CJic4g3Y6Ag

Bug 1354715 - Restore the right background color. r=spohl. a=gchang MozReview-Commit-ID: K7SF3rNPWUT

Bug 1353831 - OneOff block in search bar doesn't update text if Customize was opened at least once. r=florian. a=gchang MozReview-Commit-ID: Iu7xhCbCh6B

Bug 1352608 - Close a cursor after we're done with it r=eoger. a=gchang Removed "final" because "safeQuery" provides terrible developer ergonomics. MozReview-Commit-ID: 2F0XZnBM3Yv

Bug 1265314 - Set tabs record lastModified to clients.lastModified. r=nalexander. a=gchang MozReview-Commit-ID: 84g7WXxRwdX

No bug, Automated HPKP preload list update from host bld-linux64-spot-361 - a=hpkp-update

No bug, Automated HSTS preload list update from host bld-linux64-spot-361 - a=hsts-update

Bug 1356755 - Don't pass a null pointer to NotifyDocumentTree(). r=qdot, a=gchang

Bug 1355680 - Set progressMax to -1 when HTTP response head is not available. r=dragana, r=mayhemer, a=gchang MozReview-Commit-ID: 6xJsYFlCptC

Bug 1356183 - Let MousePosTracker use event.clientX instead of event.screenX - window.mozInnerScreenX to avoid flushing layout. r=florian, a=gchang MozReview-Commit-ID: 5jULHMnWFBW

Bug 1355414 - places.sqlite schema migration fails if an application has never used the bookmarks service. r=past, a=gchang MozReview-Commit-ID: 13YXf2On75J

Bug 1241066 - Fix mozRtt always 0 or 1. r=jib, a=gchang My shortest patch to date. MozReview-Commit-ID: 8r3ZrGUk40D

Bug 1347835 - NativeKey should dispatch keypress events even if WM_KEYDOWN is processed by IME but followed by printable WM_(SYS)CHAR messages. r=m_kato, a=gchang Some IME may handle WM_KEYDOWN message before application and may set the keycode value to VK_PROCSSKEY but not do actually. Similarly, IME may handle WM_KEYDOWN message and replace following WM_CHAR messages with different characters. Therefore, even if WM_KEYDOWN message comes with VK_PROCESSKEY, NativeKey shouldn't stop dispatching keypress events if it detects following printable char messages. MozReview-Commit-ID: DcC2qgcLDrQ

Bug 1351359 - Make nsGridContainerFrame call ReflowInput::SetBResize(true) because of auto-block-size swapping between measuring reflows and regular reflows. r=mats, a=gchang This fixes the failure of layout/reftests/css-grid/grid-min-max-content-sizing-002.html with the primary patch in bug 1308876 (which causes a child whose parent is dirty to pick up the dirty bit from the parent only the first reflow of the child if the parent reflows the child multiple times). A simplified testcase for that failure is https://bugzilla.mozilla.org/attachment.cgi?id=8849771 . The failure was caused by an error in height calculation of the first <x> in the test. The div that is the parent of that x has a definite height (presumably due to rules in grid), and the x has a specified height. The div gets three reflows: two measuring reflows (from MinContentContribution and then from MaxContentContribution) and then a final reflow from nsGridContainerFrame::ReflowInFlowChild. Prior to the primary patch in this bug, the div was marked dirty on all three reflows, but with it it is marked dirty only on the first. This means that, without the block-resize flag, the div optimizes away the reflow of its children, since ShouldReflowAllKids returns false because IsBResize() is false, even though NS_FRAME_CONTAINS_RELATIVE_BSIZE is correctly set. In order to fix this, we need to make sure the BResize flag on the reflow state in at least some cases (see the comments in the patch for when, and for how the cases could be optimized in the future). Note that: * when the dirty bit is set on the grid container, the new behavior (with the combination of the patches) is strictly more efficient than the old, since we will sometimes do non-dirty reflows on the grid items (with the b-resize flag) * when the dirty bit is *not* set on the grid container, the new behavior is less efficient than the old, since we will set the b-resize flag when we did not do so before. However, this slowdown fixes existing bugs such as the one in the reftest. Given that I was able to construct a reftest that triggers the failure without the changes from bug 1308876, I've moved this to a separate bug. Without the patch, grid-measuring-reflow-resize-dynamic-001.html fails, but grid-measuring-reflow-resize-static-001.html passes. With the patch both tests pass. (And without the patch, doing a text zoom on the dynamic test fixes the layout error.) MozReview-Commit-ID: JQOdVTQIkU0

Bug 1356786 - Add waitUntil in browser_webconsole_shows_reqs_in_netmonitor.js. r=ntim, a=test-only MozReview-Commit-ID: 1cLwVeaortC

Bug 1345650 - use https://%(server)s/ for shield/self-support URLs to not break web-platform-tests on beta, and set shield-recipe-client.api_url to a working value in browser_RecipeRunner.js, a=test-only

Bug 1351197 - disable test_TelemetryModules.js for crashing by hitting the network on beta, a=test-only

Bug 1351053. P2 - fix data race in accessing mAudioDemuxer/mVideoDemuxer, r=jya, a=jcristau 1. http://searchfox.org/mozilla-central/rev/944f87c575e8a0bcefc1ed8efff10b34cf7a5169/dom/media/MediaFormatReader.cpp#755 mAudioDemuxer is modified on the task queue thread of MFR in Shutdown() called by MFR::Shutdown(). 2. http://searchfox.org/mozilla-central/rev/944f87c575e8a0bcefc1ed8efff10b34cf7a5169/dom/media/MediaFormatReader.cpp#1082 mAudioDemuxer is read on the task queue thread of DemuxerProxy in DemuxerProxy::NotifyDataArrived(). MozReview-Commit-ID: 8qLAzNKjuzb

Bug 1331507 remove release/beta restriction on injecting sidebar button, r=kmag, a=jcristau MozReview-Commit-ID: 9pzyaWQGFn5

No bug, Automated HPKP preload list update from host bld-linux64-spot-324 - a=hpkp-update

No bug, Automated HSTS preload list update from host bld-linux64-spot-324 - a=hsts-update

No bug, Automated HPKP preload list update from host bld-linux64-spot-305 - a=hpkp-update

No bug, Automated HSTS preload list update from host bld-linux64-spot-305 - a=hsts-update

Bug 1353347 - wasm baseline, properly chop int32 constants to 32 bits. r=luke, a=jcristau

Bug 1354619 - Allow nsWindow::SetTransparencyMode when it preserves current mode. r=mconley, a=gchang MozReview-Commit-ID: FEqPTGlpbjE

Bug 1356091 - Allow the compositor process with the software compositor. r=dvander, a=gchang MozReview-Commit-ID: Ljt73heC0k4

Bug 1349741 - Part 2: Reftest. r=mstange, a=gchang MozReview-Commit-ID: AOKD7BeAUNl

Bug 1349741 - Part 1: Correct the transform matrix of a context that pass to nsFilterInstance::PaintFilteredFrame. r=mstange, a=gchang MozReview-Commit-ID: 9BISQehlSgG

Bug 1354245 - Don't call CaptivePortalService::Start() in SetConnectivityInternal if the captive portal service is disabled via pref. r=mcmanus, a=jcristau MozReview-Commit-ID: 4xCkBgWDZ2b

Bug 1337947 - Fix tab strip item offsets for RTL. r=maliu, a=gchang MozReview-Commit-ID: HrOETozOSfW

Bug 1337947 - Fix tab strip divider drawing for RTL. r=maliu, a=gchang MozReview-Commit-ID: LITy483QhlW

Bug 1354924 - Fix mousewheel multiplier prefs for external mouse devices on OS X. r=botond, a=gchang The code in the OS X widget was calling ReceiveInputEvent on IAPZCTreeManager with a ScrollWheelInput, which would bypass the multiplier code. This modifies the widget to use a WidgetWheelEvent instead, so that it goes through the IAPZCTreeManager multiplier handling for wheel inputs. Other platforms already send wheel events in WidgetWheelEvent format so they don't have this problem. MozReview-Commit-ID: 5gOOGnfD87W

Bug 1350718 - Part 3: Name the special value -1 used to indicate a new tab should be appended. r=maliu, a=gchang

Bug 1350718 - Part 2: Remove the add animation for the TabsLayout list. r=maliu, a=gchang Based on what I'm seeing, if you call scrollToPosition and that causes you to "scroll into view" (remember, scrollToPosition doesn't actually scroll, it just redraws the new position) one or more positions, then RecyclerView runs the add animation on all those views "scrolled onto screen", which, for the list view's slide-in-from-the-right add animation, looks silly (I think). [Caveat: RecyclerView sometimes keeps one offscreen view ready to go, which doesn't seem to get the add animation.] In non open-tab-from-another-app-with-the-tabs-tray-already-open operations this was never an issue because either those animations are hidden by the panel being animated into view when the panel opens and we scroll to the selected position [at least that's my guess], or we only scroll by at most one, as in the case of a tab close or undo close. But in the open-a-tab-and-scroll-to-it-while-the-tabs-tray-is-already-open case that we can get with opening a tab from another app, the add animation runs for however many tabs "need to be added" between the current position and the new tab; sometimes the animation still gets hidden if the new tabs get added quickly enough when fennec reloads [again, my guess], but on my device I always see the animations if I open a tab in tab queue and then reopen Fennec by hand, whereas on an emulator I see the animations in additional external-app-open cases as well.

Bug 1350718 - Part 1: Scroll to a tab added to the tabs tray by an external app. r=maliu, a=gchang If another app opens a link in Fennec, and Fennec restores itself in a state where the tabs tray is already open, we need to scroll to the newly added tab since it gets added offscreen (not to mention the scroll position restored when we open is unconstrained (it's whatever the user left it at before they switched apps)). This introduces one small change in behavior: 1) Use a gridded tabs tray; 2) Fill more tabs than will fit in the tray; 3) Put more than one tab on the last row; 4) Scroll so that the last row is partially, but not fully, hidden; 5) Close the last tab and then undo the close. In that case we now scroll the last row fully into view, whereas previously we maintained the old (partially hidden) scroll position. (If you undo close any tab other than the last on the final row then you still get the old behavior.) Note that this fixes the case where the other app adds a *new* tab in Fennec with the tabs tray open; it's (currently) also possible to open a link in an already existing tab with the tabs tray open - that's bug 1353226.

Bug 1353694. r=jonco, a=jcristau

Bug 1349883 - Part 4: Strengthen precondition of ReturnDecodedData(). r=jya, a=gchang MozReview-Commit-ID: 2jLNlrHEE0b

Bug 1349883 - Part 3: Resolve decode promise according to buffer status. r=jya, a=gchang MozReview-Commit-ID: GwCPHXW0fqK

Bug 1349883 - Part 2: Reveal more input buffer status to callbacks. r=esawin, r=jya, a=gchang Promise based MediaDataDecoder expects one response per request, but ICodecCallbacks was not designed that way. onInputExhausted() is called only when there are none or just a few input buffers waiting to be queued, and onOutput() is called as soon as output buffers are available. It means these 2 kinds of events are usually interleaved and hard to align with pending promises. Reporting each input buffer status makes it easier for RemoteDataDecoder to resolve promise properly. MozReview-Commit-ID: 3FoPzXruRnh

Bug 1349883 - Part 1: Combine output buffer index and sample records. r=esawin, r=jya, a=gchang MozReview-Commit-ID: ERbFUoyvVhZ

Bug 1346235 - Part 4: Recycle unpopulated input samples. r=jchen, a=gchang MozReview-Commit-ID: Lr0aSQ0OfRg

Bug 1346235 - Part 3: Forward shared memory allocation error to peer process. r=jchen, a=gchang MozReview-Commit-ID: KDIMylv9tVt

Bug 1346235 - Part 2: Translate memory error to I/O failure. r=jchen, a=gchang MozReview-Commit-ID: F2ca0q2JQp

Bug 1346235 - Part 1: Translate native error to Java exception. r=jchen, a=gchang MozReview-Commit-ID: BcpfJLSzTvk

No bug, Automated HPKP preload list update from host bld-linux64-spot-089 - a=hpkp-update

No bug, Automated HSTS preload list update from host bld-linux64-spot-089 - a=hsts-update

Bug 1345225 - Check if webRequest filters overlap with host permissions, r=kmag, a=gchang MozReview-Commit-ID: 1tMHynv9FBO

Bug 1356292 - Switch back to triggering principal checks for add-on install permission checking to allow url bar triggered installs to proceed. r=bz, r=aswan, a=lizzard MozReview-Commit-ID: 4HXgGqSGkN0

Bug 1354294 - Preserve IsCallable and IsConstructor when nuking wrappers. r=evilpie, a=lizzard

Bug 1337578 - Mark atoms when cloning functions. r=bhackett, a=gchang

Bug 1354395 - Always bind symbols at load time for ctypes libraries. r=bsmedberg, a=gchang When symbols are not bound at load time, missing symbols can lead to NULL derefs or jumps to 0x0 at runtime, crashing the process running the corresponding JS code, which is rather undesirable. So, prevent libraries that have missing symbols to load at all through ctypes.

Bug 1354395 - Update sixgill to a rebuild that links against GMP statically. r=sfink, a=gchang The mercurial revision of sixgill listed in the manifest doesn't exist, so I took what looks like corresponds to the last change to the tooltool manifests, in order to avoid any other difference than GMP linkage. This was built manually on a one-click-loaner.

Bug 1349275 - Refactor `moveInfobar` function. r=pbro, a=lizzard - Added `getViewportDimensions` - Added `getComputedStylePropertyValue` to `CanvasFrameAnonymousContentHelper` - Refactored totally `moveInfobar` to works with both APZ enabled and new positioned absolutely highlighters - Updated `AutoRefreshHighlighter` for having a `scrollUpdate` method. - Updated tests MozReview-Commit-ID: 5m31ZzRzLXr

Bug 1356123 - Add missing locales package to the lint docker image. r=gps, a=NPOTB For some reason, the locales package is not installed anymore during the docker image build, which leads to the locale-gen command failing, since it's not there.

No bug, Automated blocklist update from host bld-linux64-spot-368 - a=blocklist-update

No bug, Automated HPKP preload list update from host bld-linux64-spot-368 - a=hpkp-update

No bug, Automated HSTS preload list update from host bld-linux64-spot-368 - a=hsts-update

Bug 1355070 - Add Burmese(my) to shipped-locales r=Pike a=gchang MozReview-Commit-ID: JRRJY9TpRgN

Bug 1350972 - Avoid crash when AutoInvalidateSelection dtor doesn't have commonAncestor. r=smaug a=gchang

Bug 1350525 - Storage Inspector should trim port from hosts for cookies r=pbro a=gchang MozReview-Commit-ID: FfsfiXTt96o

Bug 1349096 - Prevent download panel from moving up and down with download start animation. r=Paolo a=gchang MozReview-Commit-ID: E6zQUXcBWWP

Bug 1346623 - Part 2: Add an API to get computed style values through an AnonymousContent object. r=bholley a=gchang MozReview-Commit-ID: Dbvuk16CjFT

Bug 1340934 - Enable flash async drawing on release channels. r=bsmedberg a=gchang MozReview-Commit-ID: 6SmdWRrGmEI

Bug 1337933 - Make the tabs grid layout decorations work with RTL. r=maliu a=gchang This patch affects all grided tabs panel layouts: tablets, phones in landscape mode, and phones in portrait mode with the "Compact tabs" setting on. MozReview-Commit-ID: 5cqVJA57ARu

Bug 1355259 - Only filter out udp candidates if force_tcp is set. r=mjf, a=gchang MozReview-Commit-ID: AKv0N74epZ1

Bug 1344401 - Increase wait time in test_formless_submit_navigation_negative, on Android only. r=mattn, a=test-only

Bug 1353736 - Add whitelist entries to placate eslint. a=bustage

Bug 1353216 - Disable certificate transparency processing for performance concerns. r=jcj, a=lizzard MozReview-Commit-ID: 2k7FwC5RCWC

Bug 1346392 - Force non-spdy on sticky auth connections. r=dragana, a=lizzard MozReview-Commit-ID: IkYTVgetKsn

Backed out changeset 11a4906850b2 (bug 1322554) for frequent crashes. a=jcristau

Bug 1192818 - Skip browser_mediaPlayback_mute.js on Linux debug due to frequent failures. a=test-only

Bug 1090198 - Disable intermittent websockets/constructor/014.html on e10s debug. a=test-only

Bug 1139795 - Disable /html/dom/dynamic-markup-insertion/opening-the-input-stream/010.html due to bad timing. r=jgraham, a=test-only MozReview-Commit-ID: 49YuAQjjYoH

Bug 1355624 - Make Mercurial require TLS 1.2+ connections. r=dustin, a=NPOTB Mercurial uses the latest version of TLS that is both supported by Python and the server. In automation, the servers we care about should all support TLS 1.2. The Python side is trickier. Modern versions of Python (typically 2.7.9+) support TLS 1.1 and 1.2. Mercurial will default to allowing TLS 1.1+ - explicitly disallowing TLS 1.0. However, legacy versions of Python don't support TLS 1.1+, so Mercurial will allow TLS 1.0+ rather than prevent connections at all. TLS 1.0 is borderline secure these days. I think it is a bug for TLS 1.0 to be used anywhere in the Firefox release process. This simple patch changes our default Mercurial config in TaskCluster to require TLS 1.2+ for all https:// communications. For modern Python versions, this effectively prevents potential downgrade attacks to TLS 1.1 (connections before should have negotiated the use of TLS 1.2). I expect this change to break things. Finding and fixing automation that isn't capable of speaking TLS 1.1+ should be encouraged. MozReview-Commit-ID: 876YpL5vB3T

Bug 1353736 - Add regression test for JSON viewer after updating triggeringPrincipal. r=gijs, a=test-only

Bug 1355009 - Temporarily disable usage of simplified update ui. r=ato, a=test-only As long as update tests do not support the new simplified update ui it has to be kept disabled. MozReview-Commit-ID: 4fC0CYhp7Pc

Bug 1355009 - Harden update tests with better error messages. r=ato, a=test-only To ensure better failure messages a refactoring of checks has to be done. It includes the following changes: * No further checks for a follow-up (watershed) update. It's not supported and as such doesn't need assertions (bug 1353717) * Checks for fallback updates have to be made to ensure that an invalidated partial/complete update does not cause an upgrade of Firefox during the restart. MozReview-Commit-ID: CLb0aXoIur2

Bug 1355009 - Force update tests to only allow a single update. r=ato, a=test-only There was never a need to run a multiple-update step in the past, and as we agreed a while ago it is not something we want to do in the future. It means that watershed releases will have to be tested by issuing multiple update tests. MozReview-Commit-ID: 7cmK3gEOkv1

Bug 1355471 - Test timeout field before value's typing. r=whimboo, a=test-only The error message returned when unmarshalling the timeout configuration object with invalid input is misleading, because it checks the typing of the value before the field name. This patch changes Marionette to run the type assertion for the value after each case in the switch statement has been evaluated, ensuring that the field is valid before asserting its value. It also adds a few unit tests to verify this behaviour. Fixes: https://github.com/mozilla/geckodriver/issues/633 MozReview-Commit-ID: LVjTyUacD0s

Bug 1355404 - Make single locale use the unified repo r=nthomas a=release MozReview-Commit-ID: KvYjJhbxCzs

no bug - Bumping Fennec l10n changesets r=release a=l10n-bump bg -> ['android', 'android-api-15', 'android-multilocale'] kab -> ['android', 'android-api-15', 'android-multilocale']

Bug 1355190 - Add "bg" and "kab" in Fennec maemo-locale for multi-locale builds. r=flod, a=gchang

Bug 1353204 - Use entire spec for origin for Thunderbird. r=bz, r=bholley, a=gchang

Bug 1354090: P4. Only assume we have an init segment with moov box. r=gerald, a=gchang By ISOBMFF spec, an init segment is made of an ftyp and a moov box. However the ftyp box serve little purpose as such and only the moov atom contains essential information. Some streams incorrectly add ftyp box all accross the content, despite the ISOBMFF spec stating (4.3.1): Box Type: `ftyp’ Container: File Mandatory: Yes Quantity: Exactly one (but see below) Additionally, with this change the ftyp box may not be present in content following earlier spec. So we will now be able to play them. MozReview-Commit-ID: KijlV5pPLby

Bug 1354090: P3. Properly offset the media and init byte ranges. r=gerald, a=gchang The init and media segment byte ranges were not offset by the amount of bytes currently parsed. Whenever a new init segment signature was seen we would recreate a container parser. This would lead to invalid offsets later. MozReview-Commit-ID: 8U7kTa7SK8O

Bug 1354090: P2. Add operator+(val). r=gerald, a=gchang Such that: [x, y] + a = [x+a, y+a] MozReview-Commit-ID: HlyI9KmVtTg

Bug 1354090: P1. Fix coding style. r=gerald, a=gchang MozReview-Commit-ID: ByBHvxM3kfc

Bug 1344839 - Don't allow transparent top-level windows on Windows. r=jimm, a=gchang MozReview-Commit-ID: 6bXHqDuUMw6

Bug 1344839 - Don't allow transparent top-level windows on GTK. r=karlt, a=gchang MozReview-Commit-ID: 9UP1UYcaF2F

Bug 1317594 - Vendor latest version of robustcheckout extension. a=NPOTB File copied verbatim from changeset 249a47720ddcf896a9f07600c429a1b4492b805e from the version-control-tools repository. The updated extension now detects and retries after network failures where it didn't before. This should cut down on the number of intermittent failures. MozReview-Commit-ID: 2bFLcGEARTJ

Backed out changeset f4b12d9682b3 (bug 1317594) for breaking compat with Mercurial 3.7.

Bug 1329294 - Part 2: Enable content temp file test on Windows. r=haik, a=lizzard

Bug 1329294 - Part 1: Ensure Content Temp Dir is created before use. r=bsmedberg, a=lizzard

Bug 1349055 - Stencil clear value needs to mask to be an unsigned integer. r=jgilbert, a=gchang MozReview-Commit-ID: 48PE2bHh87L

Bug 1350243 - Combine e10s and non-e10s code paths when reacting to datalist updates. r=MattN, a=gchang MozReview-Commit-ID: 1O3wAzzwWAR

Bug 1351301 - Test that '.' is not required to accept subdomains in auth::URIMatchesPrefPattern. r=mayhemer, a=gchang MozReview-Commit-ID: 4FXBDSo1bVG

Bug 1351301 - Don't require '.' to accept subdomains in *.auth.trusted-uris preferences. r=valentin, a=gchang

Bug 1345355 - Add a gtest for a pinch with zero span but changing focus. r=botond, a=gchang MozReview-Commit-ID: 8AstF1QmBzw

Bug 1345355 - Allow pinch gestures with a zero span change but a nonzero focus change to scroll. r=botond, a=gchang It appears that some touchpad devices send us "touch" events (i.e. WM_TOUCH on Windows) but with all touch points having the same coordinates. This ends up getting detected as a zero-span pinch gesture in APZ, which short-circuits early and doesn't really get processed. Therefore even if the focus point changes we don't do any corresponding scroll. This patch shifts things around a little so that the short-circuit doesn't happen quite so early, and we still scroll when the focus point changes, even if the span is zero. MozReview-Commit-ID: 3CaQN1MsM8y

Bug 1349419 - Do fallible memory alloc and return failure to avoid OOM crash. r=mcmanus, a=gchang

Bug 1353610 - Add null checks. r=bobbyholley, a=gchang

Bug 1350177 - Refactor a thread-safe ExpirationTracker for the use in SurfaceCache. r=froydnj, r=mystor, r=aosmond, a=gchang

Backed out 4 changesets (bug 1192818) for causing bug 1346872 and other regressions. a=gchang Backed out changeset 8ceb12241231 (bug 1192818) Backed out changeset be2f9c0ee32e (bug 1192818) Backed out changeset ede7e1b58095 (bug 1192818) Backed out changeset 53900c65dbe1 (bug 1192818)

Bug 1342941 - Implement the geometry editor toggle in the new box model. r=zer0, a=gchang

Bug 1351340 - Check null pointer before using. r=bagder, a=lizzard MozReview-Commit-ID: LJ0FtjZTKq3

Bug 1295815 - Retry synthesizeNativeMouseMove until mousemove happens. r=xidorn, a=test-only MozReview-Commit-ID: 9e10lJbHzDd

Bug 1348547 - ReleaseCachedProcesses() between aboutdebugging tests. r=gabor, a=test-only

Bug 1345932 - Show warning info in about:debugging#workers if multi-e10s is on. r=ochameau, a=gchang If multi e10s is enabled, service worker debugging is unavailable. We temporarily disabled the start/push/debug buttons in about:debugging for all service workers if multi-e10s is on. A warning section is also displayed on top of about:debugging, letting the user know that they can set dom.ipc.processCount to 1 in order to force a single content process and restore debugging capabilities. MozReview-Commit-ID: IPAlbOq2Hij

Bug 1317594 - Vendor latest version of robustcheckout extension. a=NPOTB File copied verbatim from changeset e0d30b04dac6bcd36b57c711d9c5b1c280f63390 from the version-control-tools repository. The updated extension now detects and retries after network failures where it didn't before. This should cut down on the number of intermittent failures. MozReview-Commit-ID: 2bFLcGEARTJ

Backed out changeset 9e4c49409f40 (bug 1342433) for causing bug 1349847. a=lizzard

Bug 1355534 - Revert to longer insecure login field contextual warning string. r=johannh a=lizzard MozReview-Commit-ID: F2hmNQGLvo8

No bug, Automated HPKP preload list update from host bld-linux64-spot-304 - a=hpkp-update

No bug, Automated HSTS preload list update from host bld-linux64-spot-304 - a=hsts-update

Bug 1351188 - Disable test262 String.prototype.normalize tests when normalize isn't available, r=shu, a=test-only

bug 1318919 - fix beta to release config. r=jlorenzo a=release DONTBUILD

Bug 1337641 - Temporarily disable an assert in cubeb duplex. r=achronop a=NPOTB MozReview-Commit-ID: 28Bx6H7fL6o

Bug 1353975 - Fix nullPrincipal edge case. r=bz, r=mcmanus, a=lizzard

Bug 1353649 - Update tzdata in ICU data files to 2017b. r=Waldo, a=lizzard

Bug 1338867 - Strip username/password after strip reader mode url prefix. r=sebastian, a=lizzard MozReview-Commit-ID: KCr7cBdetq7

Bug 1183300 - Intermittent dom/security/test/csp/test_upgrade_insecure.html. disable on win7. r=ckerschb, r=gbrown, a=test-only MozReview-Commit-ID: AslnFrYGOVw

Bug 1353910 - Fix intermittents in webrtc tests while removing code duplication. r=florian, a=test-only MozReview-Commit-ID: 83XSsnQ0bBY

No bug, Automated HPKP preload list update from host bld-linux64-spot-305 - a=hpkp-update

No bug, Automated HSTS preload list update from host bld-linux64-spot-305 - a=hsts-update

Backed out changeset ecbabf4c7f8d (bug 1353975) for bustage

Bug 1354310 - Ensure to use page load timeout for Wait().until() in unit tests. r=ato a=test-only Tests which have to wait for a page being loaded should always use a timeout as set via self.marionette.timeout.page_load. MozReview-Commit-ID: HFTOYy6WYNk

Bug 1354323 - Accept string text for sendKeysToDialog command; r=automatedtester a=test-only As a follow-up to https://bugzilla.mozilla.org/show_bug.cgi?id=1354323, this makes the Marionette sendKeysToDialog command take a text field with a string, instead of the value field with an array of strings. The relevant WebDriver specification change is https://github.com/w3c/webdriver/pull/889. Fixes: https://github.com/mozilla/geckodriver/issues/607 MozReview-Commit-ID: AY52pAK2582

Bug 1354233 - Add stream to m0RTTStreams after streamId is set. r=mcmanus a=gchang

Bug 1353980 - Don't try to hide popup notifications in fullscreen mode. r=Paolo a=lizzard MozReview-Commit-ID: 5VWHooAIMRT

Bug 1353975 - Fix nullPrincipal edge case. r=bz, r=mcmanus a=lizzard

Bug 1353917 - Skip the downloading page in the update wizard if the update is already staged. r=rstrong a=gchang The update wizard's fallback mode (where it appears on startup to download and apply a complete update if applying a partial update has failed) begins to download and stage the update without waiting for the user to confirm. This means it can get ahead of itself and actually finish downloading and staging before the user can click OK to bring up the progress page. The progress page does not expect this and errors out in a way that breaks the otherwise valid update. This change therefore skips over that page if the update is finished staging when it would normally be shown. MozReview-Commit-ID: E6P2UKO4C9h

Bug 1353638 - P2: Remove a testcase. r=bkelly a=lizzard

Bug 1353638 - P1: Hide useCache registrationOptions. r=bkelly a=gchang

Bug 1352778 - Do not reset triggeringPrincipal but only principalToInherit within loadInfo when forced to. r=gijs a=gchang

Bug 1352380 - Make sure gradient stops have ordered mPosition variables. r=dbaron a=lizzard MozReview-Commit-ID: DsP5Q6qmvH2

Bug 1349406 - Avoid reflows for arrow-typed download indicators by using stack instead of position: absolute. r=Paolo a=gchang MozReview-Commit-ID: KLsYueSWKId

Bug 1348936 - Fix BasicPlanarYCbCrImage::CopyData() r=nical a=gchang

Bug 1347646 Part 2: Use CreateRenderingContext in nsSimplePageSequenceFrame::PrePrintNextPage to ensure recorder is used for canvases. r=jwatt a=lizzard Calling CreateReferenceRenderingContext currently returns a non-recording DrawTarget even when the nsIDeviceContextSpec within the nsDeviceContext has a DrawEventRecorder, due to bug 1354624. Since we've already called BeginPage we can call CreateRenderingContext instead though and that will give us a recording DrawTarget as we require to record the canvas drawing.

Bug 1347646 Part 1: Backout 4849ef8c9a34 for causing OOM issues when printing PDFs. r=jwatt, r=lsalzman, r=tobytailor a=lizzard

Bug 1344174 -Add measure for memory distribution among child processes. r=chutten a=gchang

Bug 1342002 - Border color of overriden items in Rule view looks lighter than font color in dark theme. r=gl a=gchang MozReview-Commit-ID: CazYLmKktx

Bug 1273265 - Allow long file names in specific cases. r=jimm a=lizzard MozReview-Commit-ID: 3tl5Iihxs27

Bug 1353088: Ensure we don't attempt to process garbage data. r=gerald,a=lizzard MozReview-Commit-ID: AryC34eLjOg

No bug, Automated HPKP preload list update from host bld-linux64-spot-303 - a=hpkp-update

No bug, Automated HSTS preload list update from host bld-linux64-spot-303 - a=hsts-update

Bug 1349879 - Intl object is hidden on Android beta and release channel. r=snorp,Waldo a=test-only Some interface tests into mochitest is failure on android beta and release. test_serviceworker_interfaces.js has nonReleaseAndroid flag, so we should use it into test_interfaces.js and test_worker_interfaces.js MozReview-Commit-ID: A1aHrTXwGil

No bug, Automated HPKP preload list update from host bld-linux64-spot-388 - a=hpkp-update

No bug, Automated HSTS preload list update from host bld-linux64-spot-388 - a=hsts-update

Bug 1352681 - Make SAB rawBuffer refcounting for structured clone more sophisticated. r=sfink, a=dveditz

Bug 1352681 - Overflow checking on SAB reference count. r=sfink, a=dveditz

Bug 1343642 - Ensure to grab nsFrameSelection before calling its methods unless calling only const methods. r=smaug, a=abillings

Bug 1354625 - Use mozilla-unified for upstream for all Firefox builds. r=aki, a=NPOTB Without the use of an upstream the first repo cloned on a machine will be cached. If a subsequent job references a different repo, it may have to pull thousands of changesets because those changesets aren't part of the initially-cloned repo. This is why the --upstream feature to robustclone exists and is why it uses the mozilla-unified repo. The mozilla-unified repo is a superset of central, aurora, beta, release, etc. So by cloning it, you get changesets for all of the repos at the time of the clone. When a subsequent job comes along and requests a changeset from a different repo, you likely only need to fetch data for a handful of changesets, not thousands. This change adds the upstream url config for all fx_desktop_build jobs, ensuring it is used. A redundant config entry for the try repo has been removed as well. MozReview-Commit-ID: 3EL7aSXS4AG

Bug 1354625 - Use mozilla-unified for single_locale try builds upstream. r=aki, a=NPOTB I think I initially defined these and I think I know why I used mozilla-central (it had to do with try not advertising bundles and mozilla-unified being generaldelta when other repos were not). Those reasons are no longer valid and we should be using mozilla-unified everywhere. MozReview-Commit-ID: CFaZspU6A5M

No bug, Automated blocklist update from host bld-linux64-spot-308 - a=blocklist-update

No bug, Automated HPKP preload list update from host bld-linux64-spot-308 - a=hpkp-update

No bug, Automated HSTS preload list update from host bld-linux64-spot-308 - a=hsts-update

Bug 1353333 - [push-apk] release: activate rollout percentage r=aki a=release MozReview-Commit-ID: 6SXbfREPBVS

Bug 1321418 - Update menu items using the correct ID. r=sebastian, a=lizzard Regression caused by an incorrect ID used to update menu items.

Bug 1347617 - In SVG ConvolveMatrix code, bail out if kernelUnitLength is negative or zero. r=dholbert, a=lizzard

Bug 1346630 - Disable test_peerConnection_captureStream_canvas_webgl.html on Mw android 4.2/17. r=dminor, a=test-only

Bug 1281639 - Disable failing tests for Mochitest Media failures on Android. r=gbrown, a=test-only

Bug 1347954 - Disable dom/media/test/test_video_to_canvas.html. r=gbrown, a=test-only

Bug 1232305 - Disable failing Mochitest DOM Media tests on Android - follow up 1. r=gbrown, a=test-only

Bug 1354224 - Upgrade to Mercurial 4.1.2. r=dustin, a=gchang This is a pretty straightforward change. Just bumping package versions and hashes. MozReview-Commit-ID: CaVjM0JHYKi

Bug 1345873 - Part 1: Make flex-item sorting code treat placeholders as having the default value of 'order' / 'box-ordinal-group'. r=mats, a=lizzard This should allow their flex-item siblings to reorder themselves as-needed for honoring their own 'order' values, without unnecessarily reordering the placeholders. MozReview-Commit-ID: aCBQER5r2G

Bug 1345873 - Part 0: Minor cleanup in existing test_flexbox_order mochitests. a=test-only This patch: - does s/var/let/ to upgrade to modern JS best-practices. - Wraps some lines that are too long. - Changes loops to "for (let foo of [...])" rather than foreach+function-pointer. - Changes some copypasted cleanup code to use a loop instead (which will be especially useful in a forthcoming version of this test that'll add another thing to clean up). MozReview-Commit-ID: DWK8jFbfqeB

Bug 1350661 - Extract layout attributes into styles in order to separate api 15 and 17 style tree. r=sebastian, a=gchang __Device_Configuration__|__o_Applied_Style_______________ ldrtl-v17 v17 v15 | o o o | UrlBar.Entry \ | | | -----o | | UrlBar.V17.Entry(start/end) \ | | \ o | UrlBar.V15.Entry(left/right) \ | | --o | UrlBar.Base.Entry(original style) Though Android support RTL since API level 17(JB_MR1), it's really buggy at that moment. This patch fix a severe UI layout attribute bug, which only happen on android 4.2 in RTL language context: If view attributes "start/end" and "left/right" are both written in a view layout xml, they will both be applied and cause UI abnormal. In API 18 and above, "left" will be ignored if "start" also exist. For example, as below show, alignLeft and alignStart are both exist in ImageView. On android 4.2 with RTL context, it's width will both align Left and Start(Right), cause the symptom that ImageView have the same width and cover on the view "back." ``` <ImageView android:id="@+id/url_bar_entry" android:layout_alignLeft="@+id/back" android:layout_alignStart="@+id/back" ``` MozReview-Commit-ID: JptLuWX2w15

Bug 1348576 - e10s is never enabled for non-official release builds. r=mconley, a=gchang MozReview-Commit-ID: DaqGyeGdgzD

Bug 1353660 - Proxy destruction of nsHashPropertyBag's hash table to the main thread. r=mccr8, a=gchang We need this because the stored values in the hash table may themselves be main-thread only objects, and destroying them off the main thread will cause crashes.

Bug 1353789 - Unship deployment-checker. r=Felipe, r=jcj, a=lizzard MozReview-Commit-ID: 3Q5v97JVs2X

Bug 1353439 - Cc and Ci are not defined in printUtils.js so use Components full names. r=jhector, a=gchang

Bug 1333858 - Intersect should be fallible on overflow. r=daoshengmu, a=gchang MozReview-Commit-ID: 6lmIKKyXXah

Bug 1350442 - Remove redundant storeDone from the RepositorySession class. r=rnewman, a=gchang Confusion between storeDone() and storeDone(long end) resulted in certain sessions (bookmarks and form history) not overriding the current method. As a result, their final "flush the queue" methods weren't being called by the buffering middleware. This patch removes the storeDone(long end) method, making such confusion a non-issue. Given that a lot of sessions tend to build up buffers which they need to then flush after a storeDone() call, passing in a timestamp into that method doesn't make sense. Instead, let's supply a default implementation in RepositorySession which calls onStoreCompleted(endTimestamp) with current time, and allow sessions to override this method and own the onStoreCompleted(endTimestamp) call. MozReview-Commit-ID: 84o7aAL8RPC

Bug 1350442 - Use the main storeWorkQueue for failures in the buffering middleware. r=rnewman, a=gchang MozReview-Commit-ID: ByZtR5zSKZB

Bug 1339588 - Part 3: Don't break the nsTimer/nsTimerImpl cycle during Fire. r=froydnj, a=dveditz MozReview-Commit-ID: J6TNJqGsBv4

Bug 1339588 - Part 2: Help prevent nullptr crashes due to misuse of the timer API. r=froydnj, a=dveditz MozReview-Commit-ID: BXCGYWnFqSj

Bug 1339588 - Part 1: Simplify nsTimerEvent::Cancel, since there's no need to release the nsTimerImpl itself. r=froydnj, a=dveditz MozReview-Commit-ID: 6br6DaDqxR0

Bug 1306848 - Skip TestTimeouts.test_search_timeout_found on Android. r=ato, a=test-only

Bug 1317121 - Skip TestTimeouts.test_search_timeout_found_settimeout on Android. r=ato, a=test-only

Bug 1353016 - Add more test chunks for linux debug mochitest-bc. r=jmaher, a=test-only

Bug 1344486 - Intermittent TEST-UNEXPECTED-PASS | /html/semantics/scripting-1/the-script-element/nomodule-set-on-async-classic-script.html. fail early if noModule. r=jgraham, a=test-only MozReview-Commit-ID: K8qMR6fICXH

Bug 1341980 - Intermittent browser/components/sessionstore/test/browser_819510_perwindowpb.js - disable on osx. r=gbrown, a=test-only MozReview-Commit-ID: J3IL8cHAmau

Bug 1322554 - Interpose kernel32!BaseThreadInitThunk to add verification of thread start addresses. r=dmajor, a=gchang MozReview-Commit-ID: CvqZ3gXJyvo

Bug 1353543 - Baldr: soften LookupAsmJSModuleInCache check. r=bbouvier, a=gchang MozReview-Commit-ID: 7g55zJzU7pF

Bug 1353618 - Optimize storage of MozPromise::mThenValues. r=gerald, a=gchang 1. Set the capacity to 1 when IsExclusive is true. 2. Set it to 3 because Try shows we never have more the 3 elements when IsExclusive is false. MozReview-Commit-ID: dmIeE6ZTeh

Bug 1345422 - Do not hardcode the sfx package and installer path. r=mshal, a=gchang

Bug 1353742 - Upgrade Firefox 54 to NSS 3.30.1. a=gchang

Bug 1083140 - Intermittent devtools/client/projecteditor/test/browser_projecteditor_contextmenu_01.js. disable on asan. r=jryans, a=test-only MozReview-Commit-ID: HUXF2MUXqfi

Bug 1348174 - Test that deviceIds are stable for same origin and differ across origins. r=jesup, a=test-only MozReview-Commit-ID: LEkHzgUZwqd

Bug 1348174 - Convert test_enumerateDevices.html to async/await. r=ng, a=test-only MozReview-Commit-ID: 7kWiwD0E5tO

No bug, Automated HPKP preload list update from host bld-linux64-spot-309 - a=hpkp-update

No bug, Automated HSTS preload list update from host bld-linux64-spot-309 - a=hsts-update

bug 1354115: remove special WebSense rules for non-blacklisted and unknown WebSense users. r=jlorenzo, a=release

Bug 1351964 - Use UI thread for "Tabs:TabsOpened" event. r=sebastian, a=lizzard Use the UI thread for handling "Tabs:TabsOpened", so we don't race with other tab events that are already handled on the UI thread.

Bug 1352926 - Make sure to check the right character for being semicolon or comma. r=farre, a=dveditz MozReview-Commit-ID: DOb1qYb8Tz4

Bug 1328001 - Fix focusring on devtools buttons. r=jryans, r=ntim, a=lizzard MozReview-Commit-ID: 2SA3ZLLIE8E

Bug 1347075 - Validate insertion index for a child move. r=yzen, a=lizzard

Bug 1347758 - Let outer window decides when need to resume the tab. r=ehsan, a=gchang

Bug 1345653 - Handle document unload when dispatching actions. r=ato, a=test-only This fixes the reported hang that occurs after a pointer click action resulting in navigation. MozReview-Commit-ID: A9SBhextVLH

Bug 1332279 - Include keyCode in KeyboardEvents synthesized for key actions. r=ato, a=test-only The key dispatch functions now pass the raw key to event.js, which determines the keyCode for the event. Note the change in Normalized Key Value for Enter versus Return. The browser throws an exception when the event key attribute is set to "Return" and KEY_NON_PRINTABLE_KEY is set, which implies that the key value isn't valid. Changing it to Enter fixes the issue. MozReview-Commit-ID: 831f4EcqI1P

Bug 1352367 - part2: [push-apk] deactivate dry-run option on beta r=aki a=release MozReview-Commit-ID: BUm8aJVw7Dm

Bug 1349650 - [css-grid] Crashtest. a=NPOTB MozReview-Commit-ID: LNuu5leq887

Bug 1349650 - [css-grid] Fix a bogus assertion. r=dholbert a=NPOTB When mArray is used it contains all grid container children, including placeholders. aGridItemCount always excludes placeholders. MozReview-Commit-ID: 2RVFOdVvTq

Bug 1348776 - Part 1: Fix a crash in the profiler when MOZ_PROFILING is undefined. r=mstange, a=gchang

Bug 1353373 - Fix Edge readinglist imports. r=jaws, a=lizzard MozReview-Commit-ID: Fu9n3FIrxf6

Bug 1345108 - Reduce resolution for timestamp in crashreporter's crash pings. r=ted, a=gchang MozReview-Commit-ID: 5BA51mWlkqW

Bug 1348250 - Change TelemetryStorage to load crash pending pings outside of the profile. r=chutten, a=gchang MozReview-Commit-ID: 1kqdGgEwXvI

Bug 1349421 - Update handling of values derived from chunk duration in OpusTrackEncoder. r=derf, a=gchang This changeset updates the handling of the variables based on chunk.GetDuration() within the OpusTrackEncoder. This changeset alters places where overflow could have taken place previously. It also adds asserts with the dual purpose of defense and documentation for future developers. MozReview-Commit-ID: 28vmAfE84ik

Bug 1349421 - Adjust handling of chunk duration in opus encoder. r=jesup, a=gchang For very long chunks, the opus encoder could run into issues with processing. This changeset seeks to address that by using CheckedInt to prevent a potential int overflow when encountering extremely long durations. MozReview-Commit-ID: 7uLZAARLf5w

Bug 1351465 - shield-recipe-client: Unconditionally set up logging in bootstrap. r=Gijs, a=gchang MozReview-Commit-ID: 5W7UdpsLyYd

Bug 1345153 - Suppress valgrind warnings for PR_SetEnv in SetupCrashReporterDirectory. r=njn, a=gchang MozReview-Commit-ID: 4vV2EHsgTVT

Bug 1345153 - When the pingsender fails to send a ping, persist it to disk so that it can be sent later via regular telemetry. r=chutten, r=ted, a=gchang Currently we hand over a crash ping to the pingsender via a pipe; if the pingsender fails to send the ping we rely on the CrashManager assembling and sending one instead. Since the crashmanager is not aware of whether the ping was sent or not this causes duplication on the server side. To solve this problem we save the ping to disk instead, read it from the pingsender and delete the file only if the ping was sent. In this scenario the CrashManager will know that a ping was already sent and will not send a new one. This patch removes all the code used to deal with pipes between the telemetry, crashreporter and pingsender code and also tries to cut down the amount of platform-specific code we have in this machinery. MozReview-Commit-ID: ASm2jnDagCK

Bug 1351456 - Don't assume that window.load will be called after the pref has been flipped. r=pehrsons, a=test-only MozReview-Commit-ID: 2r6d6a5vIa

Bug 1350887 - Warn when non-loopback connections are allowed; r=whimboo, a=test-only MozReview-Commit-ID: LG8f5q5QbD6

Bug 1350887 - Propagate errors thrown when starting server.TCPListener; r=whimboo, a=test-only Errors thrown are printed to console and there is no point in having a custom catch to print it. This also makes it possible to start Marionette programmatically without worrying about disappearing errors. MozReview-Commit-ID: GGhyCyYqJg

Bug 1350887 - Bind sacrificial goat to ephemeral port; r=whimboo, a=test-only Port 666 is in the protected port range and can not normally be bound to unless the process is running with sudo permissions. We can instead bind to port 0, which will give us a system-defined port in the epemeral range. MozReview-Commit-ID: Ld6BDMhtbck

Bug 1350887 - Fall back to deprecated pref if it exists; r=whimboo, a=test-only This is a follow-up to address a fallout caused by bug 1344748 whereby deprecated preferences relevant to Marionette are no longer being picked up. This is preventing trace logs from being emitted in CI. The old logic related to falling back to a deprecated preference is faulty in that it the preferred, new preference always exists through the power of testing/marionette/prefs.js. This patch introduces a new helper method getPref that first looks at whether the preferred pref is set, and only falls back to the deprecated if it isn't set and the deprecation preference exists. MozReview-Commit-ID: 8DeawLAELyK

Bug 1350887 - Include Marionette prefs amongst defaults; r=ted, r=whimboo, a=test-only The Marionette component ships in Firefox, but is not enabled by default. We want to facilitate activating Marionette at runtime by flipping the marionette.enabled preference, and showing the Marionette related preferences in about:config helps discoverability. It is also useful to rely on the preferences' default values so that they do not have to be hardcoded in the component. When Marionette is enabled by setting marionette.enabled to true, a set of recommended automation preferences found in testing/marionette/server.js are set if the user has not overriden/user-defined one of them and marionette.prefs.recommended is true (default). When Marionette is stopped, the altered preferences are reset. MozReview-Commit-ID: 3HLnEI0TEBB

Bug 1350887 - Skip recommended Marionette prefs for reftests; r=whimboo, a=test-only The recommended Marionette preferences for use in automation are not so useful when testing the browser itself. Many of the tests for Firefox have highly specific requirements, and setting the recommended Marionette preferences could for these mean a deterioration of test coverage. The majority of test harnesses utilising Marionette use it to install unsigned add-ons at runtime, and these preferences are not relevant in this context. To avoid an unfortunate and unintended regression in test coverage, we skip using the recommended Marionette preferences when we test the browser itself. MozReview-Commit-ID: 4DeHZfQEX7d

Bug 1350887 - Add general overview documentation to Marionette; r=whimboo, a=test-only MozReview-Commit-ID: HwMOQVxwIDN

Bug 1350887 - Ensure Marionette prefs file has sensible name. r=maja_zf, a=test-only Files appended to JS_PREFERENCE_FILES are moved into the objdir/dist/bin/defaults/pref directory, shared with default global preferences from other parts of Gecko. To ensure Marionette's preference file ends up in this directory with a sensible name, we put it in testing/marionette/prefs/marionette.js so that it ends up in the objdir as dist/bin/defaults/pref/marionette.js. MozReview-Commit-ID: 9YJ7vysDjSJ