Difference between revisions of "OSD700-DPS911-Winter2012"

From CDOT Wiki
Jump to: navigation, search
m
(→‎OSD700 and DPS911 Class Information (Winter 2012))
 
(38 intermediate revisions by 7 users not shown)
Line 16: Line 16:
 
     <td>cadecairos</td>
 
     <td>cadecairos</td>
 
     <td>[https://twitter.com/#!/ChrisDeCairos @ChrisDeCairos]</td>
 
     <td>[https://twitter.com/#!/ChrisDeCairos @ChrisDeCairos]</td>
−
     <td>[http://www.chrisdecairos.ca My Blog]</td>
+
     <td>[https://www.chrisdecairos.ca My Blog]</td>
 
     <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=686137 Bug 686137]</s> Resolved :D<br />
Line 24: Line 24:
 
[https://bugzilla.mozilla.org/show_bug.cgi?id=601032 Bug 601032]<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=517363 Bug 517363]<br />
−
[https://bugzilla.mozilla.org/show_bug.cgi?id=618737 Bug 618737]
+
[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>
 
     </td>
 
   </tr>
 
   </tr>
Line 34: Line 38:
 
     <td>
 
     <td>
 
[https://bugzilla.mozilla.org/show_bug.cgi?id=686370 Bug 686370]<br/>
 
[https://bugzilla.mozilla.org/show_bug.cgi?id=686370 Bug 686370]<br/>
−
[https://bugzilla.mozilla.org/show_bug.cgi?id=702161 Bug 702161]<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/>
 
[https://bugzilla.mozilla.org/show_bug.cgi?id=708814 Bug 708814]<br/>
−
[https://bugzilla.mozilla.org/show_bug.cgi?id=680321 Bug 680321]<br/>
+
<s>[https://bugzilla.mozilla.org/show_bug.cgi?id=680321 Bug 680321]</s><br/>
−
[https://bugzilla.mozilla.org/show_bug.cgi?id=708553 Bug 708553]<br/>
 
−
[https://bugzilla.mozilla.org/show_bug.cgi?id=722788 Bug 722788]<br/>
 
 
     </td>
 
     </td>
 
   </tr>
 
   </tr>
Line 53: Line 55:
 
   <tr>
 
   <tr>
 
     <td>Matthew Schranz</td>
 
     <td>Matthew Schranz</td>
−
     <td>Moznet: mjschranz / freenode: mjschranz</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>
 
<s>[https://bugzilla.mozilla.org/show_bug.cgi?id=698384 Bug 698384]</s><br/>
 
<s>[https://bugzilla.mozilla.org/show_bug.cgi?id=698384 Bug 698384]</s><br/>
 
<s>[https://bugzilla.mozilla.org/show_bug.cgi?id=698385 Bug 698385]</s><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/>
 
<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/>
 
[https://bugzilla.mozilla.org/show_bug.cgi?id=720768 Bug 720768]<br/>
−
[https://bugzilla.mozilla.org/show_bug.cgi?id=725289 Bug 725289]
+
<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 73: Line 78:
 
     <td>
 
     <td>
 
<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=716107 Bug 716107]</s> (too complicated as of now; read bug comments)<br/>
−
[https://bugzilla.mozilla.org/show_bug.cgi?id=601535 Bug 601535]<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 83: Line 89:
 
     <td>[http://mbuttu.wordpress.com Blog]</td>
 
     <td>[http://mbuttu.wordpress.com Blog]</td>
 
     <td>
 
     <td>
 +
 
Popcorn Maker<br/>
 
Popcorn Maker<br/>
−
[https://webmademovies.lighthouseapp.com/projects/80723-popcorn-maker/tickets/67-dont-let-a-user-commit-a-blank-field-for-timeline-media-change#ticket-67-7 Bug 67]<br/>
+
<s>[https://webmademovies.lighthouseapp.com/projects/80723/tickets/281-basic-template-has-2-descriptions-in-manifest Bug 281]</s><br/>
−
[https://webmademovies.lighthouseapp.com/projects/80723/tickets/157-layer-numbering-issue-on-import Bug 157]<br/>
 
−
[https://webmademovies.lighthouseapp.com/projects/65733/tickets/254-much-confusion-re-apply-and-ok Bug 254]<br/>
 
−
[https://webmademovies.lighthouseapp.com/projects/65733/tickets/256-timecodes-should-round-to-1-decimal Bug 256]<br/>
 
−
[https://webmademovies.lighthouseapp.com/projects/65733/tickets/257-add-ds_store-to-gitignore Bug 257]<br/>
 
  
−
[https://webmademovies.lighthouseapp.com/projects/80723/tickets/281-basic-template-has-2-descriptions-in-manifest Bug 281]<br/><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/>
 
Popcorn Maker FCP<br/>
Line 97: Line 112:
  
 
Test Swarm
 
Test Swarm
−
* Make it work on iOS
+
* <s>Make it work on iOS</s>
 +
 
 
     </td>
 
     </td>
 
   </tr>  
 
   </tr>  
Line 125: Line 141:
 
     <td>[http://diogogmt.wordpress.com/ Blog]</td>
 
     <td>[http://diogogmt.wordpress.com/ Blog]</td>
 
     <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=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=715141 Bug 715141]</s><br/>
−
[https://bugzilla.mozilla.org/show_bug.cgi?id=718133 Bug 718133]<br/>
+
<s>[https://bugzilla.mozilla.org/show_bug.cgi?id=718133 Bug 718133]<br/></s>
−
[https://bugzilla.mozilla.org/show_bug.cgi?id=713383 Bug 713383]<br />
+
<s>[https://bugzilla.mozilla.org/show_bug.cgi?id=713383 Bug 713383]<br /></s>
−
[https://bugzilla.mozilla.org/show_bug.cgi?id=714071 Bug 714071]
+
[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>
 
     </td>
 
   </tr>
 
   </tr>
Line 138: Line 162:
 
     <td>[https://brattahlid.wordpress.com/ Blog]</td>
 
     <td>[https://brattahlid.wordpress.com/ Blog]</td>
 
     <td>
 
     <td>
−
[https://bugzilla.mozilla.org/show_bug.cgi?id=705234 Bug 705234]<br/>
+
<s>[https://bugzilla.mozilla.org/show_bug.cgi?id=705234 Bug 705234]</s><br/>
−
[https://bugzilla.mozilla.org/show_bug.cgi?id=620164 Bug 620164]<br/>
+
<s>[https://bugzilla.mozilla.org/show_bug.cgi?id=620164 Bug 620164]</s><br/>
−
Considering one other possibility
+
<s>[https://bugzilla.mozilla.org/show_bug.cgi?id=500784 Bug 500784]</s><br/>
 
     </td>
 
     </td>
 
   </tr>
 
   </tr>
Line 153: Line 177:
 
* <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>
 
* <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>
  
 
* <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>
 
* <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>
Line 197: Line 221:
 
* <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=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=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=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>
  
 
* <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>
 
* <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>

Latest revision as of 12:22, 9 May 2012

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 Twitter Blog Bugs
Christopher De Cairos / cadecairos cadecairos @ChrisDeCairos My Blog

Bug 686137 Resolved :D
Bug 715323 Resolved
Bug 711839 This was fixed by somebody else
Bug 711742
Bug 601032
Bug 517363
Bug 618737
Bug 726904<- hacking on this
Bug 736400<- hacking on this
Bug 665598<- hacking on this
Bug 665395<- hacking on this

David Seifried dseif @dcseifried Blog

Bug 686370
Bug 702161
 Bug 708814
Bug 680321
Scott Downe moznet: scott / freenode: sdowne @ScottDowne My Blog

Bug 677121
Bug 677122
Matthew Schranz mjschranz @mjschranz Blog

Bug 698303
 Bug 698381
 Bug 698384
Bug 698385
Bug 718274
Bug 720768
Bug 725289
Bug 686370
Bug 686913
 Bug 723020

Ching Wei Tseng (Steven) Anachid @sttseng Blog

Bug 716107 (too complicated as of now; read bug comments)
Bug 601535 (Resolved)
Bug 620159 (Resolved in Bug 601535)
Bug 633602 (the test suite)

Mohammed Buttu mbuttu @mbuttu Blog

Popcorn Maker
Bug 281


Butter
Bug 253
Bug 252
Bug 256
Bug 257
Bug 390
Bug 410
Bug 434
Bug 458
Bug 459
Bug 463
Bug 493

Popcorn Maker FCP

  • Create a DMG file that will allow users to install Popcorn Maker FCP once downloaded to the Applications folder by dragging and dropping


Test Swarm

  • Make it work on iOS
Raymond Hung rhung @Raymond_Hung Blog Bug 633602 - Mouselock the tests in particular
Anurag Bhatnagar abhatnagar @anuragbh Blog

Bug 1392 - Processing.js
Bug 689374 - Mozilla Firefox
Diogo Golovanevsky Monteiro diogogmt @diogogmt Blog

Bug 581863
Bug 581866
Bug 633602
Bug 712225
Bug 715141
Bug 718133
 Bug 713383
 Bug 713608
Bug 714071
 Bug 728893
 Bug 724108
Bug 735031

Abhishek Bhatnagar abhatnagar1 @abhishekToronto Blog

Bug 705234
Bug 620164
Bug 500784



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 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

Retrieved from "https://wiki.cdot.senecapolytechnic.ca/w/index.php?title=OSD700-DPS911-Winter2012&oldid=85078"