Difference between revisions of "OSD700-DPS911-Winter2012"
(Added bug list from cpearce and jdm) |
(→OSD700 and DPS911 Class Information (Winter 2012)) |
||
(101 intermediate revisions by 10 users not shown) | |||
Line 13: | Line 13: | ||
</tr> | </tr> | ||
<tr> | <tr> | ||
− | <td>Christopher De Cairos</td> | + | <td>Christopher De Cairos / [[user:Cadecairos|cadecairos]]</td> |
<td>cadecairos</td> | <td>cadecairos</td> | ||
<td>[https://twitter.com/#!/ChrisDeCairos @ChrisDeCairos]</td> | <td>[https://twitter.com/#!/ChrisDeCairos @ChrisDeCairos]</td> | ||
− | <td>[ | + | <td>[https://www.chrisdecairos.ca My Blog]</td> |
− | <td>[https://bugzilla.mozilla.org/show_bug.cgi?id=686137 686137]</td> | + | <td> |
+ | <s>[https://bugzilla.mozilla.org/show_bug.cgi?id=686137 Bug 686137]</s> Resolved :D<br /> | ||
+ | <s>[https://bugzilla.mozilla.org/show_bug.cgi?id=715323 Bug 715323]</s> Resolved<br /> | ||
+ | <s>[https://bugzilla.mozilla.org/show_bug.cgi?id=711839 Bug 711839]</s> This was fixed by somebody else<br /> | ||
+ | [https://bugzilla.mozilla.org/show_bug.cgi?id=711742 Bug 711742]<br /> | ||
+ | [https://bugzilla.mozilla.org/show_bug.cgi?id=601032 Bug 601032]<br /> | ||
+ | [https://bugzilla.mozilla.org/show_bug.cgi?id=517363 Bug 517363]<br /> | ||
+ | [https://bugzilla.mozilla.org/show_bug.cgi?id=618737 Bug 618737]<br /> | ||
+ | [https://bugzilla.mozilla.org/show_bug.cgi?id=726904 Bug 726904]<- hacking on this <br /> | ||
+ | [https://bugzilla.mozilla.org/show_bug.cgi?id=736400 Bug 736400]<- hacking on this<br /> | ||
+ | [https://bugzilla.mozilla.org/show_bug.cgi?id=665598 Bug 665598]<- hacking on this<br /> | ||
+ | [https://bugzilla.mozilla.org/show_bug.cgi?id=665395 Bug 665395]<- hacking on this | ||
+ | </td> | ||
</tr> | </tr> | ||
<tr> | <tr> | ||
− | <td>David Seifried</td> | + | <td>[http://zenit.senecac.on.ca/wiki/index.php/OSD700-DPS911-Winter2012/dseif David Seifried]</td> |
<td>dseif</td> | <td>dseif</td> | ||
<td>[https://twitter.com/#!/dcseifried @dcseifried]</td> | <td>[https://twitter.com/#!/dcseifried @dcseifried]</td> | ||
<td>[http://www.dseifried.wordpress.com Blog]</td> | <td>[http://www.dseifried.wordpress.com Blog]</td> | ||
− | <td>[https://bugzilla.mozilla.org/show_bug.cgi?id=686370 686370]</td> | + | <td> |
+ | [https://bugzilla.mozilla.org/show_bug.cgi?id=686370 Bug 686370]<br/> | ||
+ | <s>[https://bugzilla.mozilla.org/show_bug.cgi?id=702161 Bug 702161]<br/></s> | ||
+ | [https://bugzilla.mozilla.org/show_bug.cgi?id=708814 Bug 708814]<br/> | ||
+ | <s>[https://bugzilla.mozilla.org/show_bug.cgi?id=680321 Bug 680321]</s><br/> | ||
+ | </td> | ||
</tr> | </tr> | ||
<tr> | <tr> | ||
Line 31: | Line 48: | ||
<td>[https://twitter.com/#!/ScottDowne @ScottDowne]</td> | <td>[https://twitter.com/#!/ScottDowne @ScottDowne]</td> | ||
<td>[http://scottdowne.wordpress.com My Blog]</td> | <td>[http://scottdowne.wordpress.com My Blog]</td> | ||
− | <td>[https://bugzilla.mozilla.org/show_bug.cgi?id=677121 677121] | + | <td> |
+ | [https://bugzilla.mozilla.org/show_bug.cgi?id=677121 Bug 677121]<br/> | ||
+ | [https://bugzilla.mozilla.org/show_bug.cgi?id=677122 Bug 677122]<br/> | ||
+ | </td> | ||
</tr> | </tr> | ||
<tr> | <tr> | ||
<td>Matthew Schranz</td> | <td>Matthew Schranz</td> | ||
− | <td> | + | <td>mjschranz</td> |
<td>[http://twitter.com/mjschranz @mjschranz]</td> | <td>[http://twitter.com/mjschranz @mjschranz]</td> | ||
<td>[http://mschranz.wordpress.com Blog]</td> | <td>[http://mschranz.wordpress.com Blog]</td> | ||
<td> | <td> | ||
− | [https://bugzilla.mozilla.org/show_bug.cgi?id=698303 Bug 698303]<br/> | + | <s>[https://bugzilla.mozilla.org/show_bug.cgi?id=698303 Bug 698303]<br/></s> |
− | [https://bugzilla.mozilla.org/show_bug.cgi?id=698381 Bug 698381]<br/> | + | <s>[https://bugzilla.mozilla.org/show_bug.cgi?id=698381 Bug 698381]<br/></s> |
− | [https://bugzilla.mozilla.org/show_bug.cgi?id=698384 Bug 698384]<br/> | + | <s>[https://bugzilla.mozilla.org/show_bug.cgi?id=698384 Bug 698384]</s><br/> |
− | [https://bugzilla.mozilla.org/show_bug.cgi?id=698385 Bug 698385]<br/> | + | <s>[https://bugzilla.mozilla.org/show_bug.cgi?id=698385 Bug 698385]</s><br/> |
+ | <s>[https://bugzilla.mozilla.org/show_bug.cgi?id=718274 Bug 718274]</s><br/> | ||
+ | [https://bugzilla.mozilla.org/show_bug.cgi?id=720768 Bug 720768]<br/> | ||
+ | <s>[https://bugzilla.mozilla.org/show_bug.cgi?id=725289 Bug 725289]</s><br/> | ||
+ | [https://bugzilla.mozilla.org/show_bug.cgi?id=686370 Bug 686370]<br/> | ||
+ | <s>[https://bugzilla.mozilla.org/show_bug.cgi?id=686913 Bug 686913]<br/></s> | ||
+ | [https://bugzilla.mozilla.org/show_bug.cgi?id=723020 Bug 723020] | ||
</td> | </td> | ||
</tr> | </tr> | ||
Line 51: | Line 77: | ||
<td>[http://steventseng.wordpress.com Blog]</td> | <td>[http://steventseng.wordpress.com Blog]</td> | ||
<td> | <td> | ||
− | [https://bugzilla.mozilla.org/show_bug.cgi?id=716107 Bug 716107] ( | + | <s>[https://bugzilla.mozilla.org/show_bug.cgi?id=716107 Bug 716107]</s> (too complicated as of now; read bug comments)<br/> |
+ | <s>[https://bugzilla.mozilla.org/show_bug.cgi?id=601535 Bug 601535]</s> (Resolved)<br/> | ||
+ | <s>[https://bugzilla.mozilla.org/show_bug.cgi?id=601535 Bug 620159]</s> (Resolved in Bug 601535)<br/> | ||
[https://bugzilla.mozilla.org/show_bug.cgi?id=633602 Bug 633602] (the test suite) | [https://bugzilla.mozilla.org/show_bug.cgi?id=633602 Bug 633602] (the test suite) | ||
</td> | </td> | ||
Line 60: | Line 88: | ||
<td> [http://twitter.com/mbuttu @mbuttu] </td> | <td> [http://twitter.com/mbuttu @mbuttu] </td> | ||
<td>[http://mbuttu.wordpress.com Blog]</td> | <td>[http://mbuttu.wordpress.com Blog]</td> | ||
− | <td></td> | + | <td> |
+ | |||
+ | Popcorn Maker<br/> | ||
+ | <s>[https://webmademovies.lighthouseapp.com/projects/80723/tickets/281-basic-template-has-2-descriptions-in-manifest Bug 281]</s><br/> | ||
+ | |||
+ | |||
+ | Butter<br/> | ||
+ | [https://webmademovies.lighthouseapp.com/projects/65733/tickets/253-layer-numbering-issue-on-import Bug 253]<br/> | ||
+ | [https://webmademovies.lighthouseapp.com/projects/65733/tickets/252-dont-let-a-user-commit-a-blank-field-for-timeline-media-change Bug 252]<br/> | ||
+ | <s>[https://webmademovies.lighthouseapp.com/projects/65733/tickets/256-timecodes-should-round-to-1-decimal Bug 256]</s><br/> | ||
+ | <s>[https://webmademovies.lighthouseapp.com/projects/65733/tickets/257-add-ds_store-to-gitignore Bug 257]</s><br/> | ||
+ | <s>[https://webmademovies.lighthouseapp.com/projects/65733-butter/tickets/390-setting-trackevent-out-time-video-duration-causes-issues Bug 390]</s><br/> | ||
+ | <s>[https://webmademovies.lighthouseapp.com/projects/65733-butter/tickets/410-change-all-occurrences-of-const-to-var Bug 410]</s><br/> | ||
+ | [https://webmademovies.lighthouseapp.com/projects/65733-butter/tickets/434-let-users-move-track-events-together Bug 434]<br/> | ||
+ | [https://webmademovies.lighthouseapp.com/projects/65733/tickets/458-protect-from-invalid-times-in-trackevents Bug 458]<br/> | ||
+ | [https://webmademovies.lighthouseapp.com/projects/65733-butter/tickets/459-fix-the-error-message-given-for-invalid-start-and-end-times-for-track-events Bug 459]<br/> | ||
+ | [https://webmademovies.lighthouseapp.com/projects/65733-butter/tickets/463-implement-undoredo-backend#ticket-463-3 Bug 463]<br/> | ||
+ | [https://webmademovies.lighthouseapp.com/projects/65733-butter/tickets/493-if-a-track-event-is-double-clicked-and-the-editor-is-already-opened-focus-the-editor Bug 493]<br/> | ||
+ | |||
+ | Popcorn Maker FCP<br/> | ||
+ | * Create a DMG file that will allow users to install Popcorn Maker FCP once downloaded to the Applications folder by dragging and dropping<br/> | ||
+ | |||
+ | |||
+ | Test Swarm | ||
+ | * <s>Make it work on iOS</s> | ||
+ | |||
+ | </td> | ||
</tr> | </tr> | ||
<tr> | <tr> | ||
Line 69: | Line 123: | ||
<td>[https://bugzilla.mozilla.org/show_bug.cgi?id=633602 Bug 633602 - Mouselock] the tests in particular</td> | <td>[https://bugzilla.mozilla.org/show_bug.cgi?id=633602 Bug 633602 - Mouselock] the tests in particular</td> | ||
</tr> | </tr> | ||
+ | <tr> | ||
+ | <td>Anurag Bhatnagar</td> | ||
+ | <td>abhatnagar</td> | ||
+ | <td> [http://twitter.com/anuragbh @anuragbh] </td> | ||
+ | <td>[http://anuragbhatnagar.com Blog]</td> | ||
+ | <td> | ||
+ | [https://processing-js.lighthouseapp.com/projects/41284/tickets/1392-filterblur-shifts-contents-of-blurred-image Bug 1392 - Processing.js] <br/> | ||
+ | [https://bugzilla.mozilla.org/show_bug.cgi?id=689374 Bug 689374 - Mozilla Firefox]<br/> | ||
+ | |||
+ | |||
+ | </td> | ||
+ | </tr> | ||
+ | <tr> | ||
+ | <td>Diogo Golovanevsky Monteiro</td> | ||
+ | <td>diogogmt</td> | ||
+ | <td> [https://twitter.com/#!/diogogmt @diogogmt] </td> | ||
+ | <td>[http://diogogmt.wordpress.com/ Blog]</td> | ||
+ | <td> | ||
+ | |||
+ | [https://bugzilla.mozilla.org/show_bug.cgi?id=581863 Bug 581863]<br /> | ||
+ | [https://bugzilla.mozilla.org/show_bug.cgi?id=581866 Bug 581866]<br /> | ||
+ | [https://bugzilla.mozilla.org/show_bug.cgi?id=633602 Bug 633602]<br/> | ||
+ | [https://bugzilla.mozilla.org/show_bug.cgi?id=712225 Bug 712225]<br /> | ||
+ | <s>[https://bugzilla.mozilla.org/show_bug.cgi?id=715141 Bug 715141]</s><br/> | ||
+ | <s>[https://bugzilla.mozilla.org/show_bug.cgi?id=718133 Bug 718133]<br/></s> | ||
+ | <s>[https://bugzilla.mozilla.org/show_bug.cgi?id=713383 Bug 713383]<br /></s> | ||
+ | [https://bugzilla.mozilla.org/show_bug.cgi?id=713608 Bug 713608]<br /> | ||
+ | <s>[https://bugzilla.mozilla.org/show_bug.cgi?id=714071 Bug 714071]<br /></s> | ||
+ | <s>[https://bugzilla.mozilla.org/show_bug.cgi?id=728893 Bug 728893]<br /></s> | ||
+ | [https://bugzilla.mozilla.org/show_bug.cgi?id=724108 Bug 724108]<br /> | ||
+ | <s>[https://bugzilla.mozilla.org/show_bug.cgi?id=735031 Bug 735031]<br /></s> | ||
+ | </td> | ||
+ | </tr> | ||
+ | <tr> | ||
+ | <td>Abhishek Bhatnagar</td> | ||
+ | <td>abhatnagar1</td> | ||
+ | <td> [https://twitter.com/#!/abhishekToronto @abhishekToronto] </td> | ||
+ | <td>[https://brattahlid.wordpress.com/ Blog]</td> | ||
+ | <td> | ||
+ | <s>[https://bugzilla.mozilla.org/show_bug.cgi?id=705234 Bug 705234]</s><br/> | ||
+ | <s>[https://bugzilla.mozilla.org/show_bug.cgi?id=620164 Bug 620164]</s><br/> | ||
+ | <s>[https://bugzilla.mozilla.org/show_bug.cgi?id=500784 Bug 500784]</s><br/> | ||
+ | </td> | ||
+ | </tr> | ||
</table> | </table> | ||
+ | |||
+ | |||
+ | <br/> | ||
==Possible Video/Fullscreen Bugs== | ==Possible Video/Fullscreen Bugs== | ||
− | |||
===Small and easy bugs=== | ===Small and easy bugs=== | ||
− | * [https://bugzilla.mozilla.org/show_bug.cgi?id=715323 Bug 715323] nsBuiltinDecoderReader::NotifyDataArrived() should have a 64bit offset parameter. Easy first bug to get acquainted with our process. | + | * <s>[https://bugzilla.mozilla.org/show_bug.cgi?id=715323 Bug 715323] nsBuiltinDecoderReader::NotifyDataArrived() should have a 64bit offset parameter. Easy first bug to get acquainted with our process.</s> |
− | * [https://bugzilla.mozilla.org/show_bug.cgi?id=620159 Bug 620159] nsVorbisState::Time needs to handle failure from return from MulOverflow. (This may conflict with [https://bugzilla.mozilla.org/show_bug.cgi?id=601535 Bug 601535]) | + | * <s>[https://bugzilla.mozilla.org/show_bug.cgi?id=620159 Bug 620159] nsVorbisState::Time needs to handle failure from return from MulOverflow. (This may conflict with [https://bugzilla.mozilla.org/show_bug.cgi?id=601535 Bug 601535])</s> |
− | * [https://bugzilla.mozilla.org/show_bug.cgi?id=620164 Bug 620164] nsTheoraState::MaxKeyframeOffset doesn't need to use MulOverflow | + | * <s> [https://bugzilla.mozilla.org/show_bug.cgi?id=620164 Bug 620164] nsTheoraState::MaxKeyframeOffset doesn't need to use MulOverflow (This may conflict with [https://bugzilla.mozilla.org/show_bug.cgi?id=601535 Bug 601535])</s> |
− | (This may conflict with [https://bugzilla.mozilla.org/show_bug.cgi?id=601535 Bug 601535]) | ||
− | * [https://bugzilla.mozilla.org/show_bug.cgi?id=711839 Bug 711839] ---disable-ogg configure option breaks build after [https://bugzilla.mozilla.org/show_bug.cgi?id=689834 bug 689834] | + | * <s>[https://bugzilla.mozilla.org/show_bug.cgi?id=711839 Bug 711839] ---disable-ogg configure option breaks build after [https://bugzilla.mozilla.org/show_bug.cgi?id=689834 bug 689834]</s> |
− | * [https://bugzilla.mozilla.org/show_bug.cgi?id=702161 Bug 702161] videocontrols.xml has anonymous function event listeners that are added but never removed | + | * <s>[https://bugzilla.mozilla.org/show_bug.cgi?id=702161 Bug 702161] videocontrols.xml has anonymous function event listeners that are added but never removed</s> |
− | [https://bugzilla.mozilla.org/show_bug.cgi?id=704326 Bug 704326] Standalone audio files should have an intrinsic size so they don't look awkward | + | * <s> [https://bugzilla.mozilla.org/show_bug.cgi?id=704326 Bug 704326] Standalone audio files should have an intrinsic size so they don't look awkward </s> |
− | [https://bugzilla.mozilla.org/show_bug.cgi?id=713383 Bug 713383] Directly viewed images should be printed with a transparent background on the body | + | * <s>[https://bugzilla.mozilla.org/show_bug.cgi?id=713383 Bug 713383] Directly viewed images should be printed with a transparent background on the body</s> |
− | [https://bugzilla.mozilla.org/show_bug.cgi?id=681548 Bug 681548] Visual refresh of controls (needs graphics from Stephen Horlander first) | + | * [https://bugzilla.mozilla.org/show_bug.cgi?id=681548 Bug 681548] Visual refresh of controls (needs graphics from Stephen Horlander first) |
===Easy but not as small=== | ===Easy but not as small=== | ||
− | * [https://bugzilla.mozilla.org/show_bug.cgi?id=601535 Bug 601535] content/media should use CheckedInt.h | + | * <s>[https://bugzilla.mozilla.org/show_bug.cgi?id=601535 Bug 601535] content/media should use CheckedInt.h: Rework content/media/ to use CheckedInt to guard against integer overflow instead of the custom int overflow checking code defined in VideoUtils.h (I already suggested this to Steven Tseng, so you might want to check that he hasn't accepted this before suggesting other students tried working on it).</s> |
− | Rework content/media/ to use CheckedInt to guard against integer overflow instead of the custom int overflow checking code defined in VideoUtils.h (I already suggested this to Steven Tseng, so you might want to check that he hasn't accepted this before suggesting other students tried working on it). | ||
− | * [https://bugzilla.mozilla.org/show_bug.cgi?id=705234 Bug 705234] Inconsistent use of "full screen" and "full-screen" across browser and DOM strings, should use "fullscreen" instead Sometimes in strings we use "full-screen", "full-screen mode" and "fullscreen". We should use "fullscreen" instead. This is mostly string changes, but we actually need to rename the strings as well, otherwise localizers won't notice that a change occurred (they may or may not need to). | + | * <s>[https://bugzilla.mozilla.org/show_bug.cgi?id=705234 Bug 705234] Inconsistent use of "full screen" and "full-screen" across browser and DOM strings, should use "fullscreen" instead Sometimes in strings we use "full-screen", "full-screen mode" and "fullscreen". We should use "fullscreen" instead. This is mostly string changes, but we actually need to rename the strings as well, otherwise localizers won't notice that a change occurred (they may or may not need to).</s> |
===Not quite as easy=== | ===Not quite as easy=== | ||
− | * [https://bugzilla.mozilla.org/show_bug.cgi?id=689374 Bug 689374] Add specialized audio controls for small-dimension media | + | * <s>[https://bugzilla.mozilla.org/show_bug.cgi?id=689374 Bug 689374] Add specialized audio controls for small-dimension media</s> |
− | * [https://bugzilla.mozilla.org/show_bug.cgi?id=715141 Bug 715141] Using keyboard to switch zoom level when viewing images directly | + | * <s>[https://bugzilla.mozilla.org/show_bug.cgi?id=715141 Bug 715141] Using keyboard to switch zoom level when viewing images directly </s> |
− | * [https://bugzilla.mozilla.org/show_bug.cgi?id=708814 Bug 708814] Should fade out videocontrols even if there's no mouse movement | + | * <s>[https://bugzilla.mozilla.org/show_bug.cgi?id=708814 Bug 708814] Should fade out videocontrols even if there's no mouse movement Video controls, all in Javascript, I'm keen to see this one done.</s> |
− | Video controls, all in Javascript, I'm keen to see this one done. | ||
− | * [https://bugzilla.mozilla.org/show_bug.cgi?id=516811 Bug 516811] "load" events for poster image loads bubble out | + | * [https://bugzilla.mozilla.org/show_bug.cgi?id=516811 Bug 516811] "load" events for poster image loads bubble out (Fix should be similar to that for bug 715469, but may involve some C++ level DOM manipulation) |
− | (Fix should be similar to that for bug 715469, but may involve some C++ level DOM manipulation) | ||
− | * [https://bugzilla.mozilla.org/show_bug.cgi?id=680321 Bug 680321] Media preload state should reset in resource selection algorithm | + | * <s>[https://bugzilla.mozilla.org/show_bug.cgi?id=680321 Bug 680321] Media preload state should reset in resource selection algorithm Changes need to be made to nsHTMLMediaElement.cpp, so should be "easy" to get your head wrapped around it...</s> |
− | Changes need to be made to nsHTMLMediaElement.cpp, so should be "easy" to get your head wrapped around it... | ||
− | * [https://bugzilla.mozilla.org/show_bug.cgi?id=517363 Bug 517363] Preserve video poster aspect ratio when scaling | + | * <s>[https://bugzilla.mozilla.org/show_bug.cgi?id=517363 Bug 517363] Preserve video poster aspect ratio when scaling (Would be good to see this fixed!)</s> |
− | Would be good to see this fixed! | ||
− | * [https://bugzilla.mozilla.org/show_bug.cgi?id=711742 Bug 711742] HTML5 video's "Save Video As" saves an empty .webm file if you have not actually played the video. The patch that causes this to regress has been identified, so that should give clues of how to fix it. | + | * <s>[https://bugzilla.mozilla.org/show_bug.cgi?id=711742 Bug 711742] HTML5 video's "Save Video As" saves an empty .webm file if you have not actually played the video. The patch that causes this to regress has been identified, so that should give clues of how to fix it.</s> |
− | * [https://bugzilla.mozilla.org/show_bug.cgi?id=601032 Bug 601032] test_play_twice fails with sound disabled | + | * <s>[https://bugzilla.mozilla.org/show_bug.cgi?id=601032 Bug 601032] test_play_twice fails with sound disabled</s> |
− | * [https://bugzilla.mozilla.org/show_bug.cgi?id=618737 Bug 618737] The <video> element develops a tabindex attribute out of nowhere | + | * <s>[https://bugzilla.mozilla.org/show_bug.cgi?id=618737 Bug 618737] The <video> element develops a tabindex attribute out of nowhere (I'm not the best mentor for this one, Boris Zbarsky probably is, but it sounds simple enough to fix...)</s> |
− | (I'm not the best mentor for this one, Boris Zbarsky probably is, but it sounds simple enough to fix...) | ||
* [https://bugzilla.mozilla.org/show_bug.cgi?id=712836 Bug 712836] Reloading live stream causes HTMLMediaElement.error.code to be copied to new clone of resource | * [https://bugzilla.mozilla.org/show_bug.cgi?id=712836 Bug 712836] Reloading live stream causes HTMLMediaElement.error.code to be copied to new clone of resource | ||
− | * [https://bugzilla.mozilla.org/show_bug.cgi?id=708553 Bug 708553] Hovered element state is not relinquished when entering and exiting fullscreen mode. Requires digging into how Firefox manages event state for :hover. | + | * <s>[https://bugzilla.mozilla.org/show_bug.cgi?id=708553 Bug 708553] Hovered element state is not relinquished when entering and exiting fullscreen mode. Requires digging into how Firefox manages event state for :hover.</s> |
+ | |||
+ | * <s>[https://bugzilla.mozilla.org/show_bug.cgi?id=500784 Bug 500784] Video/audio files over 2^31 bytes in length are unseekable. We'd need changes made to the nsMediaCache to support 64bit file sizes, along with changes to the nsBuiltinDecoderReader-subclass seek implementations. We care more about seeking in large WebM files rather than in Ogg files, i.e. don't go to the trouble of changing the Ogg seeking code, that'll be painful! We can file a follow up for that work.</s> | ||
− | * [https://bugzilla.mozilla.org/show_bug.cgi?id= | + | * <s>[https://bugzilla.mozilla.org/show_bug.cgi?id=714071 Bug 714071] The Show Statistics setting is not preserved when toggling the full screen mode. Fullscreen and <video> in one bug! Your students will love this one! I have no idea why this is happening, maybe they won't love it.</s> |
− | |||
− | * [https://bugzilla.mozilla.org/show_bug.cgi?id= | + | * <s>[https://bugzilla.mozilla.org/show_bug.cgi?id=677122 Bug 677122] Changing media fragment portion of URL in video documents doesn't update start/end times</s> |
− | + | ==Link to MouseLock Bug (Bug 633602)== | |
+ | [[Implementing the Mouse Lock API in Firefox]]<br> | ||
+ | [[Mouse Lock API Test Tracker]] |
Latest revision as of 13:22, 9 May 2012
Contents
OSD700 and DPS911 Class Information (Winter 2012)
The only caveat about creating a twitter list of everyone that people can subscribe to is I can't put myself in it! Alas it's better than nothing. Go here to subscribe.
Name | IRC | Blog | Bugs | |
---|---|---|---|---|
Christopher De Cairos / cadecairos | cadecairos | @ChrisDeCairos | My Blog |
|
David Seifried | dseif | @dcseifried | Blog | |
Scott Downe | moznet: scott / freenode: sdowne | @ScottDowne | My Blog | |
Matthew Schranz | mjschranz | @mjschranz | Blog |
|
Ching Wei Tseng (Steven) | Anachid | @sttseng | Blog |
|
Mohammed Buttu | mbuttu | @mbuttu | Blog |
Popcorn Maker
Popcorn Maker FCP
|
Raymond Hung | rhung | @Raymond_Hung | Blog | Bug 633602 - Mouselock the tests in particular |
Anurag Bhatnagar | abhatnagar | @anuragbh | Blog |
Bug 1392 - Processing.js
|
Diogo Golovanevsky Monteiro | diogogmt | @diogogmt | Blog |
Bug 581863 |
Abhishek Bhatnagar | abhatnagar1 | @abhishekToronto | Blog |
Possible Video/Fullscreen Bugs
Small and easy bugs
-
Bug 715323 nsBuiltinDecoderReader::NotifyDataArrived() should have a 64bit offset parameter. Easy first bug to get acquainted with our process.
-
Bug 620159 nsVorbisState::Time needs to handle failure from return from MulOverflow. (This may conflict with Bug 601535)
-
Bug 620164 nsTheoraState::MaxKeyframeOffset doesn't need to use MulOverflow (This may conflict with Bug 601535)
-
Bug 711839 ---disable-ogg configure option breaks build after bug 689834
-
Bug 702161 videocontrols.xml has anonymous function event listeners that are added but never removed
-
Bug 704326 Standalone audio files should have an intrinsic size so they don't look awkward
-
Bug 713383 Directly viewed images should be printed with a transparent background on the body
- Bug 681548 Visual refresh of controls (needs graphics from Stephen Horlander first)
Easy but not as small
-
Bug 601535 content/media should use CheckedInt.h: Rework content/media/ to use CheckedInt to guard against integer overflow instead of the custom int overflow checking code defined in VideoUtils.h (I already suggested this to Steven Tseng, so you might want to check that he hasn't accepted this before suggesting other students tried working on it).
-
Bug 705234 Inconsistent use of "full screen" and "full-screen" across browser and DOM strings, should use "fullscreen" instead Sometimes in strings we use "full-screen", "full-screen mode" and "fullscreen". We should use "fullscreen" instead. This is mostly string changes, but we actually need to rename the strings as well, otherwise localizers won't notice that a change occurred (they may or may not need to).
Not quite as easy
-
Bug 689374 Add specialized audio controls for small-dimension media
-
Bug 715141 Using keyboard to switch zoom level when viewing images directly
-
Bug 708814 Should fade out videocontrols even if there's no mouse movement Video controls, all in Javascript, I'm keen to see this one done.
- Bug 516811 "load" events for poster image loads bubble out (Fix should be similar to that for bug 715469, but may involve some C++ level DOM manipulation)
-
Bug 680321 Media preload state should reset in resource selection algorithm Changes need to be made to nsHTMLMediaElement.cpp, so should be "easy" to get your head wrapped around it...
-
Bug 517363 Preserve video poster aspect ratio when scaling (Would be good to see this fixed!)
-
Bug 711742 HTML5 video's "Save Video As" saves an empty .webm file if you have not actually played the video. The patch that causes this to regress has been identified, so that should give clues of how to fix it.
-
Bug 601032 test_play_twice fails with sound disabled
-
Bug 618737 The <video> element develops a tabindex attribute out of nowhere (I'm not the best mentor for this one, Boris Zbarsky probably is, but it sounds simple enough to fix...)
- Bug 712836 Reloading live stream causes HTMLMediaElement.error.code to be copied to new clone of resource
-
Bug 708553 Hovered element state is not relinquished when entering and exiting fullscreen mode. Requires digging into how Firefox manages event state for :hover.
-
Bug 500784 Video/audio files over 2^31 bytes in length are unseekable. We'd need changes made to the nsMediaCache to support 64bit file sizes, along with changes to the nsBuiltinDecoderReader-subclass seek implementations. We care more about seeking in large WebM files rather than in Ogg files, i.e. don't go to the trouble of changing the Ogg seeking code, that'll be painful! We can file a follow up for that work.
-
Bug 714071 The Show Statistics setting is not preserved when toggling the full screen mode. Fullscreen and <video> in one bug! Your students will love this one! I have no idea why this is happening, maybe they won't love it.
-
Bug 677122 Changing media fragment portion of URL in video documents doesn't update start/end times
Link to MouseLock Bug (Bug 633602)
Implementing the Mouse Lock API in Firefox
Mouse Lock API Test Tracker