Security/Reviews/Firefox4: Difference between revisions

From MozillaWiki
Jump to navigation Jump to search
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/AudioAPI Security Review|Audio API]]
| David Humphrey
|
|
|-
|-
| [[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]] ({{bug|462957}}, {{bug|570904}})
| [[Security/Reviews/Firefox4/Media Buffer Security Review|Media Buffer API]]&nbsp;
|
| <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
|-
| [[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
| <br>
| faaborg, bsmith
|-
| [[Security/Reviews/Firefox4/New Theme Security Review|New Theme (code/bindings)]]
| gavin
| <br>
| <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]
|
|-
| [[Security/Reviews/Firefox4/Update Billboard Security Review|Update billboard changes]]
| rs
| Sent email request to security@mozilla.org on 10/18
| <br>
|-
| [[Firefox/Projects/Action_based_on_update_xml_after_app_update|Update: Post-update actions]]
| rs
| <br>
| <br>
|}
|}


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