TPE-Layout/2016-11-01

Team Affairs

• David Baron visit on the week of 11/7.
  • Plan your discussion sessions ahead.
• Your training budget matters.
  • Can we buy physical books in favor of proficiency of soft/hard skills?
  • Can we apply multiple times for difference seminars/conferences ?
  • Your suggestions & feedback are welcomed.
• Global Wellness Program FAQ
  • Eligible Expense in 2016
  • Exception Request guideline

Individual Contributor

Astley Chen

• bug 1301638 - Using runtime pref to enable/ disable CSS mask properties
  • r+ - use fail-if to handle relevant test cases.
• bug 1304636 - [css-grid][css-flexbox] A {flex,grid} item's min-{width,height} "auto" value should be reported as "auto" in getComputedStyle()
  • WIP

Boris Chiou

Web Animations API

• bug 1286150 - Support paced spacing for basic shapes
  • r?
• bug 1273784 - Implement keyframe effect copy constructors
  • OOAD
• bug 1313554 - Make ShearArray brace-initialized with initial contents.
  • land
• bug 1313619 - Cannot do interpolation between two basic shapes with explicit margin-box and default reference-box
  • file

Animations

• bug 1168759 - Intermittent test_deferred_start.html
  • r

Cameron McCormack

Stylo

• bug 1288302 - Stylo: implement support for background-image: url()
• bug 1311921 - 2.04 - 2.71% tp5o Main_RSS / tp5o Private Bytes (windows7-32, windowsxp) regression on push 6861f02dadfa1863af35086da6133ebc1ceaf845 (Thu Oct 20 2016)
  • r+ | land | backout | debug

CJ Ku

• bug 1234485 - Draw image mask onto mask layer
  • r?
• bug 1305253 - In nsSVGIntegrationUtils::PaintMaskAndClipPath, manually pop up clip-path
  • r?
• bug 1310575 - Intermittent file:///C:/slave/test/build/tests/reftest/tests/layout/reftests/bugs/621253-2-externalFilter.html
  • r?
• bug 1310575 - Change the parameter of ValidateSVGFrame in nsSVGIntegrationUtils.cpp
  • r?
• bug 1313570 - Intermittent file:///C:/slave/test/build/tests/reftest/tests/layout/reftests/bugs/621253-2-externalFilter.html
  • r?
• bug 1313898 - Draw clip-path onto mask layer
  • file
• bug 1313877 - Draw SVG mask onto mask layer
  • file
• bug 1313001 - Support mask-position/mask-size OMTA
  • file
• bug 1314001 - Implement pref:layers.paintMask
  • file

Farmer Tseng

• Mortar would like to generate emf file for Windows printing
  • Write a sample code to print emf file in Windows.
• Mortar would like to generate PDF for Linux/MacOS printing
  • Studying

Jeremy Chen

Initial-letter

• bug 1310106 - Implement the rendering of floating :first-letter with CSS initial-letter property
  • OOAD | wip
  • work on a patch that doing the baseline alignment thing inside nsBlockFrame::ReflowInlineFrame()
  • will ask for feedback once the patch is ready

Layout

• bug 1313083 - Repair line layout debug flags in nsLineLayout
  • file | land

Shing Lyu

Stylo:

• drop-shadow filter CSS parser in Servo

Servo:

• Implementing flexbox auto min-size
• Fixed Servo performance test using headless mode

Tingyu Lin

shape-outside

• bug 1309467 - Implement the rendering of <shape-box> value for CSS shape-outside
  • Addressing reviewer's comments.

AccessibleCaret

• bug 1310509 - Opening new tab when text is selected does not properly dismiss the selection menu
  • r?

Layout

• bug 1313083 - Repair line layout debug flags in nsLineLayout
  • review
• bug 1313362 - Convert nsGenConList to use mozilla::LinkedList
  • file | land

Servo

• Implement keyword values in border-width #13869
  • land

Tommy Kuo

• bug 1305351 - Add ChromeCastDisplay for Presentation API.
  • land
• bug 1285870 - Show custom presentation on Chromecast.
  • land
• bug 1313601 - Disable presentation mode when taps disconnect from Chromecast on Android Notification Bar.
  • file | r?
• bug 1295087 - [Presentation WebAPI] Implement PresentationDeviceProvider for Chromecast devices on Fennec.
  • r+ by :schien | r- by :smaug
  • write test case
• bug 1305352 - [Presentation API] Implement PresentationRequestUIGlue for Fennec.
  • r-

Rule of Thumb

Status code:

• OOAD: In analysis or design phase.
• WIP: working on WIP patch.
• feedback(f): feedback a patch
• feedback?(f?): WIP is ready for feedback grant.
• feedback+(f+): Get positive feedback.
• feedback-(f-): Get negative feedback.
• review(r): review a patch.
• review?(r?): patch is under review
• review+(r+): get review+ and ready to check-in
• review-(r-): get negative review feedback. Fix problems and switch back to r? again.
• land: patch check in.
• debug: Figure out the root cause of a bug/ try to repro that bug, etc...
• file: file a bug
• PTO: take personal leave
• interview(I)
• study: Spend time on domain knowledge study, such as multimedia transport technology, compress format, real-time rendering.
• presentation
  • Write a personal blog, which is relative to work, such as articles on MozTech, or what you experience in a work week.
  • WIKI/ MDN article writing.
  • If you write a technical blog or anything relative to work, it's welcome to share it on weekly report.
  • Do a presentation of technical topics

Status code combination rule
Combine status code with "|". For example

** OOAD|WIP

Which means you work on both OOAD and WIP in the last week.

land status code
If your patch is target for specific branch, instead of central, postfix branch version. For exp

** land(1.4)