00:16:01 <jya> What is the git cinnabar github compatible repo again? 
00:18:25 <heycam> jya: see https://github.com/glandium/git-cinnabar/wiki/Mozilla:-A-git-workflow-for-Gecko-development
00:19:36 <heycam> jya: or https://wiki.mozilla.org/GitHub/Gecko_Repositories maybe
00:19:54 <jya> heycam: yeah, I know that one, don't want to check out with cinnabar though. 
00:20:06 <jya> It always times out for me on big pull. 
00:20:21 <hiro> I am seeing "A web page is slowing down your browser." message repeatedly, maybe it's the cause of the irresponsibility.
00:20:41 <heycam> jya: the instructions in https://github.com/glandium/git-cinnabar/wiki/Mozilla:-A-git-workflow-for-Gecko-development mention a faster way to do an actual cinnabar checkout
00:21:20 <jya> glandium gave me that work around. Basically pull from the repo. 
00:22:05 <jya> If I have more than 10000 commits to pull, it will time-out for me, regardless of the machine I use and the internet connection I'm on, whichever location I may be. 
00:23:03 * Mossop wonders if you can clone the hg repo locally then git clone from that
00:24:18 <jya> The other advantage of pulling from. The cinnabar compatible clone, is that I don't have to install and configure cinnabar. I'll do so automatically when I'll first run mach bootstrap 
00:35:39 <glandium> jya: it shouldn't time out anymore
00:36:20 <glandium> that was due to changes to a file that is not in the tree anymore
00:36:53 <glandium> jya: also the new faster way clones from github directly
00:38:24 <jya> glandium: it did just yesterday as I was fetching release
00:38:46 <jya> the git -c cinnabar.clone=https://github.com/glandium/gecko clone hg://hg.mozilla .org/mozilla-unified mozilla-central && cd mozilla-central way?
00:39:06 <glandium> jya: you'd need to give more details
00:39:08 <glandium> jya: yes
00:39:36 <jya> also, I pulled on my machine that hadn't been updated since December, there was 12000 commits , always the same error, so in the end i do it 3000 commits at the time
00:40:06 <jya> I mean the error are always the same and haven't changed for the time I've been experiencing those issues. Now I've changed country and it's still happening :(
00:40:45 <jya> cloning with the new way. i'm maxing out my link which is cool
00:41:28 <glandium> jya: the file was removed recently-ish, so it's plausible that your tree was just too old
00:42:00 <jya> why would it fail only with big pull, but succeed when doing 3000 at a time?
00:43:03 <glandium> jya: because it's a function of the number of changes to that one file
00:43:54 <jya> how much overhead does that add cloning the two at once like that ?
00:44:27 <glandium> cloning the two?
00:44:55 <jya> i mean using the cinnabar clone from your repo, plus the unified ?
00:45:15 <jya> (don't know the advantage of using unified over central)
00:45:32 <glandium> unified has all the branches, including esr, release, and beta
00:46:56 <jya> ah cool
00:47:05 <jya> though I just had all the repo in my .git/config set up
00:47:11 <jya> so could pull that way
00:54:36 <pulsebot> Check-in: https://hg.mozilla.org/integration/mozilla-inbound/rev/326199603b5d - Chris Pearce - Bug 1525813 - Ensure ipc::ShmemCreated message serializes shmem size as uint32_t. r=jld
01:09:14 <pulsebot> Check-in: https://hg.mozilla.org/integration/autoland/rev/760191b55ab3 - Tim Nguyen - Bug 1525739 - Remove XUL grid layout from browser/base/content/sanitize.xul. r=Gijs
01:09:57 <pulsebot> Check-in: https://hg.mozilla.org/integration/autoland/rev/d599d1a73a3a - Tim Nguyen - Bug 1525747 - Remove XUL grid layout from browser/components/preferences/sanitize.xul. r=Gijs
01:10:50 <pulsebot> Check-in: https://hg.mozilla.org/integration/autoland/rev/954c08e7350f - Mike Hommey - Bug 1525196 - Apply https://reviews.llvm.org/D57636 to clang r=dmajor
01:11:42 <pulsebot> Check-in: https://hg.mozilla.org/integration/autoland/rev/a2336e360977 - Ting-Yu Lin - Bug 1524893 - Prevent Label and Summary elements from handling click event if clicking on interactive html content. r=smaug
01:11:45 <pulsebot> Check-in: https://hg.mozilla.org/integration/autoland/rev/4d0958fba8db - Ting-Yu Lin - Bug 1524893 - Override IsInteractiveHTMLContent() for HTMLDetailsElement. r=smaug
01:12:45 <jya> glandium: clone is finished, this is the fastest pull I've ever done, it usually ran in multiple amount of hours
02:25:48 <pulsebot> Check-in: https://hg.mozilla.org/integration/mozilla-inbound/rev/4c44d5135282 - Andreea Pavel - Bug 1523562 - update wpt expectations r=emilio
02:38:46 <pulsebot> Check-in: https://hg.mozilla.org/integration/autoland/rev/1f1e882ed243 - Matt Woodrow - Bug 1524117 - Skip mix-blend mode operators in WebRender if we're sure they won't have any effect. r=gw
02:38:49 <pulsebot> Check-in: https://hg.mozilla.org/integration/autoland/rev/4224ec9c27d3 - Matt Woodrow - Bug 1524117 - Remove special handling for mix-blend modes on the root WebRender stacking context, since Gecko handles this already. r=gw
02:42:23 <pulsebot> Check-in: https://hg.mozilla.org/integration/autoland/rev/94ab0cf7c69f - Mike Hommey - Bug 1525510 - Add a trampoline to call original_init when it's not possible directly. r=froydnj
02:43:06 <pulsebot> Check-in: https://hg.mozilla.org/integration/autoland/rev/db429d8a3715 - Mark Hammond - Bug 1522381 - don't rely on Services.startup.getStartupInfo().main for telemetry session start. r=tcsc
02:43:46 <pulsebot> Check-in: https://hg.mozilla.org/integration/autoland/rev/6886f5504885 - Daniel Holbert - Bug 1526057: Adjust a test to remove reference to no-longer-recognized devtools fonteditor pref. r=gl
02:44:29 <pulsebot> Check-in: https://hg.mozilla.org/integration/autoland/rev/45fe8012bb58 - Tooru Fujisawa - Bug 1519037 - Filter out objects that is about to be finalized when iterating for non-GC purpose. r=pbone
02:45:15 <pulsebot> Check-in: https://hg.mozilla.org/integration/autoland/rev/b804c63247f5 - Adrian Wielgosik - Bug 1492629 - de-COM mozJSComponentLoader. r=mccr8
02:45:18 <pulsebot> Check-in: https://hg.mozilla.org/integration/autoland/rev/828188fd3203 - Adrian Wielgosik - Bug 1492629 - Drop ShutdownLoaders phase, move only user to ShutdownFinal. r=mccr8
02:46:05 <pulsebot> Check-in: https://hg.mozilla.org/integration/autoland/rev/2e1a490ab0e9 - Robert Strong - Bug 1525812 - Specify the correct directory for reading update-settings.ini when staging updates in updater.cpp. r=mhowell
02:46:55 <pulsebot> Check-in: https://hg.mozilla.org/integration/autoland/rev/ba9e0f7f49a3 - Andrew Halberstadt - Bug 1515746 - [lint] Remove duplicate flake8 implementation, r=egao
02:46:58 <pulsebot> Check-in: https://hg.mozilla.org/integration/autoland/rev/3c7c50fba283 - Andrew Halberstadt - Bug 1515746 - [flake8] Upgrade flake8 and dependencies, r=egao
02:47:01 <pulsebot> Check-in: https://hg.mozilla.org/integration/autoland/rev/f96c1460ffc0 - Andrew Halberstadt - Bug 1515746 - [flake8] Unsupport subdir .flake8 files and use new 'per-file-ignores' config instead, r=egao
02:47:37 <pulsebot> Check-in: https://hg.mozilla.org/integration/autoland/rev/933722dee47b - Liang-Heng Chen - Bug 1504157 - make onsize event triggered once; r=smaug
02:48:34 <pulsebot> Check-in: https://hg.mozilla.org/integration/autoland/rev/0b8097689bb5 - Andrew Halberstadt - Bug 1519472 - [taskgraph] Factor logic for adding a cache in job.common to a new function, r=tomprince
02:48:37 <pulsebot> Check-in: https://hg.mozilla.org/integration/autoland/rev/b6e19a5b0ab9 - Andrew Halberstadt - Bug 1519472 - [ci] Opt out of caching for generic-worker based Windows builds, r=tomprince
02:48:40 <pulsebot> Check-in: https://hg.mozilla.org/integration/autoland/rev/2ceeee1915ae - Andrew Halberstadt - Bug 1519472 - [taskgraph] Support generic-worker caches in run_task, r=tomprince
02:49:16 <pulsebot> Check-in: https://hg.mozilla.org/integration/autoland/rev/c7a0af90beb9 - Andrew Swan - Bug 1524325 Remove AddonEvent.needsRestart r=bzbarsky,kmag
02:50:06 <pulsebot> Check-in: https://hg.mozilla.org/integration/autoland/rev/17c5a7677d9f - Jan Henning - Bug 1496684 - Dispatch commonly expected startup notifications when opening a GeckoView window. r=snorp
02:50:52 <pulsebot> Check-in: https://hg.mozilla.org/integration/autoland/rev/0e6997e8f365 - Tom Prince - Bug 1525987: [taskgraph] Allow docker images to not be cached; r=dustin
02:50:55 <pulsebot> Check-in: https://hg.mozilla.org/integration/autoland/rev/0b0101621dcf - Tom Prince - Bug 1525987: [taskgraph] Allow docker images to be indexed as build products; r=dustin
02:51:34 <pulsebot> Check-in: https://hg.mozilla.org/integration/autoland/rev/12ad58f88888 - Sean Stangl - Bug 1522249 - ARM64: Pass atomics/basic-tests.js. r=nbp
02:52:18 <pulsebot> Check-in: https://hg.mozilla.org/integration/autoland/rev/33b3cd76b206 - Mike de Boer - Bug 1525833 - nsISearchService::addEngine does not support using a callback anymore, instead it returns a Promise. Update the searchbar's usage to reflect this. r=florian
02:53:04 <pulsebot> Check-in: https://hg.mozilla.org/integration/autoland/rev/8ec4da862bdc - Gerald Squelart - Bug 1520103 - Added missing includes in nsNativeCharsetUtils.h - r=froydnj
02:53:07 <pulsebot> Check-in: https://hg.mozilla.org/integration/autoland/rev/58654f30db8e - Gerald Squelart - Bug 1520103 - Fixed non-unified Profiler build - r=mstange
02:53:45 <pulsebot> Check-in: https://hg.mozilla.org/integration/autoland/rev/6518ca61dbd0 - Michael Kaply - Bug 1516028 - Don't allow a null originalDefaultEngine. r=daleharvey
02:54:27 <pulsebot> Check-in: https://hg.mozilla.org/integration/autoland/rev/a4247f89d070 - Dragana Damjanovic - Bug 1520483 - Return proper error if the nss layer encounters an error on the http tunnel. r=valentin,kershaw
02:55:16 <pulsebot> Check-in: https://hg.mozilla.org/integration/autoland/rev/7a9287969e24 - Jason Orendorff - Bug 1516958 - Weaken failing assertion that fails to account for nuked wrappers. r=jimb
02:55:58 <pulsebot> Check-in: https://hg.mozilla.org/integration/autoland/rev/c831a2e31841 - Jean-Yves Avenard - Bug 1525230 - Reset ImageRect when resolution change. r=bryce
02:56:41 <pulsebot> Check-in: https://hg.mozilla.org/integration/autoland/rev/a191672f28ae - Akshay Kumar - Bug 1522077 - Crash using FIDO U2F on Windows 10 insider build 1100+ r=keeler
02:57:23 <pulsebot> Check-in: https://hg.mozilla.org/integration/autoland/rev/dc44a6e74d72 - Jeff Gilbert - Bug 1523728 - Add gl.allow-high-power:true, webgl.default-low-power:false. r=lsalzman
02:58:09 <pulsebot> Check-in: https://hg.mozilla.org/integration/autoland/rev/c84c91cc9c5b - Lina Cambridge - Bug 1513434 - Remove use of `XPCOMUtils.IterSimpleEnumerator` in `nsUpdateService`. r=keeler
02:58:56 <pulsebot> Check-in: https://hg.mozilla.org/integration/autoland/rev/649fc91aa4d3 - Mike Hommey - Bug 1524467 - Use -m32/-m64 in more cases. r=froydnj
02:58:59 <pulsebot> Check-in: https://hg.mozilla.org/integration/autoland/rev/ead57b4507ac - Mike Hommey - Bug 1524467 - Fix and refactor basic_bindgen_cflags. r=froydnj
02:59:37 <pulsebot> Check-in: https://hg.mozilla.org/integration/autoland/rev/54eafeca2123 - Robert Strong - Bug 1525819 - Use relative paths for the app update test support files when possible. r=mhowell
03:00:32 <pulsebot> Check-in: https://hg.mozilla.org/integration/autoland/rev/a291f111e24e - Markus Stange - Bug 1509549 - Add a ProfilerGetSymbols module which can dump symbols with the help of a dynamically-loaded WebAssembly module. r=kmag
03:00:35 <pulsebot> Check-in: https://hg.mozilla.org/integration/autoland/rev/ba7bb5f95f43 - Markus Stange - Bug 1509549 - Use ProfilerGetSymbols for geckoProfiler WebExtension symbolication and remove all other sources of symbolication. r=kmag
03:02:20 <pulsebot> Check-in: https://hg.mozilla.org/integration/autoland/rev/f40b30f80cb1 - Markus Stange - Bug 1504101 - Use the ProfilerGetSymbols module for symbolicating profiles captured using the perf panel. r=julienw
03:02:23 <pulsebot> Check-in: https://hg.mozilla.org/integration/autoland/rev/eb8643ae792e - Markus Stange - Bug 1504101 - Add UI to the performance pane that lets the user pick an objdir for local builds. r=julienw
03:02:26 <pulsebot> Check-in: https://hg.mozilla.org/integration/autoland/rev/69e008db7e1e - Markus Stange - Bug 1504101 - When symbolicating profiles from remote targets, consult objdirs for symbol information. r=julienw
03:14:22 <pulsebot> Check-in: https://hg.mozilla.org/integration/mozilla-inbound/rev/5732522a3540 - Tooru Fujisawa - Bug 1524755 - Use ptrdiff_t instead in RangedPtr. r=jwalden
03:14:25 <pulsebot> Check-in: https://hg.mozilla.org/integration/mozilla-inbound/rev/9849e6b02bc2 - Tooru Fujisawa - Bug 1524965 - Part 0: Fix error message formatting in BinAST. r=Yoric
03:14:28 <pulsebot> Check-in: https://hg.mozilla.org/integration/mozilla-inbound/rev/7fce2598a3de - Tooru Fujisawa - Bug 1524965 - Part 1: Allocate StatementKind::Switch context in BinAST. r=Yoric
03:14:31 <pulsebot> Check-in: https://hg.mozilla.org/integration/mozilla-inbound/rev/30e403f80bfd - Tooru Fujisawa - Bug 1524965 - Part 2: Handle break/continue outside the loop properly. r=Yoric
03:14:34 <pulsebot> Check-in: https://hg.mozilla.org/integration/mozilla-inbound/rev/1143fee910c7 - Tooru Fujisawa - Bug 1524965 - Part 3: Add a script to generate testcases for BinAST with invalid content. r=Yoric
03:14:37 <pulsebot> Check-in: https://hg.mozilla.org/integration/mozilla-inbound/rev/21aedfe62db3 - Tooru Fujisawa - Bug 1524965 - Part 4: Add testcase for break/continue outside the loop. r=Yoric
03:23:52 <pulsebot> Check-in: https://hg.mozilla.org/integration/autoland/rev/321047a35ae1 - Avery Berninger - Bug 1525176 - Fixed aboutTabCrashed brandShortName, r=jaws
03:46:21 <pulsebot> Check-in: https://hg.mozilla.org/integration/autoland/rev/0ede01588c31 - Csoregi Natalia - Backed out changeset a4247f89d070 (bug 1520483) for bustage on build/src/obj-firefox/dist/include/mozilla/Logging.h:257. CLOSED TREE
04:23:58 <pulsebot> Check-in: https://hg.mozilla.org/mozilla-central/pushloghtml?startID=35515&endID=35516 - 88 changesets (bugs 1525537, 1521312, 1522436, 1516377, 1525763 and 60 other bugs)
04:29:14 <pulsebot> Check-in: https://hg.mozilla.org/integration/mozilla-inbound/pushloghtml?startID=111769&endID=111770 - 89 changesets - Merge mozilla-central to mozilla-inbound. a=merge on a CLOSED TREE
04:48:33 <jesup> So, I'm now getting "you've launched an older version of Firefox" when I started a 67.0a1 inbound local build, against one of my main "testing" profiles (which is always used with builds against inbound).  It's possible the changeset for this repo is older than the other one... but both are 67
05:15:36 <jesup> It was inbound rev 456213 (Jan 31st); the other repo used with the profile was inbound dev 457186 (Feb 5th)
05:58:48 <pulsebot> Check-in: https://hg.mozilla.org/integration/autoland/rev/c0639f601567 - Nils Ohlmeier [:drno] - Bug 1510898: disable sha1_32 in Nightly builds. r=mt
05:59:49 <pulsebot> Check-in: https://hg.mozilla.org/integration/autoland/rev/1feee32f9793 - Jared Wein - Bug 1523696 - Wait until the style attribute is applied to the dialog before sizing it. r=Gijs
06:23:19 <pulsebot> Check-in: https://hg.mozilla.org/integration/autoland/rev/7d23f7cb9de1 - Marco Zehe - Bug 1525849 - Guard against 0 columns or out of bounds indexes for ARIA grid accessibles, r=Jamie
06:32:37 <jya> glandium: in my existing mozilla-central checkout, after adding a "unified" remote (pointing to mozilla-unified), can I do something like
06:32:40 <jya> git -c cinnabar.clone=https://github.com/glandium/gecko fetch unified
06:32:53 <jya> or maybe git -c cinnabar.fetch=https://github.com/glandium/gecko fetch unified
06:33:05 <glandium> jya: no
06:33:17 <jya> must re-clone the whole lot?
06:33:33 <glandium> no, just git remote update unified
06:33:43 <jya> cause attempting to fetch release right now (need to uplift a patch) I will get timeout, always
06:35:05 <jya> yeah, but doing that fetch 54263 changeset, and when it goes to "reading and importing XXX manifests" I'll get a timeout
06:35:10 <glandium> jya: can you paste the output of git cat-file -p refs/cinnabar/metadata^ ?
06:37:07 <jya> https://irccloud.mozilla.com/pastebin/BlDtPK6R/
06:37:28 <jya> maybe time for me to ditch my setup of having multiple remotes and use unified all the way
06:37:56 <jya> would I still be able to push to inbound, or will still need another remote? (I'm guessing the later)
06:38:03 <glandium> the latter
06:39:39 <pulsebot> Check-in: https://hg.mozilla.org/integration/autoland/rev/565eefead66b - Cosmin Sabou - Backed out 2 changesets (bug 1492629) for suspicion of causing a rise in xpcshell intermittents.
06:40:57 <glandium> jya: what's your OS?
06:41:11 <jya> right now I'm on Windows
06:41:25 <jya> most of my timeouts occur on windows because I seldom use it
06:41:36 <glandium> jya: that looks like a clone of mozilla-central with maybe a few other branches?
06:41:51 <jya> this is indeed a clone on mozilla-central
06:42:10 <glandium> and pulling release times out?
06:42:17 <jya> yes.
06:42:30 <glandium> at what point does it time out?
06:42:31 <jya> I can start it and give you the exact error message in about 30 minutes
06:42:50 <jya> when it has done about 25000 manifests
06:42:50 <glandium> 30 minutes?!
06:42:58 <jya> yeah, about that
06:43:00 <glandium> git cinnabar --version
06:43:25 <jya> https://irccloud.mozilla.com/pastebin/GtTbWdJ6/
06:46:21 <glandium> jya: try setting cinnabar.experiments=store-manifest and pull release again
06:46:33 <jya> command?
06:46:48 <glandium> git -c cinnabar.experiments=store-manifest remote update $remote
06:47:22 <jya> IIRC, you got me to do that last time, and it didn't make a difference. but happy to try anything. Every time I come back on windows, i hit this issue and it takes me hours to get around it:(
06:49:36 <jya> started, takes about 1s for every 220 manifests, there are 59928 changesets to go
06:50:02 <glandium> jya: does that feel faster than before?
06:50:10 <jya> it does
06:50:24 <jya> and it's already done 30000 manifests
06:50:44 <jya> (I had timed for 10s before)
06:55:17 <jya> glandium: ah same deal
06:55:21 <jya> https://irccloud.mozilla.com/pastebin/9fPjSPY3/
06:55:42 <jya> full output:
06:55:45 <jya> https://irccloud.mozilla.com/pastebin/vZ6upc2Z/
06:56:27 <glandium> jya: try again with cinnabar.experiments=store-manifest,wire
07:04:24 <jya> https://irccloud.mozilla.com/pastebin/pXFkLb75/
07:05:27 <glandium> jya: can you graph your network throughput during the operation?
07:05:33 <jya> glandium: last time you told me it was my connection and there was nothing we could do about it. I've had this occur in Mozilla Paris, Mozilla Portland, at home in France (gigabit fibre with orange), 4G mobile and now home in Australia on a new 100/40 internet
07:05:39 <jya> sure
07:06:43 <jya> btw, it's only on windows
07:07:34 <jya> reading and importing XXX manifests doesn't touch the network.
07:08:30 <pulsebot> Check-in: https://hg.mozilla.org/integration/mozilla-inbound/rev/70d80c4a3fa0 - Chris Peterson - Bug 1507049 - Rename MOZ_CrashOOL MOZ_Crash. r=froydnj
07:08:32 <jya> well, that got me curious ttps://hg.mozilla.org/releases/mozilla-release
07:08:33 <pulsebot> Check-in: https://hg.mozilla.org/integration/mozilla-inbound/rev/426f2e129858 - Chris Peterson - Bug 1507049 - Rename GeckoCrashOOL GeckoCrash. r=froydnj
07:08:36 <jya> that's not a valid url!
07:08:36 <pulsebot> Check-in: https://hg.mozilla.org/integration/mozilla-inbound/rev/7e9ceabc7541 - Chris Peterson - Bug 1507049 - Rename MOZ_CRASH_UNSAFE_OOL MOZ_CRASH_UNSAFE. r=froydnj
07:09:32 <jya> ah yes it is :(
07:10:15 <pulsebot> Check-in: https://hg.mozilla.org/integration/autoland/rev/da71b4d4ad40 - Masayuki Nakano - Bug 1466208 - part 13: Create PresShell::EventHandler::MaybeFlushThrottledStyles() r=smaug
07:14:30 <CosminS|sheriffduty> rstrong: ping
07:15:24 <rstrong> CosminS|sheriffduty: hi
07:16:09 <jya> glandium: almost done with the reading and importing manifests page, there's been close to 0 network activity during this time, only during the first 3-4s
07:16:31 <glandium> jya: and how much time has elapsed?
07:16:40 <glandium> (between the last activity and now)
07:16:46 <CosminS|sheriffduty> rstrong: do you think this rise in xpcshell failures from here https://treeherder.mozilla.org/#/jobs?repo=autoland&resultStatus=testfailed%2Cbusted%2Cexception&searchStr=xpcshell&fromchange=828188fd32032c610c7a41c8a3651640df3bef28&tochange=17c5a7677d9f2b5487f2eb297ee27bc903c00011&selectedJob=227069582
07:16:50 <jya> about 10 minutes
07:17:07 <CosminS|sheriffduty> could have smthg to do with your changes?
07:17:22 <jya> I started the update at 18:09 
07:17:46 <glandium> jya: it would seem windows is more aggressive at buffering
07:18:02 <jya> yep, right after the manifests reading completed, the Reading and importing 30239 revisions of 4978 files started and failed almost instantly
07:18:08 <jya> and no activity then
07:18:16 <CosminS|sheriffduty> https://treeherder.mozilla.org/logviewer.html#/jobs?job_id=227069582&repo=autoland&lineNumber=6180
07:18:16 <CosminS|sheriffduty> and https://treeherder.mozilla.org/logviewer.html#/jobs?job_id=227105195&repo=autoland&lineNumber=6075
07:18:22 <jya> I mean the max of the graph is at 100Kbps 
07:18:28 <jya> that's probably me talking here
07:19:14 <glandium> jya: yeah, what happens is that until the buffered data from the initial network traffic is exhausted, things go normally, but when fresh data is needed form the server, the server is ejecting you
07:19:34 <jya> so my work around for that is to go on the web interface, go back -60000 commits, and then do git cinnabar fetch release SHA1_10000_at_a_time
07:19:55 <jya> if that fails I go 3000 at a time etc
07:20:06 <jya> buffered where?
07:20:15 <glandium> OS level, presumably
07:20:20 <rstrong> CosminS|sheriffduty: each xpcshell tests run in their own process for the test so no, I don't see how that could be caused by the patches I landed.
07:20:41 <jya> I do have 128GB on that machine, but I find it hard to believe it would cache at the OS for that long
07:20:48 <CosminS|sheriffduty> rstrong: thanks
07:20:49 <jya> but what do I know..
07:21:16 <jya> though, right after a reboot is typically when I go to fetch
07:21:33 <rstrong> CosminS|sheriffduty: looks like they started on the push prior to my push?
07:21:39 <jya> it shouldn't be buffered yet if it's never been fetched earlier right?
07:22:23 <CosminS|sheriffduty> rstrong: that was my first hunch too
07:22:42 <CosminS|sheriffduty> what to have all covered, thanks for responding
07:23:41 <rstrong> no problem... I left irccloud logged in on my phone so I get pings
07:25:27 <glandium> jya: the problem is that, say, windows buffers 2MB raw, and it takes multiple minutes for cinnabar to process 2MB (considering delta compression + compression, it's actually not so far-fetched), it can go multiple minutes without any need to fetch new data from the network
07:30:05 <glandium> jya: it's probably more than 2MB, though
07:32:17 <jya> glandium: could we have dummy connections that only keep the thing alive.?
07:34:45 <jya> I'm going to reclone the whole thing, this taking the unified repo and using the method of fetching cinnabar stuff from your repo. 
07:35:31 <glandium> jya: you can avoid recloning by doing git cinnabar rollback 0000000000000000000000000000000000000000
07:35:41 <glandium> that will reset all cinnabar metadata
07:37:10 <jya> Why would that help doing the next fetch? 
07:37:57 <glandium> jya: if you reset, you can use the fast clone thing on a fetch
08:07:18 <jya> glandium: the fast clone "thing" works with the release repo?
08:07:35 <glandium> jya: no, but if you do it with unified, you get the release branch
08:07:42 <jya> ah ok
08:08:08 <jya>  should I just get unified and work from there from now on?
08:08:33 <pulsebot> Check-in: https://hg.mozilla.org/integration/autoland/rev/f7b5792623ad - Andrea Marchesini - Bug 1525727 - CookieBehavior = 4, by default, r=Ehsan
08:08:49 <glandium> jya: maybe? but make sure not to use branches and use bookmarks instead
08:09:07 <jya> why that?
08:09:25 <glandium> because branches/default/tip switches between release, esr, beta, inbound, and central
08:09:41 <glandium> whichever was pushed last to
08:09:52 <jya> is there a guide on what do do if say I want to look at release?
08:10:02 <glandium> use bookmarks/release
08:10:13 <glandium> or still use the release remote
08:10:33 <jya> tbh I don't know what bookmarks are with git
08:10:36 <glandium> which you'll have to do if you want to manually push there
08:10:52 <glandium> jya: they're branches with a different name
08:10:59 <jya> and then I would have first to pull from the release repo
08:11:21 <glandium> jya: you'll always have to pull from the release repo, in case unified is behind
08:12:25 <jya> how far behind is unified usually?
08:12:37 <glandium> not much
08:12:44 <glandium> minutes max
08:12:57 <jya> so for release shouldn't be a problem
08:22:39 <pulsebot> Check-in: https://hg.mozilla.org/integration/autoland/rev/661d858eb94f - Jan de Mooij - Bug 1525674 part 1 - Change the enumerate proxy trap to return the jsid vector instead of an iterator. r=bzbarsky,evilpie
08:22:42 <pulsebot> Check-in: https://hg.mozilla.org/integration/autoland/rev/44a103afdb3a - Jan de Mooij - Bug 1525674 part 2 - Make js::IteratorMore (and the JIT-inlined equivalent) infallible. r=evilpie
08:22:45 <pulsebot> Check-in: https://hg.mozilla.org/integration/autoland/rev/279c4a04e2d9 - Jan de Mooij - Bug 1525674 part 3 - Rename JSObject::ITER_CLASS_NFIXED_SLOTS to PropertyIteratorObject::NUM_FIXED_SLOTS. r=evilpie
08:22:48 <pulsebot> Check-in: https://hg.mozilla.org/integration/autoland/rev/0cd43ecd2fb6 - Jan de Mooij - Bug 1525674 part 4 - Move Ion's Iterator{More,Close} code to the MacroAssembler and use it in Baseline instead of ICs. r=evilpie
08:46:57 <peterv> glandium: https://bugzilla.mozilla.org/show_bug.cgi?id=1015547#c15
08:48:52 <glandium> peterv: same conclusion
08:49:25 <peterv> right
09:00:06 <pulsebot> Check-in: https://hg.mozilla.org/integration/autoland/rev/fd700b0b82a6 - Gijs Kruitbosch - Bug 1445739 - stop invalidating the startup cache when extensions change, r=kmag
09:10:11 <pulsebot> Check-in: https://hg.mozilla.org/integration/autoland/rev/264d828518f2 - Nihanth Subramanya - Bug 1525519 - Land Firefox Monitor system add-on into browser/extensions. r=johannh
09:16:31 <pulsebot> Check-in: https://hg.mozilla.org/integration/mozilla-inbound/rev/5d720cbe1187 - Lars T Hansen - Bug 1521939 - Correct the offset for reading stack args on ARM64.  r=bbouvier
09:26:45 <pulsebot> Check-in: https://hg.mozilla.org/integration/autoland/rev/a7b62563e1ee - Jan de Mooij - Bug 1525329 - Add compartment/zone assertions to some JSObject/ShapedObject methods. r=jonco
09:30:20 <pulsebot> Check-in: https://hg.mozilla.org/integration/mozilla-inbound/rev/1431d60aedcc - Lars T Hansen - Bug 1523908 - Require zero byte encoding for memory/table index in the absence of multi-memory/multi-table. r=jseward
09:30:23 <pulsebot> Check-in: https://hg.mozilla.org/integration/mozilla-inbound/rev/a1c6c6703dfb - Lars T Hansen - Bug 1502033 - Implement partial write for memInit, memFill, memCopy. r=jseward
09:30:26 <pulsebot> Check-in: https://hg.mozilla.org/integration/mozilla-inbound/rev/e3e2baccf180 - Lars T Hansen - Bug 1502033 - Implement partial write for tableInit, tableCopy. r=jseward
09:30:29 <pulsebot> Check-in: https://hg.mozilla.org/integration/mozilla-inbound/rev/9ddffb46398b - Lars T Hansen - Bug 1502033 - Factor some tests to avoid timeouts on simulators. rs=jseward
09:49:33 <pulsebot> Check-in: https://hg.mozilla.org/mozilla-central/pushloghtml?startID=35516&endID=35517 - 49 changesets (bugs 1525196, 1524893, 1524117, 1525510, 1522381 and 29 other bugs)
09:55:36 <pulsebot> Check-in: https://hg.mozilla.org/mozilla-central/pushloghtml?startID=35517&endID=35518 - 504 changesets - Merge inbound to mozilla-central.  a=merge
09:56:32 <pulsebot> Check-in: https://hg.mozilla.org/integration/autoland/rev/3c6fcec3f2b2 - Jan de Mooij - Bug 1522837 part 9 - Update framePushed after Baseline VM call. r=tcampbell
09:58:44 <pulsebot> Check-in: https://hg.mozilla.org/integration/autoland/rev/e52aa87c1ced - Dragana Damjanovic - Bug 1520483 - Return proper error if the nss layer encounters an error on the http tunnel. r=valentin,kershaw
10:00:22 <glandium> hsivonen: did you reproduce the 2018 edition problem in a workspace+vendoring setup that is not mozilla-central?
10:00:43 <pulsebot> Check-in: https://hg.mozilla.org/releases/mozilla-beta/rev/96a4289bed4d - L10n Bumper Bot - no bug - Bumping Fennec l10n changesets r=release a=l10n-bump DONTBUILD
10:00:46 <pulsebot> Check-in: https://hg.mozilla.org/releases/mozilla-beta/rev/db241c43e783 - L10n Bumper Bot - no bug - Bumping Firefox l10n changesets r=release a=l10n-bump DONTBUILD
10:00:56 <hsivonen> glandium: not yet
10:01:06 <glandium> hsivonen: that would probably help
10:01:34 <hsivonen> glandium: indeed. I know almost nothing about workspaces, so I don't really have an idea of what could be relevant for repro
10:02:20 <glandium> hsivonen: that said, did you check there aren't other invocations of rustc for packed_simd? I've seen weird things that I need to debug about what appears to be some crates being built twice
10:03:29 <pulsebot> Check-in: https://hg.mozilla.org/integration/mozilla-inbound/rev/24775fc4e7f9 - Noemi Erli - Bug 1518370 - Disable browser_browser_languages_subdialog.js for frequent failures r=jmaher
10:04:21 <hsivonen> glandium: I didn't. I just looked at the Gnome System Monitor and hoped it updated frequently enough
10:04:38 <hsivonen> glandium: what tool should I use to dump all process creation command lines?
10:05:23 <glandium> hsivonen: strace -f -e process -s 10000
10:05:44 <glandium> -o log
10:05:58 <hsivonen> glandium: where "process" is "rustc"?
10:06:06 <glandium> no, literally process
10:06:42 <glandium> that traces all syscalls related to process management
10:07:22 <hsivonen> glandium: thanks
10:07:43 <hsivonen> whoa. slow build is slow with this one
10:09:17 <glandium> strace is heavyweight
10:14:31 <pulsebot> Check-in: https://hg.mozilla.org/integration/autoland/pushloghtml?startID=77742&endID=77743 - 505 changesets - Merge mozilla-central to autoland. a=merge CLOSED TREE
10:16:33 <pulsebot> Check-in: https://hg.mozilla.org/integration/mozilla-inbound/pushloghtml?startID=111774&endID=111775 - 51 changesets - Merge mozilla-central to inbound.  a=merge CLOSED TREE
10:26:42 <pulsebot> Check-in: https://hg.mozilla.org/mozilla-central/rev/22ca3a5f976f - Sebastian Hengst - Backed out changeset 8d7c099bbe0d (bug 1522111) for breaking SETA job schedule optimization. a=backout
10:30:13 <pulsebot> Check-in: https://hg.mozilla.org/integration/autoland/pushloghtml?startID=77743&endID=77744 - 2 changesets - Merge mozilla-central to autoland
10:32:31 <pulsebot> Check-in: https://hg.mozilla.org/integration/mozilla-inbound/pushloghtml?startID=111775&endID=111776 - 2 changesets - Merge mozilla-central to mozilla-inbound
10:33:29 <pulsebot> Check-in: https://hg.mozilla.org/integration/mozilla-inbound/rev/1a45636ad898 - Jonathan Kew - Bug 1509067 - Make font fallback properly search across multiple 'normal' faces of a family (e.g. where subsetting has been used to split a font between multiple files). r=jrmuizel
10:35:33 <pulsebot> Check-in: https://hg.mozilla.org/integration/mozilla-inbound/rev/0ae4d42ca65b - Jon Coppeard - Bug 1525663 - Make use of generic lambdas when dealing with CrossCompartmentKey variants r=sfink
10:35:36 <pulsebot> Check-in: https://hg.mozilla.org/integration/mozilla-inbound/rev/fdbb8bed650a - Jon Coppeard - Bug 1525663 - Use return type deduction to remove complex decltype expressions r=sfink
11:00:24 <pulsebot> Check-in: https://hg.mozilla.org/integration/autoland/rev/306f0162b014 - Dão Gottwald - Bug 1522784 - Adjust Quantumbar font-sizes and row padding to better match the Awesomebar. r=ntim
11:27:52 <pulsebot> Check-in: https://hg.mozilla.org/mozilla-central/rev/4bc31addf415 - Cosmin Sabou - Backed out 3 changesets (bug 1519472) for causing wrench bustages and as requested by pmoore. a=backout
11:29:57 <pulsebot> Check-in: https://hg.mozilla.org/integration/autoland/pushloghtml?startID=77745&endID=77746 - 2 changesets - Merge mozilla-central to autoland.
11:30:41 <hsivonen> glandium: In order to avoid excess slowness, I straced only a re-invocation of mach build after the failure to catch it failing the incremental build
11:31:05 <pulsebot> Check-in: https://hg.mozilla.org/integration/mozilla-inbound/pushloghtml?startID=111778&endID=111779 - 2 changesets - Merge mozilla-central to mozilla-inbound.
11:31:09 <hsivonen> there's an invocation of /home/hsivonen/.cargo/bin/rustc followed by /home/hsivonen/.rustup/toolchains/nightly-x86_64-unknown-linux-gnu/bin/rustc
11:31:29 <hsivonen> glandium: I gather the former is the rustup binary itself redicting to the second real binary
11:31:49 <hsivonen> glandium: both have --edition=2018
11:32:30 <hsivonen> hmm. some options are --foo=bar and some are --foo bar
11:32:41 <hsivonen> let's see how --edition works in a standalone build
11:34:47 <pulsebot> Check-in: https://hg.mozilla.org/integration/autoland/rev/6a43c82791d3 - Mathieu Leplatre - Bug 1525253 - Document how to unit test Remote Settings r=glasserc
11:36:42 <hsivonen> passed as --edition=2018 in the standalone case, too
11:38:41 <pulsebot> Check-in: https://hg.mozilla.org/integration/autoland/rev/37fc40cdf4ed - matthias - Bug 1524336 - Make about:config rows more compact. r=ntim
12:00:41 <pulsebot> Check-in: https://hg.mozilla.org/releases/mozilla-beta/rev/78f06587eaf3 - L10n Bumper Bot - no bug - Bumping Fennec l10n changesets r=release a=l10n-bump DONTBUILD
12:00:44 <pulsebot> Check-in: https://hg.mozilla.org/releases/mozilla-beta/rev/3ae4f6fc5da7 - L10n Bumper Bot - no bug - Bumping Firefox l10n changesets r=release a=l10n-bump DONTBUILD
12:15:37 <hsivonen> glandium: tried a minimal workspace: https://github.com/hsivonen/test_workspace . It does not reproduce the problem.
12:25:31 <pulsebot> Check-in: https://hg.mozilla.org/integration/mozilla-inbound/rev/c7041aff5ad7 - Kai Engert - Bug 1526010, Upgrade Firefox 67 to use NSPR 4.21. Beta snapshot, r=me
12:25:34 <pulsebot> Check-in: https://hg.mozilla.org/integration/mozilla-inbound/rev/5b4cd1f37414 - David Major - Bug 1520646: Use UNIFIED_SOURCES in NSPR, r=glandium
12:41:17 <masayuki> I have a question about wpt. How can I run wpt under testing/web-platform/mozilla/tests with |./match wpt product --chrome|? It claims that "Unable to find any tests at the path(s)".
12:41:43 <masayuki> oops, |./mach wpt --product chrome|, I meant.
12:49:24 <pulsebot> Check-in: https://hg.mozilla.org/integration/mozilla-inbound/rev/cab981ef743b - Olli Pettay - Bug 1506376 - Make PCompositorBridge::DidComposite high priority message, r=aosmond
12:50:22 <pulsebot> Check-in: https://hg.mozilla.org/integration/mozilla-inbound/rev/2dc80bc53119 - Andrew Comminos - Bug 1294232 - Refactor blocklisting on Linux to support the downloadable blocklist. r=jrmuizel
12:50:57 <pulsebot> Check-in: https://hg.mozilla.org/integration/autoland/rev/546288d07c7c - Masayuki Nakano - Bug 1466208 - part 14: Create PresShell::EventHandler::ComputeRootFrameToHandleEvent() r=smaug
12:57:41 <pulsebot> Check-in: https://hg.mozilla.org/integration/mozilla-inbound/rev/7ebcd074e346 - Andrew Osmond - Bug 1523410 - Add new translate/scale/zoom snapping test case. r=jrmuizel
13:04:48 <pulsebot> Check-in: https://hg.mozilla.org/integration/autoland/rev/b482c6618d72 - John Dai - Bug 1513877 - Implement send and receive methods for base WindowActor class. r=nika
13:24:04 <pulsebot> Check-in: https://hg.mozilla.org/integration/autoland/rev/8ec307e6000f - Dragana Damjanovic - Bug 1520642 - Cancel dns requests before cleaning up db. r=kershaw
13:39:55 <padenot> dmajor, what would you recommend to do exploratory profiling on Windows 10 ?
13:40:06 <padenot> like, I know there is an issue but I don't know where
13:40:12 <pulsebot> Check-in: https://hg.mozilla.org/integration/autoland/rev/013ef288b97c - Dragana Damjanovic - Bug 1516806 - Remove SPDY_VERSION2 telemetry probe. r=kershaw
13:48:15 <freddy> are there guidelines/howtos for introducing a new pref?
13:50:46 <Aryx> freddy: no afaik. anything specific in mind?
13:52:35 <freddy> I'm adding code that shouldn't run unless a pref is set :-) that pref is new
13:52:49 <freddy> I'm just gonna look at patterns from other patches. there are enough. :)
14:07:42 <dmajor> padenot: personally I use Windows Performance Analyzer. others may recommend Gecko Profiler.
14:08:46 <pulsebot> Check-in: https://hg.mozilla.org/integration/mozilla-inbound/rev/42763a2bd759 - Noemi Erli - Backed out 2 changesets (bug 1526010, bug 1520646) mass mochitest failures UPGRADE_NSPR_RELEASE CLOSED TREE
14:12:33 <padenot> dmajor, ot
14:12:39 <padenot> dmajor. it's scary, but I think it'
14:12:45 <padenot> s finally time I learn this set of tools
14:12:59 <padenot> ETW and xperf and those things
14:13:15 <padenot> gecko profiler doesn't work for me because I wouldn't know which thread to capture
14:13:21 <dmajor> padenot: I don't know if it's any comfort, but it used to be so much worse back when it was called xperf :)
14:14:03 <padenot> I'm spoiled by tracecompass  on linux and instruments on osx unfortunately, it's going to be hard, but it's not like there is a choice here
14:14:29 <padenot> and gecko profiler of course, which is amazing when it can be used
14:14:34 <pulsebot> Check-in: https://hg.mozilla.org/integration/autoland/rev/7e05ecb002cf - Dão Gottwald - Bug 1526259 - Stop setting a negative y-offset when opening the quantumbar panel. r=mak
14:14:39 <dmajor> padenot: this is probably the best collection of guides https://tinyurl.com/etwcentral
14:15:36 <padenot> dmajor, yes, i've been reading quite a few blogpost from this guy, he has amazing stories sometimes
14:15:45 <padenot> dmajor, thanks for this
14:16:00 <pulsebot> Check-in: https://hg.mozilla.org/integration/autoland/rev/bcbe1dc28725 - Chris H-C - Bug 1313327 - Flatten the Telemetry Event enums. r=janerik
14:16:03 <pulsebot> Check-in: https://hg.mozilla.org/integration/autoland/rev/2ad12527e422 - Chris H-C - Bug 1313327 - Add a C++ API for recording Telemetry Events. r=janerik
14:16:06 <pulsebot> Check-in: https://hg.mozilla.org/integration/autoland/rev/9bf817ce243c - Chris H-C - Bug 1313327 - Test C++ Events API r=janerik
14:18:39 <padenot> now if my vsc++ redist install would finish it would be amazing
14:18:58 <dmajor> padenot: let me know if anything seems too complex and I can tell you what parts to ignore :) it's one of those things where the basics will get you a long way but then there's a bunch of extra situation-specific tricks
14:19:04 <padenot> right
14:19:18 <padenot> I'll have a go at it, and ping if I face issues
14:19:51 <pulsebot> Check-in: https://hg.mozilla.org/integration/autoland/rev/1fecce7afe30 - Liang-Heng Chen - Bug 1486258 - Tests for privacy.spoof_english r=zbraniecki
14:19:54 <padenot> what I'd like is something like lttng where I can know, for example, what caused a thread to be de-scheduled, or what thread was in the queue for a mutex, how long it was taken, that kind of things
14:20:28 <padenot> but simply a good sampling profiler I can trust would nice, for some other stuff
14:22:11 <dmajor> padenot: for simple "using too many cycles" there's "CPU usage (Sampled)" but for the threading-related stuff, "CPU Usage (Precise)" is very good https://randomascii.wordpress.com/2012/05/11/the-lost-xperf-documentationcpu-scheduling/
14:23:05 <padenot> ok, reading this one first
14:28:35 <pulsebot> Check-in: https://hg.mozilla.org/integration/mozilla-inbound/rev/0468366cf474 - Gabriel Luong - Bug 1525988 - Don't render invisible declarations in the new rules view. r=mtigley
14:31:07 <Standard8> Is there an issue with treeherder?
14:31:19 <Standard8> Looks like I’m seeing a 500 attempting to get push data
14:33:47 <Aryx> Standard8: wfm. any special push in mind?
14:33:58 <Standard8> Aryx: https://treeherder.mozilla.org/#/jobs?repo=try&tochange=6dbb1ed8b56238c79f9a13aa639f498101926bc4
14:34:24 <Standard8> specifically https://treeherder.mozilla.org/api/project/try/push/?full=true&count=10&tochange=6dbb1ed8b56238c79f9a13aa639f498101926bc4
14:34:32 <Standard8> is the one that gives 500
14:36:18 <Aryx> https://treeherder.mozilla.org/#/jobs?repo=try&revision=6a145a28be6db6c67c9d3b49aa38ede46939e734 works
14:36:41 <Aryx> https://treeherder.mozilla.org/#/jobs?repo=try&tochange=6a145a28be6db6c67c9d3b49aa38ede46939e734&fromchange=b58092cf3195b640f1db217194cadab385fa6358
14:36:46 <Aryx> wants a fromchange?
14:39:14 <pulsebot> Check-in: https://hg.mozilla.org/integration/autoland/rev/45d561eacd00 - Dustin J. Mitchell - Bug 1524845 - download data about previous crons, actions in parallel r=aki
14:39:59 <Standard8> Aryx: huh, dunno where that tochange came from, can you try this one https://treeherder.mozilla.org/#/jobs?repo=try&revision=6dbb1ed8b56238c79f9a13aa639f498101926bc4
14:40:46 <Standard8> oh
14:40:57 <Standard8> Aryx: oh, I think I see, something posted the wrong link on the bug
14:41:50 <Aryx> Standard8: can you link me to that? th removed the non-pushhead changeset support in the url
14:42:14 <Aryx> went through hgmo, that automatically picks the correct revision
14:42:22 <Standard8> Aryx: https://bugzilla.mozilla.org/show_bug.cgi?id=1526228#c2 - I think that was posted automatically when I did ./mach try in hg repo
14:44:37 <pulsebot> Check-in: https://hg.mozilla.org/integration/autoland/rev/2a4f6bb7524a - Michael Ratcliffe - Bug 1524319 - Alpha values in #RRGGBBAA format can lose accuracy r=nchevobbe
14:49:12 <pulsebot> Check-in: https://hg.mozilla.org/integration/mozilla-inbound/rev/0669f92557f3 - Noemi Erli - Backed out changeset 2dc80bc53119 (bug 1294232) for xpcshell failures in test_gfxBlacklist_No_Comparison.js
14:53:20 <Aryx> Standard8: ok, needs either fixing in th or a bzext update
14:53:26 <Aryx> will create a bug
14:53:38 <Standard8> thanks
14:54:24 <pulsebot> Check-in: https://hg.mozilla.org/integration/autoland/rev/60f9b60f1f4f - Chris Martin - Bug 1347710 - Re-enable GPU sandbox level 1 r=bobowen
15:07:05 <jorendorff> nbp: we're cancelling this friday meeting, right?
15:07:26 <jorendorff> afk
15:15:01 <pulsebot> Check-in: https://hg.mozilla.org/integration/mozilla-inbound/rev/6a91ccdd5730 - Gabriel Luong - Bug 1526053 - Convert rule models to ES6 classes. r=miker
15:15:42 <pulsebot> Check-in: https://hg.mozilla.org/integration/autoland/rev/c6d1ab19c941 - Brian Grinstead - Bug 1526320 - Convert DevTools XBL test to use a test binding instead of a xul:tree;r=pbro
15:16:07 <pulsebot> Check-in: https://hg.mozilla.org/integration/mozilla-inbound/rev/eee8ce2afe60 - J.C. Jones - Bug 1523175 - land NSS 794984bb1169 UPGRADE_NSS_RELEASE, r=me
15:16:25 <pulsebot> Check-in: https://hg.mozilla.org/integration/autoland/rev/839decca3577 - J.C. Jones - Bug 1404934 - Wire-up Intermediate Preloading r=keeler
15:28:29 <pulsebot> Check-in: https://hg.mozilla.org/integration/autoland/rev/93a36a130fec - Sebastian Hengst - Bug 1526205 - set new serviceworkers mochitests (M-sw-e10s) to tier 2 because they don't run on integration (autoland and inbound) r=ahal
15:29:12 <pulsebot> Check-in: https://hg.mozilla.org/integration/autoland/rev/31b7a57cff78 - Dão Gottwald - Bug 1526233 - Strip http:// from Quantumbar result URLs. r=Standard8
15:30:02 <pulsebot> Check-in: https://hg.mozilla.org/integration/autoland/rev/6837617a1024 - Dave Hunt - Bug 1471920 - Fix Python 3 support in mozversion; r=ahal
15:51:50 <jesup> myk: ping
15:52:58 <nbp> jorendorff: right
15:59:52 <ato> dylan: Is there a way I can disable Markdown for a particular comment?
16:00:11 <dylan> ato: begin the comment with #[markdown(off)]
16:00:32 <dylan> if you want monospaced text, use ```\n````\n fences.
16:01:04 <ato> dylan: OK, thanks.
16:03:40 <pulsebot> Check-in: https://hg.mozilla.org/integration/mozilla-inbound/rev/f78d0fba26fa - Lee Salzman - Bug 1525817 - Be consistent about int for incReserve. r=rhunt
16:09:53 <myk> jesup: pong
16:12:03 <pulsebot> Check-in: https://hg.mozilla.org/integration/mozilla-inbound/rev/4e1b5ab8c44d - Noemi Erli - Backed out changeset cab981ef743b (bug 1506376) for failures in browser_appmenu.js
16:12:32 <jesup> myk: I have a patch to improve responsiveness testing in the profiler (by directly measuring Dispatch->execute times).  This means adding  dispatchTime TimeStamp attribute to nsIRunnable - and that's breaking your recent addition of moz_task:
16:12:53 <jesup> myk: https://paste.rs/w54
16:12:55 <pulsebot> Check-in: https://hg.mozilla.org/integration/autoland/rev/4171dda74df8 - arthur.iakab - Backed out changeset 1fecce7afe30 (bug 1486258) for mochitest failures on test_spoof_english.html CLOSED TREE
16:12:58 <pulsebot> Check-in: https://hg.mozilla.org/integration/autoland/rev/a91f286b5b1f - arthur.iakab - Backed out changeset c6d1ab19c941 (bug 1526320) for devtools failures on browser_inspector_highlighter-eyedropper-xul.js CLOSED TREE
16:14:20 <jesup> Either we have to move moz_task to be built on Runnable (or the UnnamedRunnable I'm adding, probably), which implements Get/SetDispatchTime, or we need to teach it's runnable impl to implement those within moz_task
16:16:18 <pulsebot> Check-in: https://hg.mozilla.org/mozilla-central/pushloghtml?startID=35520&endID=35521 - 18 changesets (bugs 1525727, 1525674, 1445739, 1525519, 1525329 and 7 other bugs)
16:16:35 <froydnj> jesup: we're going to make every runnable bigger?  just on nightly, or release/beta too?
16:17:10 <froydnj> also not enthused about more vtable entries times 2000 vtables
16:19:03 <froydnj> jesup: maybe you could land the "get everything deriving from Runnable" and then we could talk about the direction to go from there
16:20:36 <pulsebot> Check-in: https://hg.mozilla.org/integration/mozilla-inbound/pushloghtml?startID=111790&endID=111791 - 19 changesets - Merge mozilla-central to mozilla-inbound.
16:21:41 <pulsebot> Check-in: https://hg.mozilla.org/integration/autoland/rev/bac51381060d - Marco Bonardo - Bug 1519538 - Disable idle-daily notifications in tests through user.js files. r=Standard8
16:22:18 <jesup> froydnj: we can limit this to nightly/beta, like with do with the named stuff.  That would mean we wouldn't get data in profiles on release builds (or I find another way to do it, which perhaps i can do - we talked about this last fall-ish which is why I went this way (just unbitrotted and finished the initial patch).  It keeps CPU overhead down by only touching the dispatchtime stuff if the...
16:22:19 <jesup> ...profiler is on
16:22:53 <jesup> I can split into nsIRunnable->Runnable/UnnamedRunnable, and a patch to implement responsiveness
16:23:34 <pulsebot> Check-in: https://hg.mozilla.org/releases/mozilla-beta/rev/9cef500e71c6 - Mozilla Releng Treescript - No bug - Tagging a9f6e3381fb3e64a6b872efa52a46afe4dd588e8 with FIREFOX_66_0b6_RELEASE a=release CLOSED TREE DONTBUILD
16:23:37 <pulsebot> Check-in: https://hg.mozilla.org/releases/mozilla-beta/rev/fc4c1b7b7864 - Mozilla Releng Treescript - Automatic version bump CLOSED TREE NO BUG a=release DONTBUILD
16:24:06 <pulsebot> Check-in: https://hg.mozilla.org/integration/autoland/rev/45904a257981 - Mark Striemer - Bug 1514316 - Basic HTML list view for about:addons behind a pref r=aswan,jaws,flod
16:24:35 <jesup> myk: is it just that moz_task doesn't implement the attribute?  and if so, can it?  (takes a TimeStamp)
16:25:01 <myk> jesup: hmm, unsure; can you point me at the patch?
16:25:11 <pulsebot> Check-in: https://hg.mozilla.org/integration/autoland/rev/58da6f9e6cb3 - Mark Striemer - Bug 1525937 - Remove unnecessary tabindex="0"s from about:addons r=Jamie,jaws
16:25:35 <jesup> myk:  bug 1482244
16:25:44 <jesup> applies against inbound from last night
16:26:30 <pulsebot> Check-in: https://hg.mozilla.org/releases/mozilla-beta/rev/7a56e1dc2c2c - Mozilla Releng Treescript - No bug - Tagging a9f6e3381fb3e64a6b872efa52a46afe4dd588e8 with FENNEC_66_0b6_RELEASE a=release CLOSED TREE DONTBUILD
16:26:53 <jesup> froydnj: let me split, and also re-evaluate alternatives (like having a separate array where we track event dispatch times, and only populate it when the profiler is on)
16:27:39 <froydnj> jesup: I think we want everything to derive from Runnable for other reasons, so that work is valuable in its own right
16:27:59 <froydnj> jesup: tracking dispatch times in the queues themselves sounds like an interesting alternative
16:28:30 <jesup> yes; that was my initial alternative when I started looking at this
16:28:42 <froydnj> jesup: also, if you required everything in terms of Runnable, then all the thread interfaces could be moved over to work in terms of Runnable, not nsIRunnable, and then your Get/Set methods could at least become non-virtual
16:29:02 <jesup> This was slightly simpler (I thought... then I realized I the number of nsIRunnables...)
16:29:36 <pulsebot> Check-in: https://hg.mozilla.org/releases/mozilla-beta/rev/efb5186d4870 - Mozilla Releng Treescript - No bug - Tagging a9f6e3381fb3e64a6b872efa52a46afe4dd588e8 with DEVEDITION_66_0b6_RELEASE a=release CLOSED TREE DONTBUILD
16:29:41 <jesup> froydnj: yes, that's right.  My patch removes all but one or two nsIRunnables right now (and myk's new rust one)
16:30:02 <jesup> and I'm sure we could remove the rest
16:32:36 <pulsebot> Check-in: https://hg.mozilla.org/integration/autoland/rev/42b90a18e239 - Noemi Erli - Backed out changeset bac51381060d (bug 1519538) for Linting failure in config.py
16:32:45 <Fallen> Is there a way to load a js script that is actually an ES6 module from a chrome js context? Kind of like <script type="module"> but manually
16:33:30 <jesup> froydnj: right now I have a new class "UnnamedRunnable", since there were a lot of unnamed runnables out there; I could just give names (class names probably) to all the unnamed ones instead.  but we can discuss that in review
16:34:23 <jesup> gets rid of a lot of ::GetName() methods too :-)
16:39:08 <pulsebot> Check-in: https://hg.mozilla.org/integration/mozilla-inbound/rev/bcad71cf222a - Noemi Erli - Backed out 2 changesets (bug 1525988, bug 1526053) for failures in browser_animation_playback-rate-selector.js and backout conflicts
16:44:35 <myk> jesup: no, moz_task can't implement the dispatchTime attribute for the reason given in the build error ("Rust only supports [ref] / [ptr] native types"), which is a limitation of the Rust-XPCOM bindings
16:45:26 <pulsebot> Check-in: https://hg.mozilla.org/integration/autoland/rev/58b16c9dabfd - Marco Bonardo - Bug 1519538 - Disable idle-daily notifications in tests through user.js files. r=Standard8
16:46:47 <pulsebot> Check-in: https://hg.mozilla.org/integration/autoland/rev/51697f8232c5 - Mike Hommey - Bug 1525999 - Default to arm instead of armv7a for android builds. r=nalexander
16:47:03 <jesup> froydnj: patch is split, and up for feedback to you (the -> Runnable part)
16:50:19 <jesup> myk: so.... if we do decide to add an TimeStamp to runnable, what do we do?  also, even if we don't add the attribute, we may be moving all the nsIRunnable uses to Runnable for the thread interfaces - which we started and half-completed around the 57 time.  Since moz_task uses nsIRunnable....
16:51:08 <jesup> froydnj: the patch only hits ~125 files or so ;-)
16:51:24 <froydnj> jesup: tiny
16:51:58 <jesup> yeah.  (and pretty mechanical, though there are some exceptions).  emacs macros FTW
17:03:27 <pulsebot> Check-in: https://hg.mozilla.org/integration/autoland/rev/9438194cc15b - Cosmin Sabou - Bug 1481099 - Disable /navigation-timing/nav2_test_document_open.html on linux for frequent failures. r=aryx
17:09:46 <zombie> Fallen: afaik, script loading currently depends on/needs a document, so not from chrome directly
17:10:06 <zombie> *es module loading
17:10:22 <Fallen> would the xul document work?
17:10:29 <zombie> idk
17:10:41 <Fallen> I found a bug that suggests there is no way to do it right now
17:10:49 <Fallen> bug 1311728
17:23:29 <pulsebot> Check-in: https://hg.mozilla.org/integration/autoland/rev/c63ff7601477 - Drew Willcoxon - Bug 1524714 - @ by itself should show all the engine @aliases. r=mak,dao
17:25:38 <pulsebot> Check-in: https://hg.mozilla.org/integration/autoland/rev/8553239366f0 - Kartikaya Gupta - Bug 1525450 - Guard against a null controller. r=botond
17:25:45 <bz> Gijs: So ....
17:26:02 <bz> Gijs: Are you around?
17:26:42 <jesup> mossop: So, I'm now getting "you've launched an older version of Firefox" when I started a 67.0a1 inbound local build, against one of my main "testing" profiles (which is always used with builds against inbound).  The changeset for this repo is older than the other one... (Jan 31 vs Feb 5) but both are 67
17:27:03 <bz> Gijs: Have some questions about preallocated about:newtab
17:27:11 <pulsebot> Check-in: https://hg.mozilla.org/integration/mozilla-inbound/rev/b0f67b8917f4 - Olli Pettay - Bug 1506376 - Make PCompositorBridge::DidComposite high priority message, r=aosmond
17:27:12 <jesup> It was inbound rev 456213 (Jan 31st); the other repo used with the profile was inbound dev 457186 (Feb 5th)
17:28:57 <pulsebot> Check-in: https://hg.mozilla.org/integration/autoland/rev/d8e74c762135 - Ted Campbell - Bug 1526324 - Stop transcoding NeedsArgsObj flag in XDRScript r=jandem
17:29:24 <Mossop> jesup: What matters is not the revision it was built from, but the build id. Though I have been wondering if we should just bypass this entirely in development builds
17:29:56 <jesup> do buildids have to be increasing?
17:30:18 <jesup> that's a real annoyance if you have multiple builds.  Even debug and opt of the same repo/rev
17:30:27 <Mossop> They are expected to be, by default they are generated based on the build time
17:30:59 <jesup> yeah, but if I have a repo, build debug, build opt, now I can't even switch between them
17:31:38 * bz hears echoes of his conversation with mossop. ;)
17:31:53 <jesup> Also: what if a X.0.1 sec release has a problem, and you need to go back to (or try) X.0?
17:32:25 <jesup> or "can you try that in X.0b(N-1)?"
17:32:55 <Mossop> Yeah, we're wrestling with this question, probably going to be a meeting next week to figure out where we should stand
17:33:14 <tcampbell> aosmond: My shaky-text-on-reddit bug is related to being scrolled to the exact middle of the page and the scrollbar itself wiggles in size... Suggests more likely layout than graphics?
17:33:34 <jesup> So I'm screwed until then?  ;-) :-/
17:33:48 <jesup> There isn't even an override as best I can tell
17:34:01 <Mossop> jesup: --allow-downgrade
17:34:09 <tcampbell> (also happens with or without webrender)
17:35:11 <pulsebot> Check-in: https://hg.mozilla.org/integration/autoland/rev/8a05a3f36cfe - arthur.iakab - Backed out changeset bac51381060d (bug 1519538) for causing multiple failures on Windows CLOSED TREE
17:35:45 <bz> Definite echoes.  ;)
17:35:54 <jesup> bz: :-)
17:36:26 <bz> The combination of -profile and --allow-downgrade sounds like the "I take my life into my own hands" mode we were operating in before and you want now.  ;)
17:36:45 <froydnj> --I-accept-the-consequences
17:36:51 * jesup adds --allow-downgrade to all his standard "start firefox
17:36:56 <bz> --on-my-head-be-it
17:37:51 * jesup does have profiles named "release", "beta", "nightly", etc -- plus dozens of others which are almost always nightly/inbound profiles
17:38:51 <jesup> it will be annoying, though since normally I type my ./firefox commands by hand
17:39:03 <Mossop> bz, jesup: Definitely taking notes over how this is impacting folks. JoeH is very much "block any downgrade by default" and QA are pushing for some method of downgrading. We're going to try to figure out soon where we want to end up on that line
17:39:07 * jesup history helps...
17:39:21 <bz> Mossop: makes sense
17:40:14 <bz> Mossop: I think what's best for our users at large may well not be what's best for developers or serious QA
17:40:27 <jesup> Is it a downgrade if it's the same release number?  (yes, a change *could* have landed and created an incompatibility -- how often, if ever, would we uplift such a change to beta - let alone release?
17:40:39 <bz> Mossop: I'm not sure whether we need anything more than the existing override, btw
17:40:59 <bz> mossop: For the developer/QA population
17:41:48 <Mossop> jesup: On nightly yes, but as you say, less likely on beta and release
17:41:58 <jesup> My concern is we may bite ourselves with users if it's purely build-date (such as the beta N-1 thing, especially if beta N has a bug that causes a problem for people)
17:42:05 <myk> jesup: (waylaid by meeting) unsure what we do if we add TimeStamp to nsIRunnable; i don't think there's a simple fix for the Rust-XPCOM binding limitation, or nika would've implemented it in the first place; and if we move all nsIRunnables to Runnable, then it also isn't clear to me how we proceed; would need to dig into the details
17:44:08 <nika> myk: Hmm?
17:44:10 <jesup> I thought the point of this was to stop users from downgrading releases (67->66, or 67 beta to 66 release, 68 nightly -> 67 beta, etc)
17:44:14 <nika> myk: my name was mentioned
17:44:43 <jesup> nika: starting around 11:09 EST here
17:45:17 <jesup> bug 1482244
17:47:22 <nika> jesup: So, the issue is that you want to implement an interface using Rust-XPCOM which has some weird native methods on it?
17:47:47 <nika> jesup: Specifically it looks like something returning a `TimeStamp`?
17:47:47 <myk> nika: jesup has a patch to add a TimeStamp attribute to nsIRunnable, which fails because moz_task now implements that interface, and "Rust only supports [ref] / [ptr] native types"
17:47:54 <jesup> Mossop: I'd be fine if Nightly was handled differently than beta/release; we should never uplift an incompatibility to those without some real consideration (if ever).  So on beta/release, major-rev-number should be good enough.  On Nightly, we should (IMHO) do no more than warn about buildid downgrades.  
17:48:16 <nika> myk: Ah yes
17:48:45 <nika> myk: This is one of the limitations of the way that rust-xpcom works out. 
17:48:51 <jesup> and even if we don't add it, we're talking about making the xpcom thread api's take Runnables instead of nsIRunnables
17:48:55 <nika> jesup: How does your code handle JS-implemented interfaces?
17:49:13 <jesup> it's [noscript]
17:49:41 <pulsebot> Check-in: https://hg.mozilla.org/integration/autoland/rev/857e68faa705 - arthur.iakab - Backed out changeset 51697f8232c5 (bug 1525999) for causing multiple Android  failures on build/target.raptor.tests.tar.gz CLOSED TREES
17:49:43 <nika> sure, but when js implements the interface, it will give a `NS_ERROR_NOT_IMPLEMENTED` or something like that when called, is that OK?
17:49:54 <jesup> sure
17:50:39 <nika> One option here would be that I could support NS_ERROR_NOT_IMPLEMENTED return values for unsupported rust methods
17:50:44 <pulsebot> Check-in: https://hg.mozilla.org/integration/mozilla-inbound/rev/963455160329 - Brian Hackett - Bug 1525482 - Remove pauseOnDOMEvents logic, r=lsmyth.
17:51:03 <nika> So you can implement an interface with incompatible methods, but everything else throws
17:51:08 <pulsebot> Check-in: https://hg.mozilla.org/integration/mozilla-inbound/rev/95bfbc0f513d - Brian Hackett - Bug 1525685 - Remove server side breakpoint sliding code, r=jlast.
17:52:38 <jesup> froydnj: how does JS-implemented Runnables (or what Nika refers to, which are still nsIRunnables, not Runnables) affect your thoughts on the thread API?
17:53:33 <nika> i would imagine we'd need to have js-implemented runnables be special objects subclassing Runnable, and use custom conversion logic
17:53:39 <nika> rather than the default XPCWrappedJSObject
17:53:49 <nika> Which TBH may be a reasonable idea either way
17:53:57 <jesup> We might sidestep  much of this by not putting this in Runnable/nsIRunnable, and implement shadow arrays for the event queues
17:54:34 <jesup> though perhaps froydnj has other reasons to move to Runnable
17:55:11 <jesup> in any case, the first patch on that bug will get us basically to the point where there are few if any nsIRunnables in c++
17:55:32 <jesup> which was started as part of quantum dom/etc IIRC
17:56:44 <pulsebot> Check-in: https://hg.mozilla.org/integration/autoland/rev/3d8811e611d9 - Cosmin Sabou - Backed out 2 changesets (bug 1519538) for linting failures. CLOSED TREE
18:01:00 <froydnj> jesup: nika's idea (JS things either subclass, or we wrap JSRunnables in Runnable) were what I was thinking of doing
18:01:26 <jesup> froydnj: ok
18:01:57 <froydnj> (my first thought would not have been to modify xpconnect, but nika is much smarter/braver than I am)
18:01:58 * jesup my patch doesn't touch JS
18:03:46 <nika> s/smarter/more naive/
18:04:41 <nika> froydnj: The main issue currently is just that nsIRunnables can be plain-old functions in JS-land
18:09:01 <froydnj> nika: would we win any harder by implementing xpconnect magic vs. having nsIRunnable be "JS only" and wrapping those up in a separately allocated Runnable?
18:09:08 <froydnj> better naming logic?
18:09:38 <nika> I'm not sure how you'd be doing that?
18:09:54 <froydnj> doing which?
18:09:57 <nika> Would we duplicate every method taking nsIRunnable to have two implementations?
18:10:11 <nika> One for JS runnables, and one for native ones?
18:10:48 <froydnj> I think so, but ideally that would not be very many places
18:11:13 <froydnj> mostly limited to nsIThread and related, for scriptability
18:11:24 <froydnj> I am not 100% sure we can get away with that
18:12:02 <nika>   froydnj Hmm, you're right
18:12:13 <nika> there aren't many nsIRunnables in scriptable contexts
18:12:20 <froydnj> but I think moving to a world where we have "the C++ interface" and then this JS thing on the side is in some ways better than "JS interface for everything" that we have now...and AIUI what is happening in other places in the tree
18:12:56 <pulsebot> Check-in: https://hg.mozilla.org/integration/mozilla-inbound/rev/5244b5aba6e3 - Brindusan Cristian - Backed out changeset b0f67b8917f4 (bug 1506376)for bc failures at browser_appmenu.js. CLOSED TREE
18:13:01 <nika> nsIEventTarget::Dispatch, nsIPermissionManager::whenPermissionsAvailable, nsIThreadManager::DispatchToMainThread, and nsIMemoryReporterManager::minimiseMemoryUsage
18:13:11 <froydnj> and if we did a "C++-native" interface, we could look at not having QI/AddRef/Release on Runnable, as some people have requested/complained
18:13:34 <nika> Those are all 4 scriptable nsIRunnables
18:13:47 <nika> (in xpconnect)
18:13:53 <nika> (may be more in webidl)
18:13:54 <froydnj> I hadn't even heard of whenPermissionAvailable
18:14:08 <nika> Yeah, that one's my fault
18:14:10 <froydnj> is minimiseMemoryUsage scriptable?  blah
18:14:35 <nika> we should make whenPermissionsAvailable not take a nsIRunnable - it only does 'cause I was lazy
18:15:34 <nika> But yeah, I suppose ideally we'd just make nsIEventTarget::Dispatch and nsIThreadManager::DispatchToMainThread not take runnables, but instead just take jsobjects, and then have a wrapper runnable which invokes `Call` on a jsval when it is run
18:17:17 <froydnj> does that mean we can't throw errors when calling Dispatch if you hand it a non-callable value?  I assume we would throw in that case today
18:18:28 <nika> Well kinda
18:18:48 <nika> IIRC it's complicated - if the object is a jsobject, I think we just let you
18:18:57 <nika> and the callable will then raise an `NS_ERROR_NOT_IMPLEMENTED` when called
18:19:14 <nika> Because when making XPCWrappedJSObject, we don't validate that methods are implemented
18:19:16 <froydnj> yeah
18:19:25 <nika> But I think we'd throw if passed a primitive
18:19:30 <nika> Which is easy to do 
18:19:46 <froydnj> but I don't think that error would ever get seen, because errors from Run() don't really get propagated anywhere
18:19:57 <nika> Yeah
18:20:11 <nika> What I'm saying is the new logic wouldnt necessarially be worse
18:20:24 <nika> in fact we could make it better by checking that the object looks callable explicitly
18:30:49 <pulsebot> Check-in: https://hg.mozilla.org/integration/mozilla-inbound/rev/65ed3025c271 - Geoff Brown - Bug 1526113 - Use new treeherder label for Android 7.0 x86_64 tests; r=aryx
18:37:09 <bholley> Huh, I seem to be getting infinite compilation doing an optimized arm build
18:37:24 <bholley> nalexander: is this something you've seen?
18:38:01 <nalexander> bholley: yes, we have seen compiler flakiness in various configurations, including many ICEs.
18:38:16 <nalexander> bholley: but never a run-away process.  You are running Mozilla's clangs, i.e., from bootstrap?
18:38:26 <pulsebot> Check-in: https://hg.mozilla.org/integration/autoland/rev/7168320522bb - Dana Keeler - bug 1526007 - don't return early from NSSCertDBTrustDomain::FindIssuer if NSS doesn't find any candidate issuers r=jcj
18:38:34 <bholley> nalexander: yes
18:38:36 <nalexander> bholley: you might ask in #build -- I need to step out -- but I don't know an open ticket for a run-away process.
18:38:58 <nalexander> bholley: on macOS, right?  I remember froydnj wondering how tested the macOS clangs were... maybe this jogs his memory?
18:39:03 <bholley> https://reviews.llvm.org/D53190?id=169371
18:39:05 * nalexander -> hair cut, back ~1.5hrs
18:39:09 <bholley> nalexander: thanks
18:39:17 <bholley> ^ looks like it's plausibly my issue
18:39:31 <pulsebot> Check-in: https://hg.mozilla.org/integration/autoland/rev/5a7505f349c9 - Ricky Rosario - Bug 1526366 - Pref Search in PBM to ON by default in 66 Release r=nanj
18:40:17 <bholley> (I got that by web searching for selectiondag::combine, which is where the profiler shows us spending time)
18:40:34 * bholley finds https://groups.google.com/forum/#!topic/llvm-dev/kFWEyf_r6Rg
18:41:53 <bholley> I guess that latter one is pretty old, but the former is relatively recent
18:43:35 <pulsebot> Check-in: https://hg.mozilla.org/integration/autoland/rev/dec5ea7cb93f - Mark Striemer - Bug 1500147 - Part 1: Record telemetry for about:addons pages r=rpl,janerik
18:43:38 <pulsebot> Check-in: https://hg.mozilla.org/integration/autoland/rev/0a5e45e57834 - Mark Striemer - Bug 1500147 - Part 2: Record LWT telemetry in customize mode r=rpl
18:43:41 <pulsebot> Check-in: https://hg.mozilla.org/integration/autoland/rev/286129ec63de - Mark Striemer - Bug 1500147 - Part 3: Record add-on manager telemetry for page/browser actions r=rpl
18:44:33 <pulsebot> Check-in: https://hg.mozilla.org/integration/mozilla-inbound/rev/faded33ed22f - Jon Coppeard - Bug 1524946 - Ensure script private value stays alive during dynamic module import r=jandem a=dveditz
18:47:58 <bholley> this was with a --enable-debug --disable-optimize build
18:48:23 <bholley> whereas my --disable-debug --enable-optimize build worked
18:48:53 * bholley tries on linux
18:49:33 <pulsebot> Check-in: https://hg.mozilla.org/releases/mozilla-beta/rev/e4896e3e5eb2 - Ryan VanderMeulen - No bug - Unset EARLY_BETA_OR_EARLIER. a=me
18:57:24 <pulsebot> Check-in: https://hg.mozilla.org/integration/mozilla-inbound/rev/afa72a00cd22 - Brindusan Cristian - Backed out changeset 95bfbc0f513d (bug 1525685) for xpcshell failures at test_setBreakpoint-on-line-with-no-offsets.js.
18:59:05 <pulsebot> Check-in: https://hg.mozilla.org/integration/autoland/rev/91eea845d54e - Daniel Holbert - Bug 1525628 part 1: Add nsIFrame::IsBlockFrameOrSubclass() convenience accessor. r=mats
18:59:08 <pulsebot> Check-in: https://hg.mozilla.org/integration/autoland/rev/ee8803de7f47 - Daniel Holbert - Bug 1525628 part 2: Generalize block-frame special case in scrollframe baseline code, so that it includes block subclasses like `<details>`. r=mats
18:59:22 <pulsebot> Check-in: https://hg.mozilla.org/integration/mozilla-inbound/rev/6780a779387f - Petr Sumbera - Bug 1526300 js/src/gc/Memory.cpp on Solaris should use posix_madvise instead of madvise. r=sfink
19:05:13 <Gijs> anyone know if there's an easy way to take a pushlog window (ie between changes X and Y) on hgweb and see the changes to a specific subset of files?
19:05:15 <pulsebot> Check-in: https://hg.mozilla.org/releases/mozilla-beta/rev/5a11b523f8e3 - Liang-Heng Chen - Bug 1504157 - Make onsize event triggered once. r=smaug, a=test-only
19:05:18 <pulsebot> Check-in: https://hg.mozilla.org/releases/mozilla-beta/rev/9b3d629cf829 - Jan-Ivar Bruaroey - Bug 1522773 - Reject any outstanding DOMMediaStream::TrackPort::BlockSourceTrackId() promise on teardown to avoid recent debug assert permafail. r=bwc, a=RyanVM
19:05:21 <pulsebot> Check-in: https://hg.mozilla.org/releases/mozilla-beta/rev/fc9047475850 - Rob Wu - Bug 1498636 - Separate "include" variables from manifest defaults. r=ahal, a=test-only
19:07:24 <Mossop> Gijs: I know locally you can do logs at the directory level
19:07:36 <RyanVM> something a la http://benjamin.smedbergs.us/blog/2012-07-09/using-mercurial-revsets-to-search-for-changes-between-firefox-releases/ ?
19:08:06 <RyanVM> I believe hgweb can accept revsets too, but haven't ever really played with it
19:10:20 <pulsebot> Check-in: https://hg.mozilla.org/integration/autoland/rev/c53f5b9d5a75 - Andy Wingo - Bug 1526279 - Fix BigInt constructor API CallArgs usage r=terpri,jorendorff
19:10:39 <pulsebot> Check-in: https://hg.mozilla.org/integration/mozilla-inbound/rev/7922211d6dcb - André Bargull - Bug 1525352: Add VM-call for TypedArray constructor with ArrayBuffer argument. r=anba
19:12:51 <pulsebot> Check-in: https://hg.mozilla.org/integration/mozilla-inbound/rev/96c7bf80c0f2 - André Bargull - Bug 1523633: Use guardIsNumber in GetPropIRGenerator::tryAttachPrimitive to properly handle int32 values. r=anba
19:14:57 <pulsebot> Check-in: https://hg.mozilla.org/integration/mozilla-inbound/rev/f8aeed3352c0 - Paolo Amadini - Bug 1519095 - Allow changing the text of an existing notification. r=jaws
19:17:24 <Pike> can one use markdown in bugzilla user stories?
19:19:05 <pulsebot> Check-in: https://hg.mozilla.org/integration/autoland/rev/607f7e4918e9 - Michael Kaply - Bug 1525357 - Don't allow third party installs if xpinstall disabled by policy. r=kmag
19:19:20 <pulsebot> Check-in: https://hg.mozilla.org/integration/autoland/rev/b3bcf26d4dad - Gijs Kruitbosch - Bug 1252831 - remove 30boxes webcal handler as both possible and default handler, r=paolo,Dolske
19:19:22 <jesup> Gijs: well, hg log foo/bar, then filter by date I suppose
19:19:25 <Pike> Gijs: https://hg.mozilla.org/releases/mozilla-beta/help/revisions, pushdate(interval), not sure what the exact syntax for interval is
19:19:34 <jesup> or rather by rev:rev
19:19:37 <Gijs> yeah, I managed to do it locally
19:19:44 <Gijs> it turned out the pushlog was wrong, which explained a lot :)
19:19:52 <Pike> :-)
19:23:17 <pulsebot> Check-in: https://hg.mozilla.org/integration/autoland/rev/bfc4490bedbd - Robert Strong - Bug 1525858 - Use AppConstants and mozinfo where possible instead of the preprocessed test file. r=mhowell
19:24:07 <pulsebot> Check-in: https://hg.mozilla.org/integration/autoland/rev/75766cc3befd - Yuan Cheng - Bug 1519923 - Migrate about:rights to Fluent, r=jaws,flod
19:26:22 <pulsebot> Check-in: https://hg.mozilla.org/integration/mozilla-inbound/rev/9841e4b69dc2 - Jon Coppeard - Bug 1524946 - Fix rooting hazard r=me
19:27:09 <pulsebot> Check-in: https://hg.mozilla.org/integration/autoland/rev/010180a16818 - Thomas Wisniewski - Bug 1524897 - drop line-height:normal rule in html.css for <sub> and <sup> for interop. r=emilio
19:36:30 <pulsebot> Check-in: https://hg.mozilla.org/integration/autoland/rev/ba5eb152fe23 - k88hudson - Bug 1526066 - Add perf improvements, design refinements, and bug fixes to Activity Stream r=nanj
19:37:34 <dmajor> jmaher: is there a dashboard of all arm64 test failures somewhere? I don't need (but can accept) them broken down by test type. is looking at the dependency tree of bug 1522997 a good approximation?
19:37:57 <jmaher> dmajor: I just look at https://bugzilla.mozilla.org/showdependencytree.cgi?id=1522997&maxdepth=2&hide_resolved=1
19:38:24 <jmaher> dmajor: that is probably 70% of the failures- hopefully you can help get fixes for many of the bugs there
19:38:30 <dmajor> ok, thanks -- I
19:38:39 <dmajor> I'll use that as my source for picking up new investigations
19:38:44 <jmaher> dmajor: we have 1 machine hooked up to try, so if you need to run anything, let me know- itis sort of hacky atm
19:39:17 <jmaher> dmajor: cool; also note, we will have more machines hooked up to try and not have it so hacky in a few more weeks
19:39:49 <dmajor> jmaher: great! maybe by then I'll have more time to look at them :) I meant to start this week but it didn't happen
19:41:46 <jmaher> we should have a better handle on bugs on file for all failures in the next week- timing is working out well :)
19:52:59 <pulsebot> Check-in: https://hg.mozilla.org/integration/autoland/rev/79fae15dc460 - Chris Manchester - Bug 1480362 - Remove |./mach pastebin| command now that the service is no longer available. r=firefox-build-system-reviewers,nalexander
19:54:19 <jmaher> dmajor: one area in that list of bugs is there are some webgl/canvas tests that are failing- I am not sure if they are all related- if you know anything about that stuff it would be nice to get context
19:55:03 <dmajor> jmaher: unfortnuately gfx is likely one of the areas where I can be least helpful, but I'll try to take a quick look to see if it's maybe just a superficial thing
19:55:39 <jmaher> a case of your guess is as good as mine probably
20:03:07 <pulsebot> Check-in: https://hg.mozilla.org/integration/autoland/rev/8452583c0771 - Mike Conley - Bug 1523648 - Remove BUSY_TAB_ABANDONED probe and tests. r=dao
20:04:39 <pulsebot> Check-in: https://hg.mozilla.org/integration/mozilla-inbound/rev/5faca6c35b2b - Gabriel Luong - Bug 1525988 - Don't render invisible declarations in the new rules view. r=mtigley
20:09:04 <jaws> whimboo: gandalf: i added a question for both of you on https://phabricator.services.mozilla.com/D17393
20:09:25 <jaws> https://phabricator.services.mozilla.com/D17393#490605
20:13:26 <tjr> snorp: Do you know the prevalence of media decoding support for Fennec offhand?  https://developer.mozilla.org/en-US/docs/Web/HTML/Supported_media_formats says things like '[Fennec] supports [h264] in some cases, but only when a third-party decoder is available, and the device hardware can handle the profile used to encode the MP4'  but is that 90% of the
20:13:26 <tjr> time, 99%, 99.9%, etc...
20:20:59 <pulsebot> Check-in: https://hg.mozilla.org/integration/autoland/rev/731773b839bb - shindli - Backed out changeset bfc4490bedbd (bug 1525858) for xpcfailures in uriloader/exthandler/tests/unit/test_handlerService.js CLOSED TREE
20:22:29 <snorp> tjr: ah, that's super old
20:22:36 <snorp> tjr: we should be able to do h264 on every device we run on
20:23:05 <snorp> I mean I guess it's technically a true statement since we do rely on OS decoder
20:23:43 <snorp> but it's almost always going to play unless it's some super weird profile (like lossless or something)
20:24:45 <tjr> Same story with vp9, aac, and the other formats? Everything is supported same as desktop?
20:25:07 <snorp> tjr: for vp9 we probably end up using a software decoder much of the time
20:25:25 <snorp> aac is supported by the OS
20:25:31 <snorp> theora/ogg are software
20:25:39 <snorp> err vorbis
20:26:02 <snorp> tjr: h264 is really the only thing we don't have a software fallback for, IIRC, but it has been a while since I really looked
20:26:21 <snorp> tjr: jhlin is the android media guy :)
20:28:36 <rstrong> stefan_hindli|sheriffduty: were there other candidates for the failure that bug 1525858 was backed out for? I ask because xpcshell tests all run in their own process, the changes in bug 1525858 were entirely app update test only changes, and I have numerous recent try runs (around 4 AM this morning is the latest) where there were no issues.
20:29:12 <pulsebot> Check-in: https://hg.mozilla.org/integration/mozilla-inbound/rev/6370b90702b7 - Brindusan Cristian - Backed out 2 changesets (bug 1524946) for hazard build bustages.
20:30:19 <stefan_hindli|sheriffduty> rstrong: the xpcfailures seem to have started from your push
20:34:06 <rstrong> looks like marAppInUseStageSuccessComplete_unix.js failed or timed out (can't tell since there isn't logging of the parallel run) and succeeded the sequential run. Though that shouldn't pollute other tests the harness might have something that allows it.
20:43:28 <mhowell> stefan_hindli|sheriffduty: it looks like the push immediately before rstrong's is directly related to test_handlerService.js, but rstrong's definitely isn't
20:44:25 <stefan_hindli|sheriffduty> mhowell: yeah it looks like it, I'll reland rstrong's push 
20:44:47 <rstrong> Thanks mhowell !
20:45:54 <rstrong> stefan_hindli|sheriffduty: thanks for relanding!
20:46:22 <stefan_hindli|sheriffduty> rstrong: no problem, sorry for the mixup, I really thought your push was the one responsible for the xpcshell failures.
20:46:32 <rstrong> no worries
20:47:04 * rstrong remembers when I had to periodically sheriff... sheriffing can be hard
20:47:22 <stefan_hindli|sheriffduty> 😊
20:52:36 <pulsebot> Check-in: https://hg.mozilla.org/integration/autoland/rev/3c3b09d5cadd - Robert Strong - Bug 1525858 - Use AppConstants and mozinfo where possible instead of the preprocessed test file. r=mhowell
20:53:04 <smaug> florian: ping
20:53:20 <smaug> or mconley 
20:53:24 <florian> pong
20:53:54 <mconley> also pong
20:55:45 <smaug> florian: trying to understand https://searchfox.org/mozilla-central/source/browser/base/content/test/performance/head.js#68
20:56:13 <smaug> florian: any time there is any event, we mark all layout dirty?
20:56:54 <smaug> (the context here is that bug 1506376 makes already very common bug 1489259 permafail )
20:57:23 <florian> yes, the idea is to make the layout dirty before executing any script, so that any time a script does something that _could_ trigger a sync flush, it will trigger it and get caught by the test
20:57:25 <smaug> (bug 1506376 let's us paint more often if normal priority queue is flooded)
20:58:12 <smaug> florian: ahaa, so it is like approximation of "script will run"
20:58:33 <florian> yes
20:59:01 <sefeng> hey guys, I want to upload a patch to a bug, do I just do "git diff" and then upload the file?
20:59:17 <pulsebot> Check-in: https://hg.mozilla.org/integration/autoland/rev/3038495b95fc - shindli - Backed out changeset b3bcf26d4dad (bug 1252831) for xpcshell failures in uriloader/exthandler/tests/unit/test_handlerService.js CLOSED TREE
21:02:17 <kumar> Hi sefeng. You should be able to find some help with that in the #introduction channel.
21:02:18 <daleharvey> sefeng: git format-patch is usually a little nicer
21:02:41 <daleharvey> But yeh, its that simple ..... for now 
21:03:53 <sefeng> kumar daleharvey: thanks!
21:05:09 <smaug> florian: I'm having hard time to understand how this setup can work with popuppositioned
21:05:15 <smaug> which is dispatched if reflow is done
21:05:50 <smaug> should popuppositioned get similar handling as MozAfterPaint
21:06:19 <florian> I'm not familiar with popuppositioned
21:09:40 <smaug> florian: https://searchfox.org/mozilla-central/source/layout/xul/nsMenuPopupFrame.cpp#1606-1609,1614
21:11:08 <florian> if ignoring that event fixes the test, it's probably a fine thing to do
21:11:32 <smaug> florian: ok, let me push to try
21:11:50 <florian> (request review from Mike, he's more familiar than me with the reflow part of these tests)
21:13:13 <smaug> ok
21:13:17 <smaug> thanks
21:17:53 <pulsebot> Check-in: https://hg.mozilla.org/integration/autoland/rev/4e56ef85817a - Andrew Halberstadt - Bug 1526129 - Fix test failure in tryselect cram tests, r=jmaher
21:23:57 <Mossop> froydnj: Seems like this is actually wrong: https://phabricator.services.mozilla.com/differential/changeset/?ref=583281&whitespace=ignore-most. On OSX it still ends up with MOZ_HAS_REMOTE defined, it's value is '' though (literally two quote characters)
21:24:44 <emilio> smaug: :o you just made me realize that's also probably the cause of the weirdness I found when changing how overflow events work
21:24:52 <froydnj> Mossop: ah, yeah, you want `if 'gtk' in toolkit or toolkit == 'windows': return True` and that's it
21:25:12 <froydnj> sorry, I written those kind of tests scores of times
21:25:26 <Mossop> Ah it has to return None to be undefined. Ok
21:27:19 <jya> RyanVM: ping
21:27:28 <RyanVM> jya: pong
21:27:49 <jya> I see you are doing release uplift.
21:27:58 <smaug> emilio: during testing?
21:28:03 <smaug> or just this event
21:28:21 <smaug> I guess the event could cause something weird
21:28:33 <smaug> since sure something is trying to handle it
21:28:41 <smaug> and then causing more layout changes..
21:28:45 <jya> How big is the window to get https://bugzilla.mozilla.org/show_bug.cgi?id=1525230 in?
21:29:26 <RyanVM> jya: pretty simple patch - how worried would it make you doing so without it going into a beta first?
21:34:31 <jya> RyanVM: not much, otherwise the pref one line change in bug 1520200
21:34:59 <jya> Bug 1520200 is theorically safer
21:37:47 <RyanVM> jya: pref off returns us to fx64 behavior?
21:45:53 <jya> RyanVM: yes (sorry had to park the car) 
21:46:01 <RyanVM> jya: assuming that's the case (flipping the pref returns us to 64 behavior), I think I prefer that for the dot release given the short window we have here
21:46:04 <RyanVM> and the real fix for beta
21:46:09 <jya> Me too. 
21:46:15 <jya> Ok
21:46:25 <jya> I'll nominate the pref change for release then. 
21:46:32 <RyanVM> perfect, thanks
21:49:41 <stefan_hindli|sheriffduty> rstrong: it seems that the push I backed out and relanded needs to be backed out again since it has some android failures - https://treeherder.mozilla.org/#/jobs?repo=autoland&resultStatus=testfailed%2Cbusted%2Cexception&classifiedState=unclassified&fromchange=8a05a3f36cfe5c577d906f4e6fcf1f47c8650d8f&selectedJob=227261543
21:54:20 <jya> RyanVM: done
21:55:27 <rstrong> stefan_hindli|sheriffduty: those tests aren't even included with android builds or even run on android
21:56:52 <stefan_hindli|sheriffduty> https://treeherder.mozilla.org/#/jobs?repo=autoland&resultStatus=success%2Cpending%2Crunning%2Ctestfailed%2Cbusted%2Cexception&classifiedState=unclassified&fromchange=8a05a3f36cfe5c577d906f4e6fcf1f47c8650d8f&selectedJob=227265709&searchStr=android%2C4.3%2Capi16%2B%2Copt%2Cmochitests%2Ctest-android-em-4.3-arm7-api-16%2Fopt-mochitest-chrome-4%2Cm%28c4%29
21:57:46 <stefan_hindli|sheriffduty> I'm curious if the c4 on the push right before yours turn out green
21:57:52 <rstrong> stefan_hindli|sheriffduty: I looked at the log and none of my code is included
21:58:31 <rstrong> Also, android doesn't use toolkit's app update at all.
22:00:26 <rstrong> Nothing under toolkit/mozapps/update/ is included by android
22:00:27 <rstrong> https://searchfox.org/mozilla-central/source/toolkit/moz.build#25
22:03:28 <Mossop> rstrong: That's not true on nightly, though it will be soon (bug 1520248)
22:05:12 <rstrong> Mossop: Actually it is true. I almost replied to that thread because of people thinking balrog and app update are synonymous. Android / fennec used app update several years ago but they decided to create their own update mechanism that doesn't use any of toolkit/mozapps/update/.
22:05:31 <Mossop> Oh, huh
22:07:02 <rstrong> They didn't even create a new define which caused a few problems at first
22:07:06 <rstrong> bug 1152634
22:11:39 <stefan_hindli|sheriffduty> rstrong: so it looks like your push is causing those c4 failures
22:12:38 <rstrong> stefan_hindli|sheriffduty: truly, there is no way it could. The code isn't included with android and hasn't been for many many years.
22:14:07 <stefan_hindli|sheriffduty> yeah, it looks like your push is ok, it's the one before yours that seems to cause this
22:14:12 <rstrong> I verified again that the code I landed isn't in that build
22:14:25 <stefan_hindli|sheriffduty> I already backed that out so we should be fine
22:14:37 <stefan_hindli|sheriffduty> eh, sorry for wasting your time
22:15:25 <rstrong> There you go... no worries. I'm actually laughing a bit deliriously about it since I've only had a couple of hours sleep
22:16:39 <stefan_hindli|sheriffduty> yeah, you and me both, I guess it's one of those days :))
22:23:38 <bytesized> Could anyone point me towards the correct mechanism for async file writing from JS? I remember using it before, but I can't seem to remember what it is called or find it.
22:25:13 <Mossop> bytesized: OS.File
22:25:16 <lina> bytesized: https://developer.mozilla.org/en-US/docs/Mozilla/JavaScript_code_modules/OSFile.jsm
22:25:19 <lina> jinx :-)
22:25:28 <bytesized> lol, thank you both. that is it
23:34:16 <qdot> We're gonna be able to blame mozilla trees burning for global warming soon.
23:40:48 <pulsebot> Check-in: https://hg.mozilla.org/releases/mozilla-beta/rev/bc23b491541e - Oriol Brufau - Bug 1522820 - Disable browser action's "Remove Extension" if the addon can't be uninstalled. r=Felipe a=lizzard
23:40:51 <pulsebot> Check-in: https://hg.mozilla.org/releases/mozilla-beta/rev/b61d0fdc785b - Jared Wein - Bug 1523696 - Wait until the style attribute is applied to the dialog before sizing it. r=Gijs a=RyanVM
23:40:54 <pulsebot> Check-in: https://hg.mozilla.org/releases/mozilla-beta/rev/ee2b9d8d4b0b - Jared Wein - Bug 1524210 - Only clear the richlistbox selection when the box itself is clicked if the richlistbox is of multiselection type. r=Gijs a=lizzard
23:40:57 <pulsebot> Check-in: https://hg.mozilla.org/releases/mozilla-beta/rev/daa824c92e14 - Jean-Yves Avenard - Bug 1525230 - Reset ImageRect when resolution change. r=bryce a=RyanVM