TPE-Layout/2016-08-23

Weekly Report

Astley Chen

CSS Positioned Mask

• bug 1294660 - Enable mask-* longhand properties support on nightly & aurora channels.
  • r? - glandium

CSSOM

• bug 1292447 - Shrink the number of properties which return used value as resolved value for getComputedStyle
  • WIP

Boris Chiou

• Web Animations API
  • bug 1049975 - Make Animation.effect writeable
    • r+ | OOAD
    • still need to check some restyle problems.
  • bug 1290535 - Assertion failure: pacedValues[propIdx].mProperty == prop (Property mismatch), at src/dom/animation/KeyframeUtils.cpp:1557
    • land
• SVG
  • bug 1245751 - Allow href attribute without xlink on SVG elements
    • r?
    • Finish href/xlink:href on <script> element

Cameron McCormack

Reviews

• bug 1273706 - implement CSS Properties and Values API
  • r
• others

PTO

• 1 day

CJ Ku

• bug 1190881 - SVG CSS animation not working through img tag
  • WIP
• bug 1295094 - Separating nsSVGIntegrationUtils::PaintFramesWithEffects into PaintMaskAndClipPath and PaintFilter
  • WIP

Jeremy Chen

Initial-letter

• [TODO]
  • need a separated linelayout for initial letters
  • use TIGHT_HINTED_OUTLINE_EXTENTS for non-floating initial-letter as well?
• bug 1293210 - add cap height support to nsFontMetrics
  • r+ | land
• bug 1296561 - Size initial letters as CSS Inline Layout specification says
  • wip | r?
  • size calculation is much more correct with cap height support

• Bug 161070 - initial letters should respect font style setting
  • file WebKit bug

layout code refactoring

• bug 1296577 - Give nscoord_MAX a better definition to avoid potential integer overflowing
  • file
• bug 1297306 - Convert NS_STYLE_CLEAR_* to an enum class
  • file

Shing Lyu

Stylo

• Rewrite the script that generates clean stylo-gecko reftest.list

Servo

• Landed #12676: Absolutely positioned block box are collapsing margin with its previous sibling
• Fix #12824: A collapse through block flow should have the collapsed margin on its top edge
• COSCUP Talk(Aug 20-21)
• Flow Tree JSON dump & visuliazer: #12675: Pretty print the flow tree
• Help Taipei Code Sprint contributor to ramp up

Tingyu Lin

AccessibleCaret

• bug 1292904 - Can't double click to select text in text fields when AccessibleCaret is enabled
  • land
• bug 1296990 - Make AppearanceString() return nsAutoString directly
  • file | land

Layout

• bug 1291110 - The inline element overlaps with the float elements
  • r?
• bug 1297097 - Remove preference "layout.css.vertical-text.enabled"
  • file

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)