mozilla :: #developers

20 Apr 2017
00:00pulsebotCheck-in: https://hg.mozilla.org/releases/mozilla-beta/rev/2bc41df27202 - L10n Bumper Bot - no bug - Bumping Fennec l10n changesets r=release a=l10n-bump
00:03pulsebotCheck-in: https://hg.mozilla.org/integration/mozilla-inbound/rev/7492d1c0aad4 - Jeff Gilbert - Bug 1356540 - Remove unused TextureGarbageBin. - r=daoshengmu
00:12pulsebotCheck-in: https://hg.mozilla.org/mozilla-central/pushloghtml?startID=31680&endID=31681 - 78 changesets (bugs 1353420, 1354292, 1356836, 1354933, 1343396 and 44 other bugs)
00:15pulsebotCheck-in: https://hg.mozilla.org/integration/mozilla-inbound/pushloghtml?startID=89373&endID=89374 - 104 changesets - Merge m-c to inbound, a=merge
00:17pulsebotCheck-in: https://hg.mozilla.org/releases/mozilla-beta/rev/2c92847dca76 - Henrik Skupin - Bug 1356000 - Ensure unwrapped content listeners catch errors. r=ato, a=test-only
00:17pulsebotCheck-in: https://hg.mozilla.org/releases/mozilla-beta/rev/601f56f86809 - Mats Palmgren - Bug 1355873 - Add more consistent error handling and some code cleanup. r=mstange, a=gchang
00:17pulsebotCheck-in: https://hg.mozilla.org/releases/mozilla-beta/rev/5054aabe5c82 - Mats Palmgren - Bug 1357092 - Make EnsureWritablePath() always try to create a mPathBuilder, also when we don't have a valid target (i.e. for sErrorTarget). r=mstange, a=gchang
00:17pulsebotCheck-in: https://hg.mozilla.org/releases/mozilla-beta/rev/c790dc13185f - Jesse Ruderman - Bug 779426 - Crashtest. r=mats, a=test-only
00:17pulsebotCheck-in: https://hg.mozilla.org/releases/mozilla-beta/rev/927de4d78105 - Mike Hommey - Bug 1344889 - Add necessary dependendencies for generated libvpx assembly. r=mshal, a=gchang
00:17pulsebotCheck-in: https://hg.mozilla.org/releases/mozilla-beta/rev/4f8540793211 - Boris Chiou - Bug 1348322 - Fix the initial values of gVelocityCurveFunction. r=kats, a=gchang
00:17pulsebotCheck-in: https://hg.mozilla.org/releases/mozilla-beta/rev/c8b0d74c8c41 - Alessio Placitelli - Bug 1352496 - Enable sending other child processes crash pings from the CrashManager. r=ted, r=bsmedberg, a=gchang
00:17pulsebotCheck-in: https://hg.mozilla.org/releases/mozilla-beta/rev/3c1bf407a189 - Timothy Nikkel - Bug 1355898 - Rename flag SFE_WANT_FIRST_FRAME to SFE_WANT_FIRST_FRAME_IF_IMAGE to represent what it does better. r=jgilbert, a=gchang
00:17pulsebotCheck-in: https://hg.mozilla.org/releases/mozilla-beta/rev/2763894710d8 - Timothy Nikkel - Bug 1355898 - Use the first frame of animated images for texImage2d. r=jgilbert, a=gchang
00:17pulsebotCheck-in: https://hg.mozilla.org/releases/mozilla-beta/rev/9edc76120a27 - Timothy Nikkel - Bug 1355898 - Add crashtest. a=gchang
00:17pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/pushloghtml?startID=41118&endID=41119 - 79 changesets - Merge m-c to autoland, a=merge
00:27pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/ef655e427521 - J. Ryan Stinnett - Bug 1345515 - Add notifications advertising new RDM. r=ochameau
00:28jryansaswan: does this commit have the wrong bug number? https://hg.mozilla.org/integration/mozilla-inbound/rev/18f4ab471d1699f03d90a23bda5c94ae443b9908
00:29bkellymilan: is there a way for a user on FF53 to know that they have a GPU process?
00:29aswanjryans: ah ha, i was wondering why there wasn't a comment on the bug it was meant for!
00:29bkellythe about:memory reporter is only in FF54
00:29bkellyabout:support doesn't show it AFAIK
00:29aswansorry about that...
00:30bkellyor does anyone else here know about the GPU process? ^^^
00:32pulsebotCheck-in: https://hg.mozilla.org/releases/mozilla-esr52/rev/562f2490aba8 - Henrik Skupin - Bug 1355009 - Force update tests to only allow a single update. r=ato, a=test-only
00:33Mossopmrbkap: Aurora is no more now isn't it?
00:33pulsebotCheck-in: https://hg.mozilla.org/releases/mozilla-esr52/rev/efa5bc8349b3 - Henrik Skupin - Bug 1355009 - Harden update tests with better error messages. r=ato, a=test-only
00:33pulsebotCheck-in: https://hg.mozilla.org/releases/mozilla-esr52/rev/c01007911fb7 - Henrik Skupin - Bug 1355009 - Temporarily disable usage of simplified update ui. r=ato, a=test-only
00:33mrbkapMossop: I really don't know.
00:33mrbkapMossop: I guess I didn't read the emails deeply enough to figure out the process.
00:33mrbkapMossop: is it "starting immediately hg.mozilla.org/releases/mozilla-aurora is unused"?
00:35MossopI guess we'll find out
00:36mrbkapMossop: ah, you're right.
00:37mrbkapoh well.
00:41mrbkapMossop: it never occurred to me that we branched yesterday, thanks.
00:48pulsebotCheck-in: https://hg.mozilla.org/releases/mozilla-beta/pushloghtml?startID=7204&endID=7205 - 11 changesets (bugs 1352496, 1346288, 1325513, 1356392, 1345893 and 4 other bugs)
02:01pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/182ed00b19e3 - Iris Hsiao - Backed out changeset ef655e427521 (bug 1345515) for devtool test failures at responsivedesign.jsm
02:02pulsebotCheck-in: https://hg.mozilla.org/integration/mozilla-inbound/rev/f57b18615900 - Arthur Edelstein - Bug 1357247 - Fix comment for network.http.referer.hideOnionSource. r=mcmanus
02:33pulsebotCheck-in: https://hg.mozilla.org/integration/mozilla-inbound/rev/6d047b2e3e1f - L. David Baron - Bug 1357951 - Update two transforms tests that should now point to css-transforms-2 following the spec split. r=SimonSapin
02:53pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/59ba67068ce1 - JW Wang - Bug 1356506 - change the type of MediaData::mTimecode to TimeUnit since int64_t is ambiguous. r=gerald
03:19* bz tries to recall how to run jetpack test slocally again
03:19bzDo we have a mach command for this?
03:51pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/32ab1a3d736f - Matthew Wein - Bug 1331742 - Part 1 - Create and register browser_action.json r=mixedpuppy
03:51pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/c91b14d9ad2a - Matthew Wein - Bug 1331742 - Part 2 - Create a module for managing browser actions similar to PageActions.jsm r=sebastian
03:51pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/31f4919d1f6c - Matthew Wein - Bug 1331742 - Part 3 - Create and register ext-browserAction.js r=mixedpuppy
03:51pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/753e71053540 - Matthew Wein - Bug 1331742 - Part 4 - Add a position property to keep track of the menu item's position instead of using the ID r=sebastian
03:51pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/79579aab8851 - Matthew Wein - Bug 1331742 - Part 5 - Add unit tests for browserAction.onClicked r=mixedpuppy
03:55pulsebotCheck-in: https://hg.mozilla.org/integration/mozilla-inbound/rev/e2fccaa1a87a - Kershaw Chang - Bug 1355700 - Correct the count of available new connections. r=honzab
04:13philorlovely that I introduce myself to so many people with a needinfo on a "you're going to break beta, cause it to be closed, and delay the first beta release" bug
04:14philorthough I guess since my other way is "ping, you broke the tree and I'm going to back you out" there's not that much difference
04:18nigelbIt's a warning shot.
04:19bzphilor: Keeps them from getting the wrong idea. ;)
04:20pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/7457955a0eab - Boris Chiou - servo: Merge #16536 - stylo: Change the animation_type of column-count to normal (from BorisChiou:stylo/animation/column_count); r=Manishearth
04:20philorindeed, a nice tone-setter
04:25nigelbI kind of imagine philor saying that in a Liam Nelson in Taken sort of voice.
04:26* philor gasps
04:27philorsomeone touched *editor* for 55?
04:30kanrufor good reason?
04:33bzphilor: I certainly did...
04:33bzkanru: perf
04:34bzphilor: But I would not expect my changes to come to your attention....
04:34kanruyeh, I would expect so ;)
04:35philorbz: so far I don't have any guesses about who to blame for https://treeherder.mozilla.org/logviewer.html#?job_id=92882737&repo=try
04:35pulsebotCheck-in: https://hg.mozilla.org/integration/mozilla-inbound/rev/1b37b1ed3bc1 - Nicholas Nethercote - Bug 1357936 - Don't leak ProfilerIOInterposeObserver. r=mstange.
04:36ihsiao|sheriffdutymrbkap: around?
04:36bzphilor: looking
04:36philoroh, look at that, Aryeh just landed that a few weeks ago!
04:36mrbkapihsiao|sheriffduty: hi
04:36philorI was thrown off by the low bug number, been too long since I dealt with editor :)
04:37bzphilor: boy
04:37ihsiao|sheriffdutymrbkap: I'm working on uplift, your patches got conflicts, could you help to rebase? https://bugzilla.mozilla.org/show_bug.cgi?id=1349363#c46
04:37bzphilor: Is that push normal for a "yeah, we're about to go to beta" situation? :(
04:37mrbkapihsiao|sheriffduty: looking
04:37philorbz: this is 55-as-beta, 2017-06-12, but yeah, it's normal for six weeks out
04:39philorin the old world a third of them would be caught going to aurora, and then the other two-thirds aurora to beta, now we get all the !NIGHTLY_BUILD and RELEASE_OR_BETA in one shot
04:39bzphilor: We should really have infra for this
04:39bzphilor: so people notice it when landing...
04:40bzphilor: anyway, aryeh's thing... I'm surprised it depends on channel
04:40philorso, double the number of builds and tests per push? ouch, we currently run around 12-24 hour backlog for linux64 talos and 6-8 hour for 10.10, doubling the inbound/autoland load seems like a lot
04:40bzphilor: Could be a once a day thing
04:41philorthere's talk about that, but I'm not very sanguine about it actually doing *me* any good, since it will remove the step of typing 'hg push-to-try' but it won't file things for me
04:41bzFair
04:43pulsebotCheck-in: https://hg.mozilla.org/integration/mozilla-inbound/rev/8a8d9ca30319 - Boris Zbarsky - Bug 1357646 part 1. Move the setting of mType from HTMLButtonElement::ParseAttribute to HTMLButtonElement::AfterSetAttr. r=mystor
04:43pulsebotCheck-in: https://hg.mozilla.org/integration/mozilla-inbound/rev/ff8904e0ad25 - Boris Zbarsky - Bug 1357646 part 2. Move the setting of mType from HTMLMenuElement::ParseAttribute to HTMLMenuElement::AfterSetAttr. r=mystor
04:43pulsebotCheck-in: https://hg.mozilla.org/integration/mozilla-inbound/rev/b4d447189d4d - Boris Zbarsky - Bug 1357646 part 3. Move the setting of mType from HTMLMenuItemElement::ParseAttribute to HTMLMenuItemElement::AfterSetAttr. r=mystor
04:44philorI think I'm supposed to pretend that my colleagues will do some of the filing
04:44mrbkapihsiao|sheriffduty: ping?
04:44ihsiao|sheriffdutypong
04:45mrbkapihsiao|sheriffduty: Which patch did you use or how did you do the merge to beta?
04:45ihsiao|sheriffdutymrbkap: I graft from m-c
04:46mrbkapihsiao|sheriffduty: That won't apply.
04:46mrbkapihsiao|sheriffduty: In the bug, there are four patches with beta approval.
04:46ihsiao|sheriffdutymrbkap: so should I import the four attachments?
04:46mrbkapihsiao|sheriffduty: Yes.
04:47ihsiao|sheriffdutymrbkap: ok, thanks. will do again
04:47mrbkapihsiao|sheriffduty: If you prefer, I could probably give you an hg bundle or set up another repo for you to pull from.
04:48ihsiao|sheriffdutyit's fine, I can import them from the bug
04:49philorbz: and given that two days ago I was at https://treeherder.mozilla.org/#/jobs?repo=try&revision=7993ee5888d00fa25c138accd8471cf6fe00694f that push is looking pretty good!
04:51pulsebotCheck-in: https://hg.mozilla.org/releases/mozilla-beta/rev/e0340a68ed92 - Gabriele Svelto - Bug 1319071 - Make crash minidumps use the same format for filenames on Linux as on other platforms; r=ted. a=gchang
04:51pulsebotCheck-in: https://hg.mozilla.org/releases/mozilla-beta/rev/cec324879dca - Andrea Marchesini - Bug 1343172 - PBlob should not create a remoteInputStream if that already exists, r=smaug. a=gchang
04:51pulsebotCheck-in: https://hg.mozilla.org/releases/mozilla-beta/rev/9a10a8b7cf4f - Jim Chen - Bug 1354911 - Guard against null menu item names; r=sebastian. a=gchang
04:51pulsebotCheck-in: https://hg.mozilla.org/releases/mozilla-beta/rev/c8a873cdb79a - Andrea Marchesini - Bug 1356580 - Better check of PMemoryStream actor creation in Blob code, r=qdot. a=gchang
04:51pulsebotCheck-in: https://hg.mozilla.org/releases/mozilla-beta/rev/f6a1a69f8b06 - Blake Kaplan - Bug 1349363 - Centralize pref-checking code for e10s-multi control. r=Felipe. a=gchang
04:51pulsebotCheck-in: https://hg.mozilla.org/releases/mozilla-beta/rev/25da34a87bee - Blake Kaplan - Bug 1349363 - Use a centralized function to tell if e10s-multi is on. r=Felipe. a=gchang
04:51pulsebotCheck-in: https://hg.mozilla.org/releases/mozilla-beta/rev/1c0e19d64704 - Blake Kaplan - Bug 1349363 - Run an experiment in beta for e10s-multi. r=Felipe. a=gchang
04:51pulsebotCheck-in: https://hg.mozilla.org/releases/mozilla-beta/rev/bfa8f90200d1 - Blake Kaplan - Bug 1349363 - Track both .processCount and .web to get a complete view of the user's content processes. r=Felipe. a=gchang
04:52mrbkapihsiao|sheriffduty: Thanks!
04:53ihsiao|sheriffdutynp and thank you :)
04:56pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/31db93715b92 - KuoE0 - servo: Merge #16537 - Add `inline-axis` and `block-axis` aliases for `-moz-box-orient` (from KuoE0:add-inline-axis-and-block-axis-for-moz-box-orient); r=KuoE0
05:07pulsebotCheck-in: https://hg.mozilla.org/integration/mozilla-inbound/rev/7d9c92e2215c - Phil Ringnalda - Bug 1350732 - set the pref to enable dom.storageManager in a __dir__.ini instead of playing Whac-A-Test
05:11pulsebotCheck-in: https://hg.mozilla.org/integration/mozilla-inbound/rev/f7870b544dab - Ehsan Akhgari - Bug 1356913 - Remove support for the mozapptype attribute; r=billm
05:27pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/c018f36dcdad - Astley Chen - Bug 1355752 - Remove B2G wording in AccessibleCaret. r=TYLin
05:27pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/e42cf4911d08 - Astley Chen - Bug 1355752 - Remove B2G code from PresShell files. r=jrmuizel
05:27pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/1eb9604b76a9 - Astley Chen - Bug 1355752 - Remove B2G code from nsCSSFrameConstructor.cpp. r=dholbert
05:27pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/bc04a783948e - Astley Chen - Bug 1355752 - Remove B2G code from layout/build files. r=jmaher
05:27pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/16028292e2f1 - Astley Chen - Bug 1355752 - Remove B2G code from nsFormControlFrame.cpp & forms.css. r=dholbert
05:27pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/a1c51bf7ec8a - Astley Chen - Bug 1355752 - Remove B2G code from nsGfxScrollFrame.cpp. r=kats
05:27pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/7d77113f3841 - Astley Chen - Bug 1355752 - Remove B2G code from FrameLayerBuilder.cpp. r=mstange
05:56pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/648169abf928 - Henri Sivonen - Bug 1356181 - Gather telemetry about isindex usage in hope of justifying removal. r=Ehsan,francois
05:59pulsebotCheck-in: https://hg.mozilla.org/integration/mozilla-inbound/rev/fbec3b469121 - Robert Strong - Bug 1336979 - Remove unnecessary logging of the update.status contents in the maintenance service. r=mhowell
06:04pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/1f1f3f34d558 - Manish Goregaokar - Bug 1355427 - Part 3: stylo: Update test expectations; r=heycam
06:05Manishearthhm, vcs sync seems down>
06:06Manishearthif the vcs sync doesn't happen soon you may need to back out https://hg.mozilla.org/integration/autoland/rev/1f1f3f34d558a8722b7235a369ae3bc497f7f107
06:07globManishearth: hrm
06:08globManishearth: at a glance things look good on the server. digging deeper
06:11pulsebotCheck-in: https://hg.mozilla.org/integration/mozilla-inbound/rev/919b9ff342f3 - Dragana Damjanovic - Bug 1310197 - Implement fast open nspr part. r=mcmanus,bagder
06:12pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/e531bd57fe8d - Manish Goregaokar - servo: Merge #16539 - stylo: Properly support -moz-script-size-multiplier, -moz-script-level, and -moz-script-min-size (from Manishearth:stylo-scriptminsize); r=heycam
06:13globManishearth: looks like it just landed - https://hg.mozilla.org/integration/autoland/rev/e531bd57fe8d81369bccc3e648e3151d1384abf3 (i didn't change anything)
06:13Manishearthhm
06:13globindeed
06:13Manishearththat was quite slow
06:13Manishearth(but yay)
06:21globlooks like pulse messages were delayed. /me returns to researching SQS
06:26pulsebotCheck-in: https://hg.mozilla.org/integration/mozilla-inbound/pushloghtml?startID=89383&endID=89384 - 13 changesets (bug 1356843)
06:32pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/488534f69bf5 - KuoE0 - Bug 1355005 - Remove the failures mark of `box-orient` in stylo-failures.md. r=heycam
06:35pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/b06e3d6c2f78 - cu1t - servo: Merge #16472 - Fix namespaces of elements created in XML documents (from cu1t:#14095-fix-xml-doc-namespaces); r=nox
06:38pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/b01181410a8a - Valentin Gosu - Bug 1354349 - Add nsIChannel.isDocument that checks if LOAD_DOCUMENT_URI is set, or if LOAD_HTML_OBJECT_DATA and the channel has the appropriate MIME type r=mcmanus,mystor
06:38pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/8a42c7e1f799 - Valentin Gosu - Bug 1354349 - Use channel.isDocument in ContentParent::TransmitPermissionsFor r=mystor
06:44pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/7ce95e24c011 - Mark Banner - Bug 1357502 - Change eslint-plugin-mozilla tests to use the mocha test harness to improve the output. r=mossop
06:45pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/368a60db302d - Mark Banner - Bug 1357702 - Make eslint-plugin-spidermonkey-js handle paths correct on Windows to avoid Unexpected character issues on ESLint. r=mossop
07:14pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/3864467a616c - Scott Wu - Bug 1352331 - Swap the cancel and save buttons on edit profile dialog for Windows. r=lchang
07:21ihsiao|sheriffdutyheycam: xidorn not sure if you have noticed the stylo reftest failure like https://treeherder.mozilla.org/logviewer.html#?job_id=92921786&repo=autoland&lineNumber=5386 ?
07:22jdescottesTomcat|sheriffduty: ping
07:25Tomcat|sheriffdutyjdescottes: pong
07:25heycamihsiao|sheriffduty: I hadn't seen it, no...
07:25jdescottesTomcat|sheriffduty: hi! Could you back out https://hg.mozilla.org/mozilla-central/rev/26825d7a225d ?
07:25heycamihsiao|sheriffduty: it will be due to the recent font size patches
07:25Tomcat|sheriffdutyyeah sure
07:26jdescottesthanks
07:26Tomcat|sheriffdutyjdescottes: for comment 22 in the bug right ?
07:26jdescottesTomcat|sheriffduty: exactly
07:26heycamihsiao|sheriffduty: I think I'll just mark those two tests as expected fail and open a bug
07:27Pikejdescottes: thanks
07:27ihsiao|sheriffdutyheycam: ok, thanks
07:28jdescottesPike: np, sorry about the mess
07:28Tomcat|sheriffdutyjdescottes: Pike done
07:28pulsebotCheck-in: https://hg.mozilla.org/mozilla-central/rev/f6be27f5457f - Carsten "Tomcat" Book - Backed out changeset 26825d7a225d (bug 1354672) on developers request
07:28jdescottesthanks!
07:30pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/27507fdcbacd - Cameron McCormack - Bug 1355427 - Expectations adjustment followup. r=me
07:32pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/3db332e289c9 - Martin Robinson - servo: Merge #16531 - Eliminate ScrollRootId (from mrobinson:clip-id); r=glennw
07:46pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/3507329c5999 - Marco Bonardo - Bug 1357555 - Properly handle an expiration value of 0 when fetching an icon. r=adw
07:47gchangPike: hi~
07:48Pikehi
07:48Pikegreetings from Taipei ;-)
07:50pulsebotCheck-in: https://hg.mozilla.org/integration/mozilla-inbound/rev/af7955546902 - Thinker K.F. Li - Bug 1342009 - Provide a shortcut in BuildDisplayListForChild. r=mattwoodrow
07:50pulsebotCheck-in: https://hg.mozilla.org/integration/mozilla-inbound/rev/ab2aad34a967 - Munro Mengjue Chiang - Bug 1353313 - Add synchronization mechanism to CodecProxy. r=jolin
07:50gchangPike: today I want to go to build for 54 beta 1, but got some errors. it seems that 'my' locale is missing
07:50pulsebotCheck-in: https://hg.mozilla.org/integration/mozilla-inbound/rev/e2ea72d6ed2c - Tom Tung - Bug 1356466: Make checking function resolve late enough to prevent from unregistering too early. r=bkelly
07:52gchangPike: there is no "my" locale changeset in ship-it
07:55Pikegchang: I see. I'd need flod to take 'em, but he's on his way to taipei right now
07:56Pikegchang: what's the easiest way to skip them for b1?
07:57gchangPike: I'm not sure.
07:58Pikegchang: I'd be fine to back out bug 1355070, and reland it when flod ticked off the source review on l10n.m.o
07:58PikeI'd guess he'll get to it this weekend
08:01birtlesIs it because I upgraded to Windows Creators' edition or has Nightly been really bad (on Windows at least) the last few days?
08:01birtlesIt janks a lot and content processes frequently hang
08:08DexterTomcat|sheriffduty, hi! I've re-re-based the patch in bug 1352496 and left you a ni? there... could you kindly re-land it?
08:12Tomcat|sheriffdutyDexter: hi, will do
08:12Dexterthanks!
08:18pulsebotCheck-in: https://hg.mozilla.org/releases/mozilla-beta/rev/cf76e00dcd6f - Carsten "Tomcat" Book - Backed out changeset acf78d5d6ef2 (bug 1355070) on request from gchang
08:22Tomcat|sheriffdutyPike: ^
08:22Pikeyep, seen that
08:22Tomcat|sheriffdutyPike: affect l10n
08:23makTomcat|sheriffduty: is there a way to push a small change to autoland without a backout? (dumb ES failure)
08:24pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/6941882b6708 - Iris Hsiao - Backed out changeset 3507329c5999 (bug 1357555o) for eslint failure in test_expire_migrated_icons.js
08:24ihsiao|sheriffdutymak: I guess no, sorry have to back your cset out for eslint failure
08:24Tomcat|sheriffdutymak: no, we try to keep the trees clean and where asked to do the strict way
08:24makok
08:24Tomcat|sheriffdutytrees clean (autoland without manual pushes etc)
08:25ihsiao|sheriffdutymak: not sure if you need a ni?
08:26makit's an unfortunate waste of resources for just a string using ' instead of " :p
08:26makno I don't need a ni this time, thanks
08:26ihsiao|sheriffdutyok :)
08:26mak(an empty string moreover)
08:36gchangPike, Tomcat|sheriffduty thanks for doing this.
08:38freddybwell, it's also unfortunate that we dont have git/hg hooks that catch these tiny eslint violations locally
08:40Tomcat|sheriffdutydaoshengmu: ping
08:40Pikefreddyb: most of the time they're triggered by the config changing between central and autoland these days
08:41Pikeat least that's my reading here, non-scientific stats
08:42makmozreview should just run check and open issues automatically
08:42freddybwell, that would be nice
08:43Aryxthat's on the roadmap for q3, afaik
08:43pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/12f943a6d12c - Frank Serrano - Bug 1357832 - Update eula.css to use standard css border; r=dao
08:43Aryxmeanwhile, one can use https://www.oxymoronical.com/blog/2015/12/Running-ESLint-on-commit for one's own pushes
08:44freddybI should do that
08:44freddybthe git variant is a bit different though
08:45freddyb:)
08:45freddybah, the blog post links to https://bugzilla.mozilla.org/show_bug.cgi?id=1239044
09:01pulsebotCheck-in: https://hg.mozilla.org/integration/mozilla-inbound/rev/a03617861cb6 - Kevin Jones - Bug 1345090 - Modify SessionStore to restore tabs with lazy browsers. r=mikedeboer,dao
09:04pulsebotCheck-in: https://hg.mozilla.org/integration/mozilla-inbound/rev/819e87382263 - Alex Chronopoulos - Bug 1357683 - Update cubeb from upstream to 6e52314. r=padenot
09:11pulsebotCheck-in: https://hg.mozilla.org/mozilla-central/pushloghtml?startID=31682&endID=31683 - 17 changesets - merge autoland to mozilla-central a=merge
09:13pulsebotCheck-in: https://hg.mozilla.org/mozilla-central/pushloghtml?startID=31683&endID=31684 - 24 changesets - merge mozilla-inbound to mozilla-central a=merge
09:18pulsebotCheck-in: https://hg.mozilla.org/integration/mozilla-inbound/rev/4a0c24d90fc4 - Shawn Huang - Bug 1357975 - Set the pref to enable dom.storageManager in helpers.js, r=janv
09:18pulsebotCheck-in: https://hg.mozilla.org/integration/mozilla-inbound/rev/a485db46b96c - Shawn Huang - Bug 1357975 - Set pref browser.storageManager.enabled for storage browser tests, r=janv
09:31pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/45597054d4e8 - Daosheng Mu - Bug 1356452 - Part 1: Remove assertion for checking VRManager::Get(); r=kip
09:31pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/b425dbaf9394 - Daosheng Mu - Bug 1356452 - Part 2: Replace vr:: namespace with ::vr:: in OpenVR; r=kip
09:31pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/fed55d5486d0 - Daosheng Mu - Bug 1356452 - Part 3: Support changing gamepad hand property in GamepadManager; r=Lenzak
09:31pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/2e29632323c9 - Daosheng Mu - Bug 1356452 - Part 4: Add gamepad set hand function in Gamepad; r=qdot
09:31pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/ecc42c31cc28 - Daosheng Mu - Bug 1356452 - Part 5: Detect hand changing at runtime for OpenVR controllers; r=kip
09:33pulsebotCheck-in: https://hg.mozilla.org/integration/mozilla-inbound/rev/0ac28a9153c8 - Edgar Chen - Bug 1357002 - Part 1: Cache "dom.webcomponents.customelements.enabled" preference; r=wchen
09:33pulsebotCheck-in: https://hg.mozilla.org/integration/mozilla-inbound/rev/bbd2216268bc - Edgar Chen - Bug 1357002 - Part 2: Don't call SetupCustomElement() if the custom element feature is pref-ed off; r=wchen
09:36daoshengmuTomcat|sheriffduty, pong
09:38Tomcat|sheriffdutyhey daoshengmu could you take a look at bug 1358053
09:39daoshengmuTomcat|sheriffduty, sure. I will
09:39Tomcat|sheriffdutycool thanks
09:39pulsebotCheck-in: https://hg.mozilla.org/integration/mozilla-inbound/rev/5b9b767015e2 - Carsten "Tomcat" Book - Backed out changeset af7955546902 (bug 1342009) for mochitest failures
09:40pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/a6bbb991c0d2 - Marco Bonardo - Bug 1357555 - Properly handle an expiration value of 0 when fetching an icon. r=adw
09:45pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/ad7d2ae91ce6 - Marco Bonardo - Bug 1357664 - Don't delete all relations to expired icons when updating icons for a specific page. r=adw
09:52pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/91c19f2d71f3 - Michael Ratcliffe - Bug 1356223 - Support scalar telemetry probe types in Telemetry.js r=pbro
09:53pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/2fddfb975738 - cku - Bug 1357432 - Part 1. Move IsLocalRefURL to nsContentUtils to reuse this function. r=heycam
09:53pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/932320750dfe - cku - Bug 1357432 - Part 2. Implement nsSVGEffects::GetBaseURLForLocalRef to export local-ref-url-resolving logic. r=heycam
09:53pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/92f7611393f6 - cku - Bug 1357432 - Part 3. Resolve local-ref in SVGUseElement::LookupHref by nsSVGEffects::GetBaseURLForLocalRef. r=heycam
09:53pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/032d23aaffe3 - cku - Bug 1357432 - Part 4. Reftest for using local-ref as xlink:href value. r=heycam
09:56pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/7ab7f74c36a2 - Michael Ratcliffe - Bug 1352115 - Add scalars for toolbareyedropper, copyuniquecssselector and copyfullcssselector r=bsmedberg,pbro
10:00pulsebotCheck-in: https://hg.mozilla.org/integration/mozilla-inbound/rev/f7b360ab2445 - James Graham - Bug 1356227 - Add support for excluding paths when doing a wpt sync, r=ato
10:00pulsebotCheck-in: https://hg.mozilla.org/integration/mozilla-inbound/rev/de51a02a657e - James Graham - Bug 1356225 - Update web-platform-tests to revision d519fe9011da7cfce7949f7ed826e9759dc5c532, a=testonly
10:00pulsebotCheck-in: https://hg.mozilla.org/integration/mozilla-inbound/rev/34c41d2a5227 - James Graham - Bug 1347949 - Add six to wptserve dependencies, r=whimboo
10:00pulsebotCheck-in: https://hg.mozilla.org/integration/mozilla-inbound/rev/ab00229faf2c - James Graham - Bug 1318724 - Increase the HTTP connection timeout for WebDriver tests, r=ato
10:00pulsebotCheck-in: https://hg.mozilla.org/integration/mozilla-inbound/rev/e3e321ce75df - James Graham - Bug 1356225 - Use merge for upstreaming wpt commits, a=testonly
10:00pulsebotCheck-in: https://hg.mozilla.org/integration/mozilla-inbound/rev/16435cb6efc0 - James Graham - Bug 1356225 - Update web-platform-tests expected data to revision d519fe9011da7cfce7949f7ed826e9759dc5c532, a=testonly
10:00pulsebotCheck-in: https://hg.mozilla.org/integration/mozilla-inbound/rev/d11e60ed38a0 - James Graham - Bug 1356225 - Remove defunct directory from moz.build file, a=bustage
10:00pulsebotCheck-in: https://hg.mozilla.org/integration/mozilla-inbound/rev/5d4cd1bdb0c3 - James Graham - Bug 1356225 - Remove expected failure for fetch headers test, a=testonly
10:00pulsebotCheck-in: https://hg.mozilla.org/integration/mozilla-inbound/rev/1902130057ea - James Graham - Bug 1357844 - Split MathML entity test into multiple files, r=annevk
10:04pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/e8babe547652 - JW Wang - Bug 1356530 - Change the type of MediaData::mTime to TimeUnit since int64_t is ambiguous. r=kaku
10:04pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/25944baae7b0 - Anthony Ramine - servo: Merge #16511 - Implement -webkit-radial-gradient() (fixes #15441) (from nox:webkit-gradients); r=emilio
10:17pulsebotCheck-in: https://hg.mozilla.org/releases/mozilla-beta/rev/e0ef5e00268e - ffxbld - No bug, Automated blocklist update from host bld-linux64-spot-305 - a=blocklist-update
10:42pulsebotCheck-in: https://hg.mozilla.org/integration/mozilla-inbound/rev/f18e288534fb - Do Gottwald - Backed out changeset 3edd47730527
10:56pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/66be5ce5eb06 - Anthony Ramine - servo: Merge #16513 - Properly parse background-size in background longhand (fixes #15199) (from nox:background-slash-size); r=upsuper
11:00pulsebotCheck-in: https://hg.mozilla.org/releases/mozilla-beta/rev/3a505a78e1be - L10n Bumper Bot - no bug - Bumping Fennec l10n changesets r=release a=l10n-bump
11:03pulsebotCheck-in: https://hg.mozilla.org/integration/mozilla-inbound/pushloghtml?startID=89392&endID=89393 - 20 changesets - Merge mozilla-central to mozilla-inbound
11:10pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/e37099ed6eb8 - Johann Hofmann - Bug 1358083 - Rename browser/themes/shared/toolbarbuttons.inc.css to toolbarbutton-icons.inc.css. r=dao
11:19florianat which time of the day do we start building nightlies, and when do they typically become available?
11:25Aryxflorian: they start at 10am UTC, availability depends on platform (tests run), release automation
11:26florianthanks
11:27florianso the changesets that were merged to m-c about 2hours ago won't be in them?
11:29florianhmm, I got time conversion wrong, seems that merge was at 9:13am UTC.
11:40pulsebotCheck-in: https://hg.mozilla.org/integration/mozilla-inbound/rev/daed4710bbf1 - Andrea Marchesini - Bug 1353684 - nsIFileInputStream must return NS_ERROR_NOT_FOUND if Deserialize() fails, r=smaug
11:53pulsebotCheck-in: https://hg.mozilla.org/integration/mozilla-inbound/rev/d1fe2d7d463f - Tom Tromey - Bug 1346326 - implement new console.count semantics; r=baku
11:53pulsebotCheck-in: https://hg.mozilla.org/integration/mozilla-inbound/rev/e609da5f5e89 - Andrea Marchesini - Bug 1357473 - Update Console to the latest spec - part 1 - WebIDL update, r=bgrins
11:53pulsebotCheck-in: https://hg.mozilla.org/integration/mozilla-inbound/rev/92b431e3e69b - Andrea Marchesini - Bug 1357473 - Update Console to the latest spec - part 2 - Console.clear(), r=bgrins
11:53pulsebotCheck-in: https://hg.mozilla.org/integration/mozilla-inbound/rev/71cabfed1013 - Andrea Marchesini - Bug 1357473 - Update Console to the latest spec - part 3 - Console.groupEnd(), r=bgrins, r=nchevobbe
11:53pulsebotCheck-in: https://hg.mozilla.org/integration/mozilla-inbound/rev/73ca88018958 - Andrea Marchesini - Bug 1357473 - Update Console to the latest spec - part 4 - Console.trace(), r=bgrins
11:53pulsebotCheck-in: https://hg.mozilla.org/integration/mozilla-inbound/rev/e7766e5354a3 - Andrea Marchesini - Bug 1357473 - Update Console to the latest spec - part 5 - Console.time(), r=bgrins r=smaug
11:53pulsebotCheck-in: https://hg.mozilla.org/integration/mozilla-inbound/rev/ea7bd03363f8 - Andrea Marchesini - Bug 1357473 - Update Console to the latest spec - part 6 - Console.count(), r=bgrins
11:53pulsebotCheck-in: https://hg.mozilla.org/integration/mozilla-inbound/rev/8d71c8281a83 - Andrea Marchesini - Bug 1357473 - Update Console to the latest spec - part 7 - test updated, r=bgrins
11:54aygHow can I check add-on usage for some feature? Is there a searchable database of add-on code?
11:55aygsmaug: ^ is with respect to your comment here <https://bugzilla.mozilla.org/show_bug.cgi?id=1251198#c30> asking that I check for add-on usage of rarely used events.
11:59Aryxayg: hi, do you have access to https://dxr.mozilla.org/addons/source/ ?
12:08smaugayg: there is dxr for addons
12:08smaugjust select the right repo
12:13pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/76d7e4ef5e0a - nchevobbe - Bug 1357341 - Reps 0.6.0: update reps bundle from Github; r=jdescottes
12:13pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/468405fee195 - nchevobbe - Bug 1357341 - Fix VariableView with new Reps bundle; r=jdescottes
12:21aygThanks.
12:35daleharveydao hey is there a meeting now? was in 205 and didnt see anyone
12:37daodaleharvey: PhotonProgram
12:40daodaleharvey: you might want to join #photon-visual :)
12:41daleharveyah yeh I asked dolske about that but dont think it was setup yet, cheers
12:41kaiethis is a beta try build. is it sufficiently green? https://treeherder.mozilla.org/#/jobs?repo=try&revision=aeead4cac251dd0540b9d34ca4e73c5ffbc49392
12:48aygsmaug: Looks like DataContainerEvent has 0 uses according to telemetry in CreateEvent, and also no constructor, so it seems totally unused. Is it safe to remove it entirely?
12:52smaugayg: and we don&#39;t use it anymore internally?
12:52smauga11y code used to use it
12:54smaugayg: I think surkov added the event, or needed it initially
12:55surkovsmaug: recalling something like that, but if its not in use then a11y doesnt need it anymore
12:55aygsmaug: It looks like it&#39;s only used in tests.
12:56aygsmaug: testing/talos/talos/scripts/MozillaFileLogger.js looks like the only real user.
12:57aygThat could be easily replaced with CustomEvent, I&#39;d think.
12:57aygIf it&#39;s used at all.
12:57ayg(I just don&#39;t know where the code is that receives that event)
12:59ayg(testing/mochitest/tests/SimpleTest/TestRunner.js already uses CustomEvent)
13:01armenzganyone having trouble loading expensify?
13:01jmaherayg: smaug: happy to change mozillafilelogger.js as needed
13:09Tomcat|sheriffdutyemilio: ping
13:11aygjmaher|afk: Will it work if I just change the DataContainerEvent to a CustomEvent? This means the receiver needs to check .detail instead of .getData(), which should be a trivial change, but I can&#39;t find the receiver.
13:11aygListener, I guess is the proper term?
13:11aygOr whatever.
13:17marcokanru: ping
13:21pulsebotCheck-in: https://hg.mozilla.org/integration/mozilla-inbound/pushloghtml?startID=89395&endID=89396 - 17 changesets (bugs 1353475, 1353629)
13:37Tomcat|sheriffdutyxidorn: ping
13:37Tomcat|sheriffdutyxidorn: we need a fix for https://treeherder.mozilla.org/logviewer.html#?job_id=93020571&repo=autoland i guess
13:40AryxTomcat|sheriffduty: had talked to nox about this: http://logs.glob.uno/?c=mozilla%23servo#c653787
13:41Tomcat|sheriffdutyah thanks Aryx
13:41Tomcat|sheriffdutywill joing this channel too :)
13:50emilioTomcat|sheriffduty: pong
13:50aygjgraham: When a spec and test change land upstream and I want to fix Gecko to match the new spec, what should I do about test updates? Add expected fails locally that will be overwritten on next merge? Or could I just copy the test update from upstream and it hopefully shouldn&#39;t cause trouble as long as the file contents are identical?
13:50aygMaybe if I copied the exact commit from upstream?
13:51ayg(I prefer not to have to bother generating the expected failures.)
13:51emilioTomcat|sheriffduty: looking at the failures now
13:56Tomcat|sheriffdutyemilio: thanks :)
13:57pulsebotChec