Security/Reviews/Firefox4: Difference between revisions

From MozillaWiki
Jump to navigation Jump to search
No edit summary
No edit summary
Line 37: Line 37:
| jdagget/jkew  
| jdagget/jkew  
|  
|  
|
|-
| [[Security/Reviews/Firefox4/Jaegermonkey Security Review|Jägermonkey]]
| sayre
| 12/16 11am PST
|  
|  
|-
|-
Line 138: Line 133:
! review date  
! review date  
! interested
! interested
|-
| [[Security/Reviews/Firefox4/about:addons Security Review|about:addons]]
| mossop
| Wednesday Dec 15, 1pm PST; locations: 3P-PBJ and x92 - conf 215
| <br>
|-
| [[Security/Reviews/Firefox4/AddonUI Security Review|Add-on UI]]
| dietrich
| 12/16 3:30pm - 5pm, completed, [[Security/Reviews/Firefox4/AddonUI_Security_Review#Review_comments|comments]]
| <br>
|-
|-
| [[Security/Reviews/Firefox4/App Tabs Security Review|App Tabs]]  
| [[Security/Reviews/Firefox4/App Tabs Security Review|App Tabs]]  
Line 158: Line 143:
| <br>
| <br>
| faaborg, bsmith
| faaborg, bsmith
|-
| [[Security/Reviews/Firefox4/Jetpack Modules Security Review|Jetpack modules]]
| dietrich
| 12/16 3:30pm - 5pm, completed, [[Security/Reviews/Firefox4/Jetpack_Modules_Security_Review#Review_comments|comments]]
| bsmith
|-
| [[Security/Reviews/Firefox4/about:home Security Review|New Start Page]]
| mak
| done on 12/15
| <br>
|-
|-
| [[Security/Reviews/Firefox4/New Theme Security Review|New Theme (code/bindings)]]  
| [[Security/Reviews/Firefox4/New Theme Security Review|New Theme (code/bindings)]]  
Line 188: Line 163:
| <br>
| <br>
| <br>
| <br>
|-
| [[Security/Reviews/Firefox4/Sync Security Review|Sync]]
| <br>
| internal and external reviews
| faaborg, bsmith
|-
|-
| [[Security/Reviews/Firefox4/Tab Panorama Security Review|Tab Panorama]]  
| [[Security/Reviews/Firefox4/Tab Panorama Security Review|Tab Panorama]]  
Line 222: Line 192:
! date  
! date  
! review comments
! review comments
|-
| [[Security/Reviews/Firefox4/Jaegermonkey Security Review|Jägermonkey]]
| sayre
| 2010-12-16
|
|-
|-
| [[Security/Reviews/Firefox4/harfbuzz Security Review|harfbuzz]]  
| [[Security/Reviews/Firefox4/harfbuzz Security Review|harfbuzz]]  
Line 287: Line 262:
! date  
! date  
! review comments
! review comments
|-
| [[Security/Reviews/Firefox4/about:addons Security Review|about:addons]]
| mossop
| 2010-12-15
| <br>
|-
| [[Security/Reviews/Firefox4/AddonUI Security Review|Add-on UI]]
| dietrich
| 2010-12-16
| [[Security/Reviews/Firefox4/AddonUI_Security_Review#Review_comments|comments]]
|-
| [[Security/Reviews/Firefox4/Jetpack Modules Security Review|Jetpack modules]]
| dietrich
| 2010-12-16
| [[Security/Reviews/Firefox4/Jetpack_Modules_Security_Review#Review_comments|comments]]
|-
| [[Security/Reviews/Firefox4/about:home Security Review|New Start Page]]
| mak
| 2010-12-15
|
|-
| [[Security/Reviews/Firefox4/Sync Security Review|Sync]]
|
| internal and external reviews
| faaborg, bsmith
|-
|-
| [[Security/Reviews/Firefox4/Web Console Security Review|Web console]]  
| [[Security/Reviews/Firefox4/Web Console Security Review|Web console]]  

Revision as of 22:31, 12 January 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
Audio API David Humphrey
CSS calc(), CSS transitions, and CSS :-moz-any() dbaron
CSS -moz-element/document.mozSetImageElement mstange
CSS -moz-font-feature-opentype jdagget/jkew
JS animation scheduling bzbarsky
JS-ctypes dwitte bsmith
JS ES5 jwalden?
JS Proxies gal?
JS Compartments mrbkap
JS/WebGL byte arrays vlad
Media Buffer API (bug 462957, bug 570904)
New DPI System? roc
SVG as IMG/background dholbert
Tab Modal dialogs dolske bsmith
Touch Events felipe
Video driver blocklisting Joe Drew \o/
WebGL vlad
ANGLE vlad
WebM roc?
WebM 3D Matthew Gregan
XHR.mozResponseArrayBuffer vlad

Front End

feature who review date interested
App Tabs zpao
dao, faaborg
Doorhanger notifications gavin
faaborg, bsmith
New Theme (code/bindings) gavin

Port satchel to js dolske

Cascade Session Restore zpao

Deferred Session Restore zpao

Tab Panorama aza
Send email to Lucas 4 Dec
Update billboard changes rs Sent email request to security@mozilla.org on 10/18
Update: Post-update actions rs

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