Re: Getting rid of already_AddRefed?

2014-08-13 Thread Jeff Walden
On 08/12/2014 10:06 AM, Ehsan Akhgari wrote: It could also be solved with making operator T*() explicit, but neither of these options are something that we can use in our code base today. So at risk of adding yet another flavor of thing: why not introduce an already_AddRefedT sort of struct

Re: Telemetry alerts for histograms

2014-08-13 Thread Gian-Carlo Pascutto
On 12/08/2014 16:57, rviti...@mozilla.com wrote: From now on all alerts will be sent by default also to telemetry-ale...@mozilla.com. How does one read that list? I can't find anywhere on how to subscribe. -- GCP ___ dev-platform mailing list

Re: Getting rid of already_AddRefed?

2014-08-13 Thread Neil
Karl Tomlinson wrote: Aryeh Gregor writes: The compiler is required to use the move constructor (if one exists) instead of the copy constructor when constructing the return value of a function, and also when initializing an object from the return value of a function, or assigning the

Re: Evaluating adding JOSE and JWS to mozilla-central

2014-08-13 Thread Frederik Braun
Well there is https://github.com/mozilla/jwcrypto which does JWS. It is available in privileged JS through jwcrypto.jsm (i.e. resource://gre/modules/identity/jwcrypto.jsm). There's some code usage for these things in the MobileIdentityManager, Webapps and Payments jsms. On 12.08.2014 19:22,

Re: Getting rid of already_AddRefed?

2014-08-13 Thread Aryeh Gregor
On Wed, Aug 13, 2014 at 9:42 AM, Jeff Walden jwalden+...@mit.edu wrote: So at risk of adding yet another flavor of thing: why not introduce an already_AddRefedT sort of struct that *does* own an addref, *will* release on destruction if not nulled out, and does *not* explicitly convert to T*?

Re: We should drop MathML

2014-08-13 Thread peter . krautzberger
Actually, MS is very clear on their position on MathML in Internet Explorer: http://status.modern.ie/mathml?term=mathML Well, status.modern.ie was published about a year after my message. Curiously enough, according to

Re: Getting rid of already_AddRefed?

2014-08-13 Thread smaug
On 08/12/2014 06:23 PM, Aryeh Gregor wrote: On Tue, Aug 12, 2014 at 6:16 PM, Benoit Jacob jacob.benoi...@gmail.com wrote: As far as I know, the only downside in replacing already_AddRefed by nsCOMPtr would be to incur more useless calls to AddRef and Release. In the case of threadsafe i.e.

Re: Upcoming changes to Mac package layout, signing

2014-08-13 Thread Benjamin Smedberg
On 8/13/2014 3:34 AM, Philipp Kewisch wrote: Does this also affect binary extensions in any way? I'd imagine that globally installed extensions would break signing if placed incorrectly. You cannot place anything in the Firefox bundle. Any extensions, binary or not, would need to be

Re: Getting rid of already_AddRefed?

2014-08-13 Thread Aryeh Gregor
On Wed, Aug 13, 2014 at 4:11 PM, Ehsan Akhgari ehsan.akhg...@gmail.com wrote: What goal would this achieve? I don't understand why it's OK to ignore the return value of a function which returns an already AddRef'ed object from a conceptual perspective. You might want the effect of the

Re: Upcoming changes to Mac package layout, signing

2014-08-13 Thread Ben Hearsum
On 14-08-12 08:46 PM, Cameron McCormack wrote: Ben Hearsum wrote: Apple recently announced changes to how OS X applications must be packaged and signed Does this also apply if you run .app/Contents/MacOS/firefox binary manually rather than opening the .app? I'm not sure about that. I also

Re: Getting rid of already_AddRefed?

2014-08-13 Thread Ehsan Akhgari
On Wed, Aug 13, 2014 at 9:32 AM, Aryeh Gregor a...@aryeh.name wrote: On Wed, Aug 13, 2014 at 4:11 PM, Ehsan Akhgari ehsan.akhg...@gmail.com wrote: What goal would this achieve? I don't understand why it's OK to ignore the return value of a function which returns an already AddRef'ed

Passing variant Array (can contain number, string...) to JS-XPCOM from C++

2014-08-13 Thread Shanmugham Sundaram
Hi, I have to pass an array of strings (or JS::Value array) to JS-XPCOM from my C++ code. I could not find the proper example in C++ to implements this. Based on the jsFunction I am passing in JS-XPCOM, i will extract the correct number of values in the JS and call the expected function. How

Web APIs documentation meeting Friday at 10 AM PDT

2014-08-13 Thread Eric Shepherd
The Web APIs documentation meeting is Friday at 10 AM Pacific Time (see http://bit.ly/APIDocsMeeting for your time zone). Everyone's welcome to attend; if you're interested in ensuring that all Web APIs are properly documented, we'd love your input. We have an agenda, as well as details on

Re: Passing variant Array (can contain number, string...) to JS-XPCOM from C++

2014-08-13 Thread Bobby Holley
If this is from a binary component, i would advise against passing JSVals directly, because the APIs you need to manipulate are no longer exported (they're still temporarily exported in Windows IIUC, but that's changing soon). See https://bugzilla.mozilla.org/show_bug.cgi?id=978995 Instead, you

Re: Passing variant Array (can contain number, string...) to JS-XPCOM from C++

2014-08-13 Thread Shanmugham Sundaram
Thanks Bholly. regards, Shan' On Wednesday, August 13, 2014 8:48:40 PM UTC+5:30, Bobby Holley wrote: If this is from a binary component, i would advise against passing JSVals directly, because the APIs you need to manipulate are no longer exported (they're still temporarily exported in

Re: Help for new MDN page on testing frameworks (technical and editorial)

2014-08-13 Thread Sylvestre Ledru
Hello Paolo, On 13/08/2014 17:40, Paolo Amadini wrote: I've just updated the MDN page about the testing frameworks to add more details, like the process where the tests run in an e10s build, and moved everything to a table to make the information more accessible. The links to detail pages are

Re: Help for new MDN page on testing frameworks (technical and editorial)

2014-08-13 Thread Paolo Amadini
On 8/13/2014 4:55 PM, Sylvestre Ledru wrote: Too bad I didn't know about your page before, it would have saved me quite some time. Well, I just created the page :-) The work for better descriptions in mach and Treeherder is quite useful as well! By the way, do you know what is talos g1? I

External dependent tests in gecko and gaia

2014-08-13 Thread Edwin Wong
Hi dev-platform, TL;DR - Cloud Services and Quality Engineering would like to propose the creation of a directory named “external in gecko and gaia repos for externally dependent tests. This enables features married to Cloud Services such as Loop, FindMyDevice, FirefoxAccounts, and Sync to

Re: Getting rid of already_AddRefed?

2014-08-13 Thread Aryeh Gregor
On Wed, Aug 13, 2014 at 5:44 PM, Ehsan Akhgari ehsan.akhg...@gmail.com wrote: Can't you do the following instead? unused MyFunction(); // I know that I'm leaking this ref, but it's ok somehow No, because the use-case is where you don't want to leak the ref -- you want it to be released

Re: External dependent tests in gecko and gaia

2014-08-13 Thread Gregory Szorc
On 8/13/14 9:21 AM, Edwin Wong wrote: Hi dev-platform, TL;DR - Cloud Services and Quality Engineering would like to propose the creation of a directory named “external in gecko and gaia repos for externally dependent tests. This enables features married to Cloud Services such as Loop,

Re: Getting rid of already_AddRefed?

2014-08-13 Thread Jonas Sicking
On Wed, Aug 13, 2014 at 5:14 AM, Aryeh Gregor a...@aryeh.name wrote: I think this can be improved upon a bit further: just change already_AddRefed to behave more similarly to nsCOMPtr, but still not convert to T* implicitly. So for instance: * Change ~already_AddRefed to just release the

Re: Getting rid of already_AddRefed?

2014-08-13 Thread Jonas Sicking
On Wed, Aug 13, 2014 at 9:24 AM, Aryeh Gregor a...@aryeh.name wrote: No, because the use-case is where you don't want to leak the ref -- you want it to be released automatically for you. So for instance, here's a real-world bit of code from nsWSRunObject: if ((aRun-mRightType

Re: External dependent tests in gecko and gaia

2014-08-13 Thread Doug Turner
Did we ever reach agreement about moving gaia into m-c? It would make this type of problem go away, and make regression range hunting much easier. On Aug 13, 2014, at 9:21 AM, Edwin Wong edw...@mozilla.com wrote: Hi dev-platform, TL;DR - Cloud Services and Quality Engineering would like

Re: Getting rid of already_AddRefed?

2014-08-13 Thread Jeff Walden
On 08/13/2014 07:44 AM, Ehsan Akhgari wrote: If that is the goal, then I don't agree that is a useful outcome at all. I *do* wish that there were better *and* safer ways of doing more things automatically but ownership transfers are inherently unsafe operations that are expressed using

Re: External dependent tests in gecko and gaia

2014-08-13 Thread Ehsan Akhgari
On 2014-08-13, 12:21 PM, Edwin Wong wrote: Hi dev-platform, TL;DR - Cloud Services and Quality Engineering would like to propose the creation of a directory named “external in gecko and gaia repos for externally dependent tests. This enables features married to Cloud Services such as Loop,

Re: Getting rid of already_AddRefed?

2014-08-13 Thread Ehsan Akhgari
On 2014-08-13, 12:24 PM, Aryeh Gregor wrote: On Wed, Aug 13, 2014 at 5:44 PM, Ehsan Akhgari ehsan.akhg...@gmail.com wrote: Can't you do the following instead? unused MyFunction(); // I know that I'm leaking this ref, but it's ok somehow No, because the use-case is where you don't want to

Re: External dependent tests in gecko and gaia

2014-08-13 Thread Edwin Wong
Hi Greg, A few had discussed manifest vs directory separation of tests. I think the directory idea took root because it’s easily identifiable from it’s path, reducing collision with existing tests. I also agree, using directory as a form of meta data is limiting. That said, both concepts

Re: Telemetry alerts for histograms

2014-08-13 Thread Steve Fink
On 08/13/2014 08:36 AM, rviti...@mozilla.com wrote: I requested a public list available over NNTP (see Bug 1053202, mozilla.qa.telemetry-alerts). That should make everybody happy. I would sort of like a pulse notification for these, but it probably wouldn't get enough usage yet for it to be

Re: External dependent tests in gecko and gaia

2014-08-13 Thread Ehsan Akhgari
On 2014-08-13, 3:07 PM, Edwin Wong wrote: Ehsan, This wouldn’t be a top level dir, it would be as such: • [area]/[framework]/external/test_example.js • content/media/test/mochitest/external/test_loop.html • gaia/apps/findmydevice/test/marionette/external/fmd_locate.py

Re: External dependent tests in gecko and gaia

2014-08-13 Thread Ted Mielczarek
On 8/13/2014 12:21 PM, Edwin Wong wrote: Hi dev-platform, TL;DR - Cloud Services and Quality Engineering would like to propose the creation of a directory named “external in gecko and gaia repos for externally dependent tests. This enables features married to Cloud Services such as

Re: Getting rid of already_AddRefed?

2014-08-13 Thread smaug
On 08/13/2014 07:24 PM, Aryeh Gregor wrote: On Wed, Aug 13, 2014 at 5:44 PM, Ehsan Akhgari ehsan.akhg...@gmail.com wrote: Can't you do the following instead? unused MyFunction(); // I know that I'm leaking this ref, but it's ok somehow No, because the use-case is where you don't want to

Re: Intent to implement: Prerendering API

2014-08-13 Thread Ilya Grigorik
(reposting, got stuck in the queue) On Tue, Aug 12, 2014 at 3:17 PM, Jonas Sicking jo...@sicking.cc wrote: On Tue, Aug 12, 2014 at 1:04 PM, Ilya Grigorik igrigo...@gmail.com wrote: Whereas for (b) you can also do it in place by prefetching the template and then issuing a data request for

Re: Intent to implement: Prerendering API

2014-08-13 Thread Ilya Grigorik
On Tue, Aug 12, 2014 at 3:17 PM, Jonas Sicking jo...@sicking.cc wrote: On Tue, Aug 12, 2014 at 1:04 PM, Ilya Grigorik igrigo...@gmail.com wrote: Whereas for (b) you can also do it in place by prefetching the template and then issuing a data request for relevant data once you know the exact

How to convert nsIMutableArray to AtomArray

2014-08-13 Thread Yonggang Luo
In fact, I am implementing something like: [scriptable, uuid(3c2a2d26-08d6-415b-a635-24f15983bde6)] interface nsIMultilineTreeView: nsITreeView { long getCellTextPartCount(in long row, in nsITreeColumn col); AString getCellTextPart(in long row, in nsITreeColumn col, in long partIdx);

Re: Intent to implement: Prerendering API

2014-08-13 Thread Ehsan Akhgari
On 2014-08-13, 3:24 PM, Ilya Grigorik wrote: On Tue, Aug 12, 2014 at 3:17 PM, Jonas Sicking jo...@sicking.cc wrote: On Tue, Aug 12, 2014 at 1:04 PM, Ilya Grigorik igrigo...@gmail.com wrote: Whereas for (b) you can also do it in place by prefetching the template and then issuing a data

Non-backward compatible changes to JS 'let' semantics

2014-08-13 Thread Shu-yu Guo
Hello all, We are in the process of making JS 'let' semantics ES6-compliant in SpiderMonkey. I hope to land bug 1001090 sometime this month or early next month (I've been told there's a B2G uplift on Sept 1st), which is one of many for ES6 'let'-compliance. It changes 'let' semantics in two

Re: Non-backward compatible changes to JS 'let' semantics

2014-08-13 Thread Ehsan Akhgari
AFAICT gaia doesn't use let in its core code (since let is not available in non-chrome js yet) but it does use it in its build scripts etc. However, let is pretty heavily used in add-ons. CCing Jorge who can probably help with what to do with that. But have we considered not changing the

Re: External dependent tests in gecko and gaia

2014-08-13 Thread Mike Hommey
On Wed, Aug 13, 2014 at 10:52:29AM -0700, Doug Turner wrote: Did we ever reach agreement about moving gaia into m-c? It would make this type of problem go away, and make regression range hunting much easier. The gaia repo is massive, importing it in m-c would more than double m-c's size.

Re: External dependent tests in gecko and gaia

2014-08-13 Thread Doug Turner
Anyways, now that gaia revisions are tied to m-c revisions with b2g/config/gaia.json, regression range hunting should be a matter of tooling around that, now. That is excellent news. ___ dev-platform mailing list dev-platform@lists.mozilla.org

Re: Non-backward compatible changes to JS 'let' semantics

2014-08-13 Thread Shu-yu Guo
About not changing the behavior for chrome JS, the prospect of having chrome JS becoming more divergent from standard JS is unwelcome to me. Having additional, orthogonal features is one thing, but a fundamental feature with the same syntax that behaves differently depending on the context is

Re: Non-backward compatible changes to JS 'let' semantics

2014-08-13 Thread Ehsan Akhgari
On 2014-08-13, 6:02 PM, Shu-yu Guo wrote: About not changing the behavior for chrome JS, the prospect of having chrome JS becoming more divergent from standard JS is unwelcome to me. Having additional, orthogonal features is one thing, but a fundamental feature with the same syntax that

Re: Non-Trivial SpecialPowers Usage Considered Harmful

2014-08-13 Thread Ehsan Akhgari
On 2014-08-09, 10:19 PM, Bobby Holley wrote: I've mentioned this before [1], but recent checkins in mozilla-central suggest that this message could use repeating. Please don't write new tests that do tricky things with SpecialPowers.wrap and friends (like SpecialPowers.Cc and SpecialPowers.Cu,

Re: Telemetry alerts for histograms

2014-08-13 Thread rvitillo
On Wednesday, August 13, 2014 8:06:08 PM UTC+1, Steve Fink wrote: I would sort of like a pulse notification for these, but it probably wouldn't get enough usage yet for it to be worthwhile. (I'd feed it into an irc bot that notifies me + maybe a channel.)

Re: Non-Trivial SpecialPowers Usage Considered Harmful

2014-08-13 Thread Bobby Holley
On Wed, Aug 13, 2014 at 3:13 PM, Ehsan Akhgari ehsan.akhg...@gmail.com wrote: Note that anything tested in mochitest-chrome is not tested on Firefox OS at all, which is a *great* reason to avoid writing mochitest-chrome at all costs. Well, until bug 797164 is fixed. Mochitest-chrome is

Re: Telemetry alerts for histograms

2014-08-13 Thread Mike Hommey
On Wed, Aug 13, 2014 at 12:06:08PM -0700, Steve Fink wrote: On 08/13/2014 08:36 AM, rviti...@mozilla.com wrote: I requested a public list available over NNTP (see Bug 1053202, mozilla.qa.telemetry-alerts). That should make everybody happy. I would sort of like a pulse notification for

Re: Non-backward compatible changes to JS 'let' semantics

2014-08-13 Thread Mike Hommey
On Wed, Aug 13, 2014 at 06:09:31PM -0400, Ehsan Akhgari wrote: On 2014-08-13, 6:02 PM, Shu-yu Guo wrote: About not changing the behavior for chrome JS, the prospect of having chrome JS becoming more divergent from standard JS is unwelcome to me. Having additional, orthogonal features is one

Re: Non-backward compatible changes to JS 'let' semantics

2014-08-13 Thread Jorge Villalobos
On 8/13/14, 5:56 PM, Mike Hommey wrote: On Wed, Aug 13, 2014 at 06:09:31PM -0400, Ehsan Akhgari wrote: On 2014-08-13, 6:02 PM, Shu-yu Guo wrote: About not changing the behavior for chrome JS, the prospect of having chrome JS becoming more divergent from standard JS is unwelcome to me.

Re: Intent to implement: Prerendering API

2014-08-13 Thread Jonas Sicking
On Wed, Aug 13, 2014 at 12:24 PM, Ilya Grigorik igrigo...@gmail.com wrote: On Tue, Aug 12, 2014 at 3:17 PM, Jonas Sicking jo...@sicking.cc wrote: On Tue, Aug 12, 2014 at 1:04 PM, Ilya Grigorik igrigo...@gmail.com wrote: Whereas for (b) you can also do it in place by prefetching the template

FYI: Emergency try reset in process - https://bugzil.la/1053558

2014-08-13 Thread Hal Wine
tl;dr: try repo is being reset - read on for ways to access old push data Today we experienced another hg.m.o event that is not recovering similar to past events. After 2-1/2 hours with no signs of recovery, and with the recommendation of the sheriffs, we are resetting the try repo now. While

Re: Intent to implement: Prerendering API

2014-08-13 Thread Ilya Grigorik
On Wed, Aug 13, 2014 at 5:16 PM, Jonas Sicking jo...@sicking.cc wrote: On Wed, Aug 13, 2014 at 12:24 PM, Ilya Grigorik igrigo...@gmail.com wrote: On Tue, Aug 12, 2014 at 3:17 PM, Jonas Sicking jo...@sicking.cc wrote: Do you really think that Google would be able to morph a google calendar