TPE-Layout/2017-01-17

Weekly Announcement

• Layout Standup
  • Daily to-dos/status recorder and tracker to help you back trace in past weeks or months.
  • Initial coverage: WFH / People who is unable to attend daily standup. Encourage for everybody.

Weekly Sharing

• Rust
  • auto-dereferencing rules.

Weekly Status

CJ Ku

• bug 1325865 - SVG: Percentage length values in mask elements are calculated incorrectly
  • land
• bug 1325038 - Hit NS_ASSERTION in PreEffectsVisualOverflowCollector::GetPreEffectsVisualOverflowRect
  • land
• bug 872483 - reftest text-svgglyphs/svg-glyph-mask.svg fails on all platforms
  • r?
• bug 537623 - mask/clip-path/filter attribute on <use> element uses wrong coordinate space
  • WIP

Jeremy Chen

CSS Text Module Level 3

• bug 1316482 - Whitespace around the segment break should be removed before segment break transformation
  • Backout due to test failures on Win7 debug
  • re-land with skipping tests on Win7 debug for now
  • keep monitoring the intermittent bug 1300355
• bug 1056516 - presence of explicit soft-hyphen in a word should override automatic hyphenation
  • OOAD | wip

Other

• bug 1326189 - [css-ui] caret-color on visited links
  • r?

Review

• bug 1311244 - Implement the rendering of basic shape circle() for CSS shape-outside
• bug 1330630 - Intermittent 315920-26.html == http://10.0.2.2:8854/tests/layout/reftests/bugs/315920-26-ref.html | image comparison, max difference: 229, number of differing pixels: 46

Shing Lyu

Stylo

• Make Servo performance test run on Stylo
• Add custom timing test in Servo performance test for Stylo
• Re-running reftest

Servo

• Implemented display:inline-flex

Evangelism

• Coordinate NTU school project

Tingyu Lin

shape-outside

• bug 1311244 - Implement the rendering of basic shape circle() for CSS shape-outside
  • land
• bug 1330630 - Intermittent 315920-26.html == http://10.0.2.2:8854/tests/layout/reftests/bugs/315920-26-ref.html | image comparison, max difference: 229, number of differing pixels: 46
  • 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