TPE-Layout/2016-05-24

Weekly Report

Astley Chen

Offsite layout meeting

• Platform rendering sessions
  • CSS Shapes implementation
  • CSS initial-letter implementation
• 2016H2 planning
  • Layout Backlog

CSS Masking - positioned mask

• bug 1258286 - default value for mask-origin property is border-box but padding-box
  • WIP

Selector4 - unprefix :dir

• bug 1270713 - Enable corresponding wpt tests for pseudo class :dir()
  • land

Boris Chiou

• bug 1244590 - Implement KeyframeEffect spacing modes
  • r?

• bug 1274944 - Implement writable keyframeEffect spacing
  • file

• bug 1272475 - Secure bug (web animations)
  • debug

CJ Ku

• bug 1273068 - background-clip:text left ghost image after text animated
  • landed
• bug 1272970 - test_transitions_per_property.html failure after enable MOZ_ENABLE_MASK_AS_SHORTHAND
  • landed
• bug 1228280 - (mask-image) Support multiple SVG mask elements
  • landed

Jeremy Chen

Initial-letter

• bug 1223880 - Implement initial-letter (from CSS Inline Layout Module Level 3)
  • parsing & computing - 100%
  • rendering - OOAD | WIP
    • study first-letter rendering
    • study CSS Level 2 Spec. - Visual formatting model details

WebCompat

• bug 1275093 - Various webkit-text-stroke-property reftests are going to permafail when Gecko 48 merges to Beta
  • fixed by dholbert

Others

• bug 1273766 - Purge NS_*Hint inline functions for nsChangeHints
  • file | land
• bug 1274891 - convert nsRestyleHint and nsChangeHint from enum to enum class
  • file

Tingyu Lin

Shape-outside

• bug 1098939 - Implement CSS shape-outside property
  • Study 'float' layout, and add parser support.
  • WIP
• bug 1274835 - Extract functions processing basic shapes from clip-path logic
  • file | land

Others (Provide information or comments, and triage bugs)

• bug 1272831 - Add AutoRestore::Value() to retrieve the saved value
  • land

Team

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)