mozilla :: #developers

21 Apr 2017
00:04pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/afe865149e0e - Daosheng Mu - Bug 1358010 - Skip vr mochitests when running on release and non-Windows platforms; r=kip
00:11dmajornjn: have you worked much with EnsureWalkThreadReady() and friends?
00:11njndmajor: if I have I've forgotten most of it :/
00:11njndmajor: but if you have a question, fire away
00:12dmajornjn: oh, just wanted to see if you wanted to steal some reviews, since markus hasn't spent much time in those either
00:12* njn coughs
00:12njn"wanted" is a strong word
00:15pulsebotCheck-in: https://hg.mozilla.org/integration/mozilla-inbound/rev/634530ff65a6 - Blake Kaplan - Bug 1357909 - Tweak the strings explaining to the user what's happening. r=ochameau
00:20froydnjis there some way to turn off event listeners on nodes in devtools like one can in chrome?
00:24pulsebotCheck-in: https://hg.mozilla.org/integration/mozilla-inbound/rev/0f2c055d69b2 - Mats Palmgren - Bug 1357655 part 1 - [css-ui] Make 'none' the initial value for 'appearance' for web and UA compatibility. Add 'appearance:auto' to UA sheets for form controls, XUL etc where needed. r=bz
00:24pulsebotCheck-in: https://hg.mozilla.org/integration/mozilla-inbound/rev/c132d492e5ba - Mats Palmgren - Bug 1357655 part 2 - [css-ui] Add an explicit 'appearance:auto' to a couple of reftest that needs it.
00:35pulsebotCheck-in: https://hg.mozilla.org/integration/mozilla-inbound/rev/4bc74de54b5b - Blake Kaplan - Bug 1357959 - Put people who forced into e10s into the multi test group. r=Felipe
00:49pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/ab3beca99070 - Andrzej Hunt - Bug 1357630 - infer: (NULL_DEREFERENCE) add null-check to getAppEventDispatcher r=nalexander
00:50pulsebotCheck-in: https://hg.mozilla.org/integration/mozilla-inbound/rev/e064b7cde203 - Shu-yu Guo - Bug 1357483 - Stop changing the end position of parenthesized expression ParseNodes. (r=Waldo)
01:23pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/9faa446314c3 - Tom Tung - Bug 1358035: Move the check for persisting an invalid origin to test_bad_origin_directory.js and add skip beta/release flag on it since we don't check invaild origins on beta/release. r=janv
01:36pulsebotCheck-in: https://hg.mozilla.org/integration/mozilla-inbound/rev/6deb37b2d3a1 - Chris Peterson - Bug 1356843 - Fix -Wcomma warnings in js/. r=jorendorff
01:36pulsebotCheck-in: https://hg.mozilla.org/integration/mozilla-inbound/rev/bdef06c12da6 - Chris Peterson - Bug 1356843 - Enable -Wcomma clang warnings. r=glandium
02:04pulsebotCheck-in: https://hg.mozilla.org/integration/mozilla-inbound/rev/618da494f8ec - Ryan VanderMeulen - Bug 1358335 - Update pdf.js to version 1.8.243. r=bdahl
02:05pulsebotCheck-in: https://hg.mozilla.org/integration/mozilla-inbound/rev/435e638babd1 - Markus Stange - Bug 1349418 - Remove checkerboarding code and just use an opaque background color behind root scroll frames. r=kats
02:06pulsebotCheck-in: https://hg.mozilla.org/integration/mozilla-inbound/rev/3ccf396a61fe - Matt Woodrow - Bug 1349418 - Require ASRs to match before allow occlusions between layers. r=mstange
02:19pulsebotCheck-in: https://hg.mozilla.org/integration/mozilla-inbound/rev/d8fb7831a005 - Iris Hsiao - Backed out changeset 16ec1022fb84 (bug 1358230) for test_formSubmission.html time out in windows
02:27pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/aee196633a86 - Alex Touchet - servo: Merge #16509 - Add new Servo icon for Windows (from atouchet:win-ico); r=jdm
02:33pulsebotCheck-in: https://hg.mozilla.org/integration/mozilla-inbound/rev/9fd7a385411f - David Anderson - Fix a race condition between VideoBridge and Compositor thread shutdown. (bug 1356365, r=mattwoodrow)
02:44pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/afdbbb354869 - Luke Chang - Bug 1355438 - [Form Autofill] Implement an internal-only pseudo-class for highlighting elements with an autofilled value. r=heycam
02:49philortt: thank you for taking care of test_persist.js - I was just about to file on it, but decided to read the day's bugmail first :)
02:50ttphilor: np ;)
02:53pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/a5572924d69a - Daosheng Mu - Bug 1353523 - Add adjustable threshold value for VR controller trigger inputs; r=kip
03:02pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/3b8079a1c135 - Jing-wei Wu - Bug 1232439 - Part 1: Setup A/B experiment for enabling full bookmark management in Nightly. r=Grisha
03:02pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/59af50c06ae5 - Jing-wei Wu - Bug 1232439 - Part 2: Support get/update/remove bookmark record by ID in BrowserDB. r=Grisha
03:02pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/5a0b0722bb13 - Jing-wei Wu - Bug 1232439 - Part 3: Implement full-page bookmark edit dialog. r=ahunt,Grisha
03:02pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/0cbed2ce192a - Jing-wei Wu - Bug 1232439 - Part 4: Adjust bookmark list UI layout. r=ahunt
03:03pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/e81924e4cff3 - bechen - Bug 1353689 - selectors/cue : Enable testcases under */selector/* . Fix the padding and overflow issue. r=rillian
03:03pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/ebf24732e9b3 - bechen - Bug 1353689 - selectors/cue_function/italic_object: update testcase status. r=rillian
03:03pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/5179ebf1e982 - bechen - Bug 1353689 - selectors/cue_function/bold_object: update testcase status. r=rillian
03:03pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/d3ced5751998 - bechen - Bug 1353689 - selectors/cue_function/class_object: update testcase status. r=rillian
03:04pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/8d8d2824a763 - bechen - Bug 1353689 - selectors/cue_function/underline_object: update testcase status. r=rillian
03:04pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/7f2790b4c963 - bechen - Bug 1353689 - selectors/cue_function/voice_object: update testcase status. r=rillian
03:04pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/44c5f4b4cdfc - bechen - Bug 1353689 - selectors/default_styles: update testcase status. r=rillian
03:04pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/9a5b36fefb22 - bechen - Bug 1353689 - selectors/cue_function: update testcase status. r=rillian
03:04pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/80ac3ea1427e - bechen - Bug 1353689 - update /processing-model/* status. Disable tests whose result are not consistent. r=alwu
03:11pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/8c3b45902628 - Hiroyuki Ikezoe - Bug 1352628 - Skip clipped-animated-transform-1.html. r=boris
03:16pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/e7a44be96de2 - JW Wang - Bug 1357983 - remove media::Microseconds. r=gerald
03:17pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/e535df594d22 - JW Wang - Bug 1357986 - Use helper functions of TimeUnit to make code more readable. r=kaku
03:18ewongany dom peers around?
03:23ewongfwiw.. https://dxr.mozilla.org/mozilla-central/source/dom/security/nsContentSecurityManager.cpp has |#include "nisIStreamListener.h"| mentioned on both lines #4 and #8.. is this necessary?
03:30pulsebotCheck-in: https://hg.mozilla.org/integration/mozilla-inbound/rev/fcd98542af39 - Cervantes Yu - Bug 1024669 - Part 1: Annotate crash reports with thread names. r=gsvelto
03:30pulsebotCheck-in: https://hg.mozilla.org/integration/mozilla-inbound/rev/1105f666b019 - Cervantes Yu - Bug 1024669 - Part 2: Test cases for thread name annotation in the crash report. r=gsvelto
03:34pulsebotCheck-in: https://hg.mozilla.org/integration/mozilla-inbound/rev/7952b06fc0d0 - Nicholas Nethercote - Bug 1356694 - Create a ProfileGatherer in nsProfiler() if the profiler is active. r=mstange.
03:35jdmewong: it is not
03:36ewongjdm thanks. should I just file a bug and attach a patch?
03:36jdmewong: sure!
03:39* philor fears he might be a paper tiger
03:40philorI filed 55-as-beta bugs against people I don't know, who scrambled to fix them right away, and against people I do know, who scratched themselves and shifted their weight in their chair
03:49pulsebotCheck-in: https://hg.mozilla.org/integration/mozilla-inbound/rev/3b55ee79bb68 - Dragana Damjanovic dd.mozilla@gmail.com - Bug 1310197 - Fix comment.r=me
03:54ewongis dom/security part of the Security module or just the dom module?
03:54ewonglooks like it's just dom
03:58ewongdoes anyone know b_z's mozreview handle?
04:01ewongah nvm...
04:01ewongbz not entirely sure if I should've used mozreview, but I figured this is the best one for sheriffs
04:02pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/99fe1b020ce2 - Patrick McManus - Bug 1297481 - add shutdown state to protocolproxyservice r=bagder
04:25pulsebotCheck-in: https://hg.mozilla.org/integration/mozilla-inbound/rev/29e1433cdec5 - Dan Glastonbury - Bug 1346665 - P1: Enable USE_PULSE_RUST in libcubeb. r=kinetik,rillian
04:25pulsebotCheck-in: https://hg.mozilla.org/integration/mozilla-inbound/rev/d7a79e8ceb58 - Dan Glastonbury - Bug 1346665 - P2: Import rustified cubeb_pulse.c. r=kinetik,rillian
04:31pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/52e6fe988bd5 - Manish Goregaokar - Bug 1358362- stylo: Whitelist ShapeSource outparams in bindings to satisfy heap write analysis; r=bholley
04:35pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/32ae2383a6d4 - JW Wang - Bug 1348053 - add the UrgentStart flag to the channel when the loading is initiated by a user interaction for quicker network response. r=cpearce
04:39pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/f866afd3d2d2 - Cameron McCormack - Bug 1257451 - Make |mach run| use --debugger-args instead of --debugparams, like the mochitest/reftest commands do. r=gps
05:02pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/640ff6dddc6c - Masayuki Nakano - Bug 1217700 part.1 nsIWidget should return reference to IMENotificationRequests r=m_kato
05:02pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/0ea1fc4888f0 - Masayuki Nakano - Bug 1217700 part.2 IMEContentObserver should observe all possible notifications and check if it should be notified when it occurs r=m_kato
05:02pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/ab221fb1ba32 - Masayuki Nakano - Bug 1217700 part.3 Expose text change, selection change and position change notifications to nsITextInputProcessorCallback with nsITextInputProcessorNotification r=smaug
05:02pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/ca065b2e52d9 - Masayuki Nakano - Bug 1217700 part.4 Add automated tests for IMEContentObserver r=m_kato
05:19pulsebotCheck-in: https://hg.mozilla.org/integration/mozilla-inbound/rev/289645ac65c1 - Timothy Nikkel - Bug 1342567. r=aosmond a=abillings
05:19pulsebotCheck-in: https://hg.mozilla.org/integration/mozilla-inbound/rev/f577512b1a29 - Timothy Nikkel - Bug 1342567. r=aosmond a=abillings
05:38pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/7b43acb94854 - Iris Hsiao - Backed out 9 changesets (bug 1353689) for web platform reftest failures
05:52pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/7a69f0845fa1 - Hiroyuki Ikezoe - servo: Merge #16553 - Support vendor prefix keyframes rule (from hiikezoe:vendor-prefix-keyframes); r=emilio
06:02pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/049d6f1921bd - Hiroyuki Ikezoe - Bug 1356779 - Add tests for interaction between prefixed and non-prefixed @keyframes rules with the same name. r=dholbert
06:04pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/e5f7c3e9f1b9 - Kevin Chen - Bug 1356537 - Revert device reset check in EndFrame; r=dvander
06:20pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/b858187d6cb8 - Mark Hammond - Bug 1355677 - Make all Sync network requests promise based. r=kitcambridge,rnewman,tcsc
06:29pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/b7a058d98d2d - Boris Chiou - servo: Merge #16554 - stylo: Bug 1332633 - Implement ComputeDistance trait (from BorisChiou:stylo/animation/compute_distance); r=emilio
06:32pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/f902d870c642 - Boris Chiou - Bug 1332633 - Add Servo_AnimationValues_ComputeDistance. r=emilio
06:44pulsebotCheck-in: https://hg.mozilla.org/integration/mozilla-inbound/rev/2414ce702247 - Ehsan Akhgari - Bug 1358004 - Remove the REMOTE_JAR_PROTOCOL_USED telemetry; r=jduell
07:16ewongckerschb: re: https://reviewboard.mozilla.org/r/132274/ is it sufficient to update the commit message via |hg qrefresh -m "... r=ckerschb" and then do another push mozreview?
07:27pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/f0127a117bcb - btian - Bug 1357638 - Use Preferences::AddBoolVarCache() to accelerate dom.forms.inputmode access in IMEStateManager::SetIMEState(), r=masayuki
07:28pulsebotCheck-in: https://hg.mozilla.org/integration/mozilla-inbound/rev/3a7c9793ce9d - Morris Tseng - Bug 1358066 - Remove the painted layer invalidate region calculation if the painted layer has been optimized to color or image layer. r=mattwoodrow
07:44pulsebotCheck-in: https://hg.mozilla.org/releases/mozilla-beta/pushloghtml?startID=7210&endID=7211 - 13 changesets (bugs 1346825, 1356820, 1350324, 1350925, 1352387 and 2 other bugs)
07:50pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/b2c250009e4b - Mark Banner - Bug 1348097 - Fix intermittent failures caused by Experiments.jsm by ensuring the preference monitoring is in sync. r=gfritzsche
08:00pulsebotCheck-in: https://hg.mozilla.org/releases/mozilla-beta/rev/dd92c1f26a08 - L10n Bumper Bot - no bug - Bumping Fennec l10n changesets r=release a=l10n-bump
08:05pulsebotCheck-in: https://hg.mozilla.org/integration/mozilla-inbound/rev/7a446a56598f - Jan de Mooij - Bug 1357680 part 1 - Track Ion-inlined scripts explicitly so we can inline functions with unknown properties. r=bhackett
08:07ckerschbewong: I am not sure actually, but I would say yes
08:07ckerschbewong: worst case we set the checkin-needed keyword
08:07ewongckerschb: I think I'll do that instead. :)
08:08ckerschbewong: what I meant is, do an 'hg qrefresh -m "...r=ckerschb" and if it doesn't work we set the checkin-needed keyword :-)
08:08ewongah ok..
08:12pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/4eaa5414d10e - Johan Lorenzo - Bug 1354038 - [push-apk] taskgraph: Use rollout and deactivate dry-run on release r=aki
08:13freddyb...to use autoland in mozreview you need L3, if not someone else can autoland if you request checkin-needed via bugzilla
08:13ewongckerschb: ok.. now that I've done the qrefresh..
08:14ewongI thought sheriffs were the only ones who can push autoland
08:18ewongif anyone has a better understanding of mozreview here, can someone point out if https://reviewboard.mozilla.org/r/132272/ is ready for autoland?
08:19ewongwhen c_kerschb did the r+ apparently it added the r=ckersch_b to it
08:19ewongit being the commit message
08:21Standard8ewong: Im not sure what the issue is?
08:21Standard8mozreview will automatically add r=<whoever> or change r?<whoever> to r=<whoever>
08:22Standard8saves pushing another update ;-)
08:24ewongStandard8: not an issue, per se.. just curious if I can autoland.. I think what threw me off a bit was the &quot;please update the commit message&quot; but now that I&#39;ve confirmed it&#39;s just &quot;add the r=ckerschb&quot; then yeah.. I think there isn&#39;t an issue.
08:24ewongif ^ makes any sense.
08:25Standard8ewong: thats a check the commit message is ok, and update if necessary
08:26ewongah ok
08:27ewongjust pushed it to try
08:30ewongStandard8: thanks!
08:32pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/934221c412fc - Alexandre Poirot - Bug 1348761 - Fix object inspection performances against objects with large Typed Arrays. r=bgrins
08:34pulsebotCheck-in: https://hg.mozilla.org/releases/mozilla-beta/rev/937f0773ea76 - Johan Lorenzo - Bug 1354038 - [push-apk] taskgraph: Use rollout and deactivate dry-run on release r=aki a=release
08:37Tomcat|sheriffdutybaku: ping
08:37bakuTomcat|sheriffduty: pong
08:38Tomcat|sheriffdutyhey baku seems with uplifts i cause somehow a bustage on beta like https://treeherder.mozilla.org/logviewer.html#?job_id=93236469&repo=mozilla-beta
08:38Tomcat|sheriffdutybaku: could this be caused by https://hg.mozilla.org/releases/mozilla-beta/rev/89318a1e562bca3d9c17987335f66f4c260cd6cd ?
08:39* baku checking
08:40bakuTomcat|sheriffduty: yes.
08:40bakuhttps://hg.mozilla.org/releases/mozilla-beta/file/89318a1e562bca3d9c17987335f66f4c260cd6cd/dom/cache/DBSchema.cpp#l2559 this must be updated.
08:41Tomcat|sheriffdutybaku: ok how we do solve this ?
08:42Tomcat|sheriffdutybacking out 89318a1e562bca3d9c17987335f66f4c260cd6cd or updating the file ?
08:42bakuTomcat|sheriffduty: let&#39;s update the file. I can give you a patch. 2 minutes.
08:43Tomcat|sheriffdutybaku: ok cool :)
08:45pulsebotCheck-in: https://hg.mozilla.org/mozilla-central/pushloghtml?startID=31686&endID=31687 - 64 changesets (bugs 1348097, 1351657, 1358197, 1357867, 1357226 and 31 other bugs)
08:46bakuTomcat|sheriffduty: https://pastebin.mozilla.org/9019585
08:48Tomcat|sheriffdutybaku: ok cool thanks!
08:48Tomcat|sheriffdutyjmaher|afk: you around ?
08:49jmaher|afkTomcat|sheriffduty: yeah, what is up
08:49* jmaher|afk around for about 30 minutes
08:49Tomcat|sheriffdutyhey jmaher|afk i try to merge m-i to m-c but run into a merge conflict
08:50Tomcat|sheriffdutyjmaher|afk: https://tomcat.pastebin.mozilla.org/9019586 - merge conflict
08:50Tomcat|sheriffdutyand caused by https://hg.mozilla.org/integration/mozilla-inbound/rev/68842e58a066 i guess
08:50Tomcat|sheriffdutybc: ^
08:52jmaher|afkTomcat|sheriffduty: ok, so bc made changes and we had some changes for disabling two tests in the same chunk
08:52jmaher|afkso in that you want the working copy
08:52Tomcat|sheriffdutyjmaher|afk: yeah and now with autoland merged first this bites with m-c
08:52Tomcat|sheriffdutyjmaher|afk: ok cool
08:53jmaher|afkalthough I wish the disabling was via skip-if
08:53jmaher|afkoh well
08:53bcWhy?
08:53bcthe skip-if = android_version # no simulcast support on android
08:54Tomcat|sheriffdutyjmaher|afk bc seems https://hg.mozilla.org/mozilla-central/rev/9bfeb0378415 is the change on autoland
08:54bcis broken isn&#39;t it? Didn&#39;t I replace that in my patch with toolkit == &#39;android&#39; ?
08:54Tomcat|sheriffdutyand https://hg.mozilla.org/mozilla-central/rev/4417d91fd6b2
08:55jmaher|afkbc: those 2 tests are skipped everywhere, so |skip-if = true| should be the solution
08:55bcjmaher|afk: skipped everywhere? or just android?
08:55jmaher|afkbc: skipped everywhere
08:55jmaher|afkbut the # is ok, then we don&#39;t lose out on skipping on android
08:57pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/ffdb24073101 - Thomas Nguyen - Bug 1353853 - Cache preferences when doing channel classify r=francois
08:57Tomcat|sheriffdutyjmaher|afk: ok thanks pushed
08:57pulsebotCheck-in: https://hg.mozilla.org/mozilla-central/pushloghtml?startID=31687&endID=31688 - 33 changesets - merge mozilla-inbound to mozilla-central a=merge
08:58Tomcat|sheriffdutylets see if this works
08:58pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/125045e56532 - Fred Lin - Bug 1356957 - call updateRequest once when update request in netmonitor-controller;r=rickychien
08:58pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/f6a1f5acfd58 - Fred Lin - Bug 1356957 - combine double updateRequest call while receive event in _onNetworkEventUpdate;r=rickychien
08:58jmaher|afkTomcat|sheriffduty: cool
09:00ihsiao|sheriffdutyjandem-away: ping
09:01Tomcat|sheriffdutyihsiao|sheriffduty: yeah need to back out
09:01ihsiao|sheriffdutyok, will do
09:03pulsebotCheck-in: https://hg.mozilla.org/integration/mozilla-inbound/rev/14b635fc67e1 - Jeff Walden - Remove WasmBaselineCompile.cpp&#39;s unused |TlsSlot{Size,Offset}| static globals. No bug, rs=shu
09:03pulsebotCheck-in: https://hg.mozilla.org/integration/mozilla-inbound/rev/f57dbdd29d41 - Jeff Walden - Bug 1351107 - Refactor the code that identifies the offset of an impermissible function name (because of strict-mode or extra-warnings restrictions) so that SyntaxParseHandler doesn&#39;t depend on a cached TokenStream&. r=arai
09:03pulsebotCheck-in: https://hg.mozilla.org/integration/mozilla-inbound/rev/1f88113d1a9d - Jeff Walden - Bug 1351107 - Remove ParseHandler::[gs]etPosition, remove SyntaxParseHandler::tokenStream now that it&#39;s unused, and make FullParseHandler::tokenStream a TokenStreamBase to not drag in a character-type dependency. r=arai
09:03Tomcat|sheriffdutybaku: thanks :) is this a follow up to a existing bug ?
09:03pulsebotCheck-in: https://hg.mozilla.org/integration/mozilla-inbound/rev/0930544a0236 - Jeff Walden - Bug 1351107 - Make ParseContext contain a TokenStreamBase&, not a TokenStream&, so that ParseContext need not be character-type-parametrized. r=arai
09:03pulsebotCheck-in: https://hg.mozilla.org/integration/mozilla-inbound/rev/6b3e668eefd5 - Jeff Walden - Bug 1351107 - Remove all use of TokenStream from the two ParseHandlers. r=arai
09:03pulsebotCheck-in: https://hg.mozilla.org/integration/mozilla-inbound/rev/5649e25c4c42 - Jeff Walden - Bug 1351107 - Split js::frontend::Nestable into a separate header in ds/ to simplify the frontend header it&#39;s in now. r=arai
09:03pulsebotCheck-in: https://hg.mozilla.org/integration/mozilla-inbound/rev/0d069bcc20b0 - Jeff Walden - Bug 1351107 - Add a |typename CharT| template parameter to Parser, SyntaxParseHandler, and FullParseHandler, *without* otherwise changing functionality. r=arai
09:03Tomcat|sheriffdutybaku: or shall i file a new bug ?
09:04pulsebotCheck-in: https://hg.mozilla.org/integration/mozilla-inbound/rev/3ccc9c855766 - Iris Hsiao - Backed out changeset 7a446a56598f (bug 1357680) for leak issue
09:04bakuTomcat|sheriffduty: I would add the patch as follow up of bug 1357799
09:04ihsiao|sheriffdutyjandem-away: ^
09:04Tomcat|sheriffdutybaku: ok sir
09:14jandem-awayihsiao|sheriffduty: oops, sorry
09:14ihsiao|sheriffdutyjandem: no worries :)
09:18pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/c5ab3b44dbf1 - Marco Bonardo - Bug 1354681 - Intermittent timeout in browser_ext_omnibox.js. r=mattw
09:21Tomcat|sheriffdutybaku: pushed and thanks again
09:21pulsebotCheck-in: https://hg.mozilla.org/releases/mozilla-beta/rev/8ad85f4bb919 - Andrea Marchesini [:baku] - Bug 1357799 - followup to fix bustage a=bustagefix
09:21bakuTomcat|sheriffduty: np
09:27pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/6d201ceaff82 - Andreas Tolfsen - Bug 1356229 - Remove marionette.forcelocal preference; r=whimboo
09:27pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/0625a195fcf5 - Andreas Tolfsen - Bug 1356229 - Prevent non-loopback connections in Marionette; r=whimboo
09:27pulsebotCheck-in: https://hg.mozilla.org/mozilla-central/rev/dd530a59750a - Henrik Skupin - Bug 1358402 - Keep &#39;marionette.defaultPrefs.enabled&#39; around as fallback. r=ato a=tomcat
09:30pulsebotCheck-in: https://hg.mozilla.org/releases/mozilla-beta/rev/ede33bdbabd8 - Henrik Skupin - Bug 1358402 - Keep &#39;marionette.defaultPrefs.enabled&#39; around as fallback. r=ato a=test-only
09:34Tomcat|sheriffdutyerm baku
09:34Tomcat|sheriffdutyanother bustage
09:34bakuTomcat|sheriffduty: same issue? same files?
09:34Tomcat|sheriffdutybaku: https://treeherder.mozilla.org/logviewer.html#?job_id=93256274&repo=mozilla-beta
09:35Tomcat|sheriffdutydifferent file
09:36Tomcat|sheriffdutySylvestre: ^
09:37bakuTomcat|sheriffduty: same way to fix it :)
09:37bakuhttps://pastebin.mozilla.org/9019591
09:37Tomcat|sheriffdutybaku: ok cool :)
09:39ihsiao|sheriffdutybaku: hi, I was tried to back out bug 1357799, but I got the artifact build busted like https://treeherder.mozilla.org/#/jobs?repo=try&revision=3afe119ea370484317c63c27bcdfb27f6d896af1&noautoclassify&selectedJob=93242562
09:39ihsiao|sheriffdutynot sure if this is also need to fix
09:41Tomcat|sheriffdutybaku: ok next try :)
09:41bakuTomcat|sheriffduty: heh
09:41pulsebotCheck-in: https://hg.mozilla.org/releases/mozilla-beta/rev/7fad67b91a08 - Andrea Marchesini [:baku] - Bug 1357799 - followup to fix another bustage a=bustagefix
09:43Tomcat|sheriffdutyihsiao|sheriffduty: i guess this is a try specific thing
09:43Tomcat|sheriffdutyhas this before i guess
09:44ihsiao|sheriffdutyok, cool
09:44Tomcat|sheriffdutyihsiao|sheriffduty: erm bug don&#39;t backout 1357799 now :)
09:44Tomcat|sheriffdutydon&#39;t backout this bug now i mean :)
09:44ihsiao|sheriffdutyyeah
09:44Tomcat|sheriffdutysince it has now 2 followups :)
09:45ihsiao|sheriffdutyI know, I was trying to figure out if it&#39;s the root casue
09:46Tomcat|sheriffdutyheh
09:50pulsebotCheck-in: https://hg.mozilla.org/integration/mozilla-inbound/pushloghtml?startID=89434&endID=89435 - 67 changesets - Merge mozilla-central to mozilla-inbound
09:57Tomcat|sheriffdutybaku: https://treeherder.mozilla.org/logviewer.html#?job_id=93265373&repo=mozilla-beta
09:57Tomcat|sheriffdutybaku: i guess we need number 3
09:57Tomcat|sheriffdutySylvestre: ping
09:58SylvestreTomcat|cloud, pong
09:58bakuTomcat|sheriffduty: ok. Let me see if we have this pattern of #ifdef() elsewhere.
09:58Tomcat|sheriffdutyhey Sylvestre seems your patch in bug 1357799 detected some problems
09:58Sylvestreok, you can backout it
09:58Sylvestrejcristau suggested bug 1358442
09:59Tomcat|sheriffdutybaku: ^ ok with backing out
09:59bakuSylvestre: well, I guess there are just some #ifdef to fix.
09:59bakuSylvestre: do you know if there is a better way to do:#if defined(DEBUG) || !defined(RELEASE_OR_BETA) || defined(EARLY_BETA_OR_EARLIER)
09:59baku?
09:59Sylvestrebaku, indeed, but how many ? :)
09:59Sylvestrebaku, I don&#39;t know :/ I guess not
09:59* baku doing grep
10:00bakuSylvestre: I think we don&#39;t have more than 3.
10:00bakuTomcat|sheriffduty: if you want to wait, I can give you 1 last patch.
10:00Sylvestrebaku, ok, we can just try to fix the last one
10:00Tomcat|sheriffdutybaku: yeah sure
10:00Sylvestreif fails again, let&#39;s backout
10:00Tomcat|sheriffdutyyeah
10:07bakuSylvestre: r+ of this: https://pastebin.mozilla.org/9019594 ?
10:12Tomcat|sheriffdutybaku: ok waiting for Sylvestre&#39;s r+ and then i push
10:13pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/pushloghtml?startID=41219&endID=41220 - 35 changesets - Merge mozilla-central to autoland
10:21Tomcat|sheriffdutySylvestre: r+?
10:35marcomhoye: ping
10:37pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/1d6502afc19f - Rajesh Kathiriya - Bug 1354513 - Removed duplicate ESLint recommended rules in formautofill r=standard8
10:45Tomcat|sheriffdutybaku: Sylvestre will backout to get a working tree again
10:47pulsebotCheck-in: https://hg.mozilla.org/releases/mozilla-beta/rev/6867fd82bb40 - Carsten &quot;Tomcat&quot; Book - Backed out changeset 7fad67b91a08 (bug 1357799)
10:47pulsebotCheck-in: https://hg.mozilla.org/releases/mozilla-beta/rev/568167f30a09 - Carsten &quot;Tomcat&quot; Book - Backed out changeset 8ad85f4bb919 (bug 1357799)
10:47pulsebotCheck-in: https://hg.mozilla.org/releases/mozilla-beta/rev/84360b114045 - Carsten &quot;Tomcat&quot; Book - Backed out changeset 89318a1e562b (bug 1357799) to unbreak the beta-tree
10:51pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/5096d1d0cff2 - Hiroyuki Ikezoe - servo: Merge #16557 - Servo does not need to support -moz-keyframes (from hiikezoe:dont-support-moz-keyframes); r=SimonSapin
11:05pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/dea1c646413a - Marco Bonardo - Bug 1358368 - Ensure we read values from a current db snapshot in places/tests/favicons/test_root_icons.js. r=standard8
11:06pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/cc853449321f - Marco Bonardo - Bug 1321533 - places/tests/unit/test_telemetry.js assumes the decay time must be positive. r=standard8
11:37Sylvestrebaku, Tomcat|sheriffduty good, (sorry, i was eating)
11:38Tomcat|sheriffdutySylvestre: sorry had to backout to get a working tree
11:38Tomcat|lunchbut good reminder to take food :)
12:26pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/8ecca35b9ae3 - JW Wang - Bug 1358401. P1 - reject the promise when failing to fetch the buffer. r=gerald
12:26pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/cd4a2197143e - JW Wang - Bug 1358401. P2 - use the &#39;once&#39; handler to remove the |firstOpen| check. r=gerald
12:26pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/02fb1ea3df4f - JW Wang - Bug 1358401. P3 - reject the promise in LoadTest() when AppendTrack() fails. r=gerald
12:26pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/f91f21db5c8e - JW Wang - Bug 1358401. P4 - reject the promise with an error string in UpdateSessionFunc(). r=gerald
13:03pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/ba94dfa2c7f3 - Gijs Kruitbosch - Bug 1352664 - temporarily increase threshold for per-compartment timing checks to avoid intermittent orange, r=Yoric
13:15pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/0972898afcd2 - nchevobbe - Bug 1358103 - Remove createFactory call on Reps; r=jdescottes
13:17Standard8freddyb: you need to mark the issue as fixed on https://reviewboard.mozilla.org/r/131980/#issue-summary
13:20Standard8unless florian is around to mark it for you ;-)
13:21florianStandard8: done
13:21Standard8thanks!
13:22pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/402503d06db8 - Frederik Braun - Bug 1358050: Add no-implied-eval rule to eslint config r=standard8
13:23pulsebotCheck-in: https://hg.mozilla.org/integration/mozilla-inbound/rev/0d56e78960ef - Jon Coppeard - Bug 867815 - Add post barrier for visitIteratorStart r=jandem
13:23pulsebotCheck-in: https://hg.mozilla.org/integration/mozilla-inbound/rev/a2a4a9a0231d - Jon Coppeard - Bug 867815 - Fix visitIteratorStart shape/group guard for unboxed objects r=bhackett
13:24gastonanyone knows whats the technology used by the new google earth so that&#39;s its reserved to chrome ?
13:24johannhI have a fresh Windows 10 install and downloaded and installed the mozilla-build package, but for some reason it includes an outdated version of mercurial (3.7.3)
13:24froydnjgaston: PNaCl
13:25gastondammit
13:25johannhso many extensions refuse to work and working is a pain
13:25johannhhas anyone encountered this?
13:26padenotjohannh, you can udate it with pip
13:26pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/38732fc0f437 - Carsten &quot;Tomcat&quot; Book - Backed out changeset f6a1f5acfd58 (bug 1356957)
13:26pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/5e4195f34934 - Carsten &quot;Tomcat&quot; Book - Backed out changeset 125045e56532 (bug 1356957) for frequent failures in browser_net_security-error.js
13:26padenotpip install --upgrade mercurial or something like that
13:26johannhpadenot: thanks! trying...
13:27johannhpadenot: yay!
13:27johannhthank you!
13:27padenotjohannh, yw
13:28pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/2d53e3a676bb - Johan Lorenzo - Bug 1355018 - Make post-release steps use the unified repo r=rail
13:36mikeratcliffeDexter: ping
13:36Dextermikeratcliffe, pong
13:36mikeratcliffeDo you have any idea why we switched from JSON to YAML when Scalars.YAML was created?
13:37mikeratcliffeSomebody keeps asking me and you created the file.
13:38Dextermikeratcliffe, we switched the format as it had some of the features that we required and that are not available in JSON (e.g. comments)
13:38mikeratcliffeAh, perfect
13:38Dexterwell, it&#39;s not really switching to be precise, as scalars didn&#39;t exist before :)
13:38mikeratcliffeNeeded something solid for some in tree docs
13:39mikeratcliffeI know, but you know what I mean.
13:39mikeratcliffeThx
13:39Dexteryou&#39;re welcome
13:39DexterThis was discussed on some bug though
13:39Dexterlet me see if I can find that bug
13:40pulsebotCheck-in: https://hg.mozilla.org/integration/mozilla-inbound/rev/954eeda43262 - Jan de Mooij - Bug 1357680 part 1 - Track Ion-inlined scripts explicitly so we can inline functions with unknown properties. r=bhackett
13:41Dextermikeratcliffe, https://docs.google.com/document/d/13mcY17RzpE3BsB4CgR6DidytWzl4tMpAEmrvf8fOSJ8/edit#heading=h.sjwwi262bwic
13:42Dextermikeratcliffe, referenced in the scalars metabug (bug 1275517)
13:42mikeratcliffeWow, that is even better... thx
13:42Dexteryou&#39;re welcome :)
13:46mhoyemarco: yo
13:46mhoye&#39;sup
13:47marcomhoye: hello, I was wondering what is the process now for choosing GSoC students
13:48marcothere are a set of proposals that were good enough for me; should I choose one of them myself?
13:49mhoyeYeah, you&#39;ll have to pick one. Students can have more than one mentor, but mentors must have only one student.
13:50marcomhoye: ok. When do I have to tell you my choice?
13:50marcomhoye: and can I give you a ranked list, just in case one of them ends up doing something else?
13:53mhoyeWe&#39;ve got a bit of administrative juggling to do, because we got 2 less slots than we asked for, but some people are listed as mentoring more than one slot, so we&#39;ve got to clear that up.
13:54mhoye(Also, I&#39;m doing this for the first time, so I&#39;ve got questions there too.)
13:54mhoyeShort version is, I&#39;ll send some email this afternoon that&#39;ll probably answer those questions.
13:54mhoyeKnock on wood, etc.
13:55pulsebotCheck-in: https://hg.mozilla.org/mozilla-central/rev/c8198aa6e767 - Carsten &quot;Tomcat&quot; Book - Backed out changeset 35ef9cd54519 (bug 1358142) for breaking android nightlys
13:58bagderblah, my lateset nightly now doesn&#39;t reload my pinned tabs when I restart, but it reloads all other tabs fine...
13:59mhoyeIs there an only-reload-pinned-tabs option under the hood somewhere?
14:00bagderthe pinned ones are totally broken, I can&#39;t navigate anywhere in them even manually
14:00bagderthey show a blank page then
14:01bagderI could unpin, close, create new, go there, pin, restart and that worked
14:01pulsebotCheck-in: https://hg.mozilla.org/integration/mozilla-inbound/pushloghtml?startID=89437&endID=89438 - 2 changesets - Merge mozilla-central to mozilla-inbound
14:02bagderand that one keeps working after multiple restarts, just not the ones that were pinned from before...
14:03pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/pushloghtml?startID=41230&endID=41231 - 2 changesets - Merge mozilla-central to autoland
14:17pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/fee151a11c16 - Thomas Nguyen - Bug 1358333 - Fix APPLICATION_REPUTATION_ALLOWLIST_MATCH and APPLICATION_REPUTATION_BLOCKLIST_MATCH telemetry r=francois
14:25marco_mhoye: sorry, I missed your reply
14:25marco_I was offline
14:29pulsebotCheck-in: https://hg.mozilla.org/releases/mozilla-beta/rev/ca38cf3eba17 - Bob Clary - Bug 1352333 - sync autophone webrtc test manifests with normal webrtc manifests, r=jmaher,dminor, a=test-only.
14:39pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/fd25c2c275fc - Ted Campbell - Bug 1353358 - Use IfThenElseEmitter when optimizing spreadcall r=jandem
14:39pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/077dfe082524 - Ted Campbell - Bug 1353358 - Support JSOP_OPTIMIZE_SPREADCALL in Ion r=jandem
14:40mhoyemarco_: Short version is, I&#39;ll send some email this afternoon that&#39;ll probably answer those questions.
14:40marco_mhoye: ok, thanks!
14:41mhoyeWell, thank me if it actually answers your questions? Otherwise it&#39;s just more email.
14:53pulsebotCheck-in: https://hg.mozilla.org/releases/mozilla-esr52/rev/83229e29b1a5 - ffxbld - No bug, Automated HSTS preload list update from host bld-linux64-spot-307 - a=hsts-update
14:53pulsebotCheck-in: https://hg.mozilla.org/releases/mozilla-esr52/rev/a64384626286 - ffxbld - No bug, Automated HPKP preload list update from host bld-linux64-spot-307 - a=hpkp-update
14:53pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/a4212d9e1d0b - Brad Werth - Bug 1302470 Part 1: Remove trailing whitespace. r=mstange
14:53pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/87ae43ab52ed - Brad Werth - Bug 1302470 Part 2: Create a IsRangeRendered function to test range visibility in the display list. r=mstange,smaug
14:53pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/1d664865e8e2 - Brad Werth - Bug 1302470 Part 3: Branch IsRangeVisible to delegate to IsRangeRendered when range is in viewport. r=masayuki
14:53pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/71de099b2eb7 - Brad Werth - Bug 1302470 Part 4: Fix the case where HTML buttons need to generate display item children when doing opaque hit tests. r=mattwoodrow
14:54pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/4fc16536ce41 - Brad Werth - Bug 1302470 Part 5: Change nsDisplayList::GetOpaqueRegion non-opaque lists to build up a region from its children. r=mattwoodro