mozilla :: #developers

17 May 2017
00:08gandalfaswan: ping
00:09aswangandalf: hi
00:09gandalfaswan: should the new langpack package be a .zip file?
00:10pulsebotCheck-in: https://hg.mozilla.org/mozilla-central/pushloghtml?startID=31830&endID=31831 - 30 changesets - Merge autoland to m-c. a=merge
00:10aswangandalf: yes, xpis are just zips
00:10gandalfyeah, I know :)
00:10pulsebotCheck-in: https://hg.mozilla.org/mozilla-central/pushloghtml?startID=31831&endID=31832 - 8 changesets - Merge inbound to m-c. a=merge
00:10pulsebotCheck-in: https://hg.mozilla.org/integration/mozilla-inbound/pushloghtml?startID=90352&endID=90353 - 35 changesets - Merge inbound to m-c. a=merge
00:10gandalfbut wasn't sure if in the new model we use .xpi's or .zip's as a name of the extension :)
00:10gandalfthanks!
00:11pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/pushloghtml?startID=42761&endID=42762 - 13 changesets - Merge inbound to m-c. a=merge
00:11aswanah, we don't care what the extension is. i think the file picker in about:addons even shows .zip by default now
00:11gandalfaswan: ok, I'm playing with the build system. Will try to produce something and upload it to the bug
00:11aswanmost installs happen via https where we just care about the content-type
00:12aswanexcellent. i see there's a backlog of comments in the bug for me to read, will get to it soon...
00:12gandalfwill you be able to help me get the code to pick it up in gecko and register into L10nRegistry and ChromeRegistry then?
00:12aswansure
00:12gandalfok
00:52pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/fcb2c384b08b - Tom Tromey - Bug 1364839 - devtools-source-map v0.5.0: Update bundle from GitHub; r=jdescottes
00:52pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/69d16243839b - Nathan Froyd - servo: Merge #16880 - provide more information when bindgen fails (from froydnj:bindgen-informative-failure); r=emilio
00:52pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/638cd9bc82f4 - Martin Robinson - servo: Merge #16865 - Properly handle scroll offsets in hit testing (from mrobinson:scroll-offsets-fix); r=emilio
00:52pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/62ff1262c090 - J. Ryan Stinnett - servo: Merge #16872 - Create a MatchingContext to group related matching args (from jryans:matching-context); r=emilio
00:52pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/81a6855ed093 - Boris Zbarsky - servo: Merge #16875 - Don't require style sharing cache invalidation for :dir (from bzbarsky:dir-no-revalidation); r=emilio
00:53pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/b44e810d45ff - Boris Zbarsky - servo: Merge #16894 - Fix the two-value form of text-overflow to work in stylo (from bzbarsky:two-value-text-overflow); r=manishearth
00:53pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/811eac54843c - Boris Zbarsky - servo: Merge #16895 - Add :-moz-is-html support for stylo (from bzbarsky:moz-is-html); r=emilio
00:53pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/6a25c340d834 - Matt Brubeck - servo: Merge #16896 - Bug 1365100 - stylo: Fix inheritance of multiple background images. r=Manishearth (from mbrubeck:image-inherit); r=Manishearth
00:53pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/e589d6b01b8d - Brian Grinstead - Bug 1355324 - Create a popup to switch between sidebars from the sidebar header;r=Gijs
00:54RyanVMinnnnnnnnnnnnnnnncominggggggggggggg
00:54mbrubeckwoot!
00:54mbrubeckI can finally land my thing and go get dinner
00:55pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/be8ef91f497c - Mike Hommey - Bug 1364613 - Disable replace-malloc on hazard builds. r=sfink
00:55pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/6e0d4ca8c7dd - Matt Brubeck - Bug 1365100 - stylo: Update test expectations. r=Manishearth
00:56pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/31710e5be104 - Dustin J. Mitchell - Bug 1363409: ensure e10s talos have -e10s in their try name r=garndt
00:58pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/23e285c5aa70 - Mark Banner - Bug 1361976 - ESLint hooks should ensure that the ESLint setup is up to date. r=mossop
00:59pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/3f6a1965ff0d - Matt R - Bug 1355523 - Convert defer into new Promise in client/styleeditor; r=tromey
01:01pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/21ae7e6b4b44 - J. Ryan Stinnett - Bug 1350036 - Clarify llvm-config detection error. r=froydnj
01:02pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/2e4d10ec3261 - Kartikaya Gupta - Bug 1364987 - Remove the pref to allow APZ with webrender. r=rhunt
01:04pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/a2fe86ec3027 - Josh Matthews - servo: Merge #16899 - Revert "Replace intrinsics::abort with process::abort" (from jdm:revertabort); r=emilio
01:05pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/a9ea66b0059b - Drew Willcoxon - Bug 1355323 - Create action menu panel with initial "copy URL" and "email link" items. r=mikedeboer
01:06pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/9c914393e8ef - Henrik Skupin - Bug 1361837 - Update restart unit tests to not activate sessionrestore. r=ato
01:07pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/cd4176474343 - Milan Sreckovic - Bug 1364146: Only reset the prefs dirty flag when we save to the default prefs file, not on a save to another file. r=bsmedberg
01:08pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/3cbe9b541b39 - Bryce Van Dyk - Bug 1364776 - Relax the restrictions on when MediaRecorder.requestData can be called. r=pehrsons
01:10pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/102ce41b51ca - Francois Marier - Bug 1364959 - Do remote application reputation lookups when downloading HTML files. r=dimi
01:11pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/799a1e86a450 - Blake Kaplan - Bug 1345990 - Switch to an options object for BTU.openNewForegroundTab. r=mconley
01:11pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/bc1e581949f7 - Blake Kaplan - Bug 1345990 - Allow creation of new tabs in their own processes. r=mconley
01:13froydnjare downloads from tooltool known to be busted? I've had problems with them on try all day
01:14glandiumfroydnj: that's why the tree has been closed for a long time
01:14pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/c9440363f82f - Felipe Gomes - Bug 1363083 - Remove TabMixPlus and Mega from e10s blocklist. r=mrbkap
01:14froydnjglandium: ah, saw autoland commits, assumed problem had been solved...
01:15glandiumfroydnj: RyanVM reopened it, not sure why
01:15froydnjglandium: a=NPOTB
01:15glandiuminbound and central are still closed
01:15pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/fcc8c00308a1 - Tom Tromey - Bug 1349354 - remove now-unused source-map-service; r=jryans
01:15RyanVMfroydnj: relops thinks they've got it fixed, yes
01:16RyanVMwe reopened autoland to see how things go as new instances spin up
01:16froydnjhm, maybe my try push was too recent to catch the fixes
01:16froydnjer, old
01:17pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/89b8c539d894 - Mike Hommey - Bug 1365182 - When both clang and gcc are installed via tooltool, pick libstdc++ from gcc. r=froydnj
01:19glandiumwait, what? autoland is not FIFO?
01:19glandiumgps: ^
01:19pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/47661994bdb1 - Mike Hommey - Bug 780562 - Take locale chrome manifest flags into account when repacking l10n. r=gps
01:19pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/c94e87a18096 - Mike Hommey - Bug 780562 - Stop relying on the "platform" chrome flag for global-platform. r=bsmedberg
01:19glandium(I most certainly pushed 1365182 before 1364613)
01:19pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/82edbfe72c47 - Mike Hommey - Bug 780562 - Remove support for the chrome "platform" flag. r=bsmedberg
01:20RyanVMfroydnj: there *might* have been a YOLO thrown in there for good measure too when deciding to reopen
01:21pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/7c0bd165daf4 - Tom Tromey - Bug 1365371 - change source map test not to require old debugger; r=jryans
01:22froydnjRyanVM: \o/
01:22pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/15a0bebc29aa - Johann Hofmann - Bug 1363732 - Remove explicit back button width in compact theme. r=dao
01:24pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/40aa1f949308 - Jean-Yves Avenard - Bug 1363618: Only parse blob starting with ADTS marker. r=kamidphish
01:26pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/61d6ee9aa5bb - Chun-Min Chang - Bug 1331763 - part1: Support EME content for AudioContext::CreateMediaElementSource; r=cpearce
01:26pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/8f2387be7e2d - Chun-Min Chang - Bug 1331763 - part2: Test cases; r=cpearce
01:27pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/4f511c0a0d70 - Mathieu Leplatre - Bug 1357116 - Load the blocklist updater module lazily r=florian,mgoodwin
01:28* glandium wonders how many jobs are running and pending for that autoland backlog
01:29pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/53244a64eed9 - Sean Lee - Bug 1365143 - Add schema version number for each Form Fill record in ProfileStorage.; r=lchang
01:30pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/79c28571253b - Gijs Kruitbosch - Bug 1364056 - don't try to use `doc` reference when it might be dead, r=baku
01:33pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/cc9c5c0a15d1 - Andreas Tolfsen - Bug 1355890 - Namespace Marionette component r=automatedtester
01:33pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/3d995af23408 - Andreas Tolfsen - Bug 1355890 - Add Marionette XPCOM interface r=maja_zf
01:33pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/04092bfa74d5 - Andreas Tolfsen - Bug 1355890 - Move robot favicon to separate file r=Gijs
01:33pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/89c597001a47 - Andreas Tolfsen - Bug 1355890 - Add remote-active system notification r=automatedtester,Gijs
01:33pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/afc3591a1646 - Andreas Tolfsen - Bug 1355890 - Add visual cue to urlbar when under remote control r=Gijs
01:33pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/05410ef975e2 - Andreas Tolfsen - Bug 1355890 - Infer identity box background-color by computed style r=maja_zf
01:34pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/2ca54e1cbf46 - Andrew McCreight - Bug 1365086, part 1 - Make nsMessageManagerScriptExecutor::GetGlobal return a JSObject pointer. r=smaug
01:34pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/61d7468cd1c1 - Andrew McCreight - Bug 1365086, part 2 - Add unlink method for nsMessageManagerScriptExecutor. r=smaug
01:34pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/d4e14b275a0f - Andrew McCreight - Bug 1365086, part 3 - Make nsMessageManagerScriptExecutor::mGlobal into a raw pointer. r=smaug
01:36pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/f3b6085985f3 - Hiroyuki Ikezoe - Bug 1363880 - Don't skip reftests that don't fail anymore on stylo. r=birtles
01:36pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/db0aa9f5cf22 - Hiroyuki Ikezoe - Bug 1363880 - Enable crashtest for SVG that no longer cause panic. r=birtles
01:36pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/437489a51017 - Hiroyuki Ikezoe - Bug 1363880 - Annotate fails-if instead of skip-if for reftests that no longer cause panic but still fail. r=birtles
01:38pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/000460ac6af2 - Tom Ritter - Bug 1365036 Lowercase Objbase.h so MinGW can find the file r=baku
01:38froydnjwow, treeherder is a lot slower to show busted jobs now
01:39froydnjI can click on a gray B and get the failure logs
01:39pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/f14fb118695e - Jan Beich - Bug 1365269 - Unbreak non-GeckoProfiler platforms after bug 1362894. r=njn
01:39glandiumfroydnj: been true for a while
01:41pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/83e7701f282e - Mike Hommey - Bug 1364358 - Remove mozjemalloc arena balancing. r=erahm
01:41pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/0be410e68c53 - Mike Hommey - Bug 1364358 - Remove mozjemalloc check for multithreading. r=erahm
01:41pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/72a43cb4e932 - Mike Hommey - Bug 1364358 - Remove mozjemalloc support for automatic multiple arenas. r=erahm
01:41pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/bda4f18e7710 - Mike Hommey - Bug 1364358 - Keep track of mozjemalloc thread-local arenas. r=erahm
01:42pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/4f0a4a4be0fc - Nevin Chen - Bug 1334110 - Add campaignId in core-ping document. r=bsmedberg,frank,sebastian
01:43pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/3885392de7d9 - Morris Tseng - Bug 1365185 - Clear extents cache after pref "gfx.font_rendering.opentype_svg.enabled" is changed. r=jfkthame
01:50pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/c2ac9d5ebe2f - Mike Hommey - Bug 1365191 - Remove #if 0 sections in mozjemalloc. r=njn
01:50pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/03aeae77fd57 - Mike Hommey - Bug 1365191 - Remove !MOZ_MEMORY sections in mozjemalloc. r=njn
01:50pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/a33773e54f5d - Mike Hommey - Bug 1365191 - Remove MALLOC_VALIDATE in mozjemalloc. r=njn
01:50pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/0f4289fed78b - Mike Hommey - Bug 1365191 - Remove MALLOC_UTRACE from mozjemalloc. r=njn
01:50pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/e246baa0d780 - Mike Hommey - Bug 1365191 - Remove MALLOC_STATS from mozjemalloc. r=njn
01:50pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/09b32bedead3 - Mike Hommey - Bug 1365191 - Remove MALLOC_FILL from mozjemalloc. r=njn
01:50pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/a0c7189d60e3 - Mike Hommey - Bug 1365191 - Remove dead code hidden behind the never set NEEDS_PTHREAD_MMAP_UNALIGNED_TSD. r=njn
01:50pulsebotCheck-in: https://hg.mozilla.org/integration/mozilla-inbound/rev/ccd1be0275e7 - Matt Howell - Bug 1355818 - Wait for staging to finish in the update wizard if downloading is done before the download page appears. r=rstrong
01:51pulsebotCheck-in: https://hg.mozilla.org/integration/mozilla-inbound/rev/714b7caf7e00 - Andr Bargull - Bug 1360220 - Replace emitRequireObjectCoercible with JSOP_CHECKOBJCOERCIBLE. r=shu
01:51pulsebotCheck-in: https://hg.mozilla.org/integration/mozilla-inbound/rev/61ea278ec204 - Andr Bargull - Bug 1361143 - Enforce lookahead restriction for "async function" in expression statements. r=shu
01:52pulsebotCheck-in: https://hg.mozilla.org/integration/mozilla-inbound/rev/8c48de4eaf82 - Thinker K.F. Li - Bug 1364926 - Fix operand type of localno annotated in the comments. r=jandem
01:52pulsebotCheck-in: https://hg.mozilla.org/integration/mozilla-inbound/rev/8aae91ffd074 - Julian Descottes - Bug 1365314 - Remove link between code mirror and document when destroying editor. r=jlast
01:57pulsebotCheck-in: https://hg.mozilla.org/integration/mozilla-inbound/rev/f35b966ff024 - Felipe Gomes - Bug 1364505 - Add a rule to favor plugin fallback content when no srcURI has been specified. r=qDot
01:57pulsebotCheck-in: https://hg.mozilla.org/integration/mozilla-inbound/rev/e30e7ee30b7c - Felipe Gomes - Bug 1364505 - Write tests for plugin favor-fallback mode. r=qDot
01:57pulsebotCheck-in: https://hg.mozilla.org/integration/mozilla-inbound/rev/ddc27dd93ec0 - Felipe Gomes - Bug 1317856 - Ensure that flashblock tests are not affected by nosrc fallback rule. r=bytesized
01:57pulsebotCheck-in: https://hg.mozilla.org/integration/mozilla-inbound/rev/12a7b0378709 - Felipe Gomes - Bug 1317856 - Make the Flash plugin Click-to-Activate by default on Nightly. r=bsmedberg
01:59pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/484c5137e19f - Mike Hommey - Bug 1365194 - Call moz_abort directly instead of using a macro to override abort. r=njn
01:59pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/d8de424663b7 - Mike Hommey - Bug 1365194 - Remove MOZ_JEMALLOC_API from _malloc_options and _malloc_message. r=njn
01:59pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/b56224bf370d - Mike Hommey - Bug 1365194 - Make `extern "C"` part of MOZ_MEMORY_API and MOZ_JEMALLOC_API. r=njn
02:00pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/4b9c2fb40653 - Mike Hommey - Bug 1365194 - Remove parts of the hacks for memalign in mozjemalloc. r=njn
02:00pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/9711f5bbda3b - Mike Hommey - Bug 1365194 - Compile mozjemalloc as C++. r=njn
02:01pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/680ad42a723c - Xidorn Quan - Bug 1341102 - Update reftest expectations.
02:02xidornRyanVM: ^ pushed
02:04RyanVMthx
02:04froydnjglandium: looking forward to all the jemalloc simplifications
02:07pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/3876e96601b1 - DimiL - Bug 1365027 - Add $httpMethod=POST to the query string of Safe Browsing pver4 requests. r=francois
02:28pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/45e204034f66 - Simon Sapin - servo: Merge #16890 - Fix serialization of namespace and universal selectors (from servo:qname); r=emilio
02:29RyanVMmhowell: https://treeherder.mozilla.org/logviewer.html#?job_id=99575184&repo=mozilla-inbound
02:32mhowellRyanVM: oops
02:33mhowellRyanVM: I can have a followup up in a few seconds
02:33RyanVMthx
02:35pulsebotCheck-in: https://hg.mozilla.org/integration/mozilla-inbound/rev/086af5d404e1 - Matt Howell - Bug 1355818 - followup to fix eslint error. r=me
02:56tobytailordbaron: ping
03:03dbarontobytailor: pong
03:03tobytailordbaron: re 1299848
03:03tobytailordbaron: you saw my comment?
03:04dbarontobytailor: yes, and it's not convincing
03:04pulsebotCheck-in: https://hg.mozilla.org/integration/mozilla-inbound/rev/fb689ff5ec19 - L. David Baron - Bug 944200 followup - Consistently use nscoord_MIN for uninitialized mFloatEdgeIStart/End. r=mats
03:05tobytailordbaron: I try again: wanted to have it a operate method cause its not really a reftest per defenition
03:05tobytailordbaron: for == and != retests, the test and reference html have to look exactly the same, but with different code behind
03:05tobytailordbaron: thats totally different for print tests
03:06tobytailordbaron: where the reference can look totally different based on printing settings
03:06dbarontobytailor: what do the tests mean then?
03:06tobytailordbaron: we basically only use the reference to describe elements that have to appear in the printed pdf
03:06tobytailordbaron: I choose it to be html for now, but could also be a json
03:08tobytailordbaron: did you see the addition to the readme?
03:08tobytailordbaron: how print retests work is thy take the test html, silently print it to a pdf, then parse the reference html (or maybe json in the future) and compare it the following way:
03:09tobytailorevery text node of the reference html must exist as real text in the output pdf
03:09tobytailorthe number of div with class page must be equal to the total number of paged in the pdf
03:10tobytailorthose 2 assertions alone, together with the option to set print-range and print-selection settings helps us to test quite a range of stuff already we never had test for before
03:10dbarontobytailor: ok, please revise the patch to make the README clearer, as I noted in the bug
03:11dbarontobytailor: I'd also like to understand *why* you chose this particular format, and why you chose to put it inside the reftest harness.
03:11tobytailordbaron: that part I agreed on :)
03:12tobytailordbaron: also agree on that we should rename the existing reftest-print thingy
03:12tobytailorprobably to reftest-paging
03:12tobytailor?
03:16dbarontobytailor: paged or paginated, not paging
03:16dbaronor just page
03:16dbaronno, probably not page
03:17tobytailordbaron: my approach also makes it possible to test our paged media css implementation
03:17tobytailordbaron: do we have tests for that at all?
03:18dbarontobytailor: not sure... but we also don't have much of @page implemented
03:19tobytailordbaron: but we implemented some of the "side-effects" of it, like not rendering headers/footers if @pages/margin is smaller than it would need to render them
03:20tobytailordbaron: or that one css feature that lets you define if background colors/images should be printed. don't remember the exact name right now.
03:23tobytailordbaron: I didn't think much about "where to put it" in the beginning. Started with trying to extend reftest-print, guess thats why I started doing it in the retest harness.
03:27pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/10841f0c5aff - Iris Hsiao - Backed out changeset 102ce41b51ca (bug 1364959) for xpcshell failures in test_ext_downloads_misc.js. a=backout
03:32joshHow can I see errors in chrome JS code for debugging purposes in an opt Firefox build? Like, if I modify some chrome JS code and mess up I want to see the error
03:32mjf|afkpulsebot: status autoland
03:32pulsebotmjf|afk: autoland is CLOSED
03:48dbarontobytailor: seems like all of those could also be tested with visual equality of the printed output like the normal reftest harness
03:49tobytailordbaron: not for things that test for text not being rasterized
03:50tobytailordbaron: but I could at least make it generating pdf for both html files and comparing them in a special way
03:50dbarontobytailor: why not -- at least assuming the visual equality comparison is accurate enough
03:50tobytailordbaron: that would come closer to a reftest
03:52pulsebotCheck-in: https://hg.mozilla.org/integration/mozilla-inbound/rev/0aae6f865e8d - David Major - Bug 1349801: Make SleepMicro a method of SamplerThread. r=mstange
03:52pulsebotCheck-in: https://hg.mozilla.org/integration/mozilla-inbound/rev/cde48b1034b6 - David Major - Bug 1349801: Make aMicroseconds unsigned to avoid a bunch of checks. r=mstange
03:52pulsebotCheck-in: https://hg.mozilla.org/integration/mozilla-inbound/rev/fdfb138fa1a1 - David Major - Bug 1349801: Allow sub-millisecond profiler intervals with a busy wait on Windows. r=mstange
03:52pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/69f2e67310e5 - Martin Robinson - servo: Merge #16795 - Don't optimize display list for projective transforms (from mrobinson:out-of-viewport-transform); r=glennw
03:53jetWhat's a good useragent-string WebExtension that works with e10s-multi?
03:55pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/1d0a2e2c167d - Ya-Chieh Wu - bug 1360494 - part 1 remove duplicate image files and reference them from pdfjs. r=evelyn
03:55pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/19c4f1bf59e3 - Ya-Chieh Wu - bug 1360494 - part 2 add mochitest to make sure the image files exist. r=evelyn
03:55ihsiao|sheriffdutyheycam: hi, could you help to check the style failures like https://treeherder.mozilla.org/logviewer.html#?job_id=99612992&repo=autoland&lineNumber=1848
03:57heycamihsiao|sheriffduty: ok
03:59pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/a9ac97cc2efa - Alastor Wu - Bug 1347791 - part1 : rename the attribute for the tab which has blocked the autoplay audio and would show the play-tab icon. r=mikedeboer
03:59pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/be04f96bf78b - Alastor Wu - Bug 1347791 - part2 : the tab would always be resumed when it goes to foreground first time. r=baku
03:59heycamihsiao|sheriffduty: looks like the expectation updates are in https://github.com/servo/servo/pull/16890#issuecomment-301871067 ; I'll apply them
03:59pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/37d295d7eec9 - Alastor Wu - Bug 1347791 - part3 : move block/resume logic from back-end to front-end. r=baku,mikedeboer
03:59pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/888dc371081a - Alastor Wu - Bug 1347791 - part4 : keep tab's block state consistent after session restore. r=mikedeboer
04:00ihsiao|sheriffdutyheycam: ok, thanks
04:00pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/6175b270de8d - Matthew Wein - Bug 1331742 - Part 1 - Create and register browser_action.json r=mixedpuppy
04:00pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/8b5ac28fb391 - Matthew Wein - Bug 1331742 - Part 2 - Create a module for managing browser actions similar to PageActions.jsm r=sebastian
04:00pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/507eb09bc63f - Matthew Wein - Bug 1331742 - Part 3 - Create and register ext-browserAction.js r=mixedpuppy
04:00pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/5b3f1f520ee3 - Matthew Wein - Bug 1331742 - Part 4 - Add support for adding and removing browser actions from the main menu r=sebastian,walkingice
04:01pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/ba79216d1f10 - Matthew Wein - Bug 1331742 - Part 5 - Add unit tests for browserAction.onClicked r=mixedpuppy
04:02pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/692a837347e3 - Cameron McCormack - Bug 1341102: Update test expectations for servo/servo#16890. r=me
04:02heycamihsiao|sheriffduty: ^
04:02ihsiao|sheriffdutythank you
04:54pulsebotCheck-in: https://hg.mozilla.org/integration/mozilla-inbound/rev/810a380512e9 - Boris Zbarsky - Bug 1189822 part 1. Switch DOMProxyHandler::EnsureExpandoObject to using normal wrapper preservation instead of RegisterDOMExpandoObject. r=peterv
04:54pulsebotCheck-in: https://hg.mozilla.org/integration/mozilla-inbound/rev/d820067e4d5f - Boris Zbarsky - Bug 1189822 part 2. Remove the DOM proxy expando object hashtable in xpconnect. r=peterv
04:54pulsebotCheck-in: https://hg.mozilla.org/integration/mozilla-inbound/rev/5ccd27cf7166 - Boris Zbarsky - Bug 1189822 part 3. Stop using EnsureExpandoObject in codegen code when we just want to preserver the wrapper for a DOM proxy. r=peterv
04:54pulsebotCheck-in: https://hg.mozilla.org/integration/mozilla-inbound/rev/793ab87511f5 - Boris Zbarsky - Bug 1189822 part 4. Fix finalization for DOM overridebuildins proxies to not clear the expando if it's for a different reflector. r=peterv, sfink
04:54pulsebotCheck-in: https://hg.mozilla.org/integration/mozilla-inbound/rev/30f6ea5ad9e6 - Boris Zbarsky - Bug 1189822 part 5. Remove the now-unnecessary DOMProxyHandler::ClearExternalRefsForWrapperRelease function. r=peterv
04:54pulsebotCheck-in: https://hg.mozilla.org/integration/mozilla-inbound/rev/159f82e6813c - Boris Zbarsky - Bug 1365362. Make Gecko_AtomAttrValue work correctly when the attr's value is not an atom. r=emilio
05:03pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/4553b67c794f - Mike Hommey - Bug 1365473 - Rename jemalloc_types.h to mozjemalloc_types.h. r=njn
05:03pulsebotCheck-in: https://hg.mozilla.org/integration/mozilla-inbound/pushloghtml?startID=90361&endID=90362 - 11 changesets (bug 1322316)
05:13pulsebotCheck-in: https://hg.mozilla.org/integration/mozilla-inbound/rev/2268a93ddfc8 - Aaron Klotz - Bug 1354077: Annotate crash reports with COM interface configuration information when marshaling fails; r=jimm
05:35whimbooalwu: good morning.. looks like your patch broke marionette tests
05:35whimboohttps://treeherder.mozilla.org/#/jobs?repo=autoland&filter-searchStr=mn&bugfiler&selectedJob=99635431
05:35whimbooits all perma failing
05:35whimboosame for firefox ui tets
05:37whimboono sheriffs around for a backout?
05:42heycamwhimboo: ihsiao|lunch should be back sometime soon
05:50alwuwhimboo : oops, sorry, I'll check what's happened
06:02pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/4887aee47d0a - JW Wang - Bug 1365484 - Remove the call to AbstractThread::GetCurrent() in Benchmark.cpp. r=gerald
06:08pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/19d0683d197d - Alan Jeffrey - servo: Merge #16876 - Renamed constellation::Frame to constellation::BrowsingContext (from asajeffrey:constellation-rename-frames); r=cbrewster
06:12pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/db2fabb21b82 - Carsten "Tomcat" Book - Backed out changeset 888dc371081a (bug 1347791)
06:12pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/76ed52c3ca1a - Carsten "Tomcat" Book - Backed out changeset 37d295d7eec9 (bug 1347791)
06:12pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/219bfc72300d - Carsten "Tomcat" Book - Backed out changeset be04f96bf78b (bug 1347791)
06:12pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/95990be385ca - Carsten "Tomcat" Book - Backed out changeset a9ac97cc2efa (bug 1347791) for test failures in browser_ext_sessions_restore.js | restored tab has the expected url - Got about:blank, expected about:config
06:12Tomcat|sheriffdutywhimboo: handled and backed out
06:13Tomcat|sheriffdutyheycam: whimboo sorry for the delay
06:26pulsebotCheck-in: https://hg.mozilla.org/integration/mozilla-inbound/rev/412168e07dd1 - Cervantes Yu - Bug 1365203 - Annotate name of base::Thread in the crash report. r=gsvelto
06:26pulsebotCheck-in: https://hg.mozilla.org/integration/mozilla-inbound/rev/2e07ae5b0b69 - Cervantes Yu - Bug 1365203 - Test cases for annotating name of base::Thread in the crash reporter. r=gsvelto
06:36pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/f1872f3e6e34 - Julien Cristau - Bug 1358420 - use staged rollout for fennec beta builds r=aki,jlorenzo
06:43pulsebotCheck-in: https://hg.mozilla.org/releases/mozilla-beta/rev/7067d6a7dc74 - Lee Salzman - Bug 1267909 - fix line heights to properly scale to requested size for color-bitmapped SFNTs. r=jfkthame a=gchang
06:44gpsglandium: autoland should be FIFO https://hg.mozilla.org/hgcustom/version-control-tools/file/tip/autoland/autoland/autoland.py#l39
06:44gpsi suppose if a request fails due to tree closed, maybe it moves to back of queue?
06:44gpsnot sure how that works :/
06:45glandiumgps: turns out there were two autoland requests, one from me and one from froydnj ; glob filed a bug
06:45* glob is working on that bug right now
06:45globgps: and yes, autoland _should_ be fifo but it isn't
06:46glob(there's a bug for that too, somewhere)
06:46glandiumglob: it's not? O_o
06:46gpsi've never had an issue with it not being FIFO
06:46gpsand i like to think i've abused it :)
06:46glandiumso, I was right to wait when I needed to land things that depended on each other
06:46globglandium: if a job can't be landed immediately (eg. tree closure) it's deferred until later, and the next job is processed
06:47globit means things can be landed out of order when the tree is reopened (which actually happened in this case)
06:47glandiumhuh
06:47KWierso|afkif something gets queued up to autoland during a closer that has "CLOSED TREE" in the commit message, would it get through immediately?
06:47KWierso|afkjumping past anything else queued up without "CLOSED TREE"?
06:47gpsi thought we checked tree status before attempting to push
06:47globKWierso|afk: no, if the tree is closed autoland will not attempt to land it
06:48globKWierso|afk: it's unconditional and cannot be overridden via the commit description
07:04whimbooTomcat|sheriffduty: thanks!
07:14pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/99ed53cfebd6 - CuriousLearner - Bug 1298803 - Add test to quit Firefox via a shortcut; r=whimboo
07:19pulsebotCheck-in: https://hg.mozilla.org/integration/mozilla-inbound/rev/8cb64c677091 - Shawn Huang - Bug 1348390 - Sort and lowercase response's header lists for getAllResponseHeaders(), r=baku
07:28pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/7d07aa43577e - Vangelis Katsikaros - Bug 1356871 - Add timing related columns. r=Honza
07:33freesamaelkarlt: Not sure if you
07:34freesamaelkarlt: Not sure if you're still around, but if do, is there any best practice for handling new GDK api?
07:34pulsebotCheck-in: https://hg.mozilla.org/integration/mozilla-inbound/rev/f7b1a4643318 - Ruturaj K. Vartak - Bug 1354504 - Add test cases for SearchBox and AutocompletePopup react components. r=ntim, jdescottes.
07:35freesamaelkarlt: gdk_screen_get_n_monitors is deprecated in 3.22, and it seems removed in master branch, so I should apply gdk_display_get_n_monitors for 3.22+. Same for some other monitor operating functions.
07:35karltfreesamael: see widget/gtk/mozgtk/mozgtk.c, but i'm not sure i understand the question
07:36karltfreesamael: use the deprecated one because we need to work against earlier versions; master is GTK 4, which is a different ABI
07:36ntimHow do I back out a patch ?
07:36freesamaelkarlt: I see. So I don't need to implement 2 versions?
07:37karltfreesamael: correct; don't do that
07:37freesamaelkarlt: Got it, thx!
07:38karltyw
07:40pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/720c38d9052e - Marco Bonardo - Bug 1344924 - Contextual onboarding for search suggestions in the awesomebar. r=adw
07:41pulsebotCheck-in: https://hg.mozilla.org/integration/mozilla-inbound/rev/2c4621d09ea9 - Tim Nguyen - Backed out changeset f7b1a4643318
07:43pulsebotCheck-in: https://hg.mozilla.org/integration/mozilla-inbound/rev/dc9185b73e07 - David Anderson - Don't use out-of-process video decoding with software compositors. (bug 1356448, r=mattwoodrow)
07:43pulsebotCheck-in: https://hg.mozilla.org/integration/mozilla-inbound/rev/87e95e385f23 - David Anderson - Bug 1356448: Allow software backed compositor process in nightly. r=dvander
07:45pulsebotCheck-in: https://hg.mozilla.org/integration/mozilla-inbound/rev/baef5b5701f9 - Ruturaj K. Vartak - Bug 1364113 - Test cases for Network Monitor autocomplete. r=jdescottes, ntim
07:47Tomcat|sheriffdutyof course trees are not closed anymore ;)
08:02pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/2dc093cc8d32 - bechen - Bug 1364185 - Call DispatchUpdateCueDisplay() in TextTrackManager::NotifyCueUpdated. r=alwu
08:08pulsebotCheck-in: https://hg.mozilla.org/integration/mozilla-inbound/rev/2a0949cc856f - gabriellesc - Bug 1301611 - Test that mutation observer is notified when keyframes are changed using setKeyframes(). r=boris
08:08pulsebotCheck-in: https://hg.mozilla.org/integration/mozilla-inbound/rev/1a5e3003f692 - Samael Wang - Bug 1363036 - Add MOZ_DIAGNOSTIC_ASSERTs for bug hunting. r=smaug
08:08pulsebotCheck-in: https://hg.mozilla.org/integration/mozilla-inbound/rev/b3f8917bd944 - Petr Sumbera - Bug 1364865 - process_util.h should use _POSIX_PATH_MAX instead of NAME_MAX. r=billm
08:09pulsebotCheck-in: https://hg.mozilla.org/integration/mozilla-inbound/rev/228a1e9bcc4b - Andr Bargull - Bug 1365387 - Optimize FinishBoundFunctionInit by delaying atomizing the function name. r=till
08:24pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/28b19b1b5695 - Sumit - servo: Merge #16883 - Mutation Observer API (from jdm:mutationobserver); r=jdm
08:35* smaug wonders what prints OK to terminal after running Nightly
08:54Tomcat|sheriffdutymak: ping
08:54Tomcat|sheriffdutymak: seems https://treeherder.mozilla.org/logviewer.html#?job_id=99691476&repo=autoland is from your push
08:54makTomcat|sheriffduty: pong (I guess it's for the mochitest browser failure)
08:54Tomcat|sheriffdutyits also in the try run
08:54Tomcat|sheriffdutyyeah
08:55Tomcat|sheriffdutyat least that crash application crashed [@ mozilla::net::nsSocketTransport::InitiateSocket]
08:55makhm, I thought I had addressed exactly that try run failure, looks like I failed
08:55Tomcat|sheriffdutyok that i back this out
08:55makpff, Tomcat|sheriffduty please backout, I'll try to reproduce it
08:55makthanks
08:56Tomcat|sheriffdutyyeah will do
08:56Tomcat|sheriffdutythanks mak
08:56maklooks like some previous test doesn't restore search suggestions prefs yet
08:57pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/9a0a00d865f4 - Carste