TPE-Layout/2016-08-30

From MozillaWiki
Jump to: navigation, search

Weekly Report

Astley Chen

CSS Positioned Mask

  • bug 1294660 - Enable mask-* longhand properties support on nightly & aurora channels.
    • landed - need follow up to disable relevant test cases on beta and after.

CSSOM

  • bug 1292447 - Shrink the number of properties which return used value as resolved value for getComputedStyle
    • r+ - working on resolving try failure and update relevant mochitest.

PDF Printing

  • Address possible bugs.

Boris Chiou

  • Web Animations API
    • bug 1049975 - Make Animation.effect writeable
      • land
    • bug 1298742 - Don't call CanThrottle if the effect is not included in EffectSet
      • r?
    • bug 1298739 - UpdateRelevence() in if (aEffect) block in Animation::SetEffectNoUpdate() should be called before SetAnimation()
      • debug
    • bug 1292001 - Make sure reversing an exist transition still works if the effect is removed/replaced
      • debug
  • SVG
    • bug 1245751 - Allow href attribute without xlink on SVG elements
      • r-

Cameron McCormack

CJ Ku

  • bug 1190881 - SVG CSS animation not working through img tag
    • r?
  • bug 1289011 - Handle clip-path: fill-box | stroke-box | view-box correctly
    • r?

Jeremy Chen

Initial-letter

  • bug 1296561 - Size initial letters as CSS Inline Layout specification says
    • r+ | land

layout code refactoring

  • bug 1297306 - Convert NS_STYLE_CLEAR_* to an enum class
    • r+ | r?
    • wait for review results from dom peer and mfbt peer

Other

  • bug 1298756 - [WebIDL] Cannot specify type for enum class
    • file

Shing Lyu

Stylo

  • Reftest patch

Servo

  • COSCUP talk
  • Modern Web Conf talk
  • Mentor new contributor at Taiwan Code Sprint

Tingyu Lin

CSS & Layout

  • bug 1291110 - The inline element overlaps with the float elements
    • r?
  • bug 1297097 - Remove preference "layout.css.vertical-text.enabled"
    • r+
    • backout due to other issues
  • bug 1297333 - Enable "layout.css.clip-path-shapes.enabled" in test suite
    • r-
  • bug 1297340 - Revise clip-path parsing function ParseClipPath()
    • file | land
  • bug 1295084 - Move nsStyleImageLayers::Position to a different scope
    • file | land

AccessibleCaret

  • bug 1298704 - Crash in nsBlockFrame::GetMinISize - stack overflow caused by AccessibleCaretEventHub::Reflow callback
    • land

Other

  • bug 1297975 - "media.decoder.heuristic.dormant.timeout" in prefs_general.js is zero on all platforms
    • file

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)