Bug 1236737 - Stop intermittent timeouts in browser_cmd_screenshot.js r=me MozReview-Commit-ID: DLlHhPMWIB5

Bug 1288401 - Inspector sidebar tabs are missing the :active styling. r=ntim MozReview-Commit-ID: 3Mhc25G9AK3

Bug 1253330 - Fix heap-tree-item-field heap-tree-item-name inline style (margin-left) in RTL. r=ntim MozReview-Commit-ID: Irz2x4P7hx4

Bug 1291268 - HiDPI image are missing from video controls on Windows and Linux. r=ralin MozReview-Commit-ID: 7cM7knEqMyV

Bug 1184461 - If a listener is not callable, assume it is an event listener object. r=past In ThreadActor._getAllEventListeners, we construct a list of handler functions for each target in the event target chain of a given event target. If a handler is an event handler object, we obtain the handler function from its handleEvent property. Otherwise, the handler function is assumed to be the handler itself. To detect whether a handler is an event handler object, we would check its class name property. This approach is fragile for several reasons: if the handler is a proxy, accessing its class name may cause the debuggee to run. Moreover, since accessing a proxy's class name requires us to enter the compartment of the proxy's handler, if the latter is a security wrapper, accessing its class name will cause security warnings in debug mode. Since a handler is either an event handler object or a function, it should be safe to assume that a handler is an event handler object if and only if it is not callable. Checking whether a proxy is callable cannot cause the debuggee to run. Moreover, it does not require us to enter the compartment of the proxy's handler, thus avoiding the security warnings we mentioned earlier.

Bug 1282611 - Recheck for captive portal after watcher is initialized. r=MattN MozReview-Commit-ID: DKUibGPvTrx

Bug 445498 - Make background of tab preview transparent on Linux. r=mdeboer

Bug 1291078 - More closely mimic non-e10s order of events when choosing <select> elements in e10s mode. r=enndeakin+6102 MozReview-Commit-ID: 4CkFUy1nLbo

Bug 1292008 - Enable the no-empty rule for eslint. r=markh MozReview-Commit-ID: EfM3YOUjClW

Bug 1241436 - Update the "email me" checkbox in accordance with the email input field in the tab crash reporter. r=Felipe MozReview-Commit-ID: EnIe7NRwMes

Bug 1291185 - Remove the DECODER_WARN_HELPER trick. r=bechen MozReview-Commit-ID: 7U1p0VOvmY3

Bug 1289561 - Implement LSB release annotation in crash reports. r=jrmuizel MozReview-Commit-ID: IEcH6K5D27H

Merge m-c to autoland

Bug 1290616 - Pass down LD to the NSS build system. r=gps as opposed to passing down LINK when cross compiling. Until bug 1290334, LINK was the variable used by the NSS build system for the linker on Windows, but it now uses LD, like on other platforms. But its default value on Windows is "link", which can pick /usr/bin/link on msys2, instead of MSVC's so, pass down LD from the Gecko build system. And do so an all platforms in all cases, because it shouldn't hurt.

Merge m-c to autoland, a=merge

Bug 1291910 - Add ability for mach eslint to accept --fix. r=Felipe MozReview-Commit-ID: 5PB2oI2jPri

Bug 1291855 - Enable the no-else-return rule for eslint. r=Felipe,kmag,mossop MozReview-Commit-ID: DYOmE6xwMJh

Bug 1287075 - Back out bug 1273250, part 4 for regressing the rendering of the Australis submenus. r=mstange MozReview-Commit-ID: 1m8lVoIX3qT

Bug 1291569 - avoid for-let-of scoping bug and temporarily disable failing tests r=aswan Due to an existing scoping bug fixed by bug 1263355, this test hasn't been running the tests we thought it did. This patch gets most of the tests running and disables the few that are not passing while we work on fixing these. MozReview-Commit-ID: 2BqO4Ap2tLU

bug 1289885 - Enable VeriSign Class 3 Public PCA - G4 for EV in PSM r=jcj MozReview-Commit-ID: GDZnZcVCNl6