TPE-Layout/2016-10-25

Individual Contributor

Cameron McCormack

Stylo

• bug 1288302 - Stylo: implement support for background-image: url()
  • r+ | land
• bug 1311921 - 2.04 - 2.71% tp5o Main_RSS / tp5o Private Bytes (windows7-32, windowsxp) regression on push 6861f02dadfa1863af35086da6133ebc1ceaf845 (Thu Oct 20 2016)
  • debug
• bug 1310939 - pass a CARGOFLAGS through to cargo invocations
  • r+ | land
• servo/rust-bindgen #110 - Give vtables and anonymous items more stable generated names
  • r+ | land

Reviews

• bug 1311598 - Stylo: split ServoBindings.h into separate files to reduce compile time of adding binding functions
  • r
• bug 1303623 - Parsing mask-clip/mask-origin: fill-box|stroke-box|view-box|no-clip values
  • r

Farmer Tseng

Printing:

• Draw key flow of showing dialog, printing and preview.
• Mortar would like to generate emf file for Windows printing
  • Study how to print emf file in Windows.
• Mortar would like to generate PDF for Linux/MacOS printing
  • Study how to print pdf file in Linux.

Jeremy Chen

Initial-letter

• bug 1310106 - Implement the rendering of floating :first-letter with CSS initial-letter property
  • OOAD | wip

layout code refactoring

• bug 1286468 - Rename line related functions in nsBlockFrame to conform to Mozilla coding style
  • r+ | land
  • push a followup based on Mats and David's feedback

Shing Lyu

Stylo:

• Land CSS Filter

Servo:

• Enabled layout tracing for flexbox (formatting patch now)
• Studied flexbox auto min-{width,height} spec
• Flexbox: Add a Auto type to the min-{width,height} type

Evangelism:

• Talk at EE dept., National Taiwan University

Tingyu Lin

shape-outside

• bug 1309467 - Implement the rendering of <shape-box> value for CSS shape-outside
  • file | r?
• bug 1291110 - The inline element overlaps with the float elements
  • r-
  • NI dbaraon for more information.
• bug 1311244 - Implement the rendering of basic shape circle() and ellipse() for CSS shape-outside
  • file

layout

• bug 1309445 - Convert FrameConstructionItemList::mItems a mozilla::LinkedList
  • land
• bug 1286468 - Rename line related functions in nsBlockFrame to conform to Mozilla coding style
  • review

MFBT

• bug 1311277 - Add move assignment support to LinkedList
  • file | land

Tommy Kuo

Presentation API

• bug 1282003 - Make nsScreenManagerAndroid support multiple screens.
  • land
• bug 1295087 - [Presentation WebAPI] Implement PresentationDeviceProvider for Chromecast devices on Fennec.
  • review- | WIP
    • Fixing the failures reported by QA.
    • Designing the test case.
• bug 1305352 - [Presentation API] Implement PresentationRequestUIGlue for Fennec.
  • (Part 1) review- | WIP
    • Discussing with :smaug
  • (Part 2) review+

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)