Security/Reviews/Firefox4: Difference between revisions
No edit summary |
|||
| (18 intermediate revisions by 6 users not shown) | |||
| Line 17: | Line 17: | ||
! who | ! who | ||
! review date | ! review date | ||
! interested | ! interested | ||
! References<br> | |||
|- | |- | ||
| [[Security/Reviews/Firefox4/CSS features Security Review|CSS calc(), CSS transitions, and CSS :-moz-any()]] | | [[Security/Reviews/Firefox4/CSS features Security Review|CSS calc(), CSS transitions, and CSS :-moz-any()]] | ||
| dbaron | | dbaron | ||
| | | <br> | ||
| | | <br> | ||
| [https://developer.mozilla.org/en/CSS/-moz-calc -moz-calc()][https://developer.mozilla.org/en/CSS/CSS_transitions transitions] [https://developer.mozilla.org/en/CSS/%3A-moz-any -moz-any()]<br> | |||
|- | |- | ||
| [[Security/Reviews/Firefox4/CSS Element Security Review|CSS -moz-element/document.mozSetImageElement]] | | [[Security/Reviews/Firefox4/CSS Element Security Review|CSS -moz-element/document.mozSetImageElement]] | ||
| mstange | | mstange | ||
| | | <br> | ||
| | | <br> | ||
| [https://developer.mozilla.org/en/DOM/document.mozSetImageElement spec]<br> | |||
|- | |- | ||
| [[Security/Reviews/Firefox4/moz-font-feature-opentype Security Review|CSS -moz-font-feature-opentype]] | | [[Security/Reviews/Firefox4/moz-font-feature-opentype Security Review|CSS -moz-font-feature-opentype]] | ||
| jdagget/jkew | | jdagget/jkew | ||
| | | <br> | ||
| | | <br> | ||
| [http://lists.w3.org/Archives/Public/www-style/2009Jun/0506.html W3C Proposal] [https://developer.mozilla.org/en/CSS/@font-face spec]<br> | |||
|- | |- | ||
| [[Security/Reviews/Firefox4/JS Animation Scheduling Security Review|JS animation scheduling]] | | [[Security/Reviews/Firefox4/JS Animation Scheduling Security Review|JS animation scheduling]] | ||
| bzbarsky | | bzbarsky | ||
| | | <br> | ||
| | | <br> | ||
| [https://developer.mozilla.org/en/DOM/Animations_using_MozBeforePaint spec]<br> | |||
|- | |- | ||
| [[Security/Reviews/Firefox4/JS-ctypes|JS-ctypes]] | | [[Security/Reviews/Firefox4/JS-ctypes|JS-ctypes]] | ||
| dwitte | | dwitte | ||
| | | <br> | ||
| bsmith | | bsmith | ||
| [https://developer.mozilla.org/en/js-ctypes spec]<br> | |||
|- | |- | ||
| [[Security/Reviews/Firefox4/JS ES5 Security Review|JS ES5]] | | [[Security/Reviews/Firefox4/JS ES5 Security Review|JS ES5]] | ||
| jwalden? | | jwalden? | ||
| | | <br> | ||
| | | <br> | ||
| [https://developer.mozilla.org/En/JavaScript/ECMAScript_5_support_in_Mozilla draft] {{bug|520696}}<br> | |||
|- | |- | ||
| [[Security/Reviews/Firefox4/JS Proxies Security Review|JS Proxies]] | | [[Security/Reviews/Firefox4/JS Proxies Security Review|JS Proxies]] | ||
| gal? | | gal? | ||
| | | <br> | ||
| | | <br> | ||
| [https://developer.mozilla.org/en/nsISupports_proxies nsISupports proxies]<br> | |||
|- | |- | ||
| [[Security/Reviews/Firefox4/JS Compartments Security Review|JS Compartments]] | | [[Security/Reviews/Firefox4/JS Compartments Security Review|JS Compartments]] | ||
| mrbkap | | mrbkap | ||
| | | <br> | ||
| | | <br> | ||
| [https://developer.mozilla.org/En/SpiderMonkey/Internals/Thread_Safety SpiderMonkey Internals: Thread Safety]<br> | |||
|- | |- | ||
| [[Security/Reviews/Firefox4/ByteArray Security Review|JS/WebGL byte arrays]] | | [[Security/Reviews/Firefox4/ByteArray Security Review|JS/WebGL byte arrays]] | ||
| vlad | | vlad | ||
| | | <br> | ||
| bsmith | | bsmith | ||
| <br> | |||
|- | |- | ||
| [[Security/Reviews/Firefox4/Media Buffer Security Review|Media Buffer API]] | | [[Security/Reviews/Firefox4/Media Buffer Security Review|Media Buffer API]] | ||
| <br> | |||
| <br> | |||
| <br> | |||
| {{bug|462957}}, {{bug|570904}} [https://developer.mozilla.org/en/Introducing_the_Audio_API_Extension spec] | |||
|- | |- | ||
| [[Security/Reviews/Firefox4/DPI Security Review|New DPI System?]] | | [[Security/Reviews/Firefox4/DPI Security Review|New DPI System?]] | ||
| roc | | roc | ||
| | | <br> | ||
| | | <br> | ||
| <br> | |||
|- | |- | ||
| [[Security/Reviews/Firefox4/SVG as IMG Security Review|SVG as IMG/background]] | | [[Security/Reviews/Firefox4/SVG as IMG Security Review|SVG as IMG/background]] | ||
| dholbert | | dholbert | ||
| | | <br> | ||
| | | <br> | ||
| <br> | |||
|- | |- | ||
| [[Security/Reviews/Firefox4/TabModal Dialogs Security Review|Tab Modal dialogs]] | | [[Security/Reviews/Firefox4/TabModal Dialogs Security Review|Tab Modal dialogs]] | ||
| dolske | | dolske | ||
| | | <br> | ||
| bsmith | | bsmith | ||
| [https://developer.mozilla.org/en/Using_tab-modal_prompts spec]<br> | |||
|- | |- | ||
| [[Security/Reviews/Firefox4/TouchEvents Security Review|Touch Events]] | | [[Security/Reviews/Firefox4/TouchEvents Security Review|Touch Events]] | ||
| felipe | | felipe | ||
| | | <br> | ||
| | | <br> | ||
| [https://developer.mozilla.org/en/DOM/Touch_events spec]<br> | |||
|- | |- | ||
| [[Security/Reviews/Firefox4/Driver Blocklisting Security Review|Video driver blocklisting]] | | [[Security/Reviews/Firefox4/Driver Blocklisting Security Review|Video driver blocklisting]] | ||
| Joe Drew \o/ | | Joe Drew \o/ | ||
| | | <br> | ||
| | | <br> | ||
| <br> | |||
|- | |- | ||
| [[Security/Reviews/Firefox4/WebGL Security Review|WebGL]] | | [[Security/Reviews/Firefox4/WebGL Security Review|WebGL]] | ||
| vlad | | vlad | ||
| | | <br> | ||
| | | <br> | ||
| [https://developer.mozilla.org/en/WebGL Spec]<br> | |||
|- | |- | ||
| [[Security/Reviews/Firefox4/ANGLE Security Review|ANGLE]] | | [[Security/Reviews/Firefox4/ANGLE Security Review|ANGLE]] | ||
| vlad | | vlad | ||
| | | <br> | ||
| | | <br> | ||
| [https://developer.mozilla.org/en/CSS/-moz-linear-gradient spec]<br> | |||
|- | |- | ||
| [[Security/Reviews/Firefox4/WebM Security Review|WebM]] | | [[Security/Reviews/Firefox4/WebM Security Review|WebM]] | ||
| roc? | | roc? | ||
| | | <br> | ||
| | | <br> | ||
| [https://developer.mozilla.org/En/HTML/Element/Video video] [https://developer.mozilla.org/En/HTML/Element/Audio audio]<br> | |||
|- | |- | ||
| [[Security/Reviews/Firefox4/WebM3D Security Review|WebM 3D]] | | [[Security/Reviews/Firefox4/WebM3D Security Review|WebM 3D]] | ||
| Matthew Gregan | | Matthew Gregan | ||
| | | <br> | ||
| | | <br> | ||
| <br> | |||
|- | |- | ||
| [[Security/Reviews/Firefox4/XHR.mozResponseArrayBuffer Security Review|XHR.mozResponseArrayBuffer]] | | [[Security/Reviews/Firefox4/XHR.mozResponseArrayBuffer Security Review|XHR.mozResponseArrayBuffer]] | ||
| vlad | | vlad | ||
| | | <br> | ||
| bsmith | | bsmith | ||
| [https://developer.mozilla.org/en/xmlhttprequest spec]<br> | |||
|} | |} | ||
| Line 133: | Line 149: | ||
! review date | ! review date | ||
! interested | ! interested | ||
|} | |} | ||
| Line 192: | Line 163: | ||
! date | ! date | ||
! review comments | ! review comments | ||
|- | |||
| [[Security/Reviews/Firefox4/AudioAPI Security Review|Audio API]] | |||
| David Humphrey | |||
| 2011-02-02 | |||
| [[Security/Reviews/Firefox4/AudioAPI Security Review#Review_comments|comments]] | |||
|- | |- | ||
| [[Security/Reviews/Firefox4/Jaegermonkey Security Review|Jägermonkey]] | | [[Security/Reviews/Firefox4/Jaegermonkey Security Review|Jägermonkey]] | ||
| Line 262: | Line 238: | ||
! date | ! date | ||
! review comments | ! review comments | ||
|- | |||
| [[Security/Reviews/Firefox4/App Tabs Security Review|App Tabs]] | |||
| zpao | |||
| Jan 19, 2011 [https://bugzilla.mozilla.org/show_bug.cgi?id=625861 tracking bug] | |||
| dao, faaborg | |||
|- | |||
| [[Security/Reviews/Firefox4/Doorhanger Security Review|Doorhanger notifications]] | |||
| gavin | |||
| Jan 19, 2011 | |||
| faaborg, bsmith | |||
|- | |- | ||
| [[Security/Reviews/Firefox4/about:addons Security Review|about:addons]] | | [[Security/Reviews/Firefox4/about:addons Security Review|about:addons]] | ||
| Line 292: | Line 278: | ||
| 2010-10-05 | | 2010-10-05 | ||
| [[Security/Reviews/Firefox4/Web Console Security Review#Review_comments|comments]] | | [[Security/Reviews/Firefox4/Web Console Security Review#Review_comments|comments]] | ||
|- | |||
| [[Security/Reviews/Firefox4/Update Billboard Security Review|Update billboard changes]] | |||
| rs | |||
| 2011-01-24 | |||
| {{Bug|628428}} | |||
|- | |||
| [[Firefox/Projects/Action_based_on_update_xml_after_app_update|Update: Post-update actions]] | |||
| rs | |||
| 2011-01-24 | |||
| <br> | |||
|- | |||
| [[Security/Reviews/Firefox4/New Theme Security Review|New Theme (code/bindings)]] | |||
| gavin | |||
| Jan 19, 2011 | |||
| <br> | |||
|- | |||
| [[Security/Reviews/Firefox4/Satchel-JS Security Review|Port satchel to js]] | |||
| dolske | |||
| <br> | |||
| <br> | |||
|- | |||
| [[Security/Reviews/Firefox4/Cascade Session Restore Security Review|Cascade Session Restore]] | |||
| zpao | |||
| Jan 19, 2011 [https://bugzilla.mozilla.org/show_bug.cgi?id=625866 tracking bug] | |||
| <br> | |||
|- | |||
| [[Security/Reviews/Firefox4/Deferred Session Restore Security Review|Deferred Session Restore]] | |||
| zpao | |||
| [https://bugzilla.mozilla.org/show_bug.cgi?id=625866 tracking bug] | |||
| <br> | |||
|- | |||
| [[Security/Reviews/Firefox4/Tab Panorama Security Review|Tab Panorama]] | |||
| aza | |||
| Jan 19, 2011 [https://bugzilla.mozilla.org/show_bug.cgi?id=625217 tracking bug] | |||
| | |||
|- | |||
|} | |} | ||
Latest revision as of 18:17, 2 March 2011
Pending Reviews
Please complete the minimal Security Review Template for your feature before scheduling your review. For small features or changes we will likely group several together into one session or handle it one-on-one. For larger topics we should schedule them for a longer timeslot. Please add that time in the date column (use Mountain View time (Pacific)).
The "who" is responsible for filling out the review template and scheduling the review. Please email security at mozilla dot org when reviews are scheduled and if times change. Please also email any other folks in the "interested" column. If not immediately after a Platform meeting, dial in will be 650-903-0800 x92 Conf# 215 (US/INTL). Default IRC backchannel is #planning.
Features that get cut from Firefox 4 should be moved to the Firefox4.next page rather than simply deleted.
Platform
All times are Mountain View local time (Pacific)
| feature | who | review date | interested | References |
|---|---|---|---|---|
| CSS calc(), CSS transitions, and CSS :-moz-any() | dbaron | -moz-calc()transitions -moz-any() | ||
| CSS -moz-element/document.mozSetImageElement | mstange | spec | ||
| CSS -moz-font-feature-opentype | jdagget/jkew | W3C Proposal spec | ||
| JS animation scheduling | bzbarsky | spec | ||
| JS-ctypes | dwitte | bsmith | spec | |
| JS ES5 | jwalden? | draft bug 520696 | ||
| JS Proxies | gal? | nsISupports proxies | ||
| JS Compartments | mrbkap | SpiderMonkey Internals: Thread Safety | ||
| JS/WebGL byte arrays | vlad | bsmith | ||
| Media Buffer API | bug 462957, bug 570904 spec | |||
| New DPI System? | roc | |||
| SVG as IMG/background | dholbert | |||
| Tab Modal dialogs | dolske | bsmith | spec | |
| Touch Events | felipe | spec | ||
| Video driver blocklisting | Joe Drew \o/ | |||
| WebGL | vlad | Spec | ||
| ANGLE | vlad | spec | ||
| WebM | roc? | video audio | ||
| WebM 3D | Matthew Gregan | |||
| XHR.mozResponseArrayBuffer | vlad | bsmith | spec |
Front End
| feature | who | review date | interested |
|---|
Completed Reviews
Please add a link to the review comments in the appropriate column. There is a section in the template that can be used for this, but it's fine to have the comments in a separate document as long as it's linked to here.
Platform
| feature | who | date | review comments |
|---|---|---|---|
| Audio API | David Humphrey | 2011-02-02 | comments |
| Jägermonkey | sayre | 2010-12-16 | |
| harfbuzz | jdagget/jkew | 2010-12-? | |
| HTML5 Parser | hsivonen | 2010-10-27 | comments |
| Desktop Notifications | dougt | 2010-10-18 | comments |
| IndexedDB | bent | 2010-09-30 | comments |
| canvas.mozGetAsFile | khuey | 2010-09-29 | |
| File API changes | sicking | 2010-09-29 | |
| FormData | sicking | 2010-09-29 | |
| HTML5 Forms | Mounir | 2010-09-29 | comments |
| Cookie changes | dwitte | 2010-09-22 | comments |
| SVG Animation (SMIL) | dholbert | 2008-09-03 | comments |
| History pushState/replaceState | jlebar | see 3.6 rev |
Front End
| feature | who | date | review comments |
|---|---|---|---|
| App Tabs | zpao | Jan 19, 2011 tracking bug | dao, faaborg |
| Doorhanger notifications | gavin | Jan 19, 2011 | faaborg, bsmith |
| about:addons | mossop | 2010-12-15 | |
| Add-on UI | dietrich | 2010-12-16 | comments |
| Jetpack modules | dietrich | 2010-12-16 | comments |
| New Start Page | mak | 2010-12-15 | |
| Sync | internal and external reviews | faaborg, bsmith | |
| Web console | kdangoor | 2010-10-05 | comments |
| Update billboard changes | rs | 2011-01-24 | bug 628428 |
| Update: Post-update actions | rs | 2011-01-24 | |
| New Theme (code/bindings) | gavin | Jan 19, 2011 | |
| Port satchel to js | dolske | ||
| Cascade Session Restore | zpao | Jan 19, 2011 tracking bug | |
| Deferred Session Restore | zpao | tracking bug | |
| Tab Panorama | aza | Jan 19, 2011 tracking bug |