16ea72b646fe6a9b089c353231283e939c4bca21: Bug 1345153 - Suppress valgrind warnings for PR_SetEnv in SetupCrashReporterDirectory. r=njn, a=gchang
Alessio Placitelli <alessio.placitelli@gmail.com> - Thu, 30 Mar 2017 08:04:11 +0200 - rev 556816
Push 52623 by bmo:jlorenzo@mozilla.com at Thu, 06 Apr 2017 09:31:27 +0000
Bug 1345153 - Suppress valgrind warnings for PR_SetEnv in SetupCrashReporterDirectory. r=njn, a=gchang MozReview-Commit-ID: 4vV2EHsgTVT
b3b878775ada1356980114ab985c5a5fe2e2d914: 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
Gabriele Svelto <gsvelto@mozilla.com> - Mon, 27 Mar 2017 12:38:39 +0200 - rev 556815
Push 52623 by bmo:jlorenzo@mozilla.com at Thu, 06 Apr 2017 09:31:27 +0000
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
eb0d361dc73c32867d5fd54dc47733994bd48e7a: Bug 1351456 - Don't assume that window.load will be called after the pref has been flipped. r=pehrsons, a=test-only
Paul Adenot <paul@paul.cx> - Mon, 03 Apr 2017 13:20:33 +0200 - rev 556814
Push 52623 by bmo:jlorenzo@mozilla.com at Thu, 06 Apr 2017 09:31:27 +0000
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
4ab26e8674260f8a1e4dc89c1e10efa2ba544c9b: Bug 1350887 - Warn when non-loopback connections are allowed; r=whimboo, a=test-only
Andreas Tolfsen <ato@mozilla.com> - Tue, 28 Mar 2017 17:06:06 +0100 - rev 556813
Push 52623 by bmo:jlorenzo@mozilla.com at Thu, 06 Apr 2017 09:31:27 +0000
Bug 1350887 - Warn when non-loopback connections are allowed; r=whimboo, a=test-only MozReview-Commit-ID: LG8f5q5QbD6
39f487bdf0a3cafd1f19d63f52b6ea3205361523: Bug 1350887 - Propagate errors thrown when starting server.TCPListener; r=whimboo, a=test-only
Andreas Tolfsen <ato@mozilla.com> - Tue, 28 Mar 2017 16:48:20 +0100 - rev 556812
Push 52623 by bmo:jlorenzo@mozilla.com at Thu, 06 Apr 2017 09:31:27 +0000
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
684f78299750cac7dbf62ea619a5417675c8519a: Bug 1350887 - Bind sacrificial goat to ephemeral port; r=whimboo, a=test-only
Andreas Tolfsen <ato@mozilla.com> - Tue, 28 Mar 2017 16:46:54 +0100 - rev 556811
Push 52623 by bmo:jlorenzo@mozilla.com at Thu, 06 Apr 2017 09:31:27 +0000
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
28a50e73dd6ddd6fb5b4bd8f2a5c505aeed88fd6: Bug 1350887 - Fall back to deprecated pref if it exists; r=whimboo, a=test-only
Andreas Tolfsen <ato@mozilla.com> - Mon, 27 Mar 2017 14:28:32 +0100 - rev 556810
Push 52623 by bmo:jlorenzo@mozilla.com at Thu, 06 Apr 2017 09:31:27 +0000
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
7ff4847d897b49ce091540d0d715e5cef3722eb6: Bug 1350887 - Include Marionette prefs amongst defaults; r=ted, r=whimboo, a=test-only
Andreas Tolfsen <ato@mozilla.com> - Wed, 29 Mar 2017 18:25:55 +0100 - rev 556809
Push 52623 by bmo:jlorenzo@mozilla.com at Thu, 06 Apr 2017 09:31:27 +0000
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
475ab9737926abfa504ef058477a4eff46bfcb48: Bug 1350887 - Skip recommended Marionette prefs for reftests; r=whimboo, a=test-only
Andreas Tolfsen <ato@mozilla.com> - Tue, 04 Apr 2017 16:34:03 +0100 - rev 556808
Push 52623 by bmo:jlorenzo@mozilla.com at Thu, 06 Apr 2017 09:31:27 +0000
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
5f2b09b35a72f85d5ea49cf72a1e767b90ebde91: Bug 1350887 - Add general overview documentation to Marionette; r=whimboo, a=test-only
Andreas Tolfsen <ato@mozilla.com> - Mon, 03 Apr 2017 15:00:05 +0100 - rev 556807
Push 52623 by bmo:jlorenzo@mozilla.com at Thu, 06 Apr 2017 09:31:27 +0000
Bug 1350887 - Add general overview documentation to Marionette; r=whimboo, a=test-only MozReview-Commit-ID: HwMOQVxwIDN
7e03d31839f7726f20769a60988462e0d659b382: Bug 1350887 - Ensure Marionette prefs file has sensible name. r=maja_zf, a=test-only
Andreas Tolfsen <ato@mozilla.com> - Thu, 30 Mar 2017 14:52:16 +0100 - rev 556806
Push 52623 by bmo:jlorenzo@mozilla.com at Thu, 06 Apr 2017 09:31:27 +0000
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
198292931177ff33a879c002dd0293ac61210f94: Bug 1350599 - Use guaranteed preloaded test domains instead of real domains in HSTS tests. r=keeler, a=test-only
Cykesiopka <cykesiopka.bmo@gmail.com> - Mon, 27 Mar 2017 22:56:38 +0800 - rev 556805
Push 52623 by bmo:jlorenzo@mozilla.com at Thu, 06 Apr 2017 09:31:27 +0000
Bug 1350599 - Use guaranteed preloaded test domains instead of real domains in HSTS tests. r=keeler, a=test-only Some of our tests currently assume that certain real domains are HSTS preloaded. While most of the time these domains are in fact preloaded, this may change during periods of maintenance or other events. To avoid this, the changes here perform the following renames: bugzilla.mozilla.org -> includesubdomains.preloaded.test login.persona.org -> includesubdomains2.preloaded.test www.torproject.org -> noincludesubdomains.preloaded.test In addition, some tests that refer to mozilla.com (but don't depend on it being preloaded) are made to refer to example.com instead to avoid referring to real domains in tests. MozReview-Commit-ID: 3987moJnKGk
82f186fc316dc3193724132ab9ef9cec2db7a78f: Bug 1291320 - Skip test_focus_after_navigation for "Unable to locate element: :focus". r=maja_zf, a=test-only
Henrik Skupin <mail@hskupin.info> - Mon, 03 Apr 2017 22:10:38 +0200 - rev 556804
Push 52623 by bmo:jlorenzo@mozilla.com at Thu, 06 Apr 2017 09:31:27 +0000
Bug 1291320 - Skip test_focus_after_navigation for "Unable to locate element: :focus". r=maja_zf, a=test-only MozReview-Commit-ID: Kd2lrH9y5US
c5f0fcd9882b6dbf2d2009fedf33fb32da19d721: Bug 1291320 - Make refresh command synchronous. r=ato, r=automatedtester, a=test-only
Henrik Skupin <mail@hskupin.info> - Mon, 27 Mar 2017 16:16:36 +0200 - rev 556803
Push 52623 by bmo:jlorenzo@mozilla.com at Thu, 06 Apr 2017 09:31:27 +0000
Bug 1291320 - Make refresh command synchronous. r=ato, r=automatedtester, a=test-only Update the refresh command to make it synchronous, and let it return once the target page has been loaded. This can be accomplished by using the loadListener object in listener.js. MozReview-Commit-ID: Lc8QoGFeQrY
512980f54f8cec06e824d424f7b1c84bca1546a2: Bug 1291320 - Refactor page load algorithm for listener framescript. r=ato, r=automatedtester, a=test-only
Henrik Skupin <mail@hskupin.info> - Tue, 28 Mar 2017 21:41:38 +0200 - rev 556802
Push 52623 by bmo:jlorenzo@mozilla.com at Thu, 06 Apr 2017 09:31:27 +0000
Bug 1291320 - Refactor page load algorithm for listener framescript. r=ato, r=automatedtester, a=test-only This refactoring allows us to re-use the same load algorithm for each command which could trigger a page load. It also takes remoteness changes into account, and waits until the load has been done. With this change we no longer check for readyState only, but observe the necessary DOM events as fired for page unloads and loads. This will help us to implement the page loading strategy later. By observing the DOM events, I also expect a small increase of performance for any kind of page load, given that we now return immediately and do not have a delay of 100ms at maximum. MozReview-Commit-ID: IVtO6KgJFES
d449920050741d86b6dece2f12d053d01f654173: Bug 1291320 - Disallow slow loading page to be put into the cache. r=ato, r=automatedtester, a=test-only
Henrik Skupin <mail@hskupin.info> - Mon, 03 Apr 2017 22:40:48 +0200 - rev 556801
Push 52623 by bmo:jlorenzo@mozilla.com at Thu, 06 Apr 2017 09:31:27 +0000
Bug 1291320 - Disallow slow loading page to be put into the cache. r=ato, r=automatedtester, a=test-only To delay the page load for our slowly served example page when using the back or forward commands, the page doesn't have to be put into the browser cache. MozReview-Commit-ID: 4xMjR3SakZn
bffb705f4ca2e1c4995d02f3392336d8afd391c4: Bug 1291320 - Refactor navigation unit tests by using non-remote pages only if necessary. r=ato, r=automatedtester, a=test-only
Henrik Skupin <mail@hskupin.info> - Tue, 14 Mar 2017 08:30:32 +0100 - rev 556800
Push 52623 by bmo:jlorenzo@mozilla.com at Thu, 06 Apr 2017 09:31:27 +0000
Bug 1291320 - Refactor navigation unit tests by using non-remote pages only if necessary. r=ato, r=automatedtester, a=test-only Using non-remote pages causes framescripts to be reloaded. We should try to avoid using those pages as much as possible, and test remoteness switches in particular tests only. This is to reduce possible hangs. MozReview-Commit-ID: ICPPkU07KQK
2448220fff6bafa99953032e8425fd63f25403bc: Bug 1347625 - Skip browser_devices_get_user_media_multi_process.js on ASAN due to leaks. a=test-only
Ryan VanderMeulen <ryanvm@gmail.com> - Wed, 05 Apr 2017 15:57:59 -0400 - rev 556799
Push 52623 by bmo:jlorenzo@mozilla.com at Thu, 06 Apr 2017 09:31:27 +0000
Bug 1347625 - Skip browser_devices_get_user_media_multi_process.js on ASAN due to leaks. a=test-only
632c2b8c17297a8d4c70fc33a518808d81fc18c7: Bug 1349581: defer nativeRegistration for android Jni to avoid thread issues r=gcp a=lizzard
Randell Jesup <rjesup@jesup.org> - Mon, 03 Apr 2017 16:58:44 -0400 - rev 556798
Push 52623 by bmo:jlorenzo@mozilla.com at Thu, 06 Apr 2017 09:31:27 +0000
Bug 1349581: defer nativeRegistration for android Jni to avoid thread issues r=gcp a=lizzard MozReview-Commit-ID: Ep0ej5HkGE3
c549338e76391ddf7725a633bad9dc240ba1ae82: No bug, Automated HPKP preload list update from host bld-linux64-spot-389 - a=hpkp-update
ffxbld - Wed, 05 Apr 2017 08:15:00 -0700 - rev 556797
Push 52623 by bmo:jlorenzo@mozilla.com at Thu, 06 Apr 2017 09:31:27 +0000
No bug, Automated HPKP preload list update from host bld-linux64-spot-389 - a=hpkp-update
(0) -300000 -100000 -30000 -10000 -3000 -1000 -300 -100 -50 -20 +20 +50 +100 +300 +1000 +3000 +10000 +30000 +100000 tip