TPE-Layout/2017-01-03

Weekly Sharing

• Bugzilla Socorro Lens
  • Example: bug 1324908

Weekly Status

Astley Chen

• bug 1324908 - [e10s] OS X printing related crashes in CoreGraphics@0x regressing in Firefox 51
  • STR found - regression patch identified by Farmer. ni? Bob Owen to follow up the fix.

Boris Chiou

Stylo - Animations

• bug 1317209 - Stylo: Interpolate servo animation values using and add to the cascade
  • study

Layout

• bug 1274891 - convert nsRestyleHint and nsChangeHint from enum to enum class
  • r?

Animations

• bug 1134538 - Assertion failure ('Infinity + -Infinity' and '-Infinity + Infinity' are undefined) with CSS transition
  • debug
  • Know the root cause, but no idea to fix this now

CJ Ku

• bug 1325849 - The text does not appear inside the button with HWA enabled
  • land
• bug 1325550 - 3.2 - 3.45% tresize (osx-10-10, windows7-32) regression on push
  • land
• bug 1324809 - CSS filter makes wrapped text disappeard
  • land
• bug 1324174 - Simplify context management logic in nsSVGIntegrationUtils::PaintMask
  • land

Farmer Tseng

• bug 1324908 - [e10s] OS X printing related crashes in CoreGraphics@0x regressing in Firefox 51
  • debug
  • clarify regression patch
• Create a XPCOM interface to show print dialog with print session.
  • WIP

Jeremy Chen

CSS Text Module Level 3

• bug 1316482 - Whitespace around the segment break should be removed before segment break transformation
  • f+ for patch set and tests
  • prepare patch sets for review

Shing Lyu

Stylo

• Rerun and triaging reftest intermittents
• Making Stylo run in Hasal

Servo

• #14685 - Implementing diaplay:inline-flex
• Setting up server for Servo and Stylo's tests

Tingyu Lin

• bug 1311244 - Implement the rendering of basic shape circle() for CSS shape-outside
  • r?
• bug 1326406 - Implement the rendering of basic shape ellipse() for CSS shape-outside
  • file
• bug 1326407 - Implement the rendering of basic shape inset() for CSS shape-outside
  • file
• bug 1326409 - Implement the rendering of basic shape polygon() for CSS shape-outside
  • file

Tommy Kuo

• bug 1273244 - Add support for {background|mask}-repeat-{x|y}
  • WIP
• bug 1229979 - The shadow of decoration line is draw in a different color when the shadow blur is not zero
  • debug

Weekly Announcement

• 2017 Global Wellness Program
  • Coverage is identical to 2016. Health, training, exercise.
  • Reimbursement via Expensify. Approve by HR directly.

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)