TPE-Layout/2018-01-16

From MozillaWiki
Jump to: navigation, search

Weekly Announcement

Weekly Sharing

Weekly Status

Astley Chen

  • bug 133787 NEW:: [Meta][UX] Redesign print preview and make it tab-modal instead of window-modal
    • Bug 133787 - set dependency to printing improvement list
  • bug 1418806 REOPENED:: Crash in core::result::unwrap_failed<T> | geckoservo::glue::Servo_AnimationCompose
    • Bug 1418806 - resolved to wontfix in FF58
  • bug 1426400 NEW:: The video preview is missing during canvas recording
    • Bug 1426400 - triaged and ni for bug resolution for FF58

Boris Chiou

  • bug 1429299 ASSIGNED:: Implement offset-distance for motion path
    • Filed bug 1429298, bug 1429299, bug 1429301, bug 1429303, and bug 1429305
  • bug 1429301 ASSIGNED:: Implement offset-rotate for motion path
    • Filed bug 1429298, bug 1429299, bug 1429301, bug 1429303, and bug 1429305
  • bug 1429298 ASSIGNED:: Implement path() function for offset-path
    • Filed bug 1429298, bug 1429299, bug 1429301, bug 1429303, and bug 1429305
    • Is writing the parser of svg path string in Rust for bug 1429298
    • Wrote a patch to parse moveto, lineto, and closepath, for svg path string in rust, for bug 1429298
  • bug 1429305 NEW:: Support compositor animations for motion path
    • Filed bug 1429298, bug 1429299, bug 1429301, bug 1429303, and bug 1429305
  • bug 1429303 NEW:: Make offset-distance and offset-rotate animatable
    • Filed bug 1429298, bug 1429299, bug 1429301, bug 1429303, and bug 1429305
  • bug 1207734 ASSIGNED:: Implement individual transform properties: the 'translate', 'scale', and 'rotate' properties
    • Reviewed a patch to rector MatrixDecomposed3D::slerp for bug 1207734 - Implement individual transform properties: the 'translate', 'scale', and 'rotate' properties

Cameron McCormack

  • bug 1429125 RESOLVED::FIXED stylo: Lack of lazy frame construction on the browser chrome could affect stylo-chrome performance
    • Reviewed bug 1429125
  • bug 1255384 VERIFIED::FIXED inIDOMUtils.getCSSValuesForProperty() is missing keyword for counter-* properties
    • Reviewed bug 1255384
  • bug 1429990 RESOLVED::FIXED Permafailing layout/style/test/chrome/test_display_mode.html | all and (display-mode: browser) should apply
    • Reviewed bug 1429990
  • bug 1427512 RESOLVED::FIXED remove XPIDL interfaces for style sheets and style rules
    • Landed bug 1427512
  • bug 1417829 RESOLVED::FIXED Remove unresolved pseudoclass which is for Custom Elements v0
    • Reviewed bug 1417829
  • bug 1429979 RESOLVED::FIXED Some clean up for ImageLoader
    • Reviewed bug 1429979
  • bug 1255401 VERIFIED::FIXED inIDOMUtils.getCSSValuesForProperty() is missing keyword for 'quotes'
    • Reviewed bug 1255401
  • bug 1427419 RESOLVED::FIXED convert inIDOMUtils to Web IDL
    • Landed bug 1427419
  • bug 1428531 RESOLVED::FIXED add a chrome-only function to DOM objects to do a cross-origin instanceof check
    • Landed bug 1428531
    • Uploaded patch for bug 1428531
  • bug 1427519 RESOLVED::FIXED remove inCSSValueSearch and related interfaces
    • Landed bug 1427519

CJ Ku

  • bug 1427109 RESOLVED::FIXED PDFium printing paths should not be enabled by default on Windows
    • Bug 1427109 - r+
    • Bug 1427109 land

Eric Tsai

  • Issue diagnosis & triage
  • Change alexa topsite script from root credential to IAM user credential

Jeremy Chen

  • bug 1429977 RESOLVED::FIXED Code comments for gfxTextRun::BreakAndMeasureText are out-of-date
    • Filed and uploaded a patch [Bug 1429977]
    • Landed Bug 1429977
  • bug 1418472 NEW:: Wrong position of github issue name on hover
    • Investigated Bug 1418472
  • bug 1428988 NEW:: Beginning of roc's blog posts covered by blue with strict Tracking Protection
    • Triage Bug 1428988
  • bug 1283222 ASSIGNED:: Unwanted line-break opportunities at out-of-flow frames [was: Commas wrapped to beginning of line with text-align: justify]
    • Added testcases and ask for review [Bug 1283222]
    • Update investigation progress [Bug 1283222]
    • Updated investigation progress, updated wip for feedback [Bug 1283222]

Louis Chang

  • bug 1428676 - -moz-appearance menulist-button should behave like -webkit-appearance: menulist-button
    • r?mats
  • bug 1429700 - Make -webkit-appearance as an alias of -moz-appearance with pref off
    • r?mats
  • bug 1429713 - Make -webkit-appearance as an alias of -moz-appearance with pref off in stylo
    • r?heycam

Ting-Yu Lin

  • bug 1430526 RESOLVED::INVALID Cannot parse SVG XML if the fill attribute contains hash '#'
    • Filed bug 1430526.
  • bug 1429313 RESOLVED::FIXED reuse |nextColIndex| to reduce unnecessary calls
    • Reviewed bug 1429313.
  • bug 1429979 RESOLVED::FIXED Some clean up for ImageLoader
    • Filed and landed bug 1429979.

Tommy Kuo

  • bug 1425088 RESOLVED::FIXED Change the dropdown frame of <select> to a top-level absolute frame
    • Landed (bug 1425088)
  • bug 1428960 RESOLVED::FIXED -moz-dropdown-list is only allowed to use universal selector
    • Filed | lande (bug 1428960)
  • bug 1428987 NEW:: The color of decoration lines in a selection is not the color that is specified
    • Filed (bug 1428987)
  • bug 1428982 RESOLVED::WONTFIX Skip the rules of anonymous boxes that use something other than a universal selector
    • Close bug (bug 1428982)
    • Filed bug (bug 1428982)
    • Upload a patch (bug 1428982)
  • bug 1429690 RESOLVED::WONTFIX Debug mode for content-select to always show the dropdown frame
    • Filed (bug 1429690)
    • Upload patch (bug 1429690)
  • bug 1428297 RESOLVED::FIXED Ignore the widget/view usage that is used to show the dropdown menu when content-select is enabled.
    • Debug for a11y test failures (bug 1428297)
    • Landed (bug 1428297)
    • Revised and uploaded patches (bug 1428297)
  • bug 1429691 NEW:: Position and reflow the dropdown frame correctly when content-select is enabled
    • Filed (bug 1429691)

Wei-Cheng Pan

  • bug 1429652 NEW:: docshell leak by beforeunload event
    • Bug 1429652, investigated, forward ni flag
  • bug 1425213 NEW:: Animation rendering problem with (ex-Twitter) Bootstrap 3 Carousel
    • Bug 1425213 - upload wip patch
    • Bug 1425213, checking the frame tree

Ya-Chieh Wu

  • bug 1429313 RESOLVED::FIXED reuse |nextColIndex| to reduce unnecessary calls
    • Bug 1429313 r+
  • bug 1416957 NEW:: use MayHaveOpacityAnimation in nsIFrame to look up if a frame may have opacity
    • Debug Bug 1416957
    • Upload patches for Bug 1416957
    • Work on bug 1416957

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