TPE-Layout/2016-08-16

From MozillaWiki
Jump to: navigation, search

Weekly Report

Astley Chen

CSS Positioned Mask

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

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
      • OOAD
    • bug 1293106 - Ignore 'spacing' if dom.animations-api.core.enabled is false
      • land
  • SVG
    • bug 1245751 - Allow href attribute without xlink on SVG elements
      • r?

Cameron

Reviews

Stylo

  • bug 1288302 - implement support for background-image: url()
    • WIP

Other

CJ Ku

  • bug 1295095 - (mask-perf) [META] Paint css mask and clip-path on mask layer
    • bug 1295094 - Separating nsSVGIntegrationUtils::PaintFramesWithEffects into PaintMaskAndClipPath and PaintFilter
      • WIP
  • bug 1235015 - A mochitest to validate the return value of nsComputedDOMStyle::DoGetMask()
    • r+
  • bug 1289011 - Handle clip-path: fill-box | stroke-box correctly
    • anyone what to take this?
  • bug 1295084 - Move nsStyleImageLayers::Position to a different scope
    • anyone what to take this?

Jeremy Chen

Initial-letter

  • bug 1223880 - Implement initial-letter (from CSS Inline Layout Module Level 3)
    • OOAD
    • study line-height (computed value vs. used value)
  • bug 1293210 - add cap height support to nsFontMetrics
    • OOAD | wip
    • can get cap height on Mac (through gfxMacFont fall back machanism)
    • need a patch to get cap height from table look-up before fallback to platform specific codes

PTO

  • PTO 1 day

Shing Lyu

Stylo

  • Creating servo-vs-gecko reftest manifest

Servo

  • Fixed #12676: Absolutely positioned block box are collaping margin with its previous sibling
  • Identified #12824: A collapse thorugh block flow should have the collapsed margin on its top edge
  • Preparing COSCUP Talk(Aug 20-21) and Modern Web Taipei Talk (Aug 24-25)
  • Helping Taipei Code Sprint contributor to ramp up

Tingyu Lin

CSS

  • bug 1293604 - Replace NS_STYLE_FLOAT_* with StyleFloat enum class
    • file | land

AccessibleCaret

Layout

  • bug 1294628 - Replace int16_t BlockReflowInput::mFlag with bit fields struct
    • file | 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)