TPE-Layout/2016-07-05

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: GenerateCSSPropsGenerated.py did not list properties in nsCSSProperty order
  • r?(mshal|ted) - broken due to bug 1272488 & 1273579.
  • bug 1272488 - Move GenerateCSSPropsGenerated.py invocation to moz.build
  • bug 1273579 - Move GenerateCSS2PropertiesWebIDL.py invocation to moz.build
• 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
  • bug 1283625 - Security bug
    • r+ | sec-approval?

• SVG
  • bug 1245751 - Allow href attribute without xlink on SVG elements
    • OOAD | WIP

CJ Ku

• 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

Initial-letter

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

Shing Lyu

• Talks
  • Prepare for COSCUP talk on Servo
  • Proposal for Modern Web Conf talk
  • Drawn a contributor map for Servo https://flic.kr/s/aHskDBNv66
• Servo Infra
  • Migrated servo-perf from Treeherder staging to production
  • Bug 10452 - Page Load Time Test Runner
    • WIP
• servo/servo
  • Rebased viewport percentage patch Bug 10104 - Only restyle nodes that uses viewport percentage units on viewport size change
  • Started bug #11302- Selection or insertion points renders incorrectly when inside ligature
  • Try building Firefox with Stylo

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

Team

2016 Q3 Goals Draft

• Draft Your Q3 Goals Here

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)