TPE-Layout/2018-01-09

From MozillaWiki
Jump to: navigation, search

Weekly Announcement

  • CultureAMP survey will close on Friday, January 12.

Weekly Sharing

Weekly Status

Astley Chen

  • bug 1357966 NEW:: Don't report resource timing values for image, -moz-image-rect, font, and cursor from cross-origin stylesheets
    • Bug 1357966 - sec-high and need work

Boris Chiou

  • Blink issue 496518
    • Checked the current implementation of motion path on Blink
  • Motion Path Module Level 1
    • Is investigating the spec of Motion Path
  • bug 1421507 RESOLVED::FIXED Throttle animations on position:fixed element which is out-of-view
    • Reviewed bug 1421507 - Throttle animations on position:fixed element which is out-of-view
  • bug 1420928 RESOLVED::FIXED stylo: animation-name and transition-property probably shouldn't be early properties.
    • Reviewed bug 1420928 - stylo: animation-name and transition-property probably shouldn't be early properties.
  • bug 1418806 REOPENED:: Crash in core::result::unwrap_failed<T> | geckoservo::glue::Servo_AnimationCompose
    • Wrote a patch to dump the alignmend size for bug 1418806

Cameron McCormack

  • bug 1427419 ASSIGNED:: convert inIDOMUtils to Web IDL
    • Uploaded new patches for bug 1427419
  • bug 1427512 ASSIGNED:: remove XPIDL interfaces for style sheets and style rules
    • Uploaded patches for bug 1427512
  • bug 1428491 NEW:: Make ServoStyleSet hold a reference to a document, not to a pres context.
    • Reviewed bug 1428491
  • bug 1428276 RESOLVED::FIXED Make ExpectedOwnerForChild deal with abs.pos. anon boxes
    • Reviewed bug 1428276
  • bug 1428339 NEW:: Make mapped attribute resolution not require a pres context.
    • Reviewed bug 1428339
  • bug 1415781 ASSIGNED:: Fix failure in layout/style/test/chrome/ relying on non-comformant Promise handling
    • Reviewed bug 1415781
  • bug 1424607 NEW:: Make sure invalidation handles ::slotted correctly.
    • Reviewed bug 1424607
  • bug 1425755 NEW:: Implement the missing selector-matching bits for ::slotted
    • Reviewed bug 1425755

CJ Ku

  • bug 1207734 Implement individual transform properties: the 'translate', 'scale', and 'rotate' properties
    • r?

Eric Tsai

Jeremy Chen

  • bug 1283222 - Unwanted line-break opportunities at out-of-flow frames [was: Commas wrapped to beginning of line with text-align: justify]
    • debug and clarify the unknowns (discussed with the reviewer during the y'allhands)

Louis Chang

  • bug 1428676 - -moz-appearance meulist-button should behave like -webkit-appearance: menulist-button
    • filed a bug
    • WIP
  • bug 1246836 - -moz-appearance button should behave like -webkit-appearance: button
    • WIP
    • review mats patch

Ting-Yu Lin

Tommy Kuo

  • bug 1428276 RESOLVED::FIXED Make ExpectedOwnerForChild deal with abs.pos. anon boxes
    • File bug and upload a patch to ask to review (bug 1428276)
    • Landed (bug 1428276)
  • bug 1428297 NEW:: Get rid of using widget/view to show the dropdown menu when content-select is enabled.
    • File bug (Bug 1428297)
    • Uploaded the patch (bug 1428297)
  • bug 1425088 NEW:: Change the dropdown frame of <select> to a top-level absolute frame
    • Make the dropdown frame be a top-level absolute frame behind a pref (bug 1425088)
    • Upload patches to ask to review (bug 1425088)
  • bug 1428701 ASSIGNED:: Allow to toggle the dropdown frame when content-select is enabled
    • Filed a bug and WIP (bug 1428701)

Wei-Cheng Pan

  • bug 1427995 NEW:: Repeatly evaling a string literal is 10x slower than Chrome
    • Filed bug 1427995

Ya-Chieh Wu

  • bug 347335 RESOLVED::INCOMPLETE css border-collapse bad rendering on huge tables
    • Close Bug 347335
  • bug 1421887 RESOLVED::FIXED [BC] |ResetBStartStart| might introduce weird borders if we don't get the row index right.
    • Land Bug 1421887
  • bug 935914 RESOLVED::DUPLICATE Collapsed borders are not drawn correctly in a table with rowspans
    • Close bug 935914

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