TPE-Layout/2016-11-29

Weekly Status

Astley Chen

• bug 1270406 - remove pseudo class :-moz-dir after :dir is shipped
  • land
  • fxsitecompat
  • Firefox 53 Release Note
• bug 1276808 - Remove prefixed -moz-text-align-last
  • r+

Boris Chiou

Stylo - Animations

• bug 1317178 - Stylo: Use to_css_single_value in Servo_DeclarationBlock_SerializeOneValue
  • land
• bug 1317179 - Stylo: Use XPCOM string bindings instead of Gecko_Utf8SliceToString
  • r+
• bug 1317208 - Stylo: Store servo computed values for animation properties
  • study

CJ Ku

• bug 1311270 - Implement rendering of mask-clip/mask-origin: fill-box|stroke-box|view-box|no-clip values
  • r?
• bug 1320364 - Fix "ASSERTION: How did we getting here, then?: 'aFrame->GetParent()->StyleContext()->GetPseudo() == nsCSSAnonBoxes::mozAnonymousBlock"
  • file

Farmer Tseng

• convert pdf to emf
  • WIP

Jeremy Chen

Initial-letter

• pause, reprioritizing

WebCompat

• bug 1319699 - Correct webkit-text-stoke and webkit-text-fill-color pages on MDN
  • file | r+ | merge
  • PR to MDN data repo.
• {whatwg/compat} Update -webkit-text-fill-color and -webkit-text-stroke
  • file

Layout

• bug 1320239 - Use nscoord for some nsStyleCoord in style structs that only carry nscoord
  • file | r?

Servo/Stylo

• stylo - implement -webkit-text-fill-color and -webkit-text-stroke
  • file | r+ | merge

Other

• bug 1319383 - Do not check watchman extension if Mercurial version >= 3.8
  • file | r?

Shing Lyu

Stylo

• #14359 - Refactor drop-shadow filter
• Implementing URL filter

Servo

• #14332 - Added an "auto" keyword for flexbox's min-size
• Addressing review comments

Evangelism

• Rust beginner tutorial planning
• Study Rust book

Tingyu Lin

shape-outside

• bug 1316549 - Fix <shape-box> value with border-radius for non-ltr direction and writing-mode
  • land

layout

• bug 1317588 - Clean up definition and macros related to mozilla::Side
  • land
• bug 1319672 - Improve document related to writing-mode in nsFloatManager
  • r+
• bug 1320014 - Clean up definition and macros related to mozilla::css::Corner
  • file
• bug 1304598 - Drop "ns" prefix in file name to make it consistent with the class name
  • r+
• bug 1311270 - Implement rendering of mask-clip/mask-origin: fill-box|stroke-box|view-box|no-clip values
  • review

AccessibleCaret

• bug 1156160 - Blur event closes AccessibleCaret but leaves visible selection on page
  • 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)