mozilla :: #developers

13 Jul 2017
00:01pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/77fac7f5d728 - Andr Bargull - Bug 1379639 - Move Number.isInteger to self-hosted code and avoid bailouts in isSafeInteger. r=jandem
00:01pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/d7a872ed8840 - Andr Bargull - Bug 1379983 - Part 1: Move InlineCharBuffer to top of file. r=jandem
00:01pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/37e091fb7f79 - Andr Bargull - Bug 1379983 - Part 2: Use InlineCharBuffer for String.fromCharCode and escape. r=jandem
00:01pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/942e95e4964d - Andr Bargull - Bug 1379983 - Part 3: Remove goto statements from Unescape function and some other clean-ups. r=jandem
00:01pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/4e27c8cad5d1 - Andr Bargull - Bug 1380087 - Add an intrinsic to test for Map and Set objects. r=till
00:04pulsebotCheck-in: https://hg.mozilla.org/integration/mozilla-inbound/rev/b049d1bf3c46 - Aaron Klotz - Bug 1380158: Use the aIID parameter supplied to mscom::ProxyStream's constructor to simplify and speed up proxy unmarshaling; r=jimm
00:05pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/a2f57517bf30 - Honza Bambas - Bug 1378370 - Kill switch preference for HTTP urgent-start prioritization. r=valentin
00:06pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/2fb8cf9807ba - Nick Alexander - Bug 1371445 - Pre: Produce and upload XML and HTML findbugs reports. r=maliu
00:06pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/91701101061d - Nick Alexander - Bug 1371445 - Pre: Fail unit tests if unlimited strength JCE is not installed. r=maliu
00:06pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/3159a025e57a - Nick Alexander - Bug 1371445 - Pre: Upload HTML checkstyle report. r=maliu
00:06pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/485b02d68d8a - Nick Alexander - Bug 1371445 - Add |mach android {lint,findbugs,checkstyle,test}| commands for running Android-specific test suites. r=gps
00:07pulsebotCheck-in: https://hg.mozilla.org/integration/mozilla-inbound/rev/90ae2fcd089a - Kris Maglione - Bug 1378727: Use the zip reader cache when reading extension locales. r=aswan
00:10pulsebotCheck-in: https://hg.mozilla.org/integration/mozilla-inbound/rev/a4490a86f11f - Kris Maglione - Bug 1380267: Fix shutdown blocker corner cases. r=aswan
00:58pulsebotCheck-in: https://hg.mozilla.org/mozilla-central/pushloghtml?startID=32163&endID=32164 - 135 changesets - Merge autoland to central, a=merge
01:00pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/6ddf3b3e56d3 - Kartikaya Gupta - Bug 1378966 - Don't cause immediate throttling of the refresh driver on repeat transactions. r=mattwoodrow
01:06pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/3f965dd570d3 - Nils Ohlmeier [:drno] - Bug 1264479: added [current|pending][Local|Remote]Description to WebIDL. r=smaug
01:06pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/6e1a4b5f7420 - Nils Ohlmeier [:drno] - Bug 1264479: added implementation for [current|pending][Local|Remote]Description. r=bwc
01:06pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/83ba04669582 - Nils Ohlmeier [:drno] - Bug 1264479: added unit tests for [current|pending][Local|Remote]Description. r=bwc
01:11dbaronNothing like fixing one of the test failures I was hitting locally by merging to tip a six year old patch that didn't land originally because I took too long to review it.
01:15pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/84de8f2d744f - Bobby Holley - servo: Merge #17701 - reuse the bloom filter and style sharing cache across traversals (from bholley:reuse_allocations); r=emilio
01:15pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/65d39277c962 - Servo VCS Sync - No bug - Revendor rust dependencies
01:22njncan I change a |wstring| parameter in an IDL method to |AUTF8String| without affecting JS/add-on compatibility? I *think* the answer is yes...
01:27pulsebotCheck-in: https://hg.mozilla.org/integration/mozilla-inbound/pushloghtml?startID=92505&endID=92506 - 138 changesets - Merge m-c to inbound, a=merge
01:28pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/pushloghtml?startID=46376&endID=46377 - 4 changesets - Merge m-c to autoland, a=merge
01:30pulsebotCheck-in: https://hg.mozilla.org/releases/mozilla-beta/rev/4a2dd6e45c4c - Ray Lin - Bug 1367875 - Add a reftest about brief dynamically chaning <audio> to a small width. r=jaws, a=test-only
01:30pulsebotCheck-in: https://hg.mozilla.org/releases/mozilla-beta/rev/f37efe938cd4 - Jared Wein - Bug 1377302 - Support fill=&#39;context-fill&#39; for WebExtension browser-actions. r=dao, r=mattw, a=lizzard
01:30pulsebotCheck-in: https://hg.mozilla.org/releases/mozilla-beta/rev/e537bfdef7c9 - Jared Wein - Bug 1379464 - Enable context-paint for mozilla-created webextensions. r=heycam, r=kmag, a=lizzard
01:30pulsebotCheck-in: https://hg.mozilla.org/releases/mozilla-beta/rev/4cd92600443e - Chris Pearce - Bug 1374475 - Don&#39;t throttle download of media files under 8MB in size. r=jwwang, a=jcristau
01:31John-Galtnjn: Yes
01:31njnJohn-Galt: thx
01:54pulsebotCheck-in: https://hg.mozilla.org/integration/mozilla-inbound/rev/e6e712904806 - Randall Barker - Bug 1379628 - Ensure pixels for Android dynamic toolbar snapshot get processed even if the toolbar is not visible so pending animations may run r=kats
02:05pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/be2c265f67c4 - Kaku Kuo - Bug 1380244 - trigger skip-to-next-key-frame quicker; r=jya
02:16pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/106e48e65b1b - Ray Lin - Bug 1378773 - Replace &quot;Options&quot; with &quot;More Options&quot; in the autofill dropdown footer to conform with the spec. r=lchang
02:38pulsebotCheck-in: https://hg.mozilla.org/integration/mozilla-inbound/pushloghtml?startID=92507&endID=92508 - 16 changesets (bugs 667079, 1379334, 1380521, 1308876)
02:46pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/ededddfadca7 - Kearwood &quot;Kip&quot; Gilbert - Bug 1287944 - Improve interaction with Oculus Home r=daoshengmu
02:56njncan an IDL |string| arg be passed non-ASCII strings from JS? Will it end up encoding as UTF8?
03:12pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/1bb454794eab - Emilio Cobos lvarez - servo: Merge #17693 - Revert &quot;Auto merge of #17633 - servo:rustup, r=nox&quot; (from emilio:revert-rustup); r=nox
03:16bz_awaynjn: did you get your string question answered?
03:16njnbz_away: no
03:17njnbz_away: I was looking at variant code, it looked like a non-ASCII JS string would be lossy-converted through a |string| argument
03:17njnbut I wasn&#39;t totally sure
03:17bznjn: |string| is lossy
03:17bznjn: It uses JS_EncodeStringToBuffer
03:17bzhttp://searchfox.org/mozilla-central/source/js/xpconnect/src/XPCConvert.cpp#548-586
03:18bznjn: which just drops the high byte of every 16-bit thing
03:18bznjn: So non-ascii will not work right at all.
03:19njnbz: exactly what I need, thank you
03:19bzno problem
03:19* bz looks forward to the day when we remove &quot;string&quot; and &quot;wstring&quot; from xpidl completely
03:19bzOh, in a debug build non-ascii will assert
03:20njnbz: nice thing about |string| is you can use C++ string literals in C++ code
03:20njnFoo(&quot;blah&quot;) is nicer than Foo(NS_LITERAL_STRING(&quot;blah&quot;))...
03:20bzSure
03:21bzNothing prevents having a noscript thing for that
03:21bz(well, I guess if the impl is in JS it does)
03:21njnbz: you might be interested in bug 1380227
03:21njnbz: I&#39;m currently reconsidering the design a little, but the basic idea holds
03:22bzfun
03:22bzSo wait. Speedometer is partially gated on warnings we emit for deprecated stuff? :(
03:23* bz wonders whether we have JS impls of nsIStringBundle
03:23bzI would think not
03:23pulsebotCheck-in: https://hg.mozilla.org/integration/mozilla-inbound/rev/ff468ed8e5a4 - Jessica Jong - Bug 1379108 - (followup) Fix localization notes for FormValidationDateTimeRangeOverflow/Underflow strings. comment-only, DONTBUILD
03:23bzSo...
03:24bznsDependentCString(aName) is not safe if aName is null
03:24bzWhich &quot;string&quot; allows....
03:25bzThis API is so sucky.
03:25bzOK, so
03:25bzGiven that you already have the string in nsAutoString form
03:25bzIs there a reason to make the return value wstring instead of AString?
03:26bzThe FormatString case is a bit harder...
03:26njnbz: I&#39;m not touching the return value
03:27bzwell, you&#39;re giving your new thing the same return value as the existing thing
03:27bzAnd I&#39;m wondering whether that&#39;s the right call. ;)
03:28njnok
03:28bz // nsTextFormatter does not use the shared nsMemory allocator.
03:28bzBald-faced lie nowadays, I think
03:28* bz files
03:28njnbz: I&#39;m now considering this design:
03:28njn[noscript] wstring GetStringFromName(in string aName)
03:28njn[binaryName(GetStringFromWName)] wstring GetStringFromName(wstring)
03:29njnthe first one is new; the second one is the old one, but renamed for C++ code
03:29bzright
03:29njnI could change the return value of the new one
03:29njnAUTF8STring?
03:29bzOr just AString
03:29njnI haven&#39;t paid any attention to how the return values are used
03:30bzProbably badly... ;)
03:30njnyeah, not sure if I want this scope creep
03:30bzI mean, in JS that&#39;s all the same
03:30bzbut in C++....
03:30bzThe main benefit there would be reducing the number of allocations we do
03:30njnsure
03:30bzRight now we allocate. Then clone. Then clone again. Then clone again.
03:30bzafaict.
03:30njnthe main changes so far on the C++ code have ben changing lots of u&quot;foo&quot; literals to &quot;foo&quot;, which was easy
03:31* bz double-checks
03:31bzI think doing a followup for the different signature is probably fine
03:33njnbz: thunderbird uses this a lot, alas
03:33njnwhich complicates things
03:33bzfwiw, filed https://bugzilla.mozilla.org/show_bug.cgi?id=1380549
03:33bzDoes it need non-mechanical changes?
03:33bzBecause if not, filing a bug on it is probably fine
03:34njnbz: depends
03:34bz(I mean, if you do whip up a patch that&#39;s cool too)
03:34njncould just mass change GetStringFromName to GetStringFromWName for no change in behaviour
03:34njncould also use the new GetSTringFromName where it makes sense, e.g. u&quot;foo&quot; literals
03:34pulsebotCheck-in: https://hg.mozilla.org/integration/mozilla-inbound/rev/eb39f860d7b3 - Tom Ritter - Bug 1380083 - Pull in upstream ICU patch that changes &#39;\&#39; to &#39;/&#39; in #include. r=andre,m_kato
03:34njnwhich is not that much harder
03:34bzThose all sound like changes Jorgk could do
03:35njnugh, this doesn&#39;t work:
03:35njn[noscript] wstring GetStringFromName(in string aName)
03:35njn [binaryName(GetStringFromWName)] wstring GetStringFromName(wstring)
03:35njnbecause the name is reused in IDL
03:37njnmaybe this will do:
03:37njn [binaryname(GetStringFromWName)] wstring GetStringFromName(in wstring aName);
03:37njn [noscript, binaryname(GetStringFromName)] wstring GetStringFromNameCpp(in string aName);
03:39pulsebotCheck-in: https://hg.mozilla.org/integration/mozilla-inbound/rev/bf8fee7089b4 - Lee Salzman - Bug 1380431 - fix DrawTargetCairo::Snapshot error note to check for surface before accessing status. r=mchang
03:42njnbz: nsXPIDLString is often used for the return value
03:42njndbaron wants to get rid of nsXPIDLString
03:47bznjn: yep
03:47njnbz: nsAString is also often used (also with a getter_copies)
03:47bzmmhm
03:47bzOur string clasess are so awesome....
03:51njnbz: in |wstring GetStringFromName(in wstring aName);|, if I change the argument to AUTF8String, does that make JS-side calls more efficient?
03:53dbaronI would think AString would have less encoding conversion than AUTF8String... although I&#39;m not sure.
03:58dbaronhmm, looks like KWierso|afk merged a perma-orange from central to inbound
03:58dbaronlet&#39;s see if it&#39;s orange on central...
03:59dbaronoh, I guess it&#39;s tier 2
03:59dbaronsince it&#39;s stylo, and from autoland
04:00dbaronah, and fixed in https://hg.mozilla.org/integration/autoland/rev/bd85336e149ef6b0db2888f1c2dc24d65a932f44
04:03bznjn: No, less efficient
04:04bznjn: since JS doesn&#39;t store UTF8
04:04bznjn: wstring/AString are more or less equivalent in terms of overhead here, I suspect
04:04njnbz: but the C++ code always converts to UTF8
04:05njnso if I changed it from wstring to AUTF8String the 16-to-8 conversion would happen in xpconnect, rather than in the C++ GetStringFromName() function
04:05njnbz: hmm, but JS does Latin1, so if the string is ASCII, presumably that&#39;ll avoid an 8-to-16-to-8 path?
04:07* bz checks
04:07bzhttp://searchfox.org/mozilla-central/source/js/xpconnect/src/XPCConvert.cpp#614-645
04:08bzLooks like we call JS::DeflateStringToUTF8Buffer
04:08* bz looks up what that does
04:08bzOK, so yes, in the ascii case it will end up just doing a copy
04:09bzAs opposed to two copies, if the callee ends up wanting it as char* anyway
04:12njnbz: cool, thanks
04:13pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/22031dceccbf - Emilio Cobos lvarez - servo: Merge #17705 - style: Avoid parsing negative padding in the padding shorthand (from emilio:neg-padding); r=bholley
04:16njneveryone&#39;s string classes suck
04:17bznjn: http://robert.ocallahan.org/2008/01/string-theory_08.html
04:18bzAnd the companion post at http://robert.ocallahan.org/2008/06/advanced-topics-in-computer-science-2_17.html of course
04:20njnmmm
04:22pulsebotCheck-in: https://hg.mozilla.org/integration/mozilla-inbound/rev/d4d99d47b9fa - Nicholas Nethercote - Bug 1379933 (part 1) - Remove Sample entries from the ProfileBuffer. r=mstange.
04:22pulsebotCheck-in: https://hg.mozilla.org/integration/mozilla-inbound/rev/00674f36f790 - Nicholas Nethercote - Bug 1379933 (part 2) - Improve ProfileBuffer&#39;s handling of labels and dynamic strings. r=mstange.
04:22pulsebotCheck-in: https://hg.mozilla.org/integration/mozilla-inbound/rev/158b7d6d7ea6 - Nicholas Nethercote - Bug 1379933 (part 3) - Start all ProfilerBufferEntry methods with an upper case letter. r=mstange.
04:22pulsebotCheck-in: https://hg.mozilla.org/integration/mozilla-inbound/rev/f7e6d96eb514 - Nicholas Nethercote - Bug 1379933 (part 4) - Start all ProfilerBuffer methods with an upper case letter. r=mstange.
04:22pulsebotCheck-in: https://hg.mozilla.org/integration/mozilla-inbound/rev/3fe4adc63baf - Nicholas Nethercote - Bug 1379933 (part 5) - Tweak ProfileBuffer arguments. r=mstange.
04:28pulsebotCheck-in: https://hg.mozilla.org/integration/mozilla-inbound/rev/a790d8b9abb1 - Yoshi Huang - Bug 1377523 - Part 1: add dummy.xul. r=smaug
04:28pulsebotCheck-in: https://hg.mozilla.org/integration/mozilla-inbound/rev/cbfb8a491544 - Yoshi Huang - Bug 1377523 - Part 2: fix for browser_toolbox_*. r=pbro
04:28pulsebotCheck-in: https://hg.mozilla.org/integration/mozilla-inbound/rev/e29e8a32baf4 - Yoshi Huang - Bug 1377523 - Part 3: fix failure in sourceeditor/. r=pbro
04:29pulsebotCheck-in: https://hg.mozilla.org/integration/mozilla-inbound/rev/e836466814de - Yoshi Huang - Bug 1377523 - Part 4: fix devtools/client/shared/test. r=pbro
04:29pulsebotCheck-in: https://hg.mozilla.org/integration/mozilla-inbound/rev/b09f9523c66f - Yoshi Huang - Bug 1377523 : Part 5: fix styleeditor/. r=pbro
04:29pulsebotCheck-in: https://hg.mozilla.org/integration/mozilla-inbound/rev/50b98c3ff6ca - Yoshi Huang - Bug 1377523 - Part 6: fix in webconsole/. r=pbro
04:41dbaronhmmm, the &quot;trigger new jobs&quot; mechanism on treeherder doesn&#39;t seem to work for me
04:41dbaronI&#39;m trying to infill some windows xpcshell runs on mozilla-inbound to determine who to backout
04:41dbaronback out
04:41bzare you logged in?
04:42dbaronyes
04:42dbaronat least in treeherder
04:42dbaronmaybe not in other things...
04:42dbaronthe devtools network panel claims that nothing is happening when I hit &quot;Trigger New Jobs&quot;
04:42dbaronin fact, when I shift-reload it claims that only a single image got loaded from the network
04:43dbaronoh, hey, they just appeared
04:43dbaronI wonder if triggering new jobs doesn&#39;t work when there&#39;s a filter applied
05:03nalexanderdbaron: it&#39;s very slow to appear for me, and seems to scale linearly (or worse) with the number of jobs triggered.
05:04squibis it still possible to make a new protocol with nsIProtocolHandler? (this is for a web extension experiment.) i can&#39;t get it to call newChannel() no matter what i do...
05:04dbaronI think it actually doesn&#39;t work if I have filters...
05:10squiboh weird. it works if i try to load it directly, but not in the url bar
05:16bz_sleepsquib: hmm?
05:16bz_sleepsquib: load it directly in what sense?
05:16bz_sleepsquib: In general, for now it&#39;s still possible, but I think we plan to not make it possible after 57....
05:16squibbz_sleep: right now i&#39;m loading it via a <script> tag in a page from my add-on
05:17bzsquib: And that works?
05:17bzsquib: but loading it in the url bar does not?
05:17squibwell, it calls newChannel2. i haven&#39;t gotten farther than that yet
05:17bzsquib: What happens in the url bar of a non-e10s window?
05:18bzok
05:18squibok, it actually loads the content i wanted now. let me try in non-e10s
05:19squibbz: works from the url bar of a non-e10s window
05:19bzok
05:20bzSounds like the issue might then be in the code that does the &quot;decide what process to load it in&quot; bits in our UI
05:20* bz has no idea what that code looks like
05:20squibyeah, maybe
05:20squibin any case, this is mostly just an experiment to allow registering an about: URL for a webextension
05:20squibi ended up writing a protocol handler because this is a rabbit hole
05:24pulsebotCheck-in: https://hg.mozilla.org/integration/mozilla-inbound/rev/d259c59353d3 - L. David Baron - Backed out changeset 90ae2fcd089a (bug 1378727) for numerous xpcshell test failures on Windows, starting with test_ext_browsingData_downloads.js.
05:29squibbz: anyway, thanks for the idea on non-e10s. i&#39;ll try to remember to file a bug about it tomorrow morning
05:32pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/d541f4f8d435 - Ray Lin - Bug 1380556 - Darken the font color of also fill note on the dropdown footer. r=lchang
05:32bzsquib: Good luck!
05:44xidornhmmmm
05:44xidornservo/servo#17701 causes panic
05:46xidornbholley: ^
05:46xidornwe probably need to backout 84de8f2d744f and 65d39277c962
06:02pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/4f0dccde5566 - Martin Robinson - servo: Merge #17694 - Upgrade to the latest version of WebRender (from glennw:webrender-update); r=glennw
06:12pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/c4b8736c4c08 - Cameron McCormack - Bug 1376964 - Part 1: Record the docshell&#39;s &quot;bypass cache&quot; flag on FontFaceSet. r=jfkthame
06:12pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/c2229c8eef89 - Cameron McCormack - Bug 1376964 - Part 2: Record the docshell&#39;s &quot;private browsing&quot; flag on FontFaceSet. r=jfkthame
06:12pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/5dc0af8dc8ff - Cameron McCormack - Bug 1376964 - Part 3: Add a generation counter to the user font cache. r=jfkthame
06:12pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/d5c07368105e - Cameron McCormack - Bug 1376964 - Part 4: Call FontLoadAllowed ahead of time and cache the results for style worker threads. r=jfkthame
06:12pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/c648fe419177 - Cameron McCormack - Bug 1376964 - Part 5: Add OMT wrapper for nsIURIs useful for font stuff. r=jfkthame
06:12pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/f2ecba994335 - Cameron McCormack - Bug 1376964 - Part 6: Use gfxFontSrcURI in the user font set and cache. r=jfkthame
06:13pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/7c316771b894 - Cameron McCormack - Bug 1376964 - Part 7: Remove unused nsIURI argument from gfxPlatform::IsFontFormatSupported. r=jfkthame
06:13pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/93d998d3eacd - Cameron McCormack - Bug 1376964 - Part 8: Add OMT wrapper for nsIPrincipals useful for font stuff. r=jfkthame
06:13pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/e1489c41dd18 - Cameron McCormack - Bug 1376964 - Part 9: Use gfxFontSrcPrincipal in the user font set and cache. r=jfkthame
06:13pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/bb098650d199 - Cameron McCormack - Bug 1376964 - Part 10: Allow style worker threads to pick fonts out of the user font cache. r=jfkthame
06:13pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/868c580d8d9d - Mark Banner - Bug 1308343 - Fix intermittent failures in browser_ContentSearch.js by ensuring we keep listeners alive and listening for loads correctly. r=adw
06:20pulsebotCheck-in: https://hg.mozilla.org/integration/mozilla-inbound/rev/5a26c718cf33 - Christoph Kerschbaumer - Bug 1380249 - Convert tests within js/xpconnect to comply with new data: URI inheritance model. r=bholley
06:47pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/a859e2a0ae06 - Marco Bonardo - Bug 1380302 - Fix -Wmaybe-uninitialized warnings in Places. r=adw
07:11pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/2c7d53158c5a - Ting-Yu Lin - Bug 1380567 - Remove RestyleManagerHandle. r=heycam
07:46pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/948166f84734 - Xidorn Quan - servo: Merge #17702 - Avoid random python address to be added to generated files (from upsuper:python-addr-in-generated); r=emilio
08:11Aryxxidorn: ok, will back out
08:16emilioAryx: I already opened a manual revert myself
08:16emilioAryx: https://github.com/servo/servo/pull/17710
08:16Aryxemilio: ok, thanks. reverting the servo change has many failing hunks
08:16emilioAryx: huh? How so?
08:16emilioAryx: it was a clean revert for me
08:18pulsebotCheck-in: https://hg.mozilla.org/integration/mozilla-inbound/rev/59d05f6cda7a - David Anderson - Add a helper for automating ClearView and clear-shader operations. (bug 1378095 part 1, r=mattwoodrow)
08:18pulsebotCheck-in: https://hg.mozilla.org/integration/mozilla-inbound/rev/cec67bb0c301 - David Anderson - Fix Advanced Layers not forcing a clear for the window theme. (bug 1378095 part 2, r=mattwoodrow)
08:19Aryxstrange, worked on second attempt
08:20Aryxjust curious: any ETA for travis to finish and the patch get merged?
08:22pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/aa8e8b3f4ce1 - Emilio Cobos lvarez - Bug 1380492: Test. r=bholley
08:23emilioAryx: well, it&#39;s waiting on another PR according to http://build.servo.org/homu/queue/servo, I can also force it
08:26pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/29ab214f96d2 - Nevin Chen - Bug 1378688 - Make Splash Screen faster by making it&#39;s background transparent. r=maliu
08:26pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/7b5d4791c970 - Nevin Chen - Bug 1378688 - Scale the image to save APK size, respect channel, and hide splash screen when the user interact with url bar. r=maliu
08:41pulsebotCheck-in: https://hg.mozilla.org/integration/mozilla-inbound/rev/4699fe6d75de - Jonathan Kew - Bug 1379444 - Use generic cmap-based font fallback if platform-specific code fails to find a usable font. r=lsalzman
08:45pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/8904c48d86ce - Jan Odvarko - Bug 1378420 - Register new pref; r=rickychien
08:59pulsebotCheck-in: https://hg.mozilla.org/integration/mozilla-inbound/rev/cadef5ef9c44 - Jon Coppeard - Bug 1380397 - Use PersistentRooted to implement XPCJSObjectHolder r=mccr8
09:02pulsebotCheck-in: https://hg.mozilla.org/integration/mozilla-inbound/rev/1a49d403a9a4 - Tristan Bourvon - Bug 1374024 - add checker to prevent dangling pointers returned by method calls
09:02pulsebotCheck-in: https://hg.mozilla.org/integration/mozilla-inbound/rev/7f768b83c928 - Tristan Bourvon - Bug 1374024 - clang-format on some files from static analysis directory. r=mystor
09:02pulsebotCheck-in: https://hg.mozilla.org/integration/mozilla-inbound/rev/1eb2623a06c8 - Tristan Bourvon - Bug 1374024 - add static-analysis annotation for methods that return dangling pointers. r=mystor
09:10pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/5a5d8de65b70 - Do Gottwald - Bug 1379938 - Rename the -moz-windows-accent-color-applies media query and make -moz-win-accentcolor available without it. r=jimm
09:18smaugwhat is pingsender.exe
09:20freddybhttp://searchfox.org/mozilla-central/source/toolkit/components/telemetry/docs/internals/pingsender.rst
09:20freddybof http://searchfox.org/mozilla-central/source/toolkit/components/telemetry/pingsender if you prefer cpp over rst :)
09:24Pikeand http://gecko.readthedocs.io/en/latest/toolkit/components/telemetry/telemetry/internals/pingsender.html
09:27pulsebotCheck-in: https://hg.mozilla.org/integration/mozilla-inbound/rev/6bbddebba266 - Carsten &quot;Tomcat&quot; Book - Backed out changeset 1eb2623a06c8 (bug 1374024)
09:27pulsebotCheck-in: https://hg.mozilla.org/integration/mozilla-inbound/rev/3115e6d4f7df - Carsten &quot;Tomcat&quot; Book - Backed out changeset 7f768b83c928 (bug 1374024)
09:27pulsebotCheck-in: https://hg.mozilla.org/integration/mozilla-inbound/rev/fc27e4fc79a3 - Carsten &quot;Tomcat&quot; Book - Backed out changeset 1a49d403a9a4 (bug 1374024) for bustage
09:28pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/6ceb427e8afa - Farmer Tseng - Bug 1380301 - Add missing &quot;#ifdef MOZ_ENABLE_SKIA_PDF&quot; to nsDeviceContextSpecWin.h r=heycam
09:28pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/621c06836799 - Farmer Tseng - Bug 1380301 - Add windows.h include to nsLookAndFeel.h r=heycam
09:44pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/851de84975c1 - Emilio Cobos lvarez - servo: Merge #17710 - Revert &quot;Auto merge of #17701 - bholley:reuse_allocations, r=emilio&quot; (from emilio:revert-bloom-sharing); r=heycam
09:44Dextersmaug, it&#39;s a small software that&#39;s used to reduce the overall ping latency (Telemetry)
09:45pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/58ef10a3dd4c - Servo VCS Sync - No bug - Revendor rust dependencies
09:45Dextersmaug, when shutting down Firefox it gets spawned to send the &quot;shutdown&quot; ping. It also sends the &quot;crash&quot; ping when Firefox crashes.
09:45smaugaha
09:46smaugjust a tiny bit annoying when testing tryserver builds, and Windows asks permission to run it after closing FF
09:46smaugbut ok, fine
09:47Dexterah, uhm, interesting. Luckily that doesn&#39;t happen on official builds ^^
09:47Dexteryou can preff it off if you need to
09:56pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/fe939858ead4 - Tom Prince - Bug 1379785 - Stop calling abspath on symbol_archive paths. r=gps
09:56pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/63df6245eaeb - Tom Prince - Bug 1380236 - Don&#39;t treat *.stylo-bindings.zip as a potential installer; r=Callek
09:59pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/9cf3756e48b3 - jwu - Bug 1366704 - Part 1: Implement ThemedListView for Photon visual refresh. r=sebastian,walkingice
09:59pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/714e8049f1a5 - Jing-wei Wu - Bug 1366704 - Part 2: Support changing theme for search suggestion custom views. r=sebastian,walkingice
09:59pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/91c6be339537 - jwu - Bug 1366704 - Part 3: Support search suggestion visual refresh in private mode. r=sebastian,walkingice
10:01pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/17425c055ef8 - Rex Lee - Bug 1378770 - [Onboarding] Sync tour should proceed with only valid Email. r=mossop
10:01pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/19f6e9eca436 - Julian_Chu - Bug 1379551 - change tab indicator height r=jwu
10:03pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/8f07c253160b - Ray Lin - Bug 1379575 - Do not show focused category in autofill notes if none of the profiles have been selected. r=MattN
10:06pulsebotCheck-in: https://hg.mozilla.org/integration/mozilla-inbound/rev/7bfcb0aeb065 - Tom Tung - Bug 1372116 - P1: Create originInfo after creating metadata if the temporay storage has been initialized for persist(). r=janv
10:06pulsebotCheck-in: https://hg.mozilla.org/integration/mozilla-inbound/rev/76538de6b9b2 - Tom Tung - Bug 1372116 - P2: Init quota for origin directlly when the metadata is just created to save time for traversing directory. r=janv
10:06pulsebotCheck-in: https://hg.mozilla.org/integration/mozilla-inbound/rev/451af4e965fb - Samuel Vargas - Bug 1376399 - Hard crash in FF 54 and Nightly 56 on Windows 7 - r=jgilbert
10:06pulsebotCheck-in: https://hg.mozilla.org/integration/mozilla-inbound/rev/122e0e90b9a1 - Samuel Vargas - Bug 1378571 - Avoid unnecessary MakeCurrentImpl calls when GL context is already current - r=jgilbert
10:23pulsebotCheck-in: https://hg.mozilla.org/integration/mozilla-inbound/rev/55471601da44 - Junior Hsu - Bug 1373198 - Part 1: disable rcwn for wpt tests which require deterministic cache behavior, r=michal
10:23pulsebotCheck-in: https://hg.mozilla.org/integration/mozilla-inbound/rev/1a52987915e7 - Junior Hsu - Bug 1373198 - Part2: disable rcwn for xpcshell tests which require deterministic cache behavior, r=michal
10:23pulsebotCheck-in: https://hg.mozilla.org/integration/mozilla-inbound/rev/3679429ca4d6 - Junior Hsu - Bug 1373198 - Part 3-1: disable rcwn for dom mochitest tests which require deterministic cache behavior, r=bz
10:23pulsebotCheck-in: https://hg.mozilla.org/integration/mozilla-inbound/rev/09b311b83042 - Junior Hsu - Bug 1373198 - Part 4: disable rcwn for necko mochitest tests which require deterministic cache behavior, r=michal
10:23pulsebotCheck-in: https://hg.mozilla.org/integration/mozilla-inbound/rev/50def55507e6 - Junior Hsu - Bug 1373198 - Part 5: disable rcwn for devtools mochitest tests which require deterministic cache behavior, r=Honza
10:23pulsebotCheck-in: https://hg.mozilla.org/integration/mozilla-inbound/rev/198cc96cc9b8 - Junior Hsu - Bug 1373198 - Part 6: disable rcwn for toolkit mochitest tests which require deterministic cache behavior, r=mixedpuppy
10:23pulsebotCheck-in: https://hg.mozilla.org/integration/mozilla-inbound/rev/212958380e64 - Junior Hsu - Bug 1373198 - Part 7: disable rcwn for wpt tests which require deterministic cache behavior, r=michal
10:23pulsebotCheck-in: https://hg.mozilla.org/integration/mozilla-inbound/rev/bc83f0af29ff - Junior Hsu - Bug 1373198 - Part 8: disable rcwn for dom mochitest tests which require deterministic cache behavior, r=smaug
10:23pulsebotCheck-in: https://hg.mozilla.org/integration/mozilla-inbound/rev/933aeb87fcbb - Junior Hsu - Bug 1373198 - Part 9: disable rcwn for xpcshell tests which require deterministic cache behavior, r=michal
10:23pulsebotCheck-in: https://hg.mozilla.org/integration/mozilla-inbound/rev/31973778f0ed - Junior Hsu - Bug 1373198 - Part 10: disable rcwn for devtools mochitest tests which require deterministic cache behavior, r=Honza
11:05pulsebotCheck-in: https://hg.mozilla.org/releases/mozilla-beta/pushloghtml?startID=7505&endID=7506 - 15 changesets (bugs 1356812, 1373061, 1374146, 1374758, 1379025 and 6 other bugs)
11:28pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/d018acce4b13 - Sebastian Hengst - Backed out changeset 942e95e4964d (bug 1379983)
11:28pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/a28d4a7126e7 - Sebastian Hengst - Backed out changeset 37e091fb7f79 (bug 1379983)
11:28pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/78ffc50433f1 - Sebastian Hengst - Backed out changeset d7a872ed8840 (bug 1379983) for failing xpcshell&#39;s toolkit/components/osfile/tests/xpcshell/test_available_free_space.js on Linux debug. r=backout
11:32Pikeif I squash a couple of commits from a mozreview patch queue, what do I do with the MozReview-Commit-IDs of each commit? add them all to the commit message, or just one, or .. ?
11:36pulsebotCheck-in: https://hg.mozilla.org/integration/mozilla-inbound/rev/f87c65cf26de - Catalin Badea - Bug 1344751 - use nsStandardURL for http and https in workers. r=baku
11:36pulsebotCheck-in: https://hg.mozilla.org/integration/mozilla-inbound/rev/9cb4e53f36aa - Catalin Badea - Bug 1344751 - Make nsStandardURL and nsIDNService available on worker threads. r=valentin
11:36pulsebotCheck-in: https://hg.mozilla.org/integration/mozilla-inbound/rev/530b1eb0a548 - Catalin Badea - Bug 1380604 - Use Killing when dispatching the sync runnable in FetchBodyConsumer::ShutDownMainThreadConsuming. r=baku
11:59pulsebotCheck-in: https://hg.mozilla.org/integration/mozilla-inbound/rev/7a156d0d1acf - Nicolas Silva - Bug 1378771 - Reduce the minimum painted layer width down to 8 pixels. r=jrmuizel
12:17pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/6177c907f345 - Emilio Cobos lvarez - servo: Merge #17707 - stylo: Waste less memory in invalidation stuff and style rules (from emilio:invalidation-less-memory); r=heycam
12:24pulsebotCheck-in: https://hg.mozilla.org/integration/mozilla-inbound/rev/2a6a73d9a560 - Jan-Ivar Bruaroey - Bug 1372509: Fix eslint warnings in about:webrtc. r=jesup
12:39pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/77c1ead3ba58 - Andrew Halberstadt - Bug 1380338 - Convert taskgraph unit tests to the |mach python-test| framework, r=dustin
12:41pulsebotCheck-in: https://hg.mozilla.org/integration/mozilla-inbound/rev/13a43361f217 - Eugen Sawin - Bug 1380071 - [1.1] Add base class for GeckoView content modules. r=jchen
12:41pulsebotCheck-in: https://hg.mozilla.org/integration/mozilla-inbound/rev/99ca35b6467b - Eugen Sawin - Bug 1380071 - [2.1] Use content module base class for the GeckoView content module. r=jchen
12:41pulsebotCheck-in: https://hg.mozilla.org/integration/mozilla-inbound/rev/9ef8c2565cb3 - Eugen Sawin - Bug 1380071 - [3.1] Use content module base class for the GeckoView scroll module. r=jchen
12:41pulsebotCheck-in: https://hg.mozilla.org/integration/mozilla-inbound/rev/852855d5d2cf - Eugen Sawin - Bug 1380071 - [4.0] Delay frame script loading to GeckoView content module registration. r=jchen
12:41pulsebotCheck-in: https://hg.mozilla.org/integration/mozilla-inbound/rev/da0754f7bb57 - Eugen Sawin - Bug 1380071 - [5.0] Delay frame script loading to GeckoView scroll module registration. r=jchen
12:56pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/812f0d47226f - Carsten &quot;Tomcat&quot; Book - Backed out changeset 4e27c8cad5d1 (bug 1380087)
12:56pulsebotCheck-in: https://hg.mozilla.org/integration/autoland/rev/0e71e43dcee8 - Carsten &quot;Tomcat&quot; Book - Backed out changeset 77fac7f5d728 (bug 1379639) for still perma failing xpcshell x6 tests
12:58pulsebotCheck-in: https://hg.mozilla.org/integration/mozilla-inbound/rev/54b1caeb09ec - Catalin Badea - Bug 1375050 - Fix crash in nsLabelsNodeList::MaybeResetRoot r=jdai
13:03pulsebotCheck-in: https://hg.mozilla.org/integration/mozilla-inbound/rev/1a0693cc418b - Tristan Bourvon - Bug 1374024 - add checker to prevent dangling pointers returned by method calls on temporaries. r=mystor
13:03pulsebotCheck-in: https://hg.mozilla.org/integration/mozilla-inbound/rev/b4a4d8144010 - Tristan Bourvon - Bug 1374024 - clang-format on some files from static analysis directory. r=mystor
13:03pulsebotCheck-in: https://hg.mozilla.org/integration/mozilla-inbound/rev/b75316c29d8c - Tristan Bourvon - Bug 1374024 - add static-analysis annotation for methods that return dangling pointers. r=mystor
13:12Tomcat|sheriffdutyAndi: ping
13:12Andihello
13:15pulsebotCheck-in: https://hg.mozilla.org/integration/mozilla-inbound/rev/32885efc9b71 - Carsten &quot;Tomcat&quot; Book - Backed out changeset b75316c29d8c (bug 1374024)
13:15pulsebotCheck-in: https://hg.mozilla.org/integration/mozilla-inbound/rev/93f2901dccd2 - Carsten &quot;Tomcat&quot; Book - Backed out changeset b4a4d8144010 (bug 1374024)
13:15pulsebotCheck-in: https://hg.mozilla.org/integration/mo