TPE-Layout/2016-08-09

From MozillaWiki
Jump to: navigation, search

Weekly Report

Astley Chen

Management

Unprefix

  • bug 1270406 - remove pseudo class :-moz-dir after :dir is shipped
  • bug 1226116 - Unprefix -moz-text-size-adjust

Boris Chiou

  • Web Animations API
    • bug 1049975 - Make Animation.effect writeable
      • r+ | r-
    • bug 1293106 - Ignore 'spacing' if dom.animations-api.core.enabled is false
      • OOAD
  • SVG
    • bug 1245751 - Allow href attribute without xlink on SVG elements
      • WIP
      • still have to write some mochitests for animate elements.

CJ Ku

  • bug 1234485 - (mask-image) Compose mask at compositor.
    • prototyping
  • bug 1288812 - When serializing a url() with the local url flag set, it must serialize as just the fragment
    • land
  • bug 1291280 - Declare nsStyleImageLayers::Layer::mSourceURI as FragmentOrURI
    • land
  • bug 1269990 - mochitests for clip-path basic shapes trigger fatal "Assertion failure: aArray1->Item(1).GetIntValue() == aArray2->Item(1).GetIntValue() (expected matching geometry-box values)
    • r?

Jeremy Chen

Initial-letter

  • bug 1223880 - Implement initial-letter (from CSS Inline Layout Module Level 3)
    • hack for size argument of initial-letter
    • uppercase vs. lowercase
  • bug 1293210 - add cap height support to nsFontMetrics
    • file | study

Shing Lyu

Stylo:

  • Reftest runner for stylo vs. non-stylo test

Servo:

 * gdb
 * rr
 * flow tree dump
 * logging

Tingyu Lin

Shape-outside

  • bug 1288626 - Add shape-outside support to the style system
    • file | land

Layout

  • bug 1291110 - The inline element overlaps with the float elements
    • file

AccessibleCaret

  • bug 1195722 - Improve text selection with touch on Windows (turn on accessible touch caret)
    • review
  • bug 1292904 - Can't double click to select text in text fields when AccessibleCaret is enabled
    • comment

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)