TPE-Layout/2017-01-24

Weekly Announcement

Recap of January Manager Meeting:

• All TDC policies on mana would be translated to English in favor of foreign employees.
• Will have waterline and new espresso machines with same standard as global offices.
• Parental leave policy for Taipei would be announced on 2017/Feb.
• Biz trip booking through Egencia will be enabled in 2017Q2. (paid by TPE office credit card for insurance cover)

Weekly Sharing

• Astley Chen
  • Quantum DOM and Event Labeling.
• Ting-Yu Lin
  • UniquePtr and Move usage.

Weekly Status

Astley Chen

• bug 1331296 - Remove support for -moz-calc()
  • land(FF53)
• bug 1321812 - DocGroup labeling
  • WIP
• A lot of meetings and management stuff.

Boris Chiou

Stylo Animations

• bug 1332633 - stylo: Implement ComputeDistance for AnimationValues
  • file
• bug 1332657 - stylo: Make transform animatable
  • file
• bug 1333310 - stylo: Refactor AnimationRule
  • file
• bug 1317209 - Stylo: Interpolate servo animation values using and add to the cascade
  • r+ | wip
• PR to update selectors
  • land

Cameron McCormack

Stylo

• bug 1331294 - stylo: value from getComputedStyle is not updated after selector matching changes
  • r+ (but getting test failures)

Stylo reviews

• bug 1331581 - Clean up nsMediaFeatureValueGetter
• bug 1328420 - stylo: implement child stylesheets
• bug 1332193 - Subproperties of border-image does not update when the the value they inherit from changes
• bug 1331903 - -moz-binding is hiding test issues and browser bugs
• bug 1332353 - Introduce more clarity into whether a stylesheet is owned by a document
• bug 851892 - Move CSS Rules to WebIDL
• bug 1328787 - Stylo: Convert Servo's animation keyframes and store them into Gecko's keyframes
• bug 1330172 - Property value with variable reference is not idempotent with assigning cssText
• bug 1317209 - Stylo: Interpolate servo animation values using and add to the cascade
• bug 1332704 - figure out what's going on with our sheet/document pointer in devtools tests
• bug 1333001 - Assertion failure: value, at BindingUtils.h:900

Other reviews

• bug 1331850 - Swap ClosestSide and FarthestSide in StyleShapeRadius.
• bug 1332550

CJ Ku

• bug 1224207 - Scaling SVG or text with filters should not blur
  • WIP

Farmer Tseng

• Mortar Printing
  • presentation - Do a presentation to Mark Mayo, Jet, Joe Hildebrand, and David Bryant.
• Feasibility study
  • Print Bitmap in Windows, unfortunately, one A4 page bitmap is around 139MB.
  • Get EMF data and save to file.
  • Validate if could create EMF DC in content process(sandboxed).
• bug 1325032 - [Discussion] The way to integrate PDFium library
  • Provide printing proposals.

Jeremy Chen

CSS Text Module Level 3

• bug 1056516 - presence of explicit soft-hyphen in a word should override automatic hyphenation
  • wip

Other

• bug 1326189 - [css-ui] caret-color on visited links
  • landed by xidorn

Shing Lyu

Stylo

• Bug 1330550 - Debug Talos crashes
• Bug 1331552 - Stylo real page diffing
• #15067 - Run stylo in servo's performance test

Servo

• #14978 - Landed the display inline-flex
• #15149 - Finding next issue to work on in flexbox

Evangelism

• Contact NTU professor for collaboration plan
• Collect list of tech conference in Asia (https://asia-confs.github.io/asia-tech-confs)

Tingyu Lin

• bug 1331850 - Swap ClosestSide and FarthestSide in StyleShapeRadius.
  • file | land
• bug 1331846 - Error when running layout/reftests/w3c-css/submitted/check-for-references.sh
  • file | land
• bug 1332180 - Assertion failure with shape-outside: url() with float
  • review
• bug 1326406 - Implement the rendering of basic shape ellipse() for CSS shape-outside
  • r?
• bug 1332105 - Convert SelectionDetails to use UniquePtr
  • land

Tommy Kuo

Layout

• bug 1229979 - The shadow of decoration line is draw in a different color when the shadow blur is not zero
  • spec discussion
• bug 1229982 - Shadow of decoration is not drawn when shadow blur is applied if text is higher than the baseline
  • debug
• Read some material (spec/articles) for grid, flex-box and multi-column.

BMO

• bug 1331302 - Restrict the max-width of the textarea of "add comment".
  • land

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