TPE-Layout/2016-11-08

Team Affairs

• David Baron visit from 11/7 ~ 11/11

Individual Contributor

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
  • land
• bug 1310575 - Change the parameter of ValidateSVGFrame in nsSVGIntegrationUtils.cpp
  • land
• bug 1313898 - Draw clip-path onto mask layer
  • WIP
• bug 1313877 - Draw SVG mask onto mask layer
  • WIP
• bug 1314001 - Implement pref:layers.paintMask
  • r+

Jeremy Chen

Initial-letter

• bug 1310106 - Implement the rendering of floating :first-letter with CSS initial-letter property
  • r?
  • 1:1 duscussion with dbaron
  • specification issues filed accordingly:
    • {css-inline} fill out initial-letter/float interaction section
    • {css-inline} spec should define behavior when 'initial-letter' and 'float' both set on same element/pseudo-element
    • {css-inline} should specify that line with initial letter is shortened relative to all floats initial letter can interact with

Shing Lyu

Stylo:

• Submitted drop-shadow patch for review

Servo

• Try to write the gfx part for drop-shadow filter, traced code
• Write and tested basic path (content size only) case for flexbox min-size:auto
• Submitted spec bug https://github.com/w3c/csswg-drafts/issues/671
• Various patch fixes and rebases

Evangelism:

• Write the reference solution for rust-tutorials.com exercises (https://github.com/shinglyu/rust-tutorial-reference-solution)
• Arranged a mentor training session for the upcoming Rust workshop in Taipei (Nov 15)

Tingyu Lin

shape-outside

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

AccessibleCaret

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

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)