mozilla :: #developers

17 Mar 2017
00:00pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/4bb7880d29e7 - Wes Kocher - Bug 1036764 - Checkstyle followup a=bustage
00:01sfinkis it possible to run a jetpack test as a mere mortal?
00:03sfinkI have the mozharness command line, but mozharness scares me. I don't want to download a browser and test suite.
00:05KWiersosfink: does https://wiki.mozilla.org/Jetpack/Testing#Running_with_mach help?
00:06KWiersootherwise you could build a time machine to when 57 is released :)
00:07sfinkooh, it looks like it's trying to. Now to cleanup my stuff to see if it can work. I was trying ./mach test, which specifically said it couldn't handle jetpack.
00:07sfinkKWierso: thanks! hopefully that'll do it
00:08KWiersoI haven't really touched jp stuff in like 3 years, so ymmv
00:08KWiersootherwise you might need to set up and use jpm?
00:08KWierso\_()_/
00:36pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/025292e3f23d - Stefano Chiodino - servo: Merge #15970 - #15842 Add gecko glue for caret-color (from StefanoChiodino:master); r=emilio
00:42dbaronf
00:43KWiersoagreed
00:43dbaronwell, it was really just two keystrokes ending up in the wrong window, but... :-)
00:44KWiersobetter than an entire password or passphrase :)
00:45dbaronwasn't even a password, just the last character of an alias for a unix command
00:45* dbaron has tons of random command aliases in .bashrc
00:45KWiersomy .hgrc is similarly littered
01:18pgadigewhile submitting a modified changeset for review, how do I ensure the files are updated or in sync with those is the main repo or original repo?
01:19pgadigethis in respect to mercurial + mozreview
01:20ehsandecoder: ping
01:22RyanVMpgadige: the parent rev I think will help?
01:22RyanVMpgadige: that'll tell you what rev your changes are on top of
01:24pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/c20d0b8a0df7 - Anthony Ramine - servo: Merge #15987 - Fix a couple of HTML parsing issues (from nox:h5e); r=Ms2ger
01:24pgadigeRyanVM: will it not get me the updated version of the files in the changeset if I do `hg pull -u`?
01:25RyanVMyou'd need to rebase your patch on top of them
01:25RyanVMhg pull --rebase
01:25RyanVMwould do it automatically
01:25pgadigeAha.. rebase!
01:25pulsebotCheck-in: https://hg.mozilla.org/integration/mozilla-inbound/rev/77b0e80d2bd3 - Ben Kelly - Bug 1347739 Don't crash if an ExtendableMessageEvent object has a null source. r=asuth
01:28pgadigeRyanVM: it worked. thank you. If at all I need to forcefully revert or sync in the updated version of a file discarding my changes, what command should I run?
01:28RyanVMhg revert path/to/file -r <rev you want to revert it back to>
01:29RyanVMnote that revert doesn&#39;t commit the changes
01:29RyanVMso don&#39;t forget that part :P
01:30pgadigeof course, yes. I shall note it.
01:49pulsebotCheck-in: https://hg.mozilla.org/integration/mozilla-inbound/rev/a6780188f669 - vincentliu - Bug 1314543 - Make EnsureGPUReady() returns bool to make sure GPU process is readay. r=dvander
01:54pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/985cd7df1397 - Makoto Kato - Bug 1347431 - Skip some xpcshell tests that is failure on Android without ENABLE_INTL_API. r=snorp
02:04pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/4a1d493b3f9b - Volodymyr M. Lisivka - servo: Merge #15990 - Layout viewer now uses local copy of jquery. See #15964 (from vlisivka:master); r=emilio
02:07pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/cacda47d7c81 - Robert Helmer - Bug 1347971 - pass oldVersion to install and startup methods for temporary add-on reloads r=aswan
02:08pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/8e72178c3893 - Masayuki Nakano - Bug 1339331 TextEventDispatcher should replace \r in composition string with \n and TextComposition should allow to input \n with composition events r=m_kato
02:19pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/4503ff6c5820 - Astley Chen - Bug 1347047 Part 1 - Remove touch caret timeout test. r=TYLin
02:20pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/7d31847c1d11 - Astley Chen - Bug 1347047 Part 2 - Remove caret timeout control from AccessibleCaretManager. r=TYLin
02:20pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/867cf8e067a7 - Astley Chen - Bug 1347047 Part 3 - Remove pref layout.accessiblecaret.timeout_ms. r=TYLin
02:30pulsebotCheck-in: https://hg.mozilla.org/integration/mozilla-inbound/rev/12ce302b8f9b - Geoff Brown - Bug 1340584 - Change reftest dm_trans default from sut to adb; r=bc
02:50pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/2d4a86ef46e9 - Henry Chang - Bug 1347657 - Use array entry as value instead of reference to avoid being invalidated by realloc. r=francois
03:02pulsebotCheck-in: https://hg.mozilla.org/integration/mozilla-inbound/rev/b1d8962e07f9 - Yura Zenevich - Bug 1335055 - improving accessibility of a tree component (keyboard and semantics). r=Honza
03:07ihsiao|sheriffdutyxidorn: hi
03:08pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/af90f70b05df - Simon Sapin - servo: Merge #15992 - Rewrite PropertyDeclaration::id to help the optimizer (from servo:id-table); r=bholley
03:08xidornihsiao|sheriffduty: yes?
03:08ihsiao|sheriffdutycould you help to check stylo failure like https://treeherder.mozilla.org/logviewer.html#?job_id=84498627&repo=autoland&lineNumber=1821
03:08pulsebotCheck-in: https://hg.mozilla.org/integration/mozilla-inbound/rev/36a2230fbad6 - JerryShih - Bug 1347164 - Serialize colors using &quot;rgba()&quot; as the color-function, if they have a non-opaque alpha channel. r=dholbert
03:08pulsebotCheck-in: https://hg.mozilla.org/integration/mozilla-inbound/rev/8af7daaf5f88 - JerryShih - Bug 1347164 - css-color computed style test. r=dholbert
03:08pulsebotCheck-in: https://hg.mozilla.org/integration/mozilla-inbound/rev/18c70bc07c6d - JerryShih - Bug 1347164 - Rewrite mochitest test_specified_value_serialization.html to look more like test_computed_style.html. r=dholbert
03:08pulsebotCheck-in: https://hg.mozilla.org/integration/mozilla-inbound/rev/bb325d01c421 - JerryShih - Bug 1347164 - css-color specified style test. r=dholbert
03:09xidornihsiao|sheriffduty: ok, will fix
03:09ihsiao|sheriffdutythanks
03:19pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/adc82cb137bf - Bobby Holley - Bug 1347399 - Use a thread-safe URI comparison in CalcStyleDifference. r=heycam
03:22pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/7a7f0db97af0 - Xidorn Quan - Bug 1341102 - Update stylo mochitest expectation for caret-color.
03:23xidornihsiao|sheriffduty: ^
03:24ihsiao|sheriffdutyxidorn: ++, thank you
03:29pulsebotCheck-in: https://hg.mozilla.org/integration/mozilla-inbound/pushloghtml?startID=88145&endID=88146 - 13 changesets (bug 1333990)
03:32pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/cfd64c2b4970 - Masayuki Nakano - Bug 1347809 Set Selection::mCalledByJS to false before moving focus in NotifySelectionListeners() because non-*JS() methods don&#39;t set it to false r=smaug
03:33pulsebotCheck-in: https://hg.mozilla.org/integration/mozilla-inbound/rev/caf3e7f1f230 - Jim Chen - Bug 1344892 - Let native calls dispatch to XPCOM event queue; r=snorp
03:47pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/c679469b5b87 - Simon Sapin - servo: Merge #15993 - Remove rust-encoding from geckolib (from servo:utf84eva); r=Manishearth
04:14pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/4d074b176f37 - Markus Stange - Bug 1329111 - Rename SharedLibraryInfo::mName to mModuleName, and consistently cut off the path on all platforms. r=marco
04:14pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/8a238c780c47 - Markus Stange - Bug 1329111 - Supply SharedLibraryInfo with the absolute module path and debug path on all platforms. r=marco
04:14pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/cdff9dc4a8ba - Markus Stange - Bug 1329111 - Record shared library architecture on Mac. r=glandium,njn
04:14pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/6cb351c611e3 - Markus Stange - Bug 1329111 - Use JSONWriter when generating the GetSharedLibraryInfo() JSON. r=njn
04:14pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/33aa51a338b1 - Markus Stange - Bug 1329111 - Change the nsIProfiler shared library information API. r=njn
04:14pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/fd8fc09c8229 - Markus Stange - Bug 1329111 - Make talos profiling symbolication compatible with the new profile format. r=jmaher
04:14pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/7e4f8900fa8c - Markus Stange - Bug 1329111 - Update devtools users of nsIProfiler.getSharedLibraryInformation. r=gregtatum
04:18pulsebotCheck-in: https://hg.mozilla.org/integration/mozilla-inbound/rev/ac0658d7d958 - Ehsan Akhgari - Bug 1348168 - Disable Mozilla custom ImageBitmap extensions that didn&#39;t go through proper API review; r=bzbarsky
04:18pulsebotCheck-in: https://hg.mozilla.org/releases/mozilla-aurora/rev/dff5f8de5eed - Ehsan Akhgari - Bug 1348168 - Disable Mozilla custom ImageBitmap extensions that didn&#39;t go through proper API review; r=bzbarsky a=dveditz
04:19pulsebotCheck-in: https://hg.mozilla.org/releases/mozilla-beta/rev/20f404abbe68 - Ehsan Akhgari - Bug 1348168 - Disable Mozilla custom ImageBitmap extensions that didn&#39;t go through proper API review; r=bzbarsky a=dveditz
04:27pulsebotCheck-in: https://hg.mozilla.org/releases/mozilla-beta/rev/484f49860df6 - Ryan VanderMeulen - Manual version bump. CLOSED TREE NO BUG a=release
04:29pulsebotCheck-in: https://hg.mozilla.org/releases/mozilla-esr52/rev/0af82b157012 - Ehsan Akhgari - Bug 1348168 - Disable Mozilla custom ImageBitmap extensions that didn&#39;t go through proper API review; r=bzbarsky a=dveditz
04:34pulsebotCheck-in: https://hg.mozilla.org/releases/mozilla-esr52/rev/4af7cd795eee - Ehsan Akhgari - Bug 1348168 - Disable Mozilla custom ImageBitmap extensions that didn&#39;t go through proper API review; r=bzbarsky a=dveditz
04:38pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/9e40bc33e620 - Cameron McCormack - Bug 1346623 - Part 1: Prevent canvas custom content from becoming NAC when reframing the root element. r=bholley
04:38pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/39ee9556d4a0 - Cameron McCormack - Bug 1346623 - Part 2: Add an API to get computed style values through an AnonymousContent object. r=bholley
04:38pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/1f9a6ce59a56 - Cameron McCormack - Bug 1346623 - Part 3: Test. r=bholley
05:08pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/253780566f3b - Cameron McCormack - Bug 1341102 - Revendor rust dependencies. r=me (CLOSED TREE)
05:15glandiummstange: wrt bug 1338530, just to be clear, you&#39;re using mercurial?
05:27pulsebotCheck-in: https://hg.mozilla.org/integration/mozilla-inbound/rev/81d70d7e0844 - kvark - Bug 1348111 - Clarify the error message. r=smaug
05:30pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/eb872c42dc7f - Thomas Nguyen - Bug 1192940 - Support referrer policy in sendBeacon r=Ehsan
05:30pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/b6252f00b471 - bechen - Bug 1344604 - Return null if there is an incomplete tag in nextToken(). r=alwu
05:30pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/c284d23cc142 - bechen - Bug 1344604 - part2: Return empty DocumentFragment if the ConvertCueToDOMTree() return null. r=alwu
05:31pulsebotCheck-in: https://hg.mozilla.org/integration/mozilla-inbound/rev/6d17c6eda45f - Franziskus Kiefer - Bug 1345368 - land NSS 37ccb22f8e51, r=me
05:31pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/64fc7575528e - Simon Sapin - servo: Merge #15996 - Fix &quot;option `Z` is unstable&quot; warning on geckolib with incremental=true (from servo:incremental-unstability); r=Manishearth
05:32pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/6c475d5fd958 - Kaku Kuo - Bug 1347836 part 1 - fix comment; r=jwwang
05:32pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/0b4a997d9fbb - Kaku Kuo - Bug 1347836 part 2 - make HTMLMediaElement::GetCurrentImage() return an already_AddRefed<layers::Image>; r=jwwang
05:34pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/fe02ae88f611 - cku - Bug 1343664 - Part 1. Correct transform matrix. r=mstange
05:34pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/44e9eace508a - cku - Bug 1343664 - Part 2. Reftest. r=mstange
05:36pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/836d16519edf - cku - Bug 1345853 - Part 1. Pass DrawResult from nsSVGPatternFrame::PaintPattern back to nsDisplaySVGGeometry::Paint. r=mstange,tnikkel
05:36pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/a46f3da8aba6 - cku - Bug 1345853 - Part 2. If the transform matrix is singular, return DrawResult::SUCCESS, instead of DrawResult::BAD_ARGS r=tnikkel
05:36pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/fd5e7c8cbb6d - cku - Bug 1345853 - Part 3. Reftest. r=tnikkel
05:41pulsebotCheck-in: https://hg.mozilla.org/integration/mozilla-inbound/rev/67c3d82f8cae - Timothy Nikkel - Bug 1343341. Fix a bug with redecoding images whose animation is finished. r=aosmond
05:43ewongany sheriffs around?
05:43RyanVMewong: kinda sorta, what&#39;s up?
05:43KWierso|afkewong: depends who&#39;s asking :)
05:43RyanVMlol
05:44ewongI&#39;m perturbed by this try push: https://treeherder.mozilla.org/#/jobs?repo=try&revision=d6c71411494d015a8a461ec60f162b857df9a5cd&selectedJob=84243120 is it releng related or my patch?
05:44ewongI&#39;m talking about the l10n tries..
05:44ewongwhy is one purple and the other one red?
05:45KWierso|afkrebase your patch onto a more current base
05:45ewongKWierso|afk: ah ok. will do that
05:45KWierso|afkI think those issues had an in-tree fix
05:45ewongKWierso|afk: thanks! gonna rebase
05:45KWierso|afkwhy one is red and one is purple is based on exit codes, no clue why they&#39;d be different
05:49ewongKWierso|afk: btw, does -p win32-l10n,win64-l10n no longer work?
05:49ewongI had to manually select the l10n options for my try push
05:51KWierso|afkno clue
05:52KWierso|afkand I&#39;m less positive that rebasing will help
05:52KWierso|afkI&#39;m not seeing any bugmail about this that I thought I saw last week
05:52ewongwell I decided to just pop the mq patches and qpush them on top of a freshly pulled source
06:17pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/1d0e54591b4d - Xidorn Quan - No bug - Update mochitest expectations.
06:19pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/d533ad7c74ea - Mark Hammond - Bug 1347373 - better handling of requests that have only non-existing items and of errors. r=kitcambridge
06:36pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/717da2199e0e - JW Wang - Bug 1348182 - default suspend-background-video to off in media mochitests. r=kaku
06:45ewongI needed to manually trigger l10n.. ;/ so win32-l10n,win64-l10n doesn&#39;t work as platforms
06:52pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/fcdbf512a1be - cku - Bug 1346265 - Part 1. Pass gfxContext to nsCSSRendering::PaintGradient. r=mstange
06:52pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/d2a3dfa4e1d2 - cku - Bug 1346265 - Part 2. Pass gfxContext to nsImageRenderer::DrawableForElement. r=mstange
06:52pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/d7b7b75a23af - cku - Bug 1346265 - Part 3. Test cases. r=mstange
07:00xidornhmmm there is a new permaleak on stylo&#39;s chrome mochitest...
07:03xidornheycam|away: that one seems to start from your anonymous content change
07:10Tomcat|sheriffdutyxidorn: the TEST-UNEXPECTED-FAIL | leakcheck | default process: 8 bytes leaked (nsStringBuffer) one ?
07:10xidornTomcat|sheriffduty: yep
07:10Tomcat|sheriffdutystarted at 1347809
07:11Tomcat|sheriffdutybug 1347809
07:11xidornTomcat|sheriffduty: no, the orange in that bug is the intermittent on test_hover
07:11xidornnot the permaleak
07:11Tomcat|sheriffdutyxidorn: so where are we with stylo issues , just catching up ..i guess autoland is now fine (only that perma leak) and m-i need that autoland fixes ..i&#39;m right ? :)
07:12xidornTomcat|sheriffduty: what&#39;s wrong with m-i?
07:12Tomcat|sheriffdutyah right
07:12Tomcat|sheriffdutyxidorn: for m-i perma failure in TEST-UNEXPECTED-FAIL | layout/style/test/test_computed_style.html | computed value of rgba(0 0 0 / 1) - got &quot;rgba(0, 0, 0, 0.5)&quot;, expected &quot;rgb(0, 0, 0)&quot;
07:12xidornTomcat|sheriffduty: errr
07:12Tomcat|sheriffdutyhttps://treeherder.mozilla.org/#/jobs?repo=mozilla-inbound&bugfiler&noautoclassify&selectedJob=84551528&filter-searchStr=stylo
07:13xidornguess I should push a fix to m-i...
07:13Tomcat|sheriffdutyxidorn: oh and you are right about that perma leak thats heycam|away :)
07:16heycamTomcat|sheriffduty: would you mind backing out my patch that caused the leak?
07:16Tomcat|sheriffdutyheycam: yeah sure
07:17heycamTomcat|sheriffduty: thank you
07:17Tomcat|sheriffdutyjust bug 1341102 or ?
07:17Tomcat|sheriffdutyaka 253780566f3b
07:17xidornTomcat|sheriffduty: don&#39;t back out the revendor :)
07:17heycamTomcat|sheriffduty: bug 1346623
07:17xidornotherwise you would see red on the tree again :)
07:18Tomcat|sheriffdutyok :)
07:18Tomcat|sheriffdutyglad i asked :P
07:19heycamxidorn: guess it could be the same issue as https://bugzilla.mozilla.org/show_bug.cgi?id=1347065
07:19Tomcat|sheriffdutyok backout done
07:19heycam(not the same test though)
07:19xidornheycam: yeah, it could be
07:19heycamcheers
07:19pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/0696366039f2 - Carsten &quot;Tomcat&quot; Book - Backed out changeset 1f9a6ce59a56 (bug 1346623)
07:19pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/2f9cdacc5359 - Carsten &quot;Tomcat&quot; Book - Backed out changeset 39ee9556d4a0 (bug 1346623)
07:19pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/6ed991361ec1 - Carsten &quot;Tomcat&quot; Book - Backed out changeset 9e40bc33e620 (bug 1346623) for perma leak in chrome-style stylo tests
07:20Tomcat|sheriffdutyxidorn: when you have a fix for m-i i can land this too
07:21xidornTomcat|sheriffduty: are you blocked on merge because of that?
07:21xidornif not... let me finish my current work
07:22Tomcat|sheriffdutyyeah
07:22Tomcat|sheriffdutyif we merge i guess m-c and autoland will go orange
07:22Tomcat|sheriffdutyxidorn: or i could backout https://treeherder.mozilla.org/#/jobs?repo=mozilla-inbound&revision=bb325d01c4219c4ecf49c2e8830eb762320358ec&filter-searchStr=stylo
07:23Tomcat|sheriffdutyseems where it started
07:23Tomcat|sheriffduty(if you think this fix it )
07:23Tomcat|sheriffdutyoh yeah this will fix it
07:23xidornTomcat|sheriffduty: I&#39;ll fix it
07:23Tomcat|sheriffdutyfailure on own tesst
07:23xidornyes
07:23Tomcat|sheriffdutyxidorn: ok so let me fix it :)
07:23xidornwait
07:23xidornwhat are you going to do?
07:23Tomcat|sheriffdutybackout
07:24xidornI&#39;m going to fix the stylo-failures.md...
07:24xidornso you don&#39;t need to backout
07:24Tomcat|sheriffdutyok
07:28xidornTomcat|sheriffduty: could you merge autoland to m-i first? I just had a relative big update of stylo-failures.md in autoland... and updating that file in m-i would lead to merge conflict later
07:29Tomcat|sheriffdutyxidorn: if you have a patch can i land it on m-c
07:29Tomcat|sheriffdutyautoland to m-i is not the recommend way :)
07:29Tomcat|sheriffdutyi got burned on this some time ago when i was doing this per accident :)
07:30Tomcat|sheriffdutyxidorn: or you could land on autoland and i will merge autoland asap
07:30Tomcat|sheriffdutyto m-c and then to m-i
07:30xidornTomcat|sheriffduty: probably you can merge autoland to m-c then to m-i, and then I fix m-i
07:31Tomcat|sheriffdutyyeah
07:31Tomcat|sheriffdutythats way better
07:31xidornTomcat|sheriffduty: fwiw, the big change is 1d0e54591b4d, and I do see conflict items
07:33Tomcat|sheriffdutyok cool
07:34Tomcat|sheriffdutyxidorn: on the backout for the leak a pgo build started and as soon this is all green will do the merge
07:35xidornTomcat|sheriffduty: how many hours would it take?
07:35pulsebotCheck-in: https://hg.mozilla.org/integration/mozilla-inbound/rev/2e2cc094e6a6 - Makoto Kato - Bug 1347818 - Part 1. Clean up documentCharacterSet not to use nsIDOM*. r=masayuki
07:35pulsebotCheck-in: https://hg.mozilla.org/integration/mozilla-inbound/rev/fa4734723b87 - Makoto Kato - Bug 1347818 - Part 2. Add documentCharacterSet test. r=masayuki
07:39Tomcat|sheriffdutyxidorn: about 3 hours
07:40Tomcat|sheriffdutyhm xidorn if we have the fix for m-i and land it to m-c and then merge m-c with this fix down to mozilla-inbound/autoland this would work too
07:40Tomcat|sheriffdutyand would fix m-i and also elimate merge conflicts if there any
07:40Tomcat|sheriffdutyor ?
07:41Tomcat|sheriffdutymerge conflicts with autoland/the bigger patch
07:41xidornTomcat|sheriffduty: the problem is, if you land the fix to m-c, m-c would bust
07:41Tomcat|sheriffdutyoh ok
07:41xidornunless you merge inbound into m-c immediately
07:41Tomcat|sheriffdutyyeah that would not work, since it also need pgo builds
07:42Tomcat|sheriffdutyxidorn: hmmm i think backing out the bustage from m-i is then maybe the best and cleanest way
07:42Tomcat|sheriffdutyand it could reland wiht the fix
07:42xidornthat works too
07:42Tomcat|sheriffdutyyeah will do it to get m-i clean
07:43* jya not happy with the clearing of r+ after a new push
07:43Tomcat|sheriffdutyand trigger a pgo build for this so we can merge the trees soon :)
07:44geraldjya: Complain in #mozreview , either it&#39;s a bug, or a new &quot;feature&quot;
07:44jyac&#39;est discute dans mozilla-dev
07:44geraldok
07:44jyatout le monde a dit que c&#39;etait une mauvaise idee
07:44jyamais ils l&#39;ont fait quand meme
07:45geraldjya: We&#39;re in #developers here, not private chat :-P
07:45jyaoh sorry, I thought i was still on #gerald :)
07:45Tomcat|sheriffduty:)
07:45jyawell, my point stand
07:45geraldNobody translate that! :-P
07:45jyaI worked to 2AM to make sure I would catch mattwoodrow on time for a review
07:46heycamjya: I just pushed a change to mozreview and it didn&#39;t clear my r+
07:46heycamjya: perhaps the change was made just for your account :)
07:46heycamjya: are you sure you didn&#39;t leave &quot;r?&quot; in the commit message for the change you pushed?
07:46jyawas made for others on #media
07:46jyaheycam: I always leave r? , I let reviewboard change the r? into r=
07:47heycamjya: I think leaving r? and re-push is the (intended, if it doesn&#39;t already work) way to ask for re-review when you yourself feel the patch needs review again
07:47pulsebotCheck-in: https://hg.mozilla.org/integration/mozilla-inbound/rev/beff3b520fb3 - Carsten &quot;Tomcat&quot; Book - Backed out changeset bb325d01c421 (bug 1347164)
07:47pulsebotCheck-in: https://hg.mozilla.org/integration/mozilla-inbound/rev/a34e80244b47 - Carsten &quot;Tomcat&quot; Book - Backed out changeset 18c70bc07c6d (bug 1347164)
07:47pulsebotCheck-in: https://hg.mozilla.org/integration/mozilla-inbound/rev/cad31f90c9b5 - Carsten &quot;Tomcat&quot; Book - Backed out changeset 8af7daaf5f88 (bug 1347164)
07:47pulsebotCheck-in: https://hg.mozilla.org/integration/mozilla-inbound/rev/71c32198e7c0 - Carsten &quot;Tomcat&quot; Book - Backed out changeset 36a2230fbad6 (bug 1347164) for perma failures in own test
07:47heycam(leaving r? in the commit message that is)
07:47jyaheycam: ah... bugger
07:47jyathat&#39;s new .
07:48jyagoing to see if re-pushing with r= will fix it
07:48jyaotherwise I&#39;ll just r+my
07:48jyaself
07:48heycamhmm, well https://bugzilla.mozilla.org/show_bug.cgi?id=1195661 isn&#39;t fixed, so perhaps it&#39;s not that..
07:48pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/1768ea38ef24 - Mark Banner - Bug 1347884 - Enable eslint cyclomatic complexity rule at existing levels. r=jaws
08:01pulsebotCheck-in: https://hg.mozilla.org/integration/mozilla-inbound/rev/e1f93ab32013 - Dragana Damjanovic - Bug 1341572 - Fix multiple HalfOpen socket for a single transaction. r=mcmanus
08:07mattwoodrowjya: I can click buttons if that helps
08:08ewongsheriff: if the l10n in https://treeherder.mozilla.org/#/jobs?repo=try&revision=5eb5695ca7f450a028227a88d2a00680bcf02384 turns red/purple like https://treeherder.mozilla.org/#/jobs?repo=try&revision=d6c71411494d015a8a461ec60f162b857df9a5cd&selectedJob=84243120, who should I ping?
08:08ewongerm s/sheriff/sheriffs/
08:09ewongI don&#39;t quite see how my patch would cause something like that.. unless some butterfly in South America hates me.
08:12pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/c8098c86e44c - Federico Padua - Bug 1344844 - Enable flake8 rule E222: &quot;multiple spaces after operator&quot;. r=Dexter
08:53pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/37d8224ed59c - Patrick Brosset - Bug 1347977 - Add docs about highlighters to /devtools/docs; r=jryans
09:13pulsebotCheck-in: https://hg.mozilla.org/integration/mozilla-inbound/rev/29e740f90f2c - Dragana Damjanovic - Bug 1348081 - if we call ProcessPendingQForEntry with considerAll=false we should return if we already found a transaction in the urgent queue. r=mayhemer
09:24pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/9a95ade0ec97 - Tom Tromey - Bug 1347961 - use #f06, not #f0c, in devtools tests; r=pbro
09:34Tomcat|sheriffdutyvliu_cloud: ping
09:39vliu_cloudTomcat|sheriffduty: pong
09:41Tomcat|sheriffdutyhey vliu_cloud there are perma failures on windows 8 like https://treeherder.mozilla.org/#/jobs?repo=mozilla-inbound&bugfiler&noautoclassify&filter-searchStr=Windows%208%20x64%20opt%20XPCShell%20(X)&fromchange=d4a8029b94ff3d364e453a9b9e0d7217de65ed14&selectedJob=84519896
09:41pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/c7df34767efb - Shing Lyu - Bug 1344132 - Make reftest run stylo-vs-gecko test without special reftest.list. r=dbaron
09:41Tomcat|sheriffdutycould this be caused by your patch ?
09:42Tomcat|sheriffdutyvliu_cloud: seems it crashes like here https://treeherder.mozilla.org/logviewer.html#?job_id=84519896&repo=mozilla-inbound&lineNumber=10069 and this contains gfx so i wonder if its realted
09:45vliu_cloudTomcat|sheriffduty: couldn&#39;t have any connection with my today&#39;s patch landed.
09:46Tomcat|sheriffdutyok
09:55pulsebotCheck-in: https://hg.mozilla.org/releases/mozilla-esr52/rev/04b6b8c1cb05 - Nick Thomas - Bug 1343524 - Split en-US beetmover config (m-r, m-esr* version) r=jlorenzo a=release DONTBUILD
09:56pulsebotCheck-in: https://hg.mozilla.org/releases/mozilla-esr52/rev/3462b933db3c - Nick Thomas - Bug 1343524 - Split en-US beetmover config (m-r, m-esr* version) r=jlorenzo a=release
09:57pulsebotCheck-in: https://hg.mozilla.org/integration/mozilla-inbound/rev/62cd40511419 - Carsten &quot;Tomcat&quot; Book - Backed out changeset a6780188f669 (bug 1314543) for suspicion this cause perma windows 7 xpcshell failures
09:58pulsebotCheck-in: https://hg.mozilla.org/releases/mozilla-esr45/rev/e570656aa352 - Nick Thomas - Bug 1343524 - Split en-US beetmover config (m-r, m-esr* version) r=jlorenzo a=release DONTBUILD
10:10pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/aecbad803701 - dimi - Bug 1346196 - URLCLASSIFIER_UPDATE_REMOTE_STATUS2 records values using empty keys. r=francois
10:10pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/06e12bb9737b - n0max - servo: Merge #15929 - Fix crop_area_bytes_length calculation and add tests (from n0max:canvas_drawimage_crop_fix); r=nox
10:12pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/e1d1f787c988 - Kaku Kuo - Bug 1347402 part 1 - call DownloadSuspended() via polymorphism; r=jwwang
10:12pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/6307b8e856ac - Kaku Kuo - Bug 1347402 part 2 - open a GetOwnerDoc() interface at the MediaDecoderOwner; r=jwwang
10:12pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/421c83786af7 - Kaku Kuo - Bug 1347402 part 3 - get owner document via polymorphism; r=jwwang
10:12pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/3ad2b6cfdc29 - Kaku Kuo - Bug 1347402 part 4 - move ConstructMediaTracks/RemoveMediaTracks to HTMLMediaElemnt; r=jwwang
10:12pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/23cfb067130f - Kaku Kuo - Bug 1347402 part 5 - create MediaElementGMPCrashHelper in HTMLMediaElement.cpp; r=cpearce,jwwang
10:12pulsebotCheck-in: https://hg.mozilla.org/integration/mozilla-inbound/rev/d913a0c7d25a - Jorg K - Bug 1347687 - Skip test if pref gecko.handlerService.defaultHandlersVersion not defined. r=alchen
10:12pulsebotCheck-in: https://hg.mozilla.org/integration/mozilla-inbound/rev/20537ebcebdb - Samael Wang - Bug 1303838 - Switch to existing target tab when clicking links. r=smaug
10:12pulsebotCheck-in: https://hg.mozilla.org/integration/mozilla-inbound/rev/2fd89162ba8d - Junior Hsu - Bug 1347470 - clean mAvailableCachedAltDataType when we get 200, r=valentin
10:20pulsebotCheck-in: https://hg.mozilla.org/releases/mozilla-aurora/rev/471cc7315f1e - Panos Astithas - Make the URL bar selected result telemetry probes opt-out (bug 1334617). r=bsmedberg a=lizzard
10:20pulsebotCheck-in: https://hg.mozilla.org/releases/mozilla-aurora/rev/4d684246e152 - Masatoshi Kimura - Bug 1345222 - Fix user-set ClearType params detection. r=mchang a=lizzard
10:21pulsebotCheck-in: https://hg.mozilla.org/integration/mozilla-inbound/rev/0635931c726c - Carsten &quot;Tomcat&quot; Book - Backed out changeset 20537ebcebdb (bug 1303838) for eslint failure
10:26pulsebotCheck-in: https://hg.mozilla.org/integration/mozilla-inbound/rev/93a2da765249 - Hannes Verschore - Backed out changeset 323c768fdc43 (bug 1331136) for failures on websites
10:29pulsebotCheck-in: https://hg.mozilla.org/releases/mozilla-beta/rev/6694dac889dd - Panos Astithas - Bug 1334617 - Make the URL bar selected result telemetry probes opt-out (bug 1334617). r=bsmedberg a=lizzard
10:29pulsebotCheck-in: https://hg.mozilla.org/releases/mozilla-beta/rev/df36e8af5a74 - Kirk Steuber - Bug 1338287 - Make nsIDocument::DocumentFlashClassification accessible from privileged JS r=bz a=lizzard
10:29pulsebotCheck-in: https://hg.mozilla.org/releases/mozilla-beta/rev/3bc5f2964d59 - Masatoshi Kimura - Bug 1345222 - Fix user-set ClearType params detection. r=mchang a=lizzard
10:29pulsebotCheck-in: https://hg.mozilla.org/releases/mozilla-beta/rev/993b574eb828 - Kirk Steuber - Bug 1345611 - Change behavior of subdocument Flash blocking to be Third-Party Flash blocking r=bsmedberg,qdot a=gchang
10:31Tomcat|sheriffdutyh4writer: is it worth to trigger nightlys when bug 1331136 merge to m-c ?
10:38pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/ed2a004bb475 - Matteo Ferretti - Bug 1332936 - hiding box model before taking the node&#39;s screenshot from inspector; r=pbro
10:45pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/fd3bfde70bcf - Hiroyuki Ikezoe - servo: Merge #16005 - Compose animation with servo&#39;s hashmap (from hiikezoe:animation-compose); r=heycam
10:48pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/10ec4edc3c63 - Hiroyuki Ikezoe - Bug 1340958 - Do not call get_animation_rules for pseudo elements other than ::before and ::after. r=heycam
10:48pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/d94b4a1a9c22 - Hiroyuki Ikezoe - Bug 1340958 - Do not call EffectCompositor::GetServoAnimationRule for print preview. r=heycam
10:48pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/efc0e046a5c1 - Hiroyuki Ikezoe - Bug 1340958 - Allocate StyleRule only if we need to compose styles. r=birtles
10:48pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/f2fdcee9cb8f - Hiroyuki Ikezoe - Bug 1340958 - Separate ComposeStyle() into servo and gecko versions. r=birtles
10:48pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/e9870de9fa97 - Hiroyuki Ikezoe - Bug 1340958 - Templatize ComposeStyle. r=birtles
10:48pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/205110b44b69 - Hiroyuki Ikezoe - Bug 1340958 - Add AnimationValueMap and expose it in FFI. r=heycam
10:48pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/82de9aee6922 - Hiroyuki Ikezoe - Bug 1340958 - Put computed values into AnimationValueMap instead of hashtable in gecko. r=heycam
10:48pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/71af5dbc19e2 - Hiroyuki Ikezoe - Bug 1340958 - Drop AnimationRule and ServoAnimationRule. r=boris
10:48pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/a01bbd72a8e4 - Hiroyuki Ikezoe - Bug 1340958 - Drop Servo_AnimationValues_Uncompute. r=boris
10:54pulsebotCheck-in: https://hg.mozilla.org/integration/mozilla-inbound/rev/156ee3ddc810 - Jeremy Chen - Bug 1343516 - clean up the mess of PropertyProvider and its implementations. r=jfkthame
10:59pulsebotCheck-in: https://hg.mozilla.org/integration/mozilla-inbound/rev/8c14b29f1788 - Dale Harvey - Bug 1347154 - Set correct csp for icon fetching. r=marcosc
11:06Tomcat|sheriffdutyshinglyu: ping
11:07Tomcat|sheriffdutyshinglyu: perma failure after your push in https://treeherder.mozilla.org/logviewer.html#?job_id=84599465&repo=autoland
11:07Tomcat|sheriffdutyxidorn: ^
11:08Tomcat|sheriffdutyalso https://treeherder.mozilla.org/logviewer.html#?job_id=84598509&repo=autoland and https://treeherder.mozilla.org/logviewer.html#?job_id=84598511&repo=autoland
11:08Tomcat|sheriffdutyand https://treeherder.mozilla.org/logviewer.html#?job_id=84598503&repo=autoland
11:09xidornthe bigger problem is the latest vcs-sync...
11:10xidornprobably not a problem, I&#39;ve seen a followup push in autoland
11:11Tomcat|sheriffdutyxidorn: yeah retrigger the faild build
11:12xidornTomcat|sheriffduty: I don&#39;t expect shinglyu&#39;s push to cause that kind of issue. I think that may need some non-trivial investigation for him, so I&#39;d suggest you back out his push if he don&#39;t object
11:13xidornI mean, if he don&#39;t respond here in a reasonable time...
11:13xidornTomcat|sheriffduty: no, you don&#39;t need to retrigger that build
11:14xidornTomcat|sheriffduty: hiro&#39;s latest push should fix the build failure of stylo
11:20Tomcat|sheriffdutyxidorn: ok will back this out and yeah builds are green now :)
11:25pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/5df20a14af5d - Paul Rouget - servo: Merge #16007 - cocoa update (from paulrouget:cocoa-update); r=KiChjang
11:26Tomcat|sheriffdutymak: thanks!
11:26Tomcat|sheriffdutywe try to be more proactive for crashes in tests thats why i set the needinfo for you
11:37pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/55ebdfc93471 - Carsten &quot;Tomcat&quot; Book - Backed out changeset c7df34767efb (bug 1344132) for stylo test failures
12:06pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/2b0ba8a8ed69 - Xidorn Quan - Bug 1346353 - Disable test_hover.html for stylo because of intermittent issue.
12:20smauggrr, the overuse of commandline params is really annoying
12:21smaugpollutes &#39;ps ax&#39; so well
12:28pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/14b74d9448cc - BharatR123 - Bug 1343827 - Removed icon for &quot;Undo Closed Tab&quot; and its references r=dao
12:45pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/d29aa1530b0d - Julian Descottes - Bug 1344504 - handle rejected promise when showing/hiding box model in markup view;r=gl
12:45pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/916169e1e3ee - Julian Descottes - Bug 1344504 - do not catch click events closing the event tooltip;r=gl
12:47pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/bb5c698bff0a - Patrick Brosset - Bug 1348258 - Use MODE.TINY for ElementNodes displayed in the grid inspector; r=nchevobbe
12:53pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/1da2122f3597 - Anthony Ramine - servo: Merge #16010 - Properly coalesce whitespace when given a reference child (fixes #15979) (from nox:h5e); r=KiChjang
12:59pulsebotCheck-in: https://hg.mozilla.org/integration/mozilla-inbound/rev/a13dc6642789 - Gabriel Luong - Bug 1346378 - Grid inspector toggle should appear for display: [inline-]grid !important declarations. r=jdescottes
13:00pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/0d33d81b075f - Julian Descottes - Bug 1348271 - attach eyedropper click event to the proper element;r=gl
13:28pulsebotCheck-in: https://hg.mozilla.org/mozilla-central/pushloghtml?startID=31511&endID=31512 - 99 changesets (bugs 1254643, 1343796, 1275958, 1341739, 1329111 and 42 other bugs)
13:33pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/78e70d84203d - cku - servo: Merge #16004 - stylo: Update clip-path and transform-box glue (from CJKu:bug-1339674); r=heycam
13:36pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/248f3609df08 - cku - Bug 1340044 - Part 1. Declare mTransformBox as StyleGeometryBox. r=heycam
13:36pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/f86b20231a00 - cku - Bug 1340044 - Part 2. Rename value in StyleGeometryBox. r=heycam
13:46pulsebotCheck-in: https://hg.mozilla.org/integration/mozilla-inbound/rev/45fe96225142 - Carsten &quot;Tomcat&quot; Book - Backed out changeset