TPE-Layout/2016-12-27

Weekly Sharing

• Printing Flow -- Farmer Tseng
  • https://docs.google.com/presentation/d/1nrms2PB_1V9bv_SzFlTV_fYIoJRCWH1Irs-eaf0aYko/edit#slide=id.p

Weekly Status

Astley Chen

• bug 1323684 - Video plays only sound, no video on Firefox mobile Nightly
  • Resolved. JW Player layout issue which is able to be reproduced on Firefox, UC.

Boris Chiou

Stylo - Animations

• bug 1317209 - Stylo: Interpolate servo animation values using and add to the cascade
  • study

Layout

• bug 1274891 - convert nsRestyleHint and nsChangeHint from enum to enum class
  • WIP | debug

Animations

• bug 1323330 - test_animation_properties.html should run with the Web Animation API pref
  • review
• bug 1312301 - Refactor type-per-property.html to be used for other procedures (i.e. addition and accumulation)
  • review
• bug 1323114 - Assertion failure: TransformFunctionsMatch(nsStyleTransformMatrix::TransformFunctionOf(a1), nsStyleTransformMatrix::TransformFunctionOf(a2)) (transform function mismatch)
  • review
• bug 1311620 - Implement keyframe/effect composite(add)
  • review

Cameron McCormack

Stylo

• bug 1324624 - stylo: mark currently failing crashtests as asserts-if(stylo)
  • r+ | land
• bug 1324620 - stylo: fix tests marked as fails-if(stylo) and asserts-if(stylo,...)
  • file
  • and filed ~40 blocking bugs for tracking individual tests

CJ Ku

• bug 1325550 - 3.2 - 3.45% tresize (osx-10-10, windows7-32) regression on push
  • land
• bug 1311270 - Implement rendering of mask-clip/mask-origin: fill-box|stroke-box|view-box|no-clip values
  • land
• bug 1313276 - Draw clip-path-basic-shape onto mask layer
  • land
• bug 1324809 - CSS filter makes wrapped text disappear
  • land
• bug 1318266 - Draw trivial clip-path onto mask layer
  • land
• bug 1320032 - Assertion failure: !mSize.IsEmpty(), at FrameLayerBuilder.cpp:162
  • land

Farmer Tseng

• bug 1321496 - Convert PDF to EMF in Windows
  • clarify why SkiaPDF generated a large size file.
• PrintJob live cycle.
  • study
• Build Mortar Project.

Jeremy Chen

CSS Text Module Level 3

• bug 1316482 - Whitespace around the segment break should be removed before segment break transformation
  • f+
  • work on w3c-css tests

Shing Lyu

Stylo

• Studying how to run Stylo in Hasal
• Updating Stylo reftest expected result

Servo

• #14685 - Study display inline-flex
• Setting up new server for Servo performance test

Evangelism

• Rust intermediate tutorial for Moz TW community
• Rust internal tutorial for Mozilla Taipei office colleague

Tingyu Lin

• bug 1324713 - clip-path: circle() with large reference box and percentage radius does not render correctly
  • file | land
• bug 1325006 - Convert NS_RADIUS_* to an enum class
  • file | land
• bug 1311244 - Implement the rendering of basic shape circle() and ellipse() for CSS shape-outside
  • WIP
  • Implementing circle()

Tommy Kuo

• bug 1273244 - Add support for background-repeat-{x|y}
  • WIP

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)