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

Cameron McCormack

Stylo

• bug 1319568 - stylo: `firefox -marionette` crashes at startup due to MOZ_CRASH("stylo: ServoStyleSets should not support XUL tree styles yet")
  • r+ | land
• bug 1293809 - stylo: Don't apply anything other than UA rules to Native Anonymous Content
  • r+ | land
• bug 1319982 - move stop-gap Servo_GetStyleVariables definition to Servo
  • r+ | land
• bug 1320000 - XBLChildrenElement::ParseAttribute doesn't delegate to its superclass
  • r?
• bug 1320423 - Crash in nsFrame::FillCursorInformationFromStyle
  • r+ | land
• Servo #14340 - implement StyleSheet.disabled
  • r+ | land

Other

• bug 1320036 - filter region computed incorrectly when targetting outer <svg> element in an HTML document
  • file

Reviews

• bug 1307357 - stylo: Implement access to CSSStyleRule
• bug 1313293 - stylo: Implement CSSStyleSheet.insertRule and deleteRule
• bug 943918 - tab-size should accept <length> values
• bug 1239100 - Implement SVGGeometryElement interface
• bug 879119 - Drop support for -moz-prefixes from cursor: zoom-in | zoom-out | grab | grabbing
• bug 1303623 - Parsing mask-clip/mask-origin: fill-box|stroke-box|view-box|no-clip values

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

• Implementation: convert pdf to emf (Windows)
  • Bruce provide a PDFium dll, I plan to read a PDF file and convert to emf by dll.
  • Now, I could convert to emf but met resolution and position problem. (WIP)

Jeremy Chen

Initial-letter

• pause, reprioritizing

CSS Text Module Level 3

• bug 1316482 - Whitespace around the segment break should be removed before segment break transformation
  • study

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?

Tommy Kuo

• bug 1273244 - Add support for background-repeat-{x|y}
  • Investigate the feasibility and the SPEC. of the work.
  • WIP

Team Affairs

• Make sure your ESTA is still VALID. ESTA

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)