TPE-Layout/2016-04-28

From MozillaWiki
Jump to: navigation, search

Weekly Report

Astley Chen

CSS Masking - positioned mask

  • bug 1258286 - default value for mask-origin property is border-box but padding-box
  • bug 1258623 - default value for mask-repeat property is no-repeat but repeat
    • WIP

Selector4 - unprefix :dir

  • bug 859301 - Unprefix css4 :dir
    • WIP - add more {JS|Reftest} test cases into WPT.

Boris Chiou

Web Animations API

CSS Animations (Performance)

  • bug 1231471 - Re-use ComputedTimingFunction objects representing cubic-bezier keyword values
    • study

CSS

  • bug 1250820 - Use scoped enum for nsCSSPseudoClasses::Type
    • r?

CJ

  • bug 1265715 - background-clip:text and transform don't interact correctly
    • land
  • bug 1266948 - text-decoration-color: currentcolor should not use value from -webkit-text-fill-color
    • land
  • bug 652991 - SVG path fill rendering can break after window.history.pushState
    • OOAD
  • bug 1267312 - A mochitest to verify that when text-decoration-color is currentColor...
    • file
  • bug 1267209 - Converting the existing flags parameter of nsLayoutUtils::PaintFrame to an enum class
    • file
    • TY land
  • bug 1266636 - Provide a function, in debug version, to dump gfxContext::Path
    • file

Jeremy Chen

  • bug 1248708 - Implement -webkit-text-stroke
    • land on FF48 \o/ \o/ \o/
    • followups
      • bug 1266743 - Stroking of the HTML text doesn't respect a mitre limit at sharp joins, whereas SVG text does
        • file
      • bug 1266752 - Extend stroking HTML text to support paint-order (like SVG text does)
        • file
  • bug 1267128 - -webkit-background-clip: text does not respect -webkit-text-stroke property
    • OOAD | WIP
      • patch
        • already have a patch for skia back-end
        • need to deal w/ cairo back-end (try shows cairo is used in this code path on Win/Linux)
      • testcase
        • already add a fragile test
        • need to enhance the robustness

Tingyu Lin

Details & Summary

  • Discuss removing the hack for summary being a list-item in www-style mailing list.

AccessibleCaret

  • bug 1262755 - Enlarge accessible carets touch area for Fennec
    • land
  • bug 1249201 - Continue showing AccessibleCaret when scrolling or panning the page
    • land
  • bug 1265750 - Some word cannot be selected via long pressing
    • review
  • bug 1266715 - Fix the space between the blinking cursor/selection highlight and the carets
    • land
  • bug 1266922 - Investigate whether to remove "visibilitychange" from ActionBarHandler.js
    • file
  • bug 1236434 - Reduce delay in hiding AccessibleCarets during zoom/scroll
    • close as WORKSFORME.
  • bug 1267929 - crash in nsRange::CloneRange
    • land
  • bug 1266782 - Update ExtendPhoneNumberSelection() method in AccessibleCaretManager for strong refs to doc, sel
    • r+
    • Need to address comments.

Others (Provide information or comments, and triage bugs)

  • bug 1267524 - Use class member initializer list for style structs
    • file | r?
  • bug 1267209 - Converting the existing flags parameter of nsLayoutUtils::PaintFrame to an enum class
    • land

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)