From MozillaWiki
Jump to: navigation, search

Weekly Report

Astley Chen

CSS Masking - positioned mask

  • bug 1283679 - mask-composite: subtract is supported as misspelled 'substract' value.
    • r?(dbaron) - waiting for review.
  • bug 1284169 - assertion failed: did not list properties in nsCSSProperty order
    • r?(mshal|ted) - broken due to bug 1272488 & 1273579.
    • bug 1272488 - Move invocation to
    • bug 1273579 - Move invocation to
  • bug 1251161 - Ship mask-* longhand properties support
    • r?(dbaron) - pending due to above bugs.

Boris Chiou

  • mfbt
    • bug 1276573 - Add a new constructor accepting two RangedPtr<T> arguments
      • r?
  • Web Animations API
  • SVG
    • bug 1245751 - Allow href attribute without xlink on SVG elements
      • OOAD | WIP


  • bug 652991- SVG path fill rendering can break after window.history.pushState
    • prototyping
  • bug 1276834 - Draw nothing when any mask-image refer to an unresolvable URL
    • r?

Jeremy Chen

layout code refactoring

  • bug 906116 - replace NS_STYLE_HINT_FRAMECHANGE with nsChangeHint_ReconstructFrame, etc.
    • r?
    • follow up
      • bug 1077851 - audit style change hints to remove unnecessary bits


  • bug 1223880 - Implement initial-letter (from CSS Inline Layout Module Level 3)
    • check feedbacks from MozLondon and re-read the spec

Shing Lyu

Tingyu Lin

AccessibleCaret on Fennec

  • bug 1283828 - Crash in nsContentUtils::ComparePoints | mozilla::AccessibleCaretManager::RestrictCaretDraggingOffsets
    • r+
  • bug 1281917

Others (Provide information or comments, and triage bugs)

  • bug 1282029 - Remove unneeded access modifiers and comments in layout/
    • land


2016 Q3 Goals Draft

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


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)