TPE-Layout/2016-11-08

Team Affairs

• David Baron visit from 11/7 ~ 11/11

Individual Contributor

Cameron McCormack

Stylo

• bug 1288302 - Stylo: implement support for background-image: url()
  • r+ | land
• bug 1310560 - stylo: support url() values in cursor
  • r+
• bug 1315155 - use nsStyleImageRequest in nsStyleContentData
  • r+
• working on mozilla-central -> incubator/stylo merge, which lead to the following bug:
• bug 1315632 - stylo: leave old style on frames when nsChangeHint_ReconstructFrame was generated
  • r+ | land

Reviews

• bug 1294299 - Stylo: Implement CSSStyleDeclaration for Element.style
  • r
• bug 1303623 - Parsing mask-clip/mask-origin: fill-box|stroke-box|view-box|no-clip values
  • r

CJ Ku

• bug 1234485 - Draw image mask onto mask layer
  • r?
• bug 1305253 - In nsSVGIntegrationUtils::PaintMaskAndClipPath, manually pop up clip-path
  • r+
• bug 1310575 - Intermittent file:///C:/slave/test/build/tests/reftest/tests/layout/reftests/bugs/621253-2-externalFilter.html
  • land
• bug 1310575 - Change the parameter of ValidateSVGFrame in nsSVGIntegrationUtils.cpp
  • land
• bug 1313898 - Draw clip-path onto mask layer
  • WIP
• bug 1313877 - Draw SVG mask onto mask layer
  • WIP
• bug 1314001 - Implement pref:layers.paintMask
  • r+

Farmer Tseng

• Mortar would like to generate emf file for Windows printing
  • Write a sample code to print emf file in Windows.
• Mortar would like to generate PDF for Linux/MacOS printing
  • Study Mac OS x API and Write a sample code to print PDF in Mac OS

Jeremy Chen

Initial-letter

• bug 1310106 - Implement the rendering of floating :first-letter with CSS initial-letter property
  • r?
  • 1:1 duscussion with dbaron
  • specification issues filed accordingly:
    • {css-inline} fill out initial-letter/float interaction section
    • {css-inline} spec should define behavior when 'initial-letter' and 'float' both set on same element/pseudo-element
    • {css-inline} should specify that line with initial letter is shortened relative to all floats initial letter can interact with

Shing Lyu

Stylo:

• Submitted drop-shadow patch for review

Servo

• Try to write the gfx part for drop-shadow filter, traced code
• Write and tested basic path (content size only) case for flexbox min-size:auto
• Submitted spec bug https://github.com/w3c/csswg-drafts/issues/671
• Various patch fixes and rebases

Evangelism:

• Write the reference solution for rust-tutorials.com exercises (https://github.com/shinglyu/rust-tutorial-reference-solution)
• Arranged a mentor training session for the upcoming Rust workshop in Taipei (Nov 15)

Tingyu Lin

shape-outside

• bug 1291110 - The inline element overlaps with the float elements
  • r+
  • Need to address reviewer's comments.
• bug 1309467 - Implement the rendering of <shape-box> value for CSS shape-outside
  • r?
  • Addressed reviewer's comments.
• Discuss spec issue related to margin-box https://github.com/w3c/csswg-drafts/issues/675

AccessibleCaret

• bug 1310509 - Opening new tab when text is selected does not properly dismiss the selection menu
  • land

MFBT

• bug 1315624 - Delete operator|| and operator&& for CastableTypedEnumResult
  • r?

Tommy Kuo

• bug 1313601 - Disable presentation mode when taps disconnect from Chromecast on Android Notification Bar.
  • land
• bug 1295087 - [Presentation WebAPI] Implement PresentationDeviceProvider for Chromecast devices on Fennec.
  • r+
  • Write test case.
• bug 1305352 - [Presentation API] Implement PresentationRequestUIGlue for Fennec.
  • r+

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)