TPE-Layout/2016-11-22

Weekly Sharing

Astley Chen

• mozconfigwrapper to manager your config variants.
• How to create an Aurora build ?
  • Upon mozilla-central - Aurora Simulation
  • Upon mozilla-aurora - Add "ac_add_options --with-branding=browser/branding/aurora" into mozconfig file.
• How to create a Beta build ?
  • Upon mozilla-central - Beta Simulation
  • Upon mozilla-beta - Add "ac_add_options --enable-official-branding" into mozconfig file.

Cameron McCormack

• Leak hunting with rr

Weekly Status

Astley Chen

• bug 1303654 - Enable basic shapes clip-path clipping on Nightly & Aurora.
  • land
• bug 1251161 - Ship CSS positioned mask support on beta & release channels
  • land

Boris Chiou

Web Animations API

• bug 1286151 - Support paced spacing for filter property
  • land
• bug 1318591 - Distance calculation for transform lists should use distance of transformed point
  • OOAD
  • Discussing with Brian and other spec editors
• bug 1317914 - Ship keyframes paced spacing
  • file

Stylo - Animations

• bug 1317178 - Stylo: Use to_css_single_value in Servo_DeclarationBlock_SerializeOneValue
  • WIP
• bug 1317179 - Stylo: Use XPCOM string bindings instead of Gecko_Utf8SliceToString
  • study

Animations

• bug 1319072 - Assertion failure: aCoeff >= 0.0 && aCoeff <= 1.0 (Coefficient should be in the range [0.0, 1.0]), atnsStyleTransformMatrix.cpp:332
  • review

Cameron McCormack

Stylo

• bug 1318238 - clear ServoNodeData from the entire document when the ServoStyleSet is shutting down
  • land
• bug 1310560 - stylo: support url() values in cursor
  • land
• more mozilla-central -> incubator/stylo merges

CJ Ku

• bug 1313898 - Draw SVG mask/clip-path onto mask layer
  • land
• bug 1318418 - Eliminate one extra copy in nsSVGIntegrationUtils::PaintMask when we have both mask and clip-path
  • file
• bug 1318266 - Draw trivial clip-path onto mask layer
  • file

Farmer Tseng

• chromium printing(from window.print and UI)
  • study
• Setup debug environment of chromium in Windows and Mac OS, and learn how to set break point in windows. (Done)

Jeremy Chen

Initial-letter

• bug 1310106 - Implement the rendering of floating :first-letter with CSS initial-letter property
  • floating rendering implementation
    • r-, parts of the code might turn out useful later
  • non-floating rendering implementation
    • bug 1318261 - Let :first-letter with initial-letter property has its own line
      • file | study
  • specification issues:
    • new
      • {css-inline} initial-letter should be clearer about how it makes its block taller
      • {css-inline} initial-letter should be clearer about how it sinks
      • {css-inline} Is initial-letter in-flow or out-of-flow?
    • pending
      • {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

Layout

• bug 1295788 - Remove layout.css.display-contents.enabled pref
  • r+ | land

Servo/Stylo

• Stylo - gecko glue code for font-size-adjust
  • r+ | merged

Shing Lyu

Stylo

• #14218 - Landed the drop-shadow filter for Stylo

Servo

• #12610 - Try fixing the getComputedStyle output for flexbox's min-size
• #14147 - Updating patche for Servo performance test
• #14194 - Updating patch for Gecko performance test

Evangelism

• Rust beginner tutorial for Moz TW community
• Study The Rust book

Tingyu Lin

shape-outside

• bug 1309467 - Implement the rendering of <shape-box> value for CSS shape-outside
  • Push a followup patch to fix spec URL in the reftests.
• bug 1316549 - Fix <shape-box> value with border-radius for non-ltr direction and writing-mode
  • r+
  • Addressing reviewer's comments.

layout

• bug 1317588 - Clean up definition and macros related to mozilla::Side
  • r?

details & summary

• bug 1271549 - Remove preference "dom.details_element.enabled" for <details> and <summary>
  • land

Tommy Kuo

• bug 1273244 - Add support for background-repeat-x/y
  • Invest SPEC and Try on Google Chrome
  • WIP

Team Affairs

• Training budget for buying technical books.
  • Company Tax ID included on your purchases.
  • Expensify with annotation of using training budget.
  • Public books management
• Hawaii All-hands
  • Per diem for 12/7~8 dinner is USD60/night. Policy is no change.
  • Expense reimbursement by 2/20.
• General expense reimbursement by 2016/12/20. (Global wellness due is 12/31)

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)