TPE-Layout/2016-10-11

Weekly Report

Team

Project Mortar

• A replacement for pdf.js to better PDF rendering support and form interaction.
• To resolve blurry issue while printing PDF due to DPI issue.
• Revamp of printing experience on FireFox.

Reference:

• iThome
• bug 1302489 - [meta] Printing Improvements Triage List

Astley Chen

• bug 1308239 - Remove CSS pref layout.css.masking.enabled
  • land - pref control for SVG mask-type prop.
• bug 1271765 - Visual refresh of media controls
  • WIP - prepare patches for review.
• bug 1301638 - Using runtime pref to enable/ disable CSS mask properties
  • WIP

Boris Chiou

Web Animations API

• bug 1272549 - Support paced spacing for transform
  • OOAD | WIP
  • Figure out how to calculate the distance between two quaternion vectors.
  • Test cases

Cameron McCormack

• bug 1288302 - Stylo: implement support for background-image: url()
  • WIP

CJ Ku

• bug 1299715 - Replace ContainerItemType::eSVGEffects with eMask and eFilter
  • r?.

Farmer Tseng

Printing:

• Trace key flow from window.print() to print engine.
• Study how to show printing dialog.

Jeremy Chen

Initial-letter

• bug 1307402 - Use a more precise bounding box for initial letter texts
  • file | land

• 

  raised initial (before)

• 

  raised initial (after)

layout code refactoring

• bug 1307728 - Convert LineReflowStatus to an enum class
  • file | land
  • followup: CSS Inline Vertical Alignment and Line Wrapping Around Floats by L. David Baron
    • PR merged
• bug 1308077 - Add compile time protection while calling nsLineBox::BreakTypeToString
  • file | land

Shing Lyu

Stylo:

• CSS Filters for Stylo

Servo

• Talked with dholbert about flexbox implementation tips
• Testing flexbox on Servo
• Evaluating the effort of migrating layout trace tool to Serde

Sharing:

• NTUEE talk
• ViewSource + RustFest brownbag
• Rust training workshop preparation

Tingyu Lin

Details & Summary

• bug 1304441 - Assertion failure: !summary || !summary->IsMainSummary()
  • land

AccessibleCaret

• bug 1307027 - The tip of caret is covered by input area
  • file | land
• bug 1304263 - Investigate whether long tap injector is needed on Windows with touch support
  • review
• bug 1307252 - Touch caret is shown when touch-keyboard or external keyboard is used
  • land

Fennec

• bug 1307100 - Floating toolbar does not hide when the selection highlight is scrolled out of viewport
  • land

Layout

• bug 1291110 - The inline element overlaps with the float elements
  • r-

Tommy Kuo

Presentation API

• bug 1282003 - Make nsScreenManagerAndroid support multiple screens.
  • land, BUT backed out.
• bug 1305351 - Add ChromeCastDisplay for Presentation API.
  • review+, waiting for the new API key for Chromecast from Mozilla account.
• bug 1285870 - Show custom presentation on Chromecast.
  • review+, blocked by bug 1305351 & bug 1282003
• bug 1295087 - [Presentation WebAPI] Implement PresentationDeviceProvider for Chromecast devices on Fennec.
  • review- | WIP
• bug 1305352 - [Presentation API] Implement PresentationRequestUIGlue for Fennec.
  • (Part 1) review- | WIP
  • (Part 2) review+

Layout

• Read the document about adding a new CSS property in the styling system.
  • https://developer.mozilla.org/en-US/docs/Mozilla/Adding_a_new_style_property

TV Support

• Help to upgrade the back-end of TV from FreeBSD to Linux.

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)