TPE-Layout/2016-08-30
From MozillaWiki
Contents
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
- bug 1049975 - Make Animation.effect writeable
- SVG
- bug 1245751 - Allow href attribute without xlink on SVG elements
- r-
- bug 1245751 - Allow href attribute without xlink on SVG elements
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)