WEBVTT: Initial Support Incoming
I have some great news. Bug 833382 and 833386, the last pieces needed to get initial support for WEBVTT into Gecko will be landing for sure next week. That being said, I hope I don't have to eat my words. It's looking really good though. Bug 833382 has gotten to the point where Boris (:bz) has conditionally r+'ed it and 833386 has reached the same point with Ralph (:rillian). Now all that's left to do for 833382 is to spin up some try builds and if it's all green, hopefully we'll be good to go. 833386 still needs to go through review with Chris Pearce (:cpearce), but I don't think that will take that long.
I've been pushing really hard on landing these two this last week and I'm ecstatic to see that we've gotten to the point where we will be landing them in the next few days. The WebVTTLoadListener in particular I'm very happy about as most of that code is mine and I've worked really hard at it. It'll feel good to land that. In the case of 833386 most of that code is Jordan Raffoul's (:jbraffoul) and Marcus Nsaad's (:msaad) and the work I did on it was just to consolidate it and get it through the last couple rounds of review. Marcus had been on vacation for a while and we really wanted to get this landed ASAP as it is blocking quite a few things so I asked Marcus if I could step in and he didn't have a problem with it (Yay Marcus!).
One of the biggest problems I was having with the tests for the
I finally figured out, when asking the right people (bz and Ms2ger, who woulda' thunk?), that it's actually impossible to test with two different prefs in one page as there is one prototype for each element on the page and the pref is only applied to it once, when the element is created for the first time. So once you've created an element underneath one pref on a page, that's it, it will behave like it is preffed that way no matter what you do. After that it was pretty quick to get through the rest of the code needed. One of the really good things about this process too is that it allowed me to find a lot of points where our current implementation is not to spec. I've filed a few bugs on those this week. I've also closed a few bugs that have been fixed with changes in the last while. I also spun off another bug for tests that we will need to implement when the WEBVTT pref finally gets removed.
Try Server Access
The other really awesome news is that I've finally got try server access! Earlier this week Daniel Holbert (:dholbert) suggessted that I should apply for it and that he would vouch for me. I probably should have applied for it sooner then this as I could have used it for sure. The process was fairly easy and I'm glad to say I now have Level 1 commit access. Woot! If your interested in applying as well check out this page that describes what you will need to do.
In accordance with this new awesomeness I also had to learn about the process of pushing to the try server. Check out this good page for more information on how to do that. You'll also probably want to look at Mercurial Manage Queue extension as it helps with managing a bunch of patches that you can move in and out of your branches easily. This really works well with my workflow of working on git and then just applying patches on my hg repository and pushing to the try server.
Until next time.- Tagged in mozilla, open-source, cdot, and seneca college.
comments powered by Disqus