#instantbird log on 06 09 2014

All times are UTC.

00:01:08 <Mic> hmm...
00:02:50 <mayanktg> When I add a Cu.reportError(conv) below the line II'm again getting a [xpconnect wrapped (nsISupports, nsIClassInfo, imIConversation, prplIConversation, nsIObserver, prplIConvIM)]
00:03:00 <clokep> qheaden: Can't wait to see it!
00:04:15 <mayanktg> It should have rather been [xpconnect wrapped (prplIConvIM)] ?
00:04:56 --> Rym has joined #instantbird
00:09:40 <-- Rym has quit (Ping timeout)
00:09:51 <Mic> Ah, maybe I got it wrong.
00:09:57 --> Rym has joined #instantbird
00:10:11 <Mic> Assign it to conv and the use QueryInterface on your new variable?
00:10:31 <Mic> let conv = this._conv;
00:10:31 <Mic> conv.QueryInterface(...); 
00:10:35 <Mic> I'm not sure.
00:10:48 <Mic> It's late here, too and I need to go to bed.
00:11:44 <mayanktg> I'm trying it. And I'll probably sleep too after a few tries..
00:11:48 <mayanktg> Ok Goodnight. :)
00:12:34 <-- Rym has quit (Ping timeout)
00:13:37 --> Rym has joined #instantbird
00:19:01 <Mic> Please post what else you tried... that might help to narrow down what the problem is.
00:25:05 <mayanktg> Mic: |conv.QueryInterface(Components.interfaces.prplIConvIM).startCall(offer.sdp)|
00:26:51 <mayanktg> Ok. I'm trying the patch on a different build just to be sure. The error identifies calling the method [prplIConvIM::startCall]  http://pastebin.instantbird.com/732416
00:29:23 <-- Mic has quit (Quit: Instantbird -- http://www.instantbird.com)
00:33:11 <-- Suiseiseki has quit (Ping timeout)
00:34:24 <-- Rym has quit (Ping timeout)
00:36:36 --> Suiseiseki has joined #instantbird
00:36:46 --> Rym has joined #instantbird
00:38:42 <-- Rym has quit (Ping timeout)
00:38:58 --> Rym has joined #instantbird
00:46:40 <-- Rym has quit (Ping timeout)
00:46:56 --> Rym has joined #instantbird
00:50:37 <mayanktg> bah :-/ It gives the same error again. I need to use QueryInterface correctly.  
00:53:50 <clokep> Can you show a complete diff?
00:53:55 <clokep> I want to see how you modified the interface.
00:55:29 <mayanktg> Yes.
00:56:45 <mayanktg> clokep: http://pastebin.instantbird.com/732455
01:01:25 <clokep> mayanktg: So what do you see? Does the print on line 568 occur?
01:02:36 <mayanktg> Yes. It occurs. as far aas I rem it outputs "conversation". wait let me check back
01:03:24 <-- Rym has quit (Ping timeout)
01:03:35 <mayanktg> Yeah . the localname is "conversation"
01:04:02 --> mconley has joined #instantbird
01:04:33 <mayanktg> I'm getting this error http://pastebin.instantbird.com/732416 at line 541 
01:04:35 --> Rym has joined #instantbird
01:05:26 * qheaden goes to bed
01:05:28 <qheaden> Goodnight
01:05:55 <clokep> Try mach configure before building.
01:06:10 <mayanktg> Ok
01:06:15 <-- qheaden has quit (Input/output error)
01:08:20 <mayanktg> I did a mach configure, now I'm building it again...
01:14:20 <-- mconley has quit (Ping timeout)
01:15:44 <-- Armada has quit (Connection reset by peer)
01:16:57 --> mayanktg-ph has joined #instantbird
01:17:07 <-- mayanktg has quit (Ping timeout)
01:21:24 <mayanktg-ph> I'll check after my build is complete whether it solves the problem..
01:27:59 --> mconley has joined #instantbird
01:37:50 <-- Rym has quit (Ping timeout)
01:43:08 <-- mconley has quit (Ping timeout)
01:55:49 --> mconley has joined #instantbird
02:02:07 <-- clokep has quit (Ping timeout)
02:11:47 --> Rym has joined #instantbird
02:29:11 <-- mconley has quit (Ping timeout)
02:30:24 <instantbot> ryanvm@gmail.com changed the Resolution on bug 1013034 from --- to FIXED.
02:30:25 <instantbot> Bug https://bugzilla.mozilla.org/show_bug.cgi?id=1013034 nor, --, ---, qheaden, RESO FIXED, Conference messages aren't broadcast
02:41:55 --> mconley has joined #instantbird
02:53:19 <-- mconley has quit (Input/output error)
03:51:32 <-- Mook has quit (Quit: Mook)
03:53:12 --> mconley has joined #instantbird
04:02:35 <-- mconley has quit (Ping timeout)
04:22:44 <-- mayanktg-ph has quit (Quit: Connection closed for inactivity)
04:29:03 <-- Rym has quit (Ping timeout)
04:29:21 --> Rym has joined #instantbird
05:01:57 <-- Rym has quit (Ping timeout)
05:02:13 --> Rym has joined #instantbird
05:06:53 --> mayanktg-ph has joined #instantbird
05:08:04 --> gerard-majax_ has joined #instantbird
05:10:47 <-- Rym has quit (Ping timeout)
05:11:03 --> Rym has joined #instantbird
05:12:22 <-- gerard-majax_ has quit (Ping timeout)
05:19:42 <-- Rym has quit (Ping timeout)
05:21:22 --> Rym has joined #instantbird
05:42:01 <-- Rym has quit (Ping timeout)
05:58:05 <-- EionRobb has quit (Quit: Leaving.)
06:01:34 --> jb has joined #instantbird
06:08:14 --> sawrubh|ib has joined #instantbird
06:18:16 --> gerard-majax_ has joined #instantbird
06:27:20 <-- gerard-majax_ has quit (Ping timeout)
06:44:06 --> EionRobb has joined #instantbird
07:13:23 --> gerard-majax_ has joined #instantbird
07:17:35 <-- gerard-majax_ has quit (Ping timeout)
07:30:07 --> Mic has joined #instantbird
07:30:07 * ChanServ sets mode +o Mic 
07:34:04 --> gerard-majax_ has joined #instantbird
07:40:00 <-- gerard-majax_ has quit (Ping timeout)
08:17:15 --> gerard-majax_ has joined #instantbird
08:22:33 <sawrubh|ib> Mic: awake?
08:22:53 <Mic> sawrubh|ib: hmm, let me check...
08:22:58 <Mic> Yeah, I think so ;)
08:23:11 <sawrubh|ib> :)
08:23:14 <sawrubh|ib> Mic: is http://mxr.mozilla.org/mozilla-central/source/toolkit/mozapps/downloads/DownloadUtils.jsm#47 how one defined a lazy getter?
08:23:20 <sawrubh|ib> s/defined/defines
08:24:45 <sawrubh|ib> or I think like this https://developer.mozilla.org/en-US/docs/Mozilla/JavaScript_code_modules/XPCOMUtils.jsm#defineLazyGetter%28%29
08:24:46 <Mic> Yes, that's a way to do it. The getter is removed and replaced with a constant value (that also has to be returned on the first call).
08:25:38 <Mic> What do you want to do?
08:26:07 <sawrubh|ib> https://bugzilla.mozilla.org/show_bug.cgi?id=1014644#c24 need to address the lazy getter feedback
08:26:09 <instantbot> Bug 1014644 nor, --, ---, saurabhanandiit, ASSI, Make FileLink work in Instantbird
08:27:20 <Mic> defineLazyGetter is the way, then.
08:27:52 <sawrubh|ib> thanks
08:28:15 <Mic> Here's an example how it is used somewhere else for localisation.
08:28:16 <Mic> https://mxr.mozilla.org/comm-central/source/im/modules/ibTagMenu.jsm#11
08:52:09 <Mic> Test 123 ;) 
08:52:25 <Mic> Cool :)
08:52:37 <Mic> This was sent from a Fx debug console :)
08:53:50 <EionRobb> lol
08:55:11 <Mic> Attach the Firefox dev tools to Instantbird (thanks, Fallen!) and call this.getTabBrowser().selectedConversation._conv.QueryInterface(Components.interfaces.prplIConvChat).sendMsg("<your message>") on the console.
08:55:26 <Mic> I mean that's obvious, isn't it? ;)
08:55:30 <EionRobb> totally
08:55:48 --> mayanktg has joined #instantbird
08:59:17 <-- Mic has quit (Input/output error)
08:59:39 <mayanktg> clokep: Hi. I had to do a |mach clobber && mach configure && mach build| to make the build pass. Even after successful build I'm getting the same error :|
09:09:35 <-- jb has quit (Ping timeout)
09:16:03 --> Mic has joined #instantbird
09:16:03 * ChanServ sets mode +o Mic 
09:17:32 --> jb has joined #instantbird
09:30:55 <-- Mic has quit (Quit: Instantbird -- http://www.instantbird.com)
09:36:11 <-- jb has quit (Ping timeout)
09:49:23 --> Armada has joined #instantbird
10:08:19 --> flo-retina has joined #instantbird
10:08:19 * ChanServ sets mode +qo flo-retina flo-retina 
10:15:39 --> mpmc has joined #instantbird
10:20:50 --> clokep has joined #instantbird
10:20:50 * ChanServ sets mode +o clokep 
10:24:00 <flo-retina> has http://log.bezut.info/instantbird/today#m1 been answered?
10:24:32 <clokep> flo-retina: He's still having trouble.
10:24:54 <sawrubh|ib> clokep: Morning!
10:24:55 <clokep> sawrubh|ib: Hmm....so I just meant define it when it gets called, but now looking at it again, I'm not so sure...
10:25:49 * sawrubh|ib is currently stuck with building the entire tree (which takes a painful 30 minutes) :(
10:26:03 <flo-retina> The reason why it's "[xpconnect wrapped (nsISupports, nsIClassInfo, imIConversation, prplIConversation, nsIObserver, prplIConvIM)]" rather than just "[xpconnect wrapped (prplIConvIM)]" is the interface flattening that happens for objects implementing nsIClassInfo
10:27:21 <clokep> sawrubh|ib: Why are you building the entire tree?
10:27:41 <flo-retina> this.getTabBrowser().selectedConversation._conv.QueryInterface(Components.interfaces.prplIConvChat).sendMsg("<your message>") seems too long
10:28:22 <sawrubh|ib> clokep: sure about what? so should I a) use the defineLazyGetter b) simply move the definition below to when it gets called c) do nothing
10:28:38 <mayanktg> flo-retina: Ok. Mic said to use it this way.... |let conv = this._conv.QueryInterface(Components.interfaces.prplIConvIM);|
10:28:54 <flo-retina> wouldn't this.getTabBrowser().selectedConversation.sendMsg("hello") work?
10:29:05 <flo-retina> mayanktg: what's the context?
10:29:05 <sawrubh|ib> clokep: nothing, I did ./mozilla/mach build im/ and it said you tree isn't configured so I thought I would have to do a full build. Turns out not, so I'm good :)
10:30:12 <mayanktg> flo-retina: I've defined a new function startCall() in the interface prplIConvIM which sends the SDP offer generated in the conversation binding..
10:30:29 --> aleth1 has joined #instantbird
10:30:47 <-- aleth1 has quit (Input/output error)
10:31:26 <flo-retina> and?
10:31:27 <mayanktg> flo-retina: I'm able to generate the message but the Line 541 of http://pastebin.instantbird.com/732455 generates that error message
10:31:30 --> aleth has joined #instantbird
10:31:30 * ChanServ sets mode +o aleth 
10:32:11 <flo-retina> "that error message"?
10:32:28 <mayanktg> flo-retina: http://pastebin.instantbird.com/732416
10:33:04 <clokep> sawrubh|ib: So I didn't mean use defineLazyGetter, just leave it how it is for now.
10:33:14 <mayanktg> Cu.reportError(this._conv); dumps |[xpconnect wrapped (nsISupports, nsIClassInfo, imIConversation, prplIConversation, nsIObserver, prplIConvIM)]|
10:33:16 <aleth> It's probably just that this.conv is a UIConv and doesn't forward the method.
10:33:35 <flo-retina> have you rebuilt after changing the .idl file?
10:34:08 <mayanktg> Yes. After adding the function I changed its uuid and rebuilt IB
10:34:33 <aleth> mayanktg: You need to add startCall here http://dxr.mozilla.org/comm-central/source/chat/components/src/imConversations.js#338
10:34:57 <aleth> Because this.conv is a UIConv object, not the prpl conversation object.
10:35:06 <flo-retina> aleth++
10:35:09 <flo-retina> that's the problem :)
10:35:15 <mayanktg> aleth: Ok!!
10:35:16 <aleth> The UIConv has to forward startCall to the prpl
10:36:36 <sawrubh|ib> clokep: also you asked me to move the IB specific strings away from cloudfile.properties, I have created a new file in im/locales called fileUpload.properties and am having to call createBundle on that too
10:37:02 <sawrubh|ib> just wanted to let you know
10:37:16 <aleth> mayanktg: (looking at the logs) you don't need any QueryInterface calls.
10:37:17 <clokep> Too many conversations. :(
10:37:39 <mayanktg> aleth: So I'll add the startCall in imConversations.js and rebuild it again? Ok I'll remove that :)
10:37:41 <clokep> sawrubh|ib: Why not fileTransfers like everything else? :-S
10:37:54 <aleth> mayanktg: Right.
10:39:55 <sawrubh|ib> clokep: because we already have fileTransfers.properties inside the preferences directory inside http://dxr.mozilla.org/comm-central/source/im/locales/en-US/chrome/instantbird/preferences. I thought a similarly named in http://dxr.mozilla.org/comm-central/source/im/locales/en-US/chrome/instantbird/ would be confusing
10:40:33 <sawrubh|ib> clokep: you won't see the file in the link above, I'm creating it in this patch btw
10:41:10 <clokep> Hmm...where tare the rest of the strings for a conversation stored?
10:41:15 <clokep> These shuld probably go there.
10:43:31 <sawrubh|ib> I guess http://dxr.mozilla.org/comm-central/source/im/locales/en-US/chrome/instantbird/conversation.properties then :)
10:46:24 <clokep> Yep.
10:56:16 <sawrubh|ib> clokep: I find the body of uploadListener to be pretty obvious, I'll try to add some comments however still
10:56:49 <-- EionRobb has quit (Quit: Leaving.)
10:57:51 <clokep> sawrubh|ib: You wrote it. :P
10:58:24 <clokep> I mostly find it obvious too, but I've read it a half dozen times at this point and looked at the TB implementation.
10:59:23 <clokep> Just pastebin something to me and I'll tell you if it's OK
10:59:35 <sawrubh|ib> one minute
11:06:32 <aleth> mayanktg: Is it working now?
11:07:53 <-- clokep has quit (Ping timeout)
11:07:57 <sawrubh|ib> clokep: http://pastebin.instantbird.com/732686
11:10:37 --> Rym has joined #instantbird
11:12:12 <mayanktg> aleth:  I added |startCall: function(aOffer)  this.target.startCall(aString),| in imConversations.js and removed startCall() from prplConversation.idl . I shouldn't have removed it from there :-/ I'm getting |startCall is not a function| . I'll fix this up..
11:12:39 <-- Rym has quit (Ping timeout)
11:12:42 <aleth> Yeah, you shouldn't have removed it. If it's not in the interface, it's not callable ;)
11:12:57 --> Rym has joined #instantbird
11:13:00 <aleth> |startCall: function(aOffer)  this.target.startCall(aString),| looks wrong to me. what's aString?!
11:13:55 <mayanktg> aleth: It should be the way its in jsProtoHelper? |startCall: function(aOffer) { throw Cr.NS_ERROR_NOT_IMPLEMENTED; },|
11:14:28 <aleth> no
11:15:44 <flo-retina> the question was "what's aString?" ;)
11:16:05 <aleth> hint: it's a very simple mistake ;)
11:16:20 <mayanktg> Yeah
11:16:25 <mayanktg> understood
11:16:27 <mayanktg> :-/
11:16:50 * sawrubh|ib will soon send flo-retina a review request
11:16:50 <mayanktg> |startCall: function(aOffer) this.target.startCall|
11:17:05 <-- nhnt11 has quit (Ping timeout)
11:17:12 <aleth> Take a deep breath and look at it again ;)
11:17:12 <flo-retina> no...
11:17:25 <flo-retina> well, yes to taking a deep breath :)
11:17:34 --> nhnt11 has joined #instantbird
11:18:33 <mayanktg> aleth, flo-retina: I'm stupid :( |function(aOffer) this.target.startCall(aOffer),|
11:18:41 <aleth> :)
11:18:41 <flo-retina> seems more likely to work
11:18:52 * flo-retina likes the deep-breath-debugging pattern :)
11:19:47 <mayanktg> I have struggled with this all night..and I'm desperate to receive the "offer" 
11:20:06 <mayanktg> I should go for lunch..feeling hungry :-|
11:20:08 <aleth> It's easy to get confused at first with all these conversation objects. What's going on is that this.conv is a UIConv (defined in imConversations.js). From the UIConv, this.target gives you the prp conversation.
11:20:27 <aleth> Lots of layers...
11:21:13 <mayanktg> Yes. Is there a doc or some reading content available for prpl?
11:22:12 <aleth> Not really. We haven't written any mdn pages for them ;)
11:22:22 <aleth> The "docs" are the comments in the interfaces.
11:22:28 <aleth> Just ask here! :D
11:22:34 <aleth> Anyway, it should work now.
11:22:47 <mayanktg> Ok..thanks a lot btw :)
11:27:13 <mayanktg> aleth: It worked! I can send the offer :D and the callee is receiving it too :D ..let me crosscheck it
11:27:43 <-- aleth has quit (Ping timeout)
11:30:03 <mayanktg> yepp
11:31:11 --> aleth has joined #instantbird
11:31:11 * ChanServ sets mode +o aleth 
11:31:26 <aleth> mayanktg: congrats :)
11:32:00 <mayanktg> Now I'll have to it for the video call button..
11:32:19 <aleth> I think there's a word missing there?
11:33:10 <mayanktg> aleth: Yes. "implement"
11:33:29 <aleth> I thought this already used the video button?
11:34:22 <mayanktg> No.. |videoCallButton.setAttribute("oncommand", "this.startCall();");| gives error |startCall is not a function|
11:35:12 <aleth> Oh, if you're using oncommand, you have to read about XUL commands.
11:35:26 <aleth> https://developer.mozilla.org/en-US/docs/Mozilla/Tech/XUL/Tutorial/Commands
11:35:49 * sawrubh|ib looks up examples to add a dropdown select menu in a notificationbx
11:35:59 <mayanktg> ok. Even on using it like videoCallButton.onclick = funtion() {...}; I was getting the same error
11:36:29 <aleth> That's because you want to add an event handler, not an attribute.
11:37:09 <aleth> You were just setting a JS property on the videoCallButton object.
11:38:35 <aleth> But if you're using commands, you're not dealing directly with events at all.
11:38:48 <aleth> Just read the page I linked to to find out what they are.
11:38:56 <mayanktg> Ok. Yeah I'm getting your point. The page says that
11:38:57 <mayanktg> :)
11:40:19 <aleth> Why are you using a command by the way?
11:41:30 <aleth> Here's where the existing conversation commands http://mxr.mozilla.org/comm-central/source/im/content/instantbird.xul#54
11:43:29 <mayanktg> aleth: I've implemented a video call button. So when I click on that button I should be able to send "offer" by initiating the startcall method in conversation binding.
11:43:57 <aleth> Right.
11:44:06 <aleth> Oh, I see Mic suggested it.
11:44:34 <mayanktg> Yeah.. ;)
11:44:35 <aleth> I was just surprised you had discovered them on your own ;)
11:45:16 <mayanktg> Mic suggested to use oncommand..you only told about command 
11:45:22 <mayanktg> :)
11:46:01 <aleth> Using a command is a bit more involved. But if you read that page you can figure it out, just add it to the existing ones.
11:49:18 <-- sawrubh|ib has quit (Connection reset by peer)
11:51:45 --> sawrubh|ib has joined #instantbird
11:52:48 <-- mayanktg has quit (Quit: Instantbird 1.6a1pre -- http://www.instantbird.com)
12:01:42 <-- sawrubh|ib has quit (Ping timeout)
12:01:49 --> clokep_work has joined #instantbird
12:01:49 * ChanServ sets mode +o clokep_work 
12:02:24 --> sawrubh|ib has joined #instantbird
12:05:00 <clokep_work> aleth: Isn't using command just setting "oncommand" instead of "onclick"? :-S
12:05:19 <aleth> No, you have to add a command element, and that's what you then refer to.
12:05:45 <aleth> iirc.
12:05:50 <-- BWMerlin has quit (Quit: BWMerlin)
12:05:58 <clokep_work> sawrubh|ib: Do you have something against blank lines? :-S You keep squishing all your code together!
12:07:29 * sawrubh|ib was a mobile phone in past life, used to small screens
12:07:56 <clokep_work> sawrubh|ib: "convBundle" should exist already somewhere, I don't think you have to recreate it.
12:08:32 <sawrubh|ib> clokep_work: I'll add more blank lines, I'm just confused most of the times, when someone might just say 'You're using too much space'
12:08:42 <clokep_work> sawrubh|ib: http://pastebin.instantbird.com/732711
12:08:52 <clokep_work> sawrubh|ib: Have we ever said that to you?
12:08:58 <clokep_work> Make it as readable as possible.
12:09:17 <sawrubh|ib> ok :)
12:09:35 <nhnt11> hello
12:10:07 <sawrubh|ib> olleh
12:11:45 <clokep_work> sawrubh|ib: So those comments are OK. The first one I'd say more of "Only handle dropped files."
12:11:55 <clokep_work> I think the cancelEvent line should have a comment.
12:12:20 <clokep_work> All comments should be full sentences (end in a period).
12:12:58 <clokep_work> I don't like the property name "uploadStartedNotif" btw.
12:13:31 <clokep_work> "Send a message with the filename and the filelink" what's "the filelink"? I htink you mean "with the filename and a link to the uploaded file."
12:13:42 <aleth> clokep_work: maybe we should add that to https://wiki.instantbird.org/Instantbird:Coding_Guidelines ;)
12:13:53 <aleth> ie. the commenting stuff
12:14:41 <clokep_work> It's in the Mozilla guide somewhere, I think.
12:16:42 <-- aleth has quit (Ping timeout)
12:17:17 --> aleth has joined #instantbird
12:17:17 * ChanServ sets mode +o aleth 
12:18:42 <sawrubh|ib> I see other non-full sentences in the code, just saying
12:19:13 <clokep_work> sawrubh|ib: That doesn't mean you should do it.
12:19:28 <sawrubh|ib> yes, I was just saying :P
12:26:27 <sawrubh|ib> clokep_work: which one do you prefer : 'Cancel the given event.' or 'This is used to cancel the given event.' ?
12:27:40 <clokep_work> sawrubh|ib: Neither, I want to know *WHY* you're doing it.
12:27:48 <clokep_work> I can tell it's cancelling the event from the function name.
12:28:09 <sawrubh|ib> ah, ok
12:28:48 <sawrubh|ib> but structure wise, the fist one will be a considered a full sentence right (because of the period) or not (because of the lack of 'This')
12:31:03 <clokep_work> "Cancel the given event." is a full sentence.
12:31:41 <sawrubh|ib> btw I add '// For an element to be a valid drop target, we must cancel the last ondragenter and ondragover events' comment to where I add the cancelEvent handler to ondragover and ondragenter, so that should explain the 'why' part
12:35:10 <-- Rym has quit (Ping timeout)
12:35:26 --> Rym has joined #instantbird
12:35:37 <-- ivan has quit (Quit: ERC Version 5.3 (IRC client for Emacs))
12:37:30 <clokep_work> :-(
12:37:52 <clokep_work> I thought you and aleth had a conversation saying that you don't even need to cancel that event there?
12:38:51 <-- Rym has quit (Ping timeout)
12:39:08 --> Rym has joined #instantbird
12:42:18 <-- mayanktg-ph has quit (Quit: )
12:42:30 --> mayanktg has joined #instantbird
12:43:23 --> ivan has joined #instantbird
12:45:18 <aleth> mayanktg: done?
12:46:45 <mayanktg> aleth: Just returned from lunch. I have added the <command> to the XUL. How should I initiate the startCall method from the command? I mean what should be |oncommand="__.startCall()"| so that startCall is called from the binding?
12:47:54 <mayanktg> this.startCall() doesn't recognize it as a function.
12:48:00 <aleth> mayanktg: Something like this should work http://mxr.mozilla.org/comm-central/source/im/content/instantbird.xul#63
12:48:42 <flo-retina> is it a recent regression that the findbar doesn't indicate the "not found" case?
12:48:42 <mayanktg> ok
12:49:03 <aleth> flo-retina: Sounds like it
12:49:31 <flo-retina> aleth: can you reproduce it too?
12:49:40 <aleth> Yes
12:49:45 <aleth> (i.e. the red colour is missing)
12:49:52 <mayanktg> aleth: Done :)
12:50:03 <sawrubh|ib> clokep_work: we discussed that the eventListener we're adding for the "select" event (in addAccountDialog.js), doesn't require adding cancelEvent since we don't want to cancel the default handler however in the case of 'drop' event, we need to cancel because the default handler would prevent the dropping of the file
12:50:07 <aleth> mayanktg: So it works now? :)
12:50:20 <mayanktg> aleth: Yes it does :)
12:50:26 <aleth> Good! :)
12:50:30 <flo-retina> aleth: wasn't there also a message in the empty space after the "Match case" button?
12:50:35 <-- Tonnes has quit (Connection reset by peer)
12:50:39 <aleth> mayanktg: What's next?
12:50:47 <aleth> flo-retina: I don't remember
12:51:07 <aleth> There is in FX though, so we should have it too.
12:51:16 --> qheaden has joined #instantbird
12:51:29 <mayanktg> aleth: What about the /call command at the xml-commands.jsm :-o . Next is to convert the xml stanza back to original offer?
12:51:43 <clokep_work> sawrubh|ib: I thought that was only for the dragging stuff, not for the dropping?
12:51:52 <clokep_work> What would block you from receiving it on the dropping?
12:52:20 <aleth> mayanktg: The call command was for the WIP so you could test things. You can't make it work easily as you need the DOMWindow to generate the offer.
12:52:39 <aleth> mayanktg: Yes, moving on to the receiving end sounds like the next step.
12:53:11 <instantbot> New Instantbird - Conversation bug 1022619 filed by florian@queze.net.
12:53:12 <instantbot> Bug https://bugzilla.mozilla.org/show_bug.cgi?id=1022619 nor, --, ---, nobody, NEW, The findbar doesn't show anything for the "not found" case
12:53:15 <mayanktg> aleth: Yes. I'm removing it then. We will only have the call initiation from the toolbarbutton then.
12:53:16 <qheaden> Hello everyone.
12:53:30 <mayanktg> aleth: Ok
12:54:27 <aleth> mayanktg: I guess you'll need an xml2sdp which does the whole thing backwards ;)
12:55:10 <aleth> mayanktg: This would be a good point at which to attach a WIP to your bug and get some feedback from Mic.
12:55:45 <mayanktg> aleth: yepp. In which i'll have to fetch the <content/> from the xml stanza and then reconstruct the offer. 
12:56:10 <mayanktg> Ok. I'm attaching a WIP after reomving the xml-commands.jsm from the patch.
12:57:00 <sawrubh|ib> clokep_work: I can't understand what that last statement means, can you explain it?
12:58:07 <clokep_work> sawrubh|ib: I don't understand why you ahve to cancel the drop event? I thought you only have to do that for the drag events.
12:58:16 <clokep_work> In other words, explain YOUR last statement.
12:58:56 <aleth> mayanktg: Well, xml2sdp should be the reverse of sdp2xml, so in the end you can do xml2sdp(sdp2xml(sdpstring)) and get the sdpstring back ;)
13:00:04 <mayanktg> Hmm.. cool :) going for it!
13:00:53 <aleth> Something like that at least. You can start with looking at where you need to handle the received stanza. Probably you'll have to add some function that handles Jingle nodes.
13:01:25 <aleth> And that will then call helper functions like xml2sdp...
13:01:41 <sawrubh|ib> clokep_work: afk for 5 minutes
13:01:47 <-- Rym has quit (Ping timeout)
13:01:49 <mayanktg> aleth: I', first adding icons for aero in the userIcon bug as Mic said yesterday. 
13:01:56 <mayanktg> *I'm
13:02:01 <aleth> Good idea, get that patch landed :)
13:02:39 <mayanktg> Yeah. I have to fix the design issue on Mac and Windows without having one :-|
13:03:18 <aleth> I doubt you can really do that. Other people will have to tell you what to change, possibly after landing.
13:04:48 <mayanktg> Hmm...me too. Anyways I'm first adding the icons for aero and sending my WIP for Mic.
13:05:00 <aleth> Yes, that should fix it for Windows ;)
13:05:04 --> Rym has joined #instantbird
13:10:18 <-- aleth has quit (Ping timeout)
13:10:45 --> aleth has joined #instantbird
13:10:45 * ChanServ sets mode +o aleth 
13:13:35 <-- Rym has quit (Ping timeout)
13:13:40 <sawrubh|ib> clokep_work: so https://developer.mozilla.org/en-US/docs/DragDrop/Drag_Operations#drop says drag event needs to be cancelled to make it a valid target, and I need to cancel the drop event otherwise the default handler will also handle it (which could be anything like in this showing the name of the file I guess)
13:13:50 --> Rym has joined #instantbird
13:14:49 <-- mayanktg has quit (Ping timeout)
13:15:22 --> mayanktg has joined #instantbird
13:20:26 <sawrubh|ib> clokep_work: also 'uploadStartedMsg' sounds good?
13:28:20 <clokep_work> sawrubh|ib: Better, yes.
13:28:42 <clokep_work> sawrubh|ib: I guess...what IS the default handler for dropping something? Is there actually one? Otherwise, I find it confusing that we would be cancelling it.
13:28:49 <clokep_work> Do we normally stop events from bubbling?
13:29:00 <flo-retina> I think there are enough people here with Macs that it shouldn't be too difficult to find someone who could help for the mac theming (mayanktg)
13:29:03 <clokep_work> (That's a question for aleth and flo-retina.)
13:29:41 <flo-retina> clokep_work: I think default handling is "nope, this is not a drop target, go away", and that's what we are canceling
13:29:47 <aleth> clokep_work: No, we don't, as I mentioned yesterday. See https://developer.mozilla.org/en-US/docs/XUL_Event_Propagation
13:29:48 <sawrubh|ib> clokep_work: so 'Varies' is your answer : https://developer.mozilla.org/en-US/docs/Web/Reference/Events/drop ;)
13:30:04 <mayanktg> flo-retina: I will ask someone to fix it up then :)
13:30:11 <clokep_work> sawrubh|ib: That's not a helpful response.
13:30:16 <flo-retina> :
13:30:29 <flo-retina> mayanktg: I'll do before r+'ing if that patch reaches my review queue again
13:30:32 <clokep_work> sawrubh|ib: So is it clear what the comment above that needs to say then?
13:31:37 <sawrubh|ib> yes, '// Need to prevent the default handlers for drop.' is what I have in mind for the call to cancelEvent in handleDroppedFiles
13:31:53 <mayanktg> flo-retina: Ok. I'll make some changes which Mic has suggested and then I'll send it for review. 
13:32:55 <clokep_work> sawrubh|ib: "Prevent the default drop handler." is more concise.
13:33:18 <clokep_work> I generally don't like things that start with "Need to" or "Check that", I like comments that say what we are doing. :)
13:33:27 <clokep_work> But that's really my own style, not anything we enforce, most likely.
13:36:16 <-- aleth has quit (Ping timeout)
13:36:52 --> aleth has joined #instantbird
13:36:52 * ChanServ sets mode +o aleth 
13:37:18 <clokep_work> sawrubh|ib: That make sense?
13:37:37 <sawrubh|ib> yes
13:41:16 <clokep_work> So what else is there to do? :)
13:41:33 <clokep_work> Btw I'd highly suggest getting the mv cloudfile patch up again before adding the notificationbox.
13:41:40 <clokep_work> So if I think it's OK I can toss the review to flo.
13:41:59 <flo-retina> nhnt11, sawrubh|ib: reminder re blogging weekly updates ;)
13:42:23 <nhnt11> flo-retina: I was going to write another post when async logs lands
13:42:30 <nhnt11> But I should write one about my Tb debugging troubles :S
13:42:43 <sawrubh|ib> I was planning one for when this patch lands :s
13:43:08 <sawrubh|ib> But I should write one about my work on getting this patch there :s
13:44:15 <flo-retina> I don't see how "when this patch lands" relates with the weekly update requirement.
13:44:29 <flo-retina> until a patch has landed, there's no accurate way to predict when it will
13:45:03 <flo-retina> of course if you are super excited when it does land, there's no reason to not blog about your excitement at that time; but that's not a weekly progress report :)
13:47:03 <nhnt11> flo-retina: Well fwiw my progress report would be "Addressed review comments" and a rant about debugging Thunderbird. But okay! :)
13:47:25 <flo-retina> you spent a week addressing review comments?
13:47:35 <flo-retina> did you learn anything interesting thanks to these review comments?
13:47:39 <flo-retina> what are you expecting to do next?
13:47:40 <nhnt11> Hmm, I also played around with the convbrowser
13:47:47 <nhnt11> Alright. Fair enough.
13:48:29 <nhnt11> Yeah I learned about async xpcshell tests and other things, I should keep track of my week
13:48:37 <nhnt11> Which is why I should blog I guess
13:48:38 <nhnt11> :]
13:52:32 <-- Even has quit (Ping timeout)
13:53:03 <clokep_work> FWIW I'd suggest nhnt11, sawrubh|ib and mayanktg blog ~ 2 a week.
13:53:08 <clokep_work> It's easy to forget what you've done after a week.
13:53:18 <flo-retina> weekly is a minimum
13:57:16 <sawrubh|ib> clokep_work: so I just did a self review and everything looks good, should I send it for review or get someone on Windows and Mac to test the styling I have done for those platforms
13:57:24 --> Even has joined #instantbird
13:57:25 * ChanServ sets mode +o Even 
13:57:30 <sawrubh|ib> otherwise I'm good for a review (and then a Try push I guess)
13:57:57 <clokep_work> sawrubh|ib: I don't have a windows machine w/ me at the moment.
13:58:02 <clokep_work> Upload it and I'll tryo n Mac though.
13:58:10 <clokep_work> After my meeting.
13:59:22 * sawrubh|ib wears his Windows-machine-detection-glasses and scans for nearby volunteers for testing
14:00:52 <sawrubh|ib> ...none found :(
14:01:05 * mayanktg bbl
14:01:17 <-- mayanktg has quit (Quit: Instantbird 1.6a1pre -- http://www.instantbird.com)
14:04:30 --> iamjayakumars has joined #instantbird
14:07:35 --> mayanktg-ph has joined #instantbird
14:07:36 <-- sawrubh|ib has quit (Ping timeout)
14:09:02 --> sawrubh|ib has joined #instantbird
14:12:33 <flo-retina> sawrubh|ib: I thought nhnt11 just explained how much space he kept for Windows :_P
14:12:59 <sawrubh|ib> *evil smile*
14:13:14 * nhnt11 cant build on windows fyi
14:13:30 <nhnt11> So if you have a try build..
14:13:47 <nhnt11> (I'm eating right now btw, brb)
14:14:00 <aleth> so the "try" in "try build" is short for "try to get someone to run this"? :P
14:15:23 * sawrubh|ib uploads first
14:16:45 <-- Rym has quit (Ping timeout)
14:17:01 --> Rym has joined #instantbird
14:17:29 <sawrubh|ib> aleth: can you take a look, just to make sure I don't do anything stupid with regard to the css and then I'll push to try
14:17:41 <sawrubh|ib> I've uploaded the patch btw
14:18:15 * sawrubh|ib goes to work on notificationbox
14:18:38 <aleth> What css? I don't have time for a full review
14:19:33 <sawrubh|ib> ok, I'll pastebin a diff between the version I just uploaded and the previous one
14:19:49 <sawrubh|ib> and I was talking about Windows and Mac CSS I wrote in this patch for the pref pane
14:21:01 <aleth> If you are just copying CSS from TB, there should be no problem.
14:21:39 <sawrubh|ib> ooh, BZ gives the option of finding interdiff cool
14:21:42 <aleth> I don't have time to look at this in detail right now, sorry.
14:22:05 <sawrubh|ib> np, I'll go work on something else anyway
14:23:56 <-- aleth has quit (Ping timeout)
14:24:30 --> aleth has joined #instantbird
14:24:30 * ChanServ sets mode +o aleth 
14:25:42 <-- Rym has quit (Ping timeout)
14:25:58 <sawrubh|ib> I also need to work on bug 954497 and setting the topic bug :/
14:25:59 --> Rym has joined #instantbird
14:26:00 <instantbot> Bug https://bugzilla.mozilla.org/show_bug.cgi?id=954497 nor, --, ---, nobody, NEW, Unsharp MUC icon in infobar on conversation top (height is 1px too large)
14:26:13 <flo-retina> that bug should be very low priority
14:26:25 <flo-retina> poke at it if you are stuck because all your other patches are waiting for reviews :)
14:26:46 <flo-retina> weren't you almost done with the topic bug?
14:26:56 <sawrubh|ib> no, notifbox is more priority right now
14:28:04 <sawrubh|ib> yes, I was nearly done but then writing the xpcshell seemed like a new things so I didn't delve into it anytime
14:29:01 <nhnt11> aleth: I meant if you have a build ready to test (since I can't build myself)
14:30:47 <flo-retina> if the patch in bug 1011226 works, I think we could land it without xpcshell test to avoid bitrot
14:30:50 <instantbot> Bug https://bugzilla.mozilla.org/show_bug.cgi?id=1011226 nor, --, ---, saurabhanandiit, NEW, Support setting the topic in XMPP MUCs
14:31:29 * sawrubh|ib goes to first spend 1 hour on that topic bug
14:31:44 <sawrubh|ib> and if it get's fixed cool, otherwise notifbox it is
14:34:43 <-- nhnt11 has quit (Ping timeout)
14:34:57 --> Tonnes has joined #instantbird
14:35:06 --> nhnt11 has joined #instantbird
14:36:57 <-- aleth has quit (Ping timeout)
14:37:40 <-- Rym has quit (Ping timeout)
14:39:12 <clokep_work> Yeah, I don't remember asking for an xpcshell test on that bug.
14:40:56 <clokep_work> bug 1021684 is unfortunate. :-\
14:40:58 <instantbot> Bug https://bugzilla.mozilla.org/show_bug.cgi?id=1021684 nor, --, ---, nobody, NEW, Update box.com Filelink implementation to new APIs
14:42:50 <sawrubh|ib> hrm, so we need to land our patch before it gets bitrotten
14:43:19 <clokep_work> That was not exactly what I was saying, but sure.
14:43:34 --> mconley has joined #instantbird
14:43:45 * sawrubh|ib was just being selfish, sorry
14:44:12 <flo-retina> you may also volunteer to fix that bug :-P
14:44:40 * sawrubh|ib finds a rock to hide under ;)
14:48:44 --> mayanktg has joined #instantbird
14:50:05 <-- nhnt11 has quit (Ping timeout)
14:50:50 --> nhnt11 has joined #instantbird
14:54:19 <-- iamjayakumars has quit (Client exited)
14:54:31 --> iamjayakumars has joined #instantbird
15:02:18 <clokep_work> sawrubh|ib: Your patch still doesn't work for me.
15:03:48 <-- mayanktg-ph has quit (Quit: )
15:04:13 <clokep_work> flo-retina, aleth: Any chance you guys can test this? :-S
15:04:20 <flo-retina> this = ?
15:05:23 <sawrubh|ib> clokep_work: 'doesn't work' = you aren't able to drop the file on the conversation area or the textbox?
15:06:00 <sawrubh|ib> flo-retina: this = the two patches in bug 1014644
15:06:02 <instantbot> Bug https://bugzilla.mozilla.org/show_bug.cgi?id=1014644 nor, --, ---, saurabhanandiit, ASSI, Make FileLink work in Instantbird
15:07:04 <clokep_work> sawrubh|ib: Nothing happens when I do, yes.
15:07:29 <sawrubh|ib> clokep_work: do you see any errors in the error console?
15:08:12 <sawrubh|ib> have you turned on the preference btw?
15:09:24 <sawrubh|ib> unlike last time (which was a long time ago) I'm (or atleast was) very confident it should work this time :(
15:09:50 <sawrubh|ib> do you see the preference pane?
15:10:55 <clokep_work> sawrubh|ib: No.
15:11:01 <clokep_work> (No errors.)
15:11:10 <sawrubh|ib> are you sure you
15:11:15 <clokep_work> sawrubh|ib: Hm...I think I flipped the pref, but it's possible I modified the wrong profile.
15:11:17 <sawrubh|ib> you've preffed on the flags
15:11:19 <clokep_work> sawrubh|ib: Yes. I'm positive.
15:11:58 <sawrubh|ib> can you check if it's the right profile, please?
15:12:16 <-- iamjayakumars has quit (Client exited)
15:12:43 <clokep_work> You need to calm down. :P
15:12:48 <clokep_work> It takes me a few minutes to do tehse things.
15:12:55 <clokep_work> And yes, it was the wrong profile.
15:13:07 <clokep_work> I don't think I have a Box.com or YouSendIt account though. :-S
15:14:16 <sawrubh|ib> Mr. User, well you'll have to wait for the protocol file transfer support </kidding>
15:14:41 <clokep_work> Or get the Dropbox addon to work in Instantbird. ;)
15:15:23 * nhnt11 will probably write a puush addon once this lands if it doesn't already exist
15:15:54 <sawrubh|ib> nhnt11: how is a puush addon related to my patch?
15:16:05 <clokep_work> Bah, Box.com requires a phone number? That's annoying...
15:16:06 <nhnt11> sawrubh|ib: For filelink?
15:16:25 <nhnt11> Drop a file, it get's sent to puush, link gets sent to the other person..
15:17:39 <nhnt11> s/get's/gets
15:17:43 <sawrubh|ib> why not land it here : http://mxr.mozilla.org/comm-central/source/mail/components/cloudfile/content/ like the others
15:18:23 <sawrubh|ib> ..because a lot of people might not want it
15:18:36 <nhnt11> That only has Box and Hightail from what I see, so I have no idea of the criteria for landing it there
15:19:02 <nhnt11> Also if it's an addon I can just spend the minimal amount of effort to make it work
15:19:15 * nhnt11 shrugs. This is a premature discussion
15:19:50 * sawrubh|ib goes back to working on the topic bug since things seem to be working for clokep_work
15:20:14 <clokep_work> Bah I can't use the dropbox add-on...firewalled at work.
15:20:28 <clokep_work> sawrubh|ib: OK so initially it seems reasonable at least. :) None of the UI was totally janked either.
15:20:30 <sawrubh|ib> use mobile data connection?
15:20:44 <clokep_work> sawrubh|ib: Tethering in the US costs INSANE amounts of money.
15:21:03 <flo-retina> is it different from just using 3G on the phone?
15:21:06 <clokep_work> The topic bug + the notificationbar stuff should get done.
15:21:07 * sawrubh|ib will keep that in mind in case he ever lands in the US
15:21:18 <clokep_work> flo-retina: Should it be? No. Is it? Yes.
15:21:42 <flo-retina> change country? :-P
15:21:53 <sawrubh|ib> hee hee hee
15:27:59 --> xangua has joined #instantbird
15:29:12 --> aleth has joined #instantbird
15:29:12 * ChanServ sets mode +o aleth 
15:33:19 <clokep_work> sawrubh|ib: Sure, ask florian for f/r.
15:37:40 <nhnt11> Oh, aleth is back
15:38:02 <nhnt11> aleth: "Would it be useful to also test this with dummy operations that resolve/reject after a timeout?" I don't think so
15:38:31 <nhnt11> But I'm investigating
15:39:15 <nhnt11> Also "shouldn't you be asserting ok at the end only if promiseB resolves/rejects correctly?" I think "yield p2.then(() => ok(false), () => ok(true));" would eliminate the need for this
15:39:19 <nhnt11> But again, investigating
15:40:23 <sawrubh|ib> clokep_work: so http://pastebin.instantbird.com/732794 is my WIP, now devel@conference.pidgin.im sends a message in the format I mention in https://bugzilla.mozilla.org/show_bug.cgi?id=1011226#c4 and in order to call setTopic and get it to the show the messages in the format I like '%s has set the topic to %s', I need to pass a setter and in order to extract that setter from the format in which pidgin is sending
15:40:26 <instantbot> Bug 1011226 nor, --, ---, saurabhanandiit, NEW, Support setting the topic in XMPP MUCs
15:40:42 <sawrubh|ib> it to me, I need to pull it from the body
15:41:51 <-- nhnt11 has quit (Ping timeout)
15:42:02 <-- gerard-majax_ has quit (Ping timeout)
15:42:23 --> nhnt11 has joined #instantbird
15:42:57 <-- Tonnes has quit (Quit: ChatZilla 0.9.90.1 [Firefox 29.0.1/20140506152807])
15:43:16 <clokep_work> sawrubh|ib: Is there a question?
15:43:39 <mayanktg> aleth: I'll make the changes as you've suggested in the bug and upload the patch soon.
15:48:52 <sawrubh|ib> clokep_work: so the alternatives are a) not pulling it from the body (since that sounds hack-ish and other channels might not do it this way) b) just showing it in the format 'The topic for <channel name> is %s'
15:49:00 <sawrubh|ib> which one do you think makes sense?
15:50:23 --> iamjayakumars has joined #instantbird
15:50:31 <mayanktg> Mic: I've added the button icons for the aero. Here's the inter diff for the same http://pastebin.instantbird.com/732800 .
15:50:49 <-- iamjayakumars has quit (Client exited)
15:52:47 --> arlolra has joined #instantbird
15:59:24 <-- nhnt11 has quit (Ping timeout)
15:59:37 --> nhnt11 has joined #instantbird
15:59:57 <-- sawrubh|ib has quit (Ping timeout)
16:00:03 --> sawrubh|ib has joined #instantbird
16:00:41 <clokep_work> sawrubh|ib: When you say "pull it from the body" do you mean the CONTENTS of the body or an attribute of the body?
16:01:55 <sawrubh|ib> contents of the body
16:02:30 <clokep_work> We probably shouldn't do that.
16:02:37 <clokep_work> What does the XEPs say?
16:04:30 <sawrubh|ib> well the XEP (http://xmpp.org/extensions/xep-0045.html#subject-mod) says that the subject modifying message shouldn't contain a body, which the pidgin server doesn;t comply to
16:07:31 * sawrubh|ib pushes to try
16:07:45 <clokep_work> Didn't one of the older RFCs or somethign say something about the topic being in the body?
16:08:30 <sawrubh|ib> I've been complying with this XEP
16:08:38 * sawrubh|ib checks the older RFS
16:09:18 <clokep_work> I understand that, but if they're using an older server that doesn't...
16:10:20 <-- nhnt11 has quit (Ping timeout)
16:11:22 --> clokep_w1 has joined #instantbird
16:11:23 --> nhnt11 has joined #instantbird
16:11:47 * clokep_w1 is now known as clokep_work1
16:12:35 <-- clokep_work has quit (NickServ (GHOST command used by clokep_work1))
16:12:40 * clokep_work1 is now known as clokep_work
16:12:52 * ChanServ sets mode +o clokep_work 
16:14:29 <-- aleth has quit (Ping timeout)
16:17:02 --> aleth has joined #instantbird
16:17:02 * ChanServ sets mode +o aleth 
16:18:30 <sawrubh|ib> clokep_work: yeah, http://xmpp.org/rfcs/rfc6121.html#message-syntax-subject says that body can be there
16:20:24 <sawrubh|ib> hrm, but it says the subject will still be in the subject element and not the body...
16:22:25 * sawrubh|ib afk for a bit
16:24:19 <clokep_work> sawrubh|ib: OK, so...it sounds to me like we shouldn't be taking it from the body.
16:25:22 <clokep_work> flo-retina: I don't think you saw my previous question of this, but do we need to also do something similar to bug 1017696?
16:25:24 <instantbot> Bug https://bugzilla.mozilla.org/show_bug.cgi?id=1017696 nor, --, seamonkey2.30, philip.chee, RESO FIXED, Use a JSON storage back-end in the Login Manager - SeaMonkey part
16:25:54 <-- aleth has quit (Ping timeout)
16:26:55 --> aleth has joined #instantbird
16:26:55 * ChanServ sets mode +o aleth 
16:30:35 --> gerard-majax_ has joined #instantbird
16:33:25 <-- gerard-majax_ has quit (Ping timeout)
16:34:15 <-- nhnt11 has quit (Input/output error)
16:34:19 --> nhnt11 has joined #instantbird
16:36:37 * sawrubh|ib -> back
16:55:15 <flo-retina> clokep_work: so it looks like they did the packaging change in im/ for us (thanks!)
16:55:32 <clokep_work> flo-retina: WOOT! :) I didn't even think to check the patch.
16:56:02 <flo-retina> clokep_work: for the async loading stuff... I think the chat/ core will need passwords near startup so the hack there copied from Firefox to suite/ wouldn't really save us much (unless the user doesn't connect accounts automatically at startup)
16:56:14 <flo-retina> summary: I think we've got nothing to do.
16:56:29 <clokep_work> OK. :)
17:06:27 <nhnt11> aleth: How would you suggest adding a zero-length log file to the test?
17:06:42 <aleth> nhnt11: You probably have to make one, then read it.
17:06:43 <nhnt11> Would just writing "" to a file work?
17:06:44 * nhnt11 doesn't think so
17:06:57 * nhnt11 does some testing
17:11:07 --> rosonline has joined #instantbird
17:12:06 <nhnt11> Cool, open {truncate: true} and then close creates a zero-length file.
17:12:29 <aleth> I guess that's the equivalent of "touch"
17:12:32 * nhnt11 was worried about a newline automatically being added or something
17:12:33 <nhnt11> yeah
17:13:31 <nhnt11> aleth: Touch doesn't truncate though :)
17:14:08 <aleth> It's more harmless I guess
17:16:02 <clokep_work> Mostly Harmless? :P
17:17:52 <aleth> As long as you bring a towel.
17:18:23 * clokep_work is glad someone got it. :)
17:18:37 * nhnt11 googles that now
17:19:53 <-- rosonline has quit (Ping timeout)
17:19:55 <clokep_work> nhnt11: Hitchhikers Guide to the Galaxy.
17:20:04 <nhnt11> got it
17:20:46 <mayanktg> aleth: I'm now using the generated id and sid, changed the variable name iqStanza to iqContent. But, 1. using |for (let contentNode of iqContent) item.addChild(contentNode)| gives the error |iqContent['@@iterator'] is not a function|
17:20:46 <mayanktg> 2. Cu.reportError(this._targetResource) returns an empty string) which is I guess because we are _sending_ the stanza and IncomingMessage() http://mxr.mozilla.org/comm-central/source/chat/protocols/xmpp/xmpp.jsm#260 isn't called yet so the value remains set to |_targetResource = ""| .
17:21:11 --> rosonline has joined #instantbird
17:21:37 <aleth> mayanktg: 1. isn't the return value of sdp2xml an array?
17:22:19 <mayanktg> It is. 
17:22:23 <aleth> 2. So the problem goes away as long as you have received a message from the target before video calling?
17:24:05 <mayanktg> Yes. We will set the _targetResource once we have received a message from the callee before we make an offer.
17:25:08 <aleth> Hmm.
17:25:25 <aleth> You have to change your current code because it will completely break if you have already received a message
17:25:51 <nhnt11> bbiab
17:26:20 <mayanktg> aleth: Yeah you are right.
17:26:42 <mayanktg> it will be barID/Instantbird/Instantbird...
17:26:50 <mayanktg> No only twice.
17:28:52 <-- aleth has quit (Ping timeout)
17:29:25 <mayanktg> Yep..I'm getting repetitive resources.  
17:30:44 <mayanktg> I'll change it up.. :)
17:31:07 * sawrubh|ib grumbles something about weird errors
17:33:02 <sawrubh|ib> btw I'm not too clear after reading https://wiki.mozilla.org/ReleaseEngineering/ThunderbirdTryServer, it says 'The Thunderbird try server is primarily for building Thunderbird. Whilst building other comm-central apps may work, this is not supported - builds and tests may fail etc. '
17:33:12 <mayanktg> aleth: fixed.
17:33:55 <sawrubh|ib> ah, wait. I don't want to build im anyways, I just want to build TB and run the tests. Cool.
17:36:07 <sawrubh|ib> clokep_work: 'try: -b o -e -p linux,linux64,macosx64,win32,win64 -u mozmill -t none' sounds like the right config with which I should push to try?
17:36:32 <clokep_work> sawrubh|ib: I don't know, use try chooser.
17:36:36 --> Rym has joined #instantbird
17:36:42 <clokep_work> sawrubh|ib: Don't build win64.
17:37:00 <sawrubh|ib> I'm using try choose, I wanted to ask which tests to run besides mozmill
17:37:12 <clokep_work> Does this even have mozmill tests?
17:37:17 <clokep_work> "this" being filelink?
17:37:23 <clokep_work> I don't thikn it does, so you should probably NOT run mozmill.
17:37:35 <sawrubh|ib> so which tests to run then?
17:37:41 <clokep_work> Whatever sort of tests it has.
17:37:51 * clokep_work isn't going to look for you.
17:42:35 <sawrubh|ib> seems like FileLink has only mozmill tests (http://dxr.mozilla.org/comm-central/source/mail/test/mozmill/cloudfile)
17:43:21 <clokep_work> Alright, then run just mozmill.
17:44:51 <-- nhnt11 has quit (Ping timeout)
17:46:09 --> nhnt11 has joined #instantbird
17:49:55 --> Mic has joined #instantbird
17:49:56 * ChanServ sets mode +o Mic 
17:51:47 <sawrubh|ib> Interesting, I didn't get a try push url on the terminal (unlike pushing to Fx)
17:51:53 <-- qheaden has quit (Quit: Instantbird 1.6a1pre -- http://www.instantbird.com)
17:52:51 <sawrubh|ib> https://tbpl.mozilla.org/?tree=Thunderbird-Try&rev=20636a880c24
17:53:00 <clokep_work> Thanks.
17:53:30 <clokep_work> Let mek now when it finishes? :)
17:54:30 * sawrubh|ib goes back to dealing with that weird 'muc.incomingMessage is not a function' error
17:57:33 <-- mconley has quit (Connection reset by peer)
17:57:51 --> mconley has joined #instantbird
17:58:02 <clokep_work> :-\
17:58:08 <clokep_work> sawrubh|ib: Are you sure that's something you need to solve?
17:58:15 <clokep_work> I'm not convinced the surrounding code is right.
17:58:22 <clokep_work> And I don't want you to waste time debugging something that's useless.
17:59:12 <-- xangua has quit (Quit: Connection closed for inactivity)
17:59:52 <sawrubh|ib> everything seems to work (the topic's getting set in both the cases, pidgin and normal ones complying with XEP), guess it can be called fixed
18:00:34 <sawrubh|ib> I just get this weird error, which doesn't seem to be affecting the output (or probably is but I'm not able to figure out the impact)
18:01:12 <clokep_work> "getting an error" means "not fixed" ;;)
18:01:31 <sawrubh|ib> hrm, I'll have to leave it for now then
18:01:37 * sawrubh|ib moves to notifbox
18:02:17 <sawrubh|ib> probably aleth might have some idea when he coems back
18:02:21 <sawrubh|ib> he usually does
18:02:50 <sawrubh|ib> he helped me fix those weird bindings issue when I was helpless :)
18:03:16 <clokep_work> sawrubh|ib: I'm pretty sure I already told you how to "solve" that issue, but I don't know if you ever tried it.
18:03:54 <sawrubh|ib> I've tried everythign you've suggested, I've tried early return, I've tried calling setTopic with a setter
18:05:48 <clokep_work> What was wrong w/ early returning?
18:05:53 <clokep_work> That it doesn't set the person?
18:06:00 <clokep_work> I thought we just discussed earlier that that isn't really possible.
18:09:46 --> Tonnes has joined #instantbird
18:12:44 <sawrubh|ib> so this is what's happening (http://pastebin.instantbird.com/732849 is the WIP for reference and http://pastebin.instantbird.com/732850 is the output with it), onMessageStanza is getting called for all message stanzas, when I join the channel muc is coming out to be 'sawrubh' which is my nick for that channel, it should instead be an Object with the methods
18:13:21 <sawrubh|ib> that's why I get that error, after that first time, muc is always an Object and has that method and hence no errors and everything works smoothly
18:14:12 <sawrubh|ib> sorry wrong pastebins
18:14:34 <clokep_work> sawrubh|ib: It's because the XMPP code doesn't crazy things where it saves your username in this._mucs and then creates the actual MUC later on.
18:14:48 <sawrubh|ib> http://pastebin.instantbird.com/732851 is the WIP
18:15:09 <sawrubh|ib> you can see I'm early returning on L66 of this
18:15:24 <clokep_work> This happens at https://mxr.mozilla.org/comm-central/source/chat/protocols/xmpp/xmpp.jsm#850
18:15:46 <clokep_work> I changed this in one of the patches I have up for review right now.
18:15:52 <clokep_work> So the MUC will always exist after you try to join it.
18:16:47 <sawrubh|ib> yeah, that's what happening. 'muc' doesn't exist the first time and hence I get that error
18:17:20 <-- rosonline has quit (Quit: Instantbird 1.5 -- http://www.instantbird.com)
18:18:11 <Mic> mayanktg: the changes for Windows/Aero won't work. That's because the files from the aero folder are going to be used for the theme (see https://mxr.mozilla.org/comm-central/source/im/themes/jar.mn#11 ) and the added lines do not add them to the aero folder. You'll need to adapt the path where the files should be stored accordingly.
18:18:24 <Mic> You can read about the structure of jar manifests here: http://dxr.mozilla.org/mozilla-central/source/build/docs/jar-manifests.rst
18:18:46 <clokep_work> sawrubh|ib: OK, so leave it for now. HOpefully it'll be easier to fix once my patch is committed.
18:18:55 <Mic> You'll also need to add the vector user image for Aero too.
18:19:05 <-- mconley has quit (User has been banned from Mozilla (no spamming here))
18:19:12 <sawrubh|ib> clokep_work: can you give me the bug so that I can add that as dependency?
18:19:17 <-- clokep_work has quit (User has been banned from Mozilla (no spamming here))
18:22:24 <mayanktg> Mic: I'll change the path accordingly. So I need to  convert the two icons into vector images and then put them into the Aero folder. Right?
18:22:58 <Mic> That's not what I wrote ;)
18:24:39 <instantbot> New Chat Core - IRC bug 1022815 filed by dtate@mozilla.com.
18:24:42 <instantbot> Bug https://bugzilla.mozilla.org/show_bug.cgi?id=1022815 cri, --, ---, nobody, NEW, Banned for messaging @mardi
18:26:06 <mayanktg> Mic: Oh.. I mixed up the two sentences. :-/ Ok I'll add the userIcon.svg image for the aero too. 
18:27:48 <sawrubh|ib> I wonder if that bug is what caused clokep_work to get disconnected 'User has been banned from Mozilla (no spamming here)'
18:28:20 <sawrubh|ib> flo-retina: ^^
18:30:00 --> mconley has joined #instantbird
18:30:40 --> sonny has joined #instantbird
18:31:09 <Mic> mconley: you left with a strange message about being banned for spamming: http://log.bezut.info/instantbird/today#m692
18:31:20 <-- Rym has quit (Ping timeout)
18:31:37 --> Rym has joined #instantbird
18:31:38 <sawrubh|ib> Mic: yeah clokep left with that message too and he has emailed me just now saying it was something weird
18:31:51 <sawrubh|ib> and that critical bug file above ^^ is also related to this
18:32:06 <Mic> Yep, I've read it.
18:32:58 <mconley> yeah, something screwy happened
18:33:01 * nhnt11 is back
18:33:02 <mconley> not my fault, I swear. :)
18:33:05 <nhnt11> Weird
18:33:31 <Mic> mconley: at least you weren't banned permanently :)
18:33:43 --> clokep_work has joined #instantbird
18:33:44 * ChanServ sets mode +o clokep_work 
18:33:56 * mconley nods
18:33:57 <clokep_work> Ah, here we go. :)
18:35:25 <clokep_work> 14:19:30 - clokep_work: (sawrubh|ib Sorry if you had told me all that before, but I didn't realize you were getting a string of your username when trying to access MUC.)
18:35:37 <clokep_work> And bug 1018771 was what was asked for.
18:35:40 <instantbot> Bug https://bugzilla.mozilla.org/show_bug.cgi?id=1018771 min, --, ---, clokep, ASSI, Use Maps and Sets in XMPP code
18:37:01 <-- Rym has quit (Ping timeout)
18:42:33 <-- sonny has quit (Ping timeout)
18:50:13 * mayanktg JAR manifests doc was a good read... :)
18:54:22 <mayanktg> Mic: Should work now http://pastebin.instantbird.com/732864 :-o
18:57:47 <-- mayanktg has quit (Ping timeout)
18:58:20 --> Rym has joined #instantbird
18:58:25 <Mic> mayanktg: other OS dependent files don't have their own folders but contain the OS name in their files names and are renamed while being packaged.
18:58:57 <Mic> Check how it's done for "tag-<OS>.png", I think we want the same here.
19:00:10 --> mayanktg has joined #instantbird
19:01:50 <nhnt11> Meh
19:01:56 <Mic> mayanktg: I left a message in the log
19:02:01 <sawrubh|ib> clokep_work: seems like notificationbox can only have a button (to select whether to upload or not) or some label. I was actually planning to have a dropdown to select the FileLink provider to upload to but that doesn't seem possible now
19:02:05 <-- Rym has quit (Connection reset by peer)
19:02:08 <mayanktg> Mic: yes I was reading it.
19:03:42 * nhnt11 's tests tell him we will need to handle asyncshutdown
19:03:45 --> Rym has joined #instantbird
19:04:11 <nhnt11> Er, we need to clear promises before shutdown is what I mean
19:05:00 <nhnt11> If I have pending logMessage promises, and the test exits (do_throw), all of these pending promises throw stuff like "JSON is undefined" and so on
19:05:32 <-- Rym has quit (Ping timeout)
19:05:42 <nhnt11> I just realized that if this happens in a test, it's possible for it to happen in a real scenario too
19:06:04 <nhnt11> flo-retina: What do you think of this? ^
19:06:15 <nhnt11> I wonder if just yielding on every promise in the map would be enough
19:06:29 <nhnt11> But that could take a while, which isn't cool during shutdown
19:06:34 <mayanktg> Mic: |skin/classic/aero/instantbird/select-file.png       (windows/select-file-aero.png)| I should rename the file in the folder since Windows file have win and aero to their file names.
19:07:04 --> Rym has joined #instantbird
19:07:17 <nhnt11> mayanktg: Are the images for aero/non-aero different?
19:07:28 <mayanktg> nhnt11: No.
19:07:41 <nhnt11> Then you don't need to rename afaik.
19:07:52 <nhnt11> (If I correctly understand what you're talking about)
19:08:20 <nhnt11> the -aero suffixes are there for image where the aero version is different from the non-aero one
19:08:20 <mayanktg> The images are same for linux, aero and non-aero. I only have a 32X32 version for retina displays.
19:08:46 <mayanktg> ok
19:09:00 <nhnt11> For example.png that is common to aero and non-aero, |skin/classic/aero/instantbird/example.png  (windows/example.png)| should do
19:09:48 <mayanktg> I understood what you said. 
19:09:58 <nhnt11> cool
19:10:00 <mayanktg> :)
19:11:11 <Mic> nhnt11: I asked mayanktg to remove the directories and store the files named similar to "tag-*.png" in the theme folder itself.
19:11:47 <nhnt11> Oh! Sorry!
19:11:55 <Mic> So it's rather |skin/classic/aero/instantbird/example.png  (example-win.png)| and so on...
19:11:58 <nhnt11> I think I just added to the confusion :(
19:12:24 <nhnt11> mayanktg: Disregard everything I said, I thought you were talking about aero/non-aero :(
19:12:25 <nhnt11> BUT
19:12:35 <nhnt11> If the linux and windows images are the same, what I said still applies possibly :)
19:13:08 <mayanktg> Sorry :( 
19:13:26 <Mic> The Windows icons look decent. I doubt that they are used for Linux too. scnr ;)
19:13:31 <nhnt11> Uh, that was my fault for responding without any context.
19:13:32 <nhnt11> Mic: lol
19:14:19 <mayanktg> The images are same for linux and windows btw. 
19:15:12 <mayanktg> Mic: So should I move use them to in the theme/ only and then use them as you said above ?
19:15:23 * nhnt11 sighs
19:15:48 <nhnt11> It looks like returning the promise from logMessage may be inevitable for tests
19:16:14 <nhnt11> Because getLogsForConversation does not do any file operations (only directory ones) and therefore is not queued
19:16:32 <nhnt11> (i.e. the getLogsForConv call will finish/yield before logMessage ever happens)
19:16:48 <nhnt11> So I either need to return a promise from logMessage and wait on that or get the path from the map and wait on /that/
19:17:00 <nhnt11> Hmm, I should possibly just use the map
19:18:13 <Mic> mayanktg: yes, you can e.g. only special case the Mac file by appending a "-mac" to it's filename and keep the Linux and Windows filename as it is?
19:18:24 <Mic> That's just an idea...
19:20:07 <mayanktg> Mic: I remember from a conversation that aleth once said we should move them to different folders because in the future we might have icons for each OS's and it will clear things. I'm not sure if it was him or someone else.
19:21:04 <mayanktg> That's why I created separate folders for linux, mac and windows. 
19:21:58 <mayanktg> Ok. I'll keep the filenames for linux and win as it it and add suffix -mac to the mac icons.
19:22:13 <mayanktg> *as it is
19:24:54 <Mic> mayanktg: that makes sense but I think it's not in the scope of this bug. Would you file a bug for what you discussed with aleth?
19:25:37 <mayanktg> Mic: I'll do that. I was just searching the logs to be sure..
19:29:35 <mayanktg> Mic: http://pastebin.instantbird.com/732892 
19:30:36 <mayanktg> ignore the rror at line 33.
19:30:54 * nhnt11 scratches his head
19:35:09 <sawrubh|ib> hrm, I need access to gBrowser to get the notificationbox thingy, this.browser doesn't seem to work
19:35:24 * sawrubh|ib looks up for some examples inside IB for notifbox
19:35:44 <nhnt11> sawrubh|ib: Do you mean the one in conversations?
19:37:28 <sawrubh|ib> yes
19:37:49 <sawrubh|ib> should I do some getParent or some such traversal on browser to get it
19:38:01 <sawrubh|ib> getElemByID prolly
19:39:27 <nhnt11> sawrubh|ib: There's a method called getBrowser() (instantbird.js, in the window's scope), may help you (or did you already have the browser)?
19:40:53 <nhnt11> (https://mxr.mozilla.org/comm-central/source/im/content/instantbird.js#66)
19:42:11 <sawrubh|ib> I'll take a look, no I didn't have the browser
19:42:48 <-- Rym has quit (Ping timeout)
19:43:04 --> Rym has joined #instantbird
19:43:10 <nhnt11> sawrubh|ib: Make sure you do a null check on the browser, not all tabs have one
19:43:16 <nhnt11> (only conversations)
19:43:42 <nhnt11> Well, others could have a browser too, but for example the awesometab doesn't.
19:44:02 <nhnt11> sawrubh|ib: To ensure the current tab is a conversation use getTabBrowser().selectedConversation
19:44:21 <nhnt11> Which will be null if the selected tab panel is not a conv
19:48:44 * nhnt11 gets rid of _queuedLogMessage and makes everything a Task
19:53:09 * mayanktg goes to write xml2sdp()
19:54:50 <-- mayanktg has quit (Ping timeout)
19:54:52 --> mayanktg has joined #instantbird
19:56:05 * nhnt11 should stop using do_throw in async tasks
20:00:52 <clokep_work> sawrubh|ib: We don't have a notification box in the conversation.
20:03:24 <sawrubh|ib> clokep_work: http://mxr.mozilla.org/comm-central/source/im/content/conversation.xml#25
20:03:50 <clokep_work> sawrubh|ib: Interesting, do we actually use that anywhere?
20:04:15 <sawrubh|ib> no where I can see
20:05:05 <clokep_work> Did nhnt11 answer your question of how to get that element?
20:05:33 <flo-retina> clokep_work: we used that for the topic bar before 1.0 (or was that before 0.2?), and didn't remove it because we thought it could be useful for stuff like eg. receiving a file
20:06:10 <sawrubh|ib> I'll take a look, I started checking #servo pfff
20:06:24 <sawrubh|ib> btw the builds are complete at https://tbpl.mozilla.org/?tree=Thunderbird-Try&rev=20636a880c24, the tests pass
20:08:14 <-- Rym has quit (Ping timeout)
20:08:32 --> Rym has joined #instantbird
20:09:32 <nhnt11> brb
20:12:15 <clokep_work> sawrubh|ib: Congrats. :)
20:12:57 <sawrubh|ib> :)
20:13:05 <clokep_work> sawrubh|ib: ftp://ftp.mozilla.org/pub/thunderbird/try-builds/saurabhanandiit@gmail.com-20636a880c24/
20:13:20 <clokep_work> I'll play w/ that on Windows tonight unless someone beats me to it.
20:13:29 * sawrubh|ib goes to paste that link in the bug
20:14:11 * sawrubh|ib eyes nhnt11's Windows partition ;)
20:15:25 <sawrubh|ib> arghh, I didn't send my custom mozbuild to try, so it's only built TB :(
20:16:14 <-- flo-retina has quit (Quit: Instantbird 1.6a1pre -- http://www.instantbird.com)
20:16:22 <sawrubh|ib> clokep_work: guess you'll have to build on Windows yourself :/
20:16:24 <-- mpmc has quit (Ping timeout)
20:17:14 <clokep_work> sawrubh|ib: I mean TB. I want to make sure it actually works.
20:17:19 <clokep_work> Tests don't necessarily mean something works.
20:17:25 <clokep_work> They should never be relied upon like that.
20:17:41 <clokep_work> qheaden: ping
20:17:42 --> mpmc has joined #instantbird
20:19:40 <-- Rym has quit (Ping timeout)
20:19:58 --> Rym has joined #instantbird
20:25:12 --> EionRobb has joined #instantbird
20:28:27 * nhnt11 falls more and more in love with Tasks
20:28:50 <EionRobb> would you have Tasks babies?
20:29:11 <nhnt11> EionRobb: ok that's a bit weird/creepy.
20:29:25 <EionRobb> that's totally a "yes"
20:38:10 <sawrubh|ib> nhnt11: how will I import instantbird.js, it's not a module and I tried adding js files in the resources section sometime back and it didn't work
20:38:34 <nhnt11> sawrubh|ib: instantbird.js is included in instantbird.xul (the conversation window)
20:38:52 <sawrubh|ib> I want to do this in conversations.xul
20:38:52 <nhnt11> just call getBrowser or getTabBrowser from the window's scope
20:39:05 <nhnt11> window.getBrowser() or just getBrowser() would probably do
20:39:15 <sawrubh|ib> err conversation.xml
20:39:18 * sawrubh|ib tries
20:39:53 <nhnt11> sawrubh|ib: https://mxr.mozilla.org/comm-central/search?string=getTabBrowser&find=im/
20:40:06 <nhnt11> Looks like you need window.getTabBrowser
20:40:20 <nhnt11> Uh, actually just getTabBrowser is used
20:40:35 <nhnt11> But there's a check |if (window.getTabBrowser)|
20:40:46 <nhnt11> Usually that's |if (getTabBrowser in window)| but whatever
20:40:59 <sawrubh|ib> ok
20:44:50 * nhnt11 needs an arbitrary date and tries to think of something clever
20:45:18 <mayanktg> sawrubh|ib: Maybe you can see how getTabBrowser is implemented here http://mxr.mozilla.org/comm-central/source/im/content/instantbird.xul#74 .
20:45:21 <nhnt11> Maybe the date of Instantbird's 1.0 release?
20:46:13 <mayanktg> sorry if it is out of context.
20:46:33 <Mook_as> nhnt11: 1998-03-27, free the lizard?
20:48:23 <nhnt11> Mook_as: What happened on that day? :S
20:48:29 <nhnt11> I couldn't find anything on Google/wikipedia
20:49:04 <nhnt11> I did find http://www.freethelizard.org/
20:49:08 <nhnt11> which is of no help :P
20:49:08 <Mook_as> nhnt11: it should be the first github result for "free the lizard" - Mozilla code was open sourced from Netscape
20:49:26 <nhnt11> Ah! I expected something like that, thanks
20:49:34 <Mook_as> (github, because they imported cvs history at some point)
20:51:16 <nhnt11> Mook_as: The book of mozilla suggests that the date is actually 1998-03-31
20:51:36 <nhnt11> https://en.wikipedia.org/wiki/The_Book_of_Mozilla#The_Book_of_Mozilla.2C_3:31
20:52:03 <Mook_as> Could be; maybe they did a commit first, and then actually opened the repo up later?
20:52:10 * nhnt11 shrugs
20:52:27 * Mook_as can't care enough to shrug
20:52:55 * nhnt11 can't care enough to say that he can't care enough to shrug.. oh wait.
20:53:02 <nhnt11> :P
20:54:13 <-- Rym has quit (Ping timeout)
20:55:15 <clokep_work> sawrubh|ib: You figure it out?
20:56:48 * nhnt11 decides to use the instantbird 1.0 release date
20:57:19 <sawrubh|ib> getNotificationBox doesn't seem to be in window.getTabBrowser(), however window.getTabBrowser exists
20:57:41 <nhnt11> getNotificationBox? Are you sure that's /supposed/ to exist?
20:58:24 <Mic> nhnt11: psst...
20:58:29 <Mic> new Date(Math.random()*1000000000000)
20:58:33 --> arlolra_ has joined #instantbird
20:58:36 <-- arlolra has quit (Connection reset by peer)
20:58:36 <-- arlolra_ has quit (Quit: arlolra_)
20:58:40 <sawrubh|ib> https://developer.mozilla.org/en-US/docs/Mozilla/Tech/XUL/notificationbox#Finding_the_current_notification_box
20:58:54 <nhnt11> Yeah that's not for Instantbird :P
20:58:59 * nhnt11 doesn't think Ib has such a method
20:59:06 <nhnt11> Mic: I need something that /isn't/ random :(
20:59:13 <nhnt11> Otherwise I'd have used new Date()
20:59:23 <Mic> Put it into a lazy getter.
20:59:27 <sawrubh|ib> or wait I can do it like this http://dxr.mozilla.org/comm-central/source/im/content/newtab.xml#330
20:59:29 <nhnt11> Mic: I need to ensure that I have a date that is not the last millisecond of a day or something
20:59:32 <nhnt11> For daily logs
20:59:46 <nhnt11> sawrubh|ib: Look at the line /above/ the line you linked
21:00:04 <clokep_work> sawrubh|ib: Did you dxr for getNotificationBox? That doesn't exist most likely. ;)
21:01:11 <nhnt11> sawrubh|ib: I think you want something like this: https://pastebin.mozilla.org/5381711
21:01:28 <sawrubh|ib> clokep_work: heh, I didn't
21:01:55 <nhnt11> Mic: It's the property of a dummy conv object I'm using in my tests :)
21:02:05 <nhnt11> (Yes it's a lazy getter)
21:08:22 <sawrubh|ib> nhnt11, clokep_work: it works
21:08:40 <nhnt11> cool :)
21:08:53 * nhnt11 just typed that message^ in his code editor -_-
21:09:10 * sawrubh|ib quickly finishes this and gets on with life
21:09:17 <sawrubh|ib> I mean the next thing
21:09:47 <sawrubh|ib> ..which will probably come from https://etherpad.mozilla.org/ib-filelink
21:10:30 <clokep_work> It will.
21:10:41 <clokep_work> I think there's some good stuff on there. :)
21:10:45 <clokep_work> The juicy parts of the project.
21:10:50 <sawrubh|ib> XMPP File Transfer seems to be the way forward!
21:10:58 <clokep_work> Unfortunately a lot of this has been not so exciting sof ar. :(
21:11:18 <sawrubh|ib> From my side? or the moving part?
21:11:26 <clokep_work> The movnig and fixing of stuff.
21:13:44 <sawrubh|ib> hrm, I kinda wonder if I should do point 3 over point 2 (on that etherpad)
21:14:22 <clokep_work> sawrubh|ib: 2 is kind of required for 3, although you could get around it by initially implementing it via a command or something.
21:14:53 <clokep_work> So you might need to do 2 + 3 at once.
21:14:59 <sawrubh|ib> so we have two big parts now, (one supposedly big part of FileLink is done I suppose) one implementing XMPP support and then a purplexpcom interface using libpurple
21:15:14 <sawrubh|ib> (and obviously the UI to accept and reject and stuff)
21:15:25 * clokep_work counts like 5 big parts in there. ;)
21:15:34 <sawrubh|ib> (and the Downloads Panel)
21:15:57 <clokep_work> So 1 is implementing the XMPP support + the interface, another is implementing that interface for purplexpcom.
21:16:10 <clokep_work> A third is all the UI stuff + figuring out the "automatic" switching from file transfer to file link.
21:16:15 <clokep_work> That mostly sums it up?
21:16:45 <sawrubh|ib> yeah, if UI is Downloads Panel + UI for deciding which method to use
21:16:45 * clokep_work is leaving in a minute.
21:17:05 <sawrubh|ib> yay, you'll reach home and have more time :)
21:17:05 <clokep_work> + adding a button to upload things.
21:17:08 <clokep_work> And probably other stuff.
21:17:14 <clokep_work> I'm going for a run first, most likely.
21:17:22 <sawrubh|ib> wow, miles to go before I sleep
21:17:35 <clokep_work> Hopefully be back before 7.
21:17:45 <sawrubh|ib> which is 4am here...
21:18:10 <sawrubh|ib> I should be awake probably...
21:18:28 * sawrubh|ib finds out the readings for 'implementing the XMPP support + the interface'
21:18:41 <sawrubh|ib> I guess I'll start with those previous bugs?
21:18:50 <sawrubh|ib> also need to file new ones
21:19:22 <-- clokep_work has quit (Ping timeout)
21:33:25 --> sonny has joined #instantbird
21:36:26 <sawrubh|ib> Mic: there?
21:37:41 <sawrubh|ib> although I've checked myself but wanted to make sure that it's not possible to add dropdowns to notificationboxes right?
21:39:09 --> jb has joined #instantbird
21:43:24 --> Rym has joined #instantbird
21:43:52 <mayanktg> Mic: Should I send the user icon bug patch for review? I have made the changes to the jar.mn, and flo said he would fix the Mac issues.
21:44:40 --> flo-retina has joined #instantbird
21:44:40 * ChanServ sets mode +qo flo-retina flo-retina 
21:45:52 <Mic> mayanktg: if you've got a newer version of the patch, then yes, please :)
21:46:47 <Mic> sawrubh|ib: you can create notification bars with any content if you like.
21:47:13 <mayanktg> Ok. I'm updating it then. The pastebin I shared for the jar.mn diff was correct http://pastebin.instantbird.com/732892 ?
21:47:16 <Mic> You'd have to write an own binding that extends normal notification bars for that.
21:47:48 <Mic> We used to have one that showed the topic on top of the conversation and allowed editing it too. So it's definitely possible.
21:48:07 <Mic> If that's what you want to do...
21:48:21 <sawrubh|ib> Mic: got any examples? btw I'm using http://mxr.mozilla.org/comm-central/source/im/content/conversation.xml#25 right now to show my notifs
21:48:46 <sawrubh|ib> hrm, sounds like a bit of work
21:48:53 <-- jb has quit (Ping timeout)
21:48:58 <sawrubh|ib> not worth the effort for a UI that's in flux
21:50:25 <-- sonny has quit (Quit: Instantbird 1.6a1pre -- http://www.instantbird.com)
21:52:38 <-- sawrubh|ib has quit (Connection reset by peer)
21:53:12 <Mic> sawrubh|ib, in case that you're curious: https://hg.mozilla.org/comm-central/rev/8541b2932cfb
21:54:00 <flo-retina> Mic: r=morian !
21:54:32 * flo-retina didn't remember that
21:55:00 --> sawrubh|ib has joined #instantbird
22:14:40 <sawrubh|ib> clokep I'm very sleepy, will carry on when I get back
22:14:43 * sawrubh|ib -> sleep
22:14:56 <-- sawrubh|ib has quit (Quit: sawrubh|ib)
22:24:11 <-- Rym has quit (Ping timeout)
22:25:18 --> Rym has joined #instantbird
22:29:48 <-- mayanktg has quit (Quit: Instantbird 1.6a1pre -- http://www.instantbird.com)
22:29:54 <-- Rym has quit (Ping timeout)
22:31:02 --> Rym has joined #instantbird
22:31:15 <nhnt11> flo-retina: By the way, after I finish these tests (which is soon but I think you might go to bed by then) I thought I'd either experiment with the convbrowser more, or start experimenting with a database. Any thoughts?
22:31:33 <nhnt11> (I'm a bit keen on trying out the database stuff at the moment)
22:31:57 <flo-retina> can you be more specific about what "trying out the database stuff" means?
22:32:00 <nhnt11> I remember we discussed that it's a good idea to test out the performance of a database if we store all messages inside it etc
22:32:32 <nhnt11> I wanted to check out how big it might get, and maybe run some benchmarks on storage/retreival with full text search enabled
22:33:04 <-- Rym has quit (Ping timeout)
22:33:46 --> Rym has joined #instantbird
22:42:19 --> clokep has joined #instantbird
22:42:19 * ChanServ sets mode +o clokep 
22:48:52 <-- Mic has quit (Quit: Instantbird -- http://www.instantbird.com)
22:49:00 <-- mconley has quit (Input/output error)
22:49:06 <clokep> Mic: Nice. :) That was before my time. ;)
22:49:29 --> mconley has joined #instantbird
22:51:15 <-- mconley has quit (Ping timeout)
22:54:11 <-- Rym has quit (Ping timeout)
22:54:21 --> Rym has joined #instantbird
22:54:48 * clokep tries sawrubh's build.
22:55:05 <flo-retina> clokep: you really never saw that topic bar?
22:55:31 <clokep> flo-retina: I just meant that commit.
22:55:37 <clokep> I was around before we had the topic header.
22:57:36 * nhnt11 tries to figure out why getNewLogFileName divides the input date by 1000
22:58:18 <nhnt11> Turns out my problems were being caused by that - any date I gave divided by 1000 was before epoch, so it returned the same name for all of my files
22:58:46 <nhnt11> I thought I'd figured this out already but apparently I haven't.
22:59:56 <flo-retina> nhnt11: timestamps in ms vs timestamps in seconds?
23:00:09 <clokep> How do you get a date before the epoch if it's a positive number? :P
23:01:17 <nhnt11> clokep: That's a good point, my reasoning for all the files being named 1970-01-16.090959+0530.json is bad then :S
23:01:48 <nhnt11> Ah, 090959...
23:01:53 * nhnt11 feels stupid
23:02:14 <nhnt11> Now I'm more confused why they all got the same name.
23:02:35 <nhnt11> 090959 = 09:09:59 right? Hours:minutes:seconds
23:03:06 <nhnt11> flo-retina: What I'm trying to understand is why it divides the input date /inside/ the date constructor
23:03:12 <nhnt11> new Date(aDate / 1000)
23:03:33 <clokep> Yes
23:03:35 <nhnt11> That means the aDate input is pre-multiplied somewhere? :S
23:03:48 <clokep> nhnt11: aDate is milliseconds
23:04:26 <nhnt11> clokep: The date constructor accepts a value in milliseconds
23:04:32 <flo-retina> isn't aDate a PRTime in micro seconds, and new Date taking a value in ms?
23:04:43 <clokep> Yes, that's it, see https://developer.mozilla.org/en-US/docs/Mozilla/Tech/XPCOM/Reference/Interface/nsIFile
23:04:47 --> wnayes has joined #instantbird
23:04:53 <clokep> lastModifiedTime
23:05:20 <clokep> oh wait, that's milliseconds too.
23:05:24 <clokep> Somewhere we use PRTime, yes. :)
23:05:29 <nhnt11> flo-retina: Wow, thanks, just saw imILogger.idl
23:05:52 * clokep remembers figuring this out when dealing w/ wnayes' patch.
23:05:55 <nhnt11> Log conversations have a PRTime startDate attribute
23:06:44 <nhnt11> Interesting, so a conversation's startDate is in microseconds
23:07:43 <-- Rym has quit (Ping timeout)
23:07:46 <nhnt11> flo-retina: Do we use PRTime to accommodate libpurple? Because JS dates are accurate to milliseconds only
23:08:04 <nhnt11> JS Date objects anyway
23:08:11 <flo-retina> nhnt11: I think that was because it's what is easy to use in xpconnect
23:08:23 <nhnt11> Hmm, unless Date objects accept fractional values
23:08:27 <nhnt11> flo-retina: Oh okay, thanks.
23:08:27 <flo-retina> I would be surprised if libpurple had more details than second
23:09:18 <clokep> nhnt11: JS doesn't even support 64-bit ints. :(
23:09:30 * nhnt11 didn't know that
23:11:22 <EionRobb> it goes up to 48-bit right?
23:11:22 <clokep> Numbers in JS are doubles IIRC.
23:11:32 <clokep> So you can only get to like yeah, 48 or 54 or something bits of precision.
23:14:36 <EionRobb> I didn't realise that bitshifting operations only worked on ints up to 32bit
23:15:15 <nhnt11> Bah, now I'm getting NaN's ...
23:15:20 <clokep> :-D
23:15:23 <clokep> nhnt11: Show some code.
23:15:27 --> Rym has joined #instantbird
23:15:43 <nhnt11> One second
23:17:19 <nhnt11> clokep: I'm about to paste, but some background: I have a dummyConv with a startDate getter that returns a date in microseconds. I pass this value while writing the logs. Reading also works fine, but the date of the conversation read from the log file is "NaN"
23:18:36 <nhnt11> https://pastebin.mozilla.org/5382194
23:20:37 <clokep> nhnt11: Where are you getting NaNs?
23:20:43 <nhnt11> I realize that the start date of the logConversation that is read from the file may not be exactly equal to the one in the dummyConv due to conversion as well as since the startDate of the logConversation is obtained by the first message in the file
23:20:54 --> mconley has joined #instantbird
23:20:57 <nhnt11> clokep: Line 62
23:21:42 <clokep> Hmm....what's the result of line 13?
23:21:54 <nhnt11> clokep: A time in millis
23:22:24 <clokep> I'm not sure. :-\
23:22:30 <clokep> Print everything otu? :P
23:22:55 <nhnt11> I just did
23:23:05 <nhnt11> It's identical to new Date(2011, 5, 28).valueOf()
23:23:07 <nhnt11> 1309199400000 vs 1309199400000
23:23:26 <nhnt11> Hmm
23:23:34 <nhnt11> It may be that MESSAGES aren't using PRTime
23:23:45 <nhnt11> And line 62 is checking properties for the messages..
23:24:09 <clokep> nhnt11: Which is the NaN?
23:24:11 <nhnt11> gah, let me check some stuff
23:24:14 <clokep> OK
23:24:37 <nhnt11> clokep: TEST-UNEXPECTED-FAIL | [...]/test_logger.js | 1309199401000 == "NaN" - See following stack: [...]
23:24:40 <nhnt11> So the first message
23:25:05 <clokep> OK, and yeah it's after you retrieve it.
23:25:17 <nhnt11> Yes
23:25:18 <nhnt11> Ah!
23:25:30 <nhnt11> logMessage does new Date(aMessage.time * 1000)
23:25:35 <nhnt11> Which makes me think aMessage.time is in seconds
23:25:48 <nhnt11> (Which coincides nicely with what flo said about libpurple ;))
23:25:49 * nhnt11 checks
23:26:41 <nhnt11> No, prplIMessage stores it as PRTime
23:26:55 <-- mconley has quit (Input/output error)
23:27:05 <clokep> nhnt11: Are you reading it wrong or something?
23:28:18 <nhnt11> No. The other properties seem to be correct
23:28:23 <nhnt11> I'll double check
23:30:48 <clokep> :-\ I'm not sure Im' helpful, sorry.
23:33:05 <nhnt11> Looks like according to this: https://mxr.mozilla.org/comm-central/source/chat/modules/jsProtoHelper.jsm#371
23:33:06 <nhnt11> time is in seconds and not millis
23:33:36 <nhnt11> flo-retina: Can you confirm this^
23:34:34 <flo-retina> prplIMessage.time seems in seconds, yes
23:35:09 <flo-retina> hmm, it's a PRTime :-S http://lxr.instantbird.org/instantbird/source/chat/components/public/prplIMessage.idl#32
23:35:11 * flo-retina is confused
23:35:14 <nhnt11> flo-retina: Exactly
23:35:23 <flo-retina> it's possible that's just mixed up
23:35:25 <nhnt11> And there's no getter for the time in the GenericMessagePrototype
23:38:00 <nhnt11> flo-retina: Should I be investigating/fixing this possible mixup or can I proceed with how it's all working currently?
23:38:44 <flo-retina> either. But I would do the latter if I were you ;)
23:39:21 <nhnt11> flo-retina: It doesn't seem to be a mix-up judging by the fact that logMessage multiplies it by 1000 before logging it
23:39:42 <nhnt11> And yeah I'm going to do the latter, I've been blocked on this for a while :]
23:40:02 <flo-retina> that .idl file should really have a comment about the unit thought
23:40:10 <nhnt11> Sure, I can add that.
23:40:29 <nhnt11> Though I have no idea of a reason for it
23:42:34 <clokep> Bah I have quesetions for sawrubh. :-S
23:44:17 <nhnt11> pfft, timezones
23:44:48 * clokep still doesn't understand these selected changes.
23:45:26 <nhnt11> clokep: selectedConversation stuff? I can answer questions related to that
23:45:36 * clokep has no idea what nhnt11 is talking about.
23:45:43 <nhnt11> When you said "selected changes"...
23:45:44 <nhnt11> nvm
23:45:57 <clokep> nhnt11: The changes to the handlers for when an account is selected in filelink.
23:46:05 <nhnt11> oh okay. not related, nvm
23:46:06 <nhnt11> sorry
23:46:56 <clokep> No problem.
23:47:56 <nhnt11> Cool, changing message time to seconds fixed everything :)
23:56:51 <-- wnayes has quit (Quit: wnayes)