mozilla :: #developers

15 Mar 2017
00:02pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/56b09fa0748c - Jared Wein - Bug 1345375 - Use the FullZoomChange event instead of browser-fullZoom observers since FullZoomChange works on MediaDocuments. r=dao
00:13pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/d5fd5117ce4e - Daosheng Mu - Bug 1346680 - Duplicate refcount on VRLayerParent/Child when the construction; r=kip
00:19pulsebotCheck-in: https://hg.mozilla.org/mozilla-central/pushloghtml?startID=31499&endID=31500 - 25 changesets - Merge inbound to central, a=merge
00:20pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/51e13fb7f254 - Mike Hommey - Bug 1345413 - Fallback to getpwuid() info when LOGNAME is not set when creating the XRemote mutex. r=froydnj
00:21pulsebotCheck-in: https://hg.mozilla.org/integration/mozilla-inbound/rev/f24f4fdc5cc8 - Ehsan Akhgari - Bug 1346987 - Part 1: Introduce the asynchronous ContentChild::AsyncOpenAnonymousTemporaryFile() API; r=billm
00:21pulsebotCheck-in: https://hg.mozilla.org/integration/mozilla-inbound/rev/34701b9ed4ba - Ehsan Akhgari - Bug 1346987 - Part 2: Stop using the PContent::Msg_OpenAnonymousTemporaryFile sync IPC message for blob storage; r=baku
00:21pulsebotCheck-in: https://hg.mozilla.org/integration/mozilla-inbound/rev/d24fa1b4553b - Ehsan Akhgari - Bug 1346987 - Part 3: Stop using the PContent::Msg_OpenAnonymousTemporaryFile sync IPC message for media encoder; r=cpearce
00:21glandiumI wish there was a "bring the tab here" option to the awesome bar when it finds an existing tab
00:23aklotzglandium: +1
00:23pulsebotCheck-in: https://hg.mozilla.org/integration/mozilla-inbound/pushloghtml?startID=88058&endID=88059 - 72 changesets - Merge m-c to inbound, a=merge
00:24pgadigeI was advised to change the commit message for a code patch in MozReview. How could I possibly do it from MozReview? Can I at all do it from there?
00:24pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/pushloghtml?startID=38874&endID=38875 - 30 changesets - Merge m-c to autoland, a=merge
00:25aklotzpgadige: AFAIK you have to change it locally, then re-push your patch
00:26pgadigeaklotz: so I should do a `hg commit` again right?
00:26pulsebotCheck-in: https://hg.mozilla.org/integration/mozilla-inbound/rev/c26f90d92f42 - David Major - Bug 1345998: Clean up some unused libs and delayloads in xul.dll. r=glandium
00:27aklotzpgadige: Well you want to edit your previous commit. So you could do a 'hg commit --amend' (assuming that you're currently updated to the commit in question)
00:47pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/da8e14542529 - Brad Werth - Bug 1343695 Part 1: Retrieve text content with GetRenderedText. r=mats
00:47pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/9bf81510d7bc - Brad Werth - Bug 1343695 Part 2: Narrow an overly restrictive assert in GetRenderedText. r=mats
00:47pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/31129448027b - Brad Werth - Bug 1343695 Part 3: Expand tests of GetClientRectsAndTexts to test more cases where rendered text differs from DOM text. r=mats
00:56pgadigeaklotz: I could rewrite the commit message using the above command. However, I am not able to push the modified changeset to review using `hg push -r ##### review`
01:00pgadigeaklotz: I figured it out. the modifications were put in a new changeset number
01:01pgadigeaklotz: Hence, `hg push review` failed on old changeset number
01:13pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/45616bdf19cd - Xidorn Quan - Bug 1341102 - Update test expectations.
01:16sarabaisdf
01:16sarabai* sorry ignore
01:49birtlesAnyone know how to install the domFuzzLite extension these days?
01:50birtlesFirefox blocks it when I try to install it despite toggling all sorts of prefs, and just dumping the xpi in obj/tmp/scratch_user/extensions doesn't seem to do it
01:51heycambirtles: are you doing it from about:addons?
01:51birtlesheycam: no just navigating to the URL
01:51heycambirtles: try about:addons, the gear button, then Install Add-On From File
01:54birtlesheycam: thanks! giving it a try now. I think I need to tweak the xpinstall signatures pref too...
01:55pulsebotCheck-in: https://hg.mozilla.org/integration/mozilla-inbound/rev/dcc6d071833b - Eric Rahm - Bug 1344305 - Switch to Base64Decode in nsHttpNTLMAuth::GenerateCredentials. r=jduell
01:55pulsebotCheck-in: https://hg.mozilla.org/integration/mozilla-inbound/rev/828637ec7e52 - Eric Rahm - Bug 1344305 - Part 2: Update size calculation for base64 encode in nsHttpNTLMAuth::GenerateCredentials. r=jduell
01:55birtlesheycam: great, that seems to work
01:55heycambirtles: great!
01:55birtlesthanks!
02:04pulsebotCheck-in: https://hg.mozilla.org/integration/mozilla-inbound/rev/9883a566c747 - Jeff Gilbert - Bug 1346442 - Update linux64 asan builds to clang3.9+libgcc4.9.4 - r=glandium
02:18pulsebotCheck-in: https://hg.mozilla.org/integration/mozilla-inbound/rev/fa6e1037d65d - Lee Salzman - Bug 1347147 - fix DrawTargetD2D1::GetImageForSurface to support uploads in device-space. r=mchang
02:36pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/b2bf3ae75d54 - John Lin - Bug 1344649 - part 1: use picture instead of display size to construct Image. r=jya
02:36pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/5eb2d62d3c61 - John Lin - Bug 1344649 - part 2: let VideoData::CreateFromImage() accept only neccessary parameters. r=jya
02:36pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/71954a8ab450 - John Lin - Bug 1344649 - part 3: rename DurationMap and turn it into a generic class. r=jya
02:36pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/ba81bdbba7e4 - John Lin - Bug 1344649 - part 4: store frame sizes in queue rather than relying on ConfigurationChanged(). r=jya
02:37pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/fcd6c84fa5b0 - John Lin - Bug 1344649 - part 5: deprecate ConfigurationChanged() once again. r=jya
02:39pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/48869a4c04a6 - JW Wang - Bug 1346678 - Label runnables in dom/media/gmp/GMPParent.cpp and its friends. r=cpearce
02:45pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/eb0006fd631c - John Lin - Bug 1345036 - label shutdown event dispatching. r=jwwang
02:45jesupglandium: ping
02:47glandiumjesup: pong
02:47pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/2a5161a3467e - JW Wang - Bug 1346679 - Label runnables in dom/media/gmp/GMPCDMCallbackProxy.cpp. r=cpearce
02:49jesupglandium: Hi. I was trying to land a cubeb import (for logging to use in tracking down audio drifts), and the import hit a "don't use this std library function" error on Try. It's trying to use std::this_thread::sleep_for(CUBEB_LOG_BATCH_PRINT_INTERVAL_MS);
02:50jesupI was wondering if that's still an operative restriction (in case it wasn't relevant since we require c++11 now)
02:50glandiumjesup: what's the exact message?
02:50jesuphttps://treeherder.mozilla.org/logviewer.html#?job_id=83836452&repo=try&lineNumber=12755
02:50jesupTEST-UNEXPECTED-FAIL | check_stdcxx | We do not want these libstdc++ symbol versions to be used: [task 2017-03-15T00:19:31.936646Z] 00:19:31 INFO - _ZNSt11this_thread11__sleep_forENSt6chrono8durationIlSt5ratioILl1ELl1EEEENS1_IlS2_ILl1ELl1000000000EEEE@GLIBCXX_3.4.18
02:52glandiumjesup: yeah no, we still support running on systems with a libstdc++ that don't have that symbol
02:52glandiumjesup: also, everything related to time in STL needs careful scrutiny
02:53glandiumjesup: last time we had something like that it was for timeouts on conditional variables (iirc), and it turns out there was no way to use a monotonic clock
02:54pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/d1dc20f9d745 - JW Wang - Bug 1345761 - Remove the unused do_GetMainThread() call. r=gerald
02:55jesupOk, thanks. Any suggestions as to alternatives? This was the code padenot wrote that's tripping us up: https://pastebin.mozilla.org/8982073
02:57glandiuma thread just for logging? urgh
02:57glandiumimagine the horror show if every subsystem did that
02:58jesupglandium: though only if cubeb:N is enabled in MOZ_LOG. And Cubeb is pretty hard-realtime code; it needs to be able to log without waiting (or very minimal/unlikely waiting)
02:59glandiumjesup: still, it's a library.
02:59jesupIt's doing this from the audio "I need N samples now" OS callbacks on OS realtime threads
02:59pulsebotCheck-in: https://hg.mozilla.org/integration/mozilla-inbound/rev/9279db79576b - Jeff Gilbert - Bug 1347289 - Remove GL context sharing support except on EAGL - r=snorp
03:01jesupI understand the concern. It would be nice to make a unified background-logger to let realtime code safely log; this has been a problem before. the WebRTC codebase has a trace-logger that's somewhat similar, and logs to a rolling 10MB file (unless we force it to throw the log over to nspr/MOZ_LOG, which causes real perf issues)
03:02jesupSo it seems like one could code up OS-specific variations using Sleep()/nanosleep() and the like
03:02jesupto avoid std::this_thread::sleep_for
03:04pulsebotCheck-in: https://hg.mozilla.org/integration/mozilla-inbound/rev/7249e4bb0187 - Phil Ringnalda - Backed out 3 changesets (bug 1346987) for media, blob, and temporaryfile failures
03:11pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/d2de79d5c39e - JW Wang - Bug 1345376 - Label runnables in dom/media/mediasink/DecodedStream.cpp. r=kikuo,pehrsons
03:16pulsebotCheck-in: https://hg.mozilla.org/integration/mozilla-inbound/rev/16f71512d587 - Boris Zbarsky - Bug 1346389. Make --enable-shared-js link again, at least for an opt mac build with intl api disabled. r=sfink, a=waldo on the gcc-specific bits.
03:16pulsebotCheck-in: https://hg.mozilla.org/integration/mozilla-inbound/rev/101bf6f61074 - Boris Zbarsky - Bug 1276186. Stop using the locale-sensitive AppendPrintf in code that is supposed to always use '.' as a decimal separator. r=dbaron
03:22pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/ae9270e12577 - Ting-Yu Lin - Bug 1273045 Part 1 - Convert UpdateCaretsHint to use EnumSet. r=mtseng
03:22pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/62ab4dd84f7e - Ting-Yu Lin - Bug 1273045 Part 2 - Update carets when scrolling in subframes without APZ. r=mtseng
03:27pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/ae9270e12577 - Ting-Yu Lin - Bug 1273045 Part 1 - Convert UpdateCaretsHint to use EnumSet. r=mtseng
03:27pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/62ab4dd84f7e - Ting-Yu Lin - Bug 1273045 Part 2 - Update carets when scrolling in subframes without APZ. r=mtseng
03:55njnugh, what is this enormous JSON blob I keep getting spewed in my terminal: "console.log: deployment-checker results"
03:57njnbug 1340617, apparently
03:58njnah, bug 1347217 is specifically about the stuff
03:59njnand it's already been fixed on autoland, hooray
04:17bznjn: the fact that console.log in these addon sdk things dumps to stdout is nuts. :(
04:17njnbz: that does sound odd, yes
04:18bznjn: and it's done so carefully....
04:23pulsebotCheck-in: https://hg.mozilla.org/integration/mozilla-inbound/rev/6ae33b63f3a2 - Nicholas Nethercote - Bug 1345262 (part 1) - Remove unnecessary assertion. r=jseward.
04:23pulsebotCheck-in: https://hg.mozilla.org/integration/mozilla-inbound/rev/6906fb64de7d - Nicholas Nethercote - Bug 1345262 (part 2) - Add profiler_{set,clear}_js_context(). r=mstange.
04:23pulsebotCheck-in: https://hg.mozilla.org/integration/mozilla-inbound/rev/7414c13a82a1 - Nicholas Nethercote - Bug 1345262 (part 3) - Split PseudoStack::sampleContext() in two. r=mstange.
04:23pulsebotCheck-in: https://hg.mozilla.org/integration/mozilla-inbound/rev/eed2dd8ea13f - Nicholas Nethercote - Bug 1345262 (part 4) - Remove and inline PseudoStack::{clearContext,flushSamplerOnJSShutdown}(). r=mstange.
04:23pulsebotCheck-in: https://hg.mozilla.org/integration/mozilla-inbound/rev/da8e7dc470c8 - Nicholas Nethercote - Bug 1345262 (part 5) - Fix how JS sampling is started/stopped by the profiler. r=mstange,djvj.
04:28pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/ec3fa55ae9e5 - Markus Stange - Bug 1341924 - Don't attempt to dispatch CheckResponsivenessTasks to non-main threads. r=njn
04:29njnmstange: with the above patches landed, I think we have fixed all the crashes/assertion failures I have been hitting frequently
04:29mstangenjn: this makes me so happy
04:30njn:)
04:30mstangenjn: we can finally change the thread selection UI so that it becomes more obvious how to profile all threads
04:30njnmstange: because it won't crash immediately if you do? :)
04:31mstangenjn: I think we were lucky that the comma trick is so non-obvious
04:31mstangeyes
04:31njnheh
04:31njnmstange: I still have a bunch more clean-ups on my todo lis
04:31njn*list
04:31njnthough I'm going to need to work on Stylo memory reporting for a bit pretty soon
04:31mstangeoh ok
04:31bznjn++
04:31* bz wants to see those numbers
04:31njnbz: don't increment me until I do it!
04:33njnmstange: we need tests for worker threads
04:34mstangenjn: yeah
04:34njnmstange: I have almost no idea about the tests, so I don't know how best to do that, even which test suite would be suitable
04:35mstangenjn: I'm looking at tools/profiler/tests/test_asm.js at the moment
04:35mstangenjn: something similar could work
04:35njnok
04:35njnyou can create workers in xpcshell?
04:35mstangeoh
04:35mstangethat I don't know
04:36njnthe skip-ifs in tools/profiler/tests/xpcshell.ini aren't confidence-inspiring
04:36mstangeheh
04:37mstangeI hadn't seen those
04:39bznjn: I'll increment again when you do it. ;)
04:39njnit's a deal!
04:39* njn wonders what other good things he can promise to do
04:43pulsebotCheck-in: https://hg.mozilla.org/integration/mozilla-inbound/rev/7012620c3aee - Cervantes Yu - Backed out changeset 5fd991c5f120 (bug 1320134) for crashing webgl on Windows. r=backout
04:43pulsebotCheck-in: https://hg.mozilla.org/integration/mozilla-inbound/rev/5389367569cc - Cervantes Yu - Backed out changeset e91e04dadeb3 (bug 1320134) for crashing webgl on Windows. r=backout
04:43pulsebotCheck-in: https://hg.mozilla.org/integration/mozilla-inbound/rev/ef6728e3e136 - Cervantes Yu - Backed out changeset 9ba65a580135 (bug 1320134) for crashing webgl on Windows. r=backout
04:51mstangenjn: http://searchfox.org/mozilla-central/source/dom/workers/test/xpcshell/test_workers.js shows how to use a worker in an xpcshell test
04:52njnnice
05:24pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/pushloghtml?startID=38885&endID=38886 - 1 changesets - servo: Merge #15949 - Don't serialize url() twice (from upsuper:filter-url); r=heycam
05:35pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/cbbd3f3e6246 - Nico Grunbaum - Bug 1343691 - fix missing rtcp stats;r=jib
06:05pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/2672f4d4b38d - Masayuki Nakano - Bug 1346499 Don't remove Ctrl nor Alt modifier state at dispatching eKeyPress event when the modifier doesn't change inputting character r=m_kato
06:35pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/e511e1218d7f - Astley Chen - Bug 1342880 : Remove class inSearchLoop from the tree. r=heycam
06:47ihsiao|sheriffdutyxidorn: heycam could you help to check the failure like https://treeherder.mozilla.org/logviewer.html#?job_id=83913246&repo=autoland&lineNumber=1826
06:48xidornihsiao|sheriffduty: will have a look
06:48ihsiao|sheriffdutythanks
06:48xidornoh, I forgot to update the expectation :/
06:50pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/b41b2d89ff02 - Xidorn Quan - Bug 1341102 - Update stylo test expectation.
06:50xidornihsiao|sheriffduty: done ^
06:50ihsiao|sheriffdutyxidorn: ++
07:41pulsebotCheck-in: https://hg.mozilla.org/integration/mozilla-inbound/rev/749de78b4831 - Michal Novotny - Bug 1342366 - Crash in nsWyciwygChannel::GetCharsetAndSource, r=honzab
07:42pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/8c0e4e52f376 - JW Wang - Bug 1344772 - set a dirty flag so we can process the notification later. r=jya
07:45pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/110bf387f390 - Jean-Yves Avenard - Bug 1346463: Disable test on windows 8 for now. r=gerald
07:46pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/22f89dff7a80 - Jean-Yves Avenard - Bug 1342913: P1. Add mochitest. r=gerald
07:46pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/cdf4db6ebcaf - Jean-Yves Avenard - Bug 1342913: P2. Terminate draining operations when possible. r=gerald
07:48pulsebotCheck-in: https://hg.mozilla.org/integration/mozilla-inbound/rev/a23edcf5b82a - Timothy Nikkel - Bug 1343341. Create state on the AnimationState object to track whether the image is decoded or not. r=aosmond
07:48pulsebotCheck-in: https://hg.mozilla.org/integration/mozilla-inbound/rev/29e2e6dd82d2 - Timothy Nikkel - Bug 1343341. Only set the animation state as discarded if we discarded the animated frames (and not a static frame from a first-frame-only decode). r=aosmond
08:06pulsebotCheck-in: https://hg.mozilla.org/integration/mozilla-inbound/rev/83eb15651f24 - Paavini Nanda - Bug 1344840 - Enable flake8 rule E231: "missing whitespace after ','". r=dexter
08:06pulsebotCheck-in: https://hg.mozilla.org/integration/mozilla-inbound/rev/267e93a16cfe - Paavini Nanda - Bug 1344842 - Enable flake8 rule E261: "at least two spaces before inline comment". r=dexter
08:06pulsebotCheck-in: https://hg.mozilla.org/integration/mozilla-inbound/rev/5a8529ec2ec2 - Paavini Nanda - Bug 1344854 - Enable flake8 rule F841: "local variable XXX is assigned to but never used". r=dexter
08:09heycamdid the J and K keybindings in treeherder change recently?
08:09heycamseems like they're opposite to what I expect
08:11KWierso|afkjgraham: is that from the autoclassify keybindings changing?
08:12KWierso|afksure looks like it from the items listed when I hit '?'
08:13heycamKWierso|afk: oh, heh
08:13heycamKWierso|afk: guess I should file a bug
08:13KWierso|afkjgraham: I will note that j/k are now listed twice, and they do have the opposite meanings
08:14pulsebotCheck-in: https://hg.mozilla.org/integration/mozilla-inbound/rev/0394f4a6e9e9 - Timothy Nikkel - Backed out changeset 29e2e6dd82d2 (bug 1343341)
08:14pulsebotCheck-in: https://hg.mozilla.org/integration/mozilla-inbound/rev/a22dff44c2ab - Timothy Nikkel - Backed out changeset a23edcf5b82a (bug 1343341)
08:15heycamKWierso|afk: jgraham: filed https://bugzilla.mozilla.org/show_bug.cgi?id=1347454
08:21KWierso|afkheycam: n/p are the same as they've always been :)
08:29pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/a6c7a9fcfc29 - Federico Padua - Bug 1344841 - Enable flake8 rule E111: "indentation is not a multiple of four". r=Dexter
08:30heycamKWierso|afk: they are unfortunately not as ingrained in my brain. I could probably even deal with them swapping. :)
08:31Tomcat|sheriffdutyheh
09:03pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/ffb8aa89b8c1 - Henry Chang - Bug 1345922 - Avoid concurrent update and take the failed beginUpdate into account. r=francois
09:14daoTomcat|sheriffduty: could you please back out bug 1345375 from autoland?
09:22Tomcat|sheriffdutydao: ok will do :)
09:24pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/0cc9dced786c - Carsten "Tomcat" Book - Backed out changeset 56b09fa0748c (bug 1345375) on request from dao
09:24daoTomcat|sheriffduty: thanks!
09:29Tomcat|sheriffdutynp
09:37pulsebotCheck-in: https://hg.mozilla.org/integration/mozilla-inbound/rev/6628c9503e36 - Iris Hsiao - Backed out changeset 4f5a93b89b2b (bug 1344892) for Mochitest rc4 failures in Android
09:48pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/dd772ad610d1 - Madeleine Chercover - Bug 1347241 - Enable linux64-ccov coverage collection for common tests set; r=gmierz,jmaher
10:23pulsebotCheck-in: https://hg.mozilla.org/releases/mozilla-beta/rev/b49638d1ba54 - ffxbld - No bug, Automated blocklist update from host bld-linux64-spot-036 - a=blocklist-update
10:34pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/487f495d8d1d - clinton - Bug 1346208 - Remove B2G-specific code from Telemetry C++ modules r=gfritzsche
10:42pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/pushloghtml?startID=38898&endID=38899 - 1 changesets - servo: Merge #15821 - Implement ImageData constructors #15671 -- 1 (from montrivo:ImDataConstructors); r=Ms2ger
11:09pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/90bb7008ea76 - cku - Bug 1331834 - Put svg/reftest.list back into alphabetical order. r=longsonr+218550
11:24pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/a5a2ead687cb - Jessica Jong - Bug 1301312 - Part 1: Localize AM/PM strings for <input type=time>. r=gandalf,mconley
11:24pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/d5be3275c987 - Jessica Jong - Bug 1301312 - Part 2: Localize placeholders for <input type=time>. r=mconley
11:24pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/66b7075919c9 - Jessica Jong - Bug 1301312 - Part 3: Order fields in <input type=time> based on locale datetime format. r=gandalf,mconley
11:24pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/8fda74ef8da6 - Jessica Jong - Bug 1301312 - Part 4: Order fields in <input type=date> based on locale datetime format. r=gandalf,mconley
11:24pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/6d303d1651cc - Jessica Jong - Bug 1301312 - Part 5: Handle input element&#39;s attribute change explicitly. r=smaug
11:24pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/f7605a47342d - steveck-chung - Bug 1338420 - Part 1: Add initialProcessData autofillSavedFieldNames set and broadcast the changes to content. r=MattN
11:24pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/80a584704a04 - steveck-chung - Bug 1338420 - Part 2: Apply form history search at startSearch and clean up redundant code. r=MattN
11:26pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/66bb0977f0ab - Prathiksha - Bug 1344713 - Replaced the waitForCertErrorLoad function in browser_aboutCertError.js with the more reliable BrowserTestUtils.waitForErrorPage function. r=johannh
11:31pulsebotCheck-in: https://hg.mozilla.org/integration/mozilla-inbound/rev/9e19d4c92575 - Jessica Jong - Bug 1346084 - Expose mozIntl.getLocaleInfo to ChromeOrXBL. r=smaug
11:31pulsebotCheck-in: https://hg.mozilla.org/integration/mozilla-inbound/rev/1a600f0ea3fb - Evelyn Hung - Bug 1347117 - Add Activate/Deactivate/ParentActivated to PContentBridge; r=billm
11:31pulsebotCheck-in: https://hg.mozilla.org/integration/mozilla-inbound/rev/8c704b823d38 - Tomislav Jovanovic - Bug 1319070 - Test matching against the principal. r=kmag
11:40AutomatedTesterbz_sleep: hey, I see that you reopened a bunch of b2g bugs. Is there a reason?
11:43KwanAutomatedTester: maybe related to http://logs.glob.uno/?a=search&c=mozilla%23developers&q=martijn&ss=13+Mar+2017&se=
11:44pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/3cc331de51d8 - flyingrub - Bug 1346739 - Update aboutTelemetry.js to async fucntion & await r=Dexter
11:44AutomatedTesterKwan: looks like its related
11:44* AutomatedTester can close some of them again as they were valid for closing
12:00pulsebotCheck-in: https://hg.mozilla.org/integration/mozilla-inbound/rev/fc94903b628a - Gabor Krizsanits - Bug 1341008 - Stop rebroadcasting BlobURLUnregistration. r=baku
12:00pulsebotCheck-in: https://hg.mozilla.org/integration/mozilla-inbound/rev/26389c6e73ff - Gabor Krizsanits - Bug 1341008 - Use the preallocated process manager by default. r=billm, r=kmag
12:01pulsebotCheck-in: https://hg.mozilla.org/mozilla-central/pushloghtml?startID=31500&endID=31501 - 46 changesets (bugs 1287660, 1345611, 1346381, 1347269, 1341102 and 28 other bugs)
12:06pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/90e25222129c - Gijs Kruitbosch - Bug 1344759 - add a unit test for Chrome bookmark imports, r=dao
12:06pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/ce47c3bc026a - Gijs Kruitbosch - Bug 1344759 - batch-insert bookmarks when importing from Edge, IE, Safari or Chrome, r=dao
12:12AryxAutomatedTester: https://bugzilla.mozilla.org/show_bug.cgi?id=497575#c9
12:12AutomatedTesterAryx: aha
12:12AutomatedTesterAryx: thanks!
12:13AutomatedTesterbz_sleep: ^ Aryx explained it all
12:13pulsebotCheck-in: https://hg.mozilla.org/mozilla-central/pushloghtml?startID=31501&endID=31502 - 73 changesets - merge mozilla-inbound to mozilla-central a=merge
12:13pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/pushloghtml?startID=38905&endID=38906 - 1 changesets - servo: Merge #15961 - Implement access to CSSNamespaceRule for stylo (from upsuper:bug1345698); r=heycam
12:13AutomatedTestersome of them are valid for closing but clearly not all
12:14pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/52ba7a56ee83 - Xidorn Quan - Bug 1345698 part 1 - Unify common code for implementing a rule type. r=heycam
12:14pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/7caba57b2d1b - Xidorn Quan - Bug 1345698 part 2 - Add bindings for namespace rule. r=heycam
12:14pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/fb56873eb923 - Xidorn Quan - Bug 1345698 part 3 - Fix a build failure for adding source. r=heycam
12:14pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/0ee93f46f578 - Xidorn Quan - Bug 1345698 part 4 - Create a new CSSNamespaceRule as a base class. r=heycam
12:14pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/3123f116d6dc - Xidorn Quan - Bug 1345698 part 5 - Add support of CSSNamespaceRule for stylo. r=heycam
12:18pulsebotCheck-in: https://hg.mozilla.org/integration/mozilla-inbound/pushloghtml?startID=88075&endID=88076 - 48 changesets - Merge mozilla-central to mozilla-inbound
12:22pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/pushloghtml?startID=38907&endID=38908 - 74 changesets - Merge mozilla-central to autoland
12:25pulsebotCheck-in: https://hg.mozilla.org/integration/mozilla-inbound/rev/c20e214d6486 - Kartikaya Gupta - Bug 1316318 - Disable APZ scrolling in subframes if the root document has visible anonymous content. r=mstange
12:25pulsebotCheck-in: https://hg.mozilla.org/integration/mozilla-inbound/rev/0e09b0ea4873 - Kartikaya Gupta - Bug 1316318 - Disable paint-skipping for elements where we disable APZ. r=mstange
12:37pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/9b6d00fa6435 - cku - Bug 1320036 - Part 1. Correct objectBoundingBox region of a filter applied to an outer SVG element. r=jwatt
12:37pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/eb2013d935b2 - cku - Bug 1320036 - Part 2. Reftest. r=jwatt
12:43Aryxgabor: hi, mass reftest failures https://treeherder.mozilla.org/#/jobs?repo=mozilla-inbound&revision=26389c6e73ff8fbfca87d5246ff3dd08cb41088b&filter-resultStatus=testfailed&filter-resultStatus=busted&filter-resultStatus=exception&filter-resultStatus=retry&filter-resultStatus=usercancel&filter-resultStatus=runnable
12:44gaborAryx: whoops... I think I forgot to run those on try :(
12:45Aryxshall i back out or do you think that&#39;s a quick, easy and safe fix?
12:46gaborAryx: Could you back it out please? I need to investigate this a bit... seems like it&#39;s the same failure for all, but I have no clue what&#39;s going on
12:46Aryxok
12:48pulsebotCheck-in: https://hg.mozilla.org/integration/mozilla-inbound/rev/121622840147 - Sebastian Hengst - Backed out changeset 26389c6e73ff (bug 1341008)
12:48pulsebotCheck-in: https://hg.mozilla.org/integration/mozilla-inbound/rev/0d5375cf0255 - Sebastian Hengst - Backed out changeset fc94903b628a (bug 1341008) for reftest failures on debug after asserting at nsLayoutStylesheetCache.cpp:253. r=backout
12:51Fallen|ptoHow is a <browser> element restricted w.r.t. content that can be loaded? I have two browser elements created by different parts of the code, one can load the url it should, the other cannot. Trying to figure out what is restricting the load
12:51Fallen|ptoboth are type=&quot;content&quot;
12:53Aryxanything in the browser console?
12:53Fallen|ptoAryx: nothing aside from the throwing loadURI call with NS_ERROR_CONTENT_BLOCKED
12:54Fallen|ptoThe failing one is created using https://dxr.mozilla.org/comm-central/source/mozilla/toolkit/components/extensions/ExtensionParent.jsm#678
12:54Fallen|ptoMaybe it has to do with the windowless browser thing
12:55Fallen|ptoThe succeeding one is created through Thunderbird&#39;s tabmail.xml
13:05pulsebotCheck-in: https://hg.mozilla.org/integration/mozilla-inbound/rev/323c768fdc43 - Tom Schuster - Bug 1331136 - Handle more StringChar cases with ropes in CacheIR. r=jandem
13:07Fallen|ptoI don&#39;t see anything that would make the windowless browser special though
13:25Fallen|ptoyep, that seems to be it. If I swap out the chrome doc then it works
13:34katstill: pong
13:36tillkats: hey, I encountered a pretty strange issue where interactions with gmail become substantially slower if I play video in the background in a specific application. Profiling shows that to be related to layer building operations, where some GL operations start taking 10x as long as without the video playing. Is that something worth filing, or do we expect
13:36tillit to go away with WebRender anyway?
13:37tillkats: I should note that I wasn&#39;t able to reproduce it when playing, say, a youtube video in the same or other browsers
13:37katstill: it&#39;s probably worth filing. depending on where in layer building it is, it might or might not be fixed by webrender
13:38katstill: although it sounds like that specific application is probably using GPU resources and therefore slowing down firefox?
13:38katstill: does the same problem appear if you use a different browser with the same background application playing video?
13:39tillkats: that&#39;s probably the case, but it doesn&#39;t cause a full GPU load or anything
13:39tillkats: no, I tested Chrome and it stays fast
13:40katstill: ok, please file something then. you might want to cc ehsan since he might be interested in such performance cliffs as well
13:41tillkats: here&#39;s a profile, fwiw https://perfht.ml/2mJg8XO
13:41tillkats: will file
13:41katstill: thx
13:42tillkats: I tried it with your WR-enabled try builds, but the tab hangs when going to gmail, so it&#39;s clearly too early for that
13:43katsi guess so. i&#39;m surprised that it hangs though, i&#39;ll look into that
13:43pulsebotCheck-in: https://hg.mozilla.org/integration/mozilla-inbound/rev/9ed72c2f16d8 - Stanford Lockhart - Bug 940275 - Part 1: Add positioning properties to box model r=gl
13:45tillkats: I can give you a profile of that, too, if that&#39;s interesting. Navigating the tab works, so capturing a profile should, too, I&#39;d hope
13:45katstill: if you have one handy, sure
13:46katstill: if you don&#39;t have one handy then don&#39;t worry about it
13:46tillkats: ok, never mind, then
13:58jyaanyone got central to build with visual studio 2017?
14:00froydnjjya: dmajor, maybe?
14:00pulsebotCheck-in: https://hg.mozilla.org/releases/mozilla-beta/rev/6825cf3f953f - L10n Bumper Bot - Bumping Fennec l10n changesets a=l10n-bump
14:01jyaguess I have to install VS 2015 on this new box.. oh well
14:02gcpthere&#39;s probably a bug...somewhere...about building with it
14:03Aryxbug 1318143
14:03jcristauhttps://bugzilla.mozilla.org/show_bug.cgi?id=vs2017
14:03jcristau:)
14:04Aryxm_kato might have built with it
14:05jyaBas told me that he built with it.. I just don&#39;t see how to start the shell with preconfigured link to vs2017
14:05BasI gave RyanVM a patch to MozBuild
14:06jdmso much empty space on the sides of bugzilla now
14:06BasNot sure if it got checked in or anything, I can give it to anyone who wants it.
14:10jyaBas: you have a script to start the shell with VS2017 ?
14:10Basjya: That is what my patch is, yes :)
14:10BasOr rather, includes.
14:10jyaso then yes I&#39;d like to get it :)
14:11jyaanyone knows how to force reinstall the rust and cargo bit ?
14:11jyai keep getting an error about std-rustc not found
14:11jyabut when I try to install it it tells me it&#39;s already installed
14:11* jya hate those builder
14:13Basjya: pastebin.mozilla.org/8982105
14:18pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/pushloghtml?startID=38909&endID=38910 - 1 changesets - servo: Merge #15956 - Fix -moz-user-select: tri-state (from upsuper:user-select-tri-state); r=heycam
14:19pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/e0a87984f613 - Federico Padua - Bug 1344832 - Enable flake8 rule E502: &quot;the backslash is redundant between brackets&quot;. r+dexter r=Dexter
14:21katsmhoye: you may want to step in on bug 1345661, it&#39;s getting kinda nuts
14:26pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/8a30570bc738 - Andi-Bogdan Postelnicu - Bug 1346725 - removed useless null check in DoSampleStackTrace. r=mstange
14:26gcpaint that just a dupe? :P
14:28gcpof bug 1247056
14:29pulsebotCheck-in: https://hg.mozilla.org/integration/mozilla-inbound/rev/fd8e7d9053dc - Dylan Roeh - Bug 1305815 - Fix false positives for x86 devices. r=snorp
14:33katsgcp: no, it&#39;s the blowback from that bug
14:34gcpyeah entitled people whining, nothing is new
14:34katsi guess we can just let it run its course
14:34katsat least it&#39;s contained to that one bug
14:35pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/a654b91969e4 - Sylvestre Ledru - Bug 1347474 - Update mach to download clang-format 5.0 r297730 r=andi
14:36katsted: so, there don&#39;t appear to be any other short-term workarounds for bug 1343625. should we land it?
14:36pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/753843fed8a9 - Sylvestre Ledru - Bug 1346737 - Use a const in the for loop declaration to avoid copy in nsGlobalWindow.cpp r=Ehsan
14:38TheOneTomcat|sheriffduty: hi. Does https://bugzilla.mozilla.org/show_bug.cgi?id=1346381 need anything to be uplifted to aurora?
14:39TheOneit already got approval, so I am wondering whether there is another step
14:39freddybTheOne: there is no other step if the patch applies cleanly
14:40TheOnefreddyb: ok thanks. I&#39;ll just be patient then :)
14:40freddybTheOne: if it doesnt, rewrite the patch so it applies, upload and carry-over approval
14:40TheOneit&#39;s not my patch, I just need it fixed on aurora :)
14:41freddybdepending on your patience, you may still do the rebase >:D
14:42TheOnemy experience with hg and mozilla-central/aurora is limited
14:42TheOneso there is a high chance of me messing up the rebase
14:45AryxSylvestre: bustage https://treeherder.mozilla.org/logviewer.html#?job_id=84029226&repo=autoland
14:47Aryxbacking it out fyi
14:48pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/b982b56e5668 - Sebastian Hengst - Backed out changeset 753843fed8a9 (bug 1346737) for build bustage. r=backout on a CLOSED TREE
14:48mhoyekats: thanks for the heads up, looking now.
14:57Tomcat|sheriffdutylooking at it
15:04Tomcat|sheriffdutyTheOne: working on uplift, just need to finish a bug comment
15:04Tomcat|sheriffduty:)
15:05TheOneTomcat|sheriffduty++
15:08RyanVMcpeterson: that plugin issue I was mentioning yesterday reproduces with async drawing off too on Beta (I can reproduce it too). That said, 64-bit aurora builds appear to be fine
15:09pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/6e99a3ae690a - Rex Lee - Bug 1287012 - [mortar][PDF] Enable select all feature. f=lchang, r=evelyn
15:11pulsebotCheck-in: https://hg.mozilla.org/releases/mozilla-esr52/rev/e1dcaa13cd12 - ffxbld - No bug, Automated HSTS preload list update from host bld-linux64-spot-034 - a=hsts-update
15:11pulsebotCheck-in: https://hg.mozilla.org/releases/mozilla-esr52/rev/0a1eb075263a - ffxbld - No bug, Automated HPKP preload list update from host bld-linux64-spot-034 - a=hpkp-update
15:11pulsebotCheck-in: https://hg.mozilla.org/releases/mozilla-esr52/rev/8d77a0620e68 - ffxbld - No bug, Automated blocklist update from host bld-linux64-spot-034 - a=blocklist-update
15:12pulsebotCheck-in: https://hg.mozilla.org/mozilla-central/rev/f2d764f52f32 - ffxbld - No bug, Automated HSTS preload list update from host bld-linux64-spot-038 - a=hsts-update
15:12pulsebotCheck-in: https://hg.mozilla.org/mozilla-central/rev/58414045173f - ffxbld - No bug, Automated HPKP preload list update from host bld-linux64-spot-038 - a=hpkp-update
15:12pulsebotCheck-in: https://hg.moz