TPE-Layout/2017-08-08

From MozillaWiki
Jump to: navigation, search

Weekly Announcement

  • 2017 Job Family Architecture 1.0 Released.
  • TDC All-hands with David Slater and John McSorley
    • 8/17 2PM.

Weekly Status

Astley Chen

  • bug 1385911 NEW:: SVG mask is broken on scroll
    • Bug 1385911 - regression patch found and ni?cku.
    • Bug 1385911 - regression window needed.
  • bug 1385419 NEW:: Parent element stretches to submit input's line-height, rather than its also-specified height.
    • Bug 1385419 - webcompat issue, regression range found and ni?dbaron.
  • bug 1385517 NEW:: inline-block/floated parent of <picture> with a srcset/2x on HiDPI monitor is twice as wide as the image
    • Bug 1385517 - regression hunting now.

Boris Chiou

  • bug 1387986 NEW:: stylo: Some distance calculation and interpolated results are not correct for stroke-dasharray in test_transitions_per_property.html
    • Filed bug 1292283, bug 1369614, bug 1387939, bug 1387941, bug 1387946, bug 1387948, bug 1387949, bug 1387951, bug 1387952, bug 1387973, bug 1387982, bug 1387986, bug 1387990 for bug 1387080 - stylo: Pass all test cases in test_transitions_per_property.html
  • bug 1292283 ASSIGNED:: stylo: Enable test_transitions_per_property.html
    • Filed bug 1292283, bug 1369614, bug 1387939, bug 1387941, bug 1387946, bug 1387948, bug 1387949, bug 1387951, bug 1387952, bug 1387973, bug 1387982, bug 1387986, bug 1387990 for bug 1387080 - stylo: Pass all test cases in test_transitions_per_property.html
  • bug 1387952 NEW:: stylo: Interpolation of clip-path (BasicShape) is not correct in test_transitions_per_property.html
    • Filed bug 1292283, bug 1369614, bug 1387939, bug 1387941, bug 1387946, bug 1387948, bug 1387949, bug 1387951, bug 1387952, bug 1387973, bug 1387982, bug 1387986, bug 1387990 for bug 1387080 - stylo: Pass all test cases in test_transitions_per_property.html
  • bug 1386864 RESOLVED::FIXED stylo: Assertion failure: IsResolved() (Resolve() must be called first), at /home/ikezoe/autoland/layout/style/nsStyleStruct.h:355
    • Investigated bug 1386864 - stylo: Assertion failure: IsResolved() (Resolve() must be called first), at /home/ikezoe/autoland/layout/style/nsStyleStruct.h:355
  • bug 1384656 RESOLVED::FIXED stylo: We use nsStyleTransformMatrix, which can enter in nsRuleNode and assume that we use a Gecko style context.
    • Landed bug 1384656 - stylo: We use nsStyleTransformMatrix, which can enter in nsRuleNode and assume that we use a Gecko style context.
  • bug 1387910 RESOLVED::FIXED stylo: getAnimations may return null for pseudo-element
    • Reviewed bug 1387910 - stylo: getAnimations may return null for pseudo-element
  • bug 1369614 ASSIGNED:: stylo: Support animating between stroke-widths that have units and those that don't
    • Filed bug 1292283, bug 1369614, bug 1387939, bug 1387941, bug 1387946, bug 1387948, bug 1387949, bug 1387951, bug 1387952, bug 1387973, bug 1387982, bug 1387986, bug 1387990 for bug 1387080 - stylo: Pass all test cases in test_transitions_per_property.html
  • bug 1387982 NEW:: stylo: The serialization of filter has unit but the expected value is unitless in test_transitions_per_property.html
    • Filed bug 1292283, bug 1369614, bug 1387939, bug 1387941, bug 1387946, bug 1387948, bug 1387949, bug 1387951, bug 1387952, bug 1387973, bug 1387982, bug 1387986, bug 1387990 for bug 1387080 - stylo: Pass all test cases in test_transitions_per_property.html
  • bug 1387973 NEW:: stylo: Some distance calculation and interpolated results are not correct for {box|text}-shadow in test_transitions_per_property.html
    • Filed bug 1292283, bug 1369614, bug 1387939, bug 1387941, bug 1387946, bug 1387948, bug 1387949, bug 1387951, bug 1387952, bug 1387973, bug 1387982, bug 1387986, bug 1387990 for bug 1387080 - stylo: Pass all test cases in test_transitions_per_property.html
  • bug 1387941 NEW:: stylo: There are lots of failures of scroll-snap-{destination, coordinate} in test_transitions_per_property.html
    • Filed bug 1292283, bug 1369614, bug 1387939, bug 1387941, bug 1387946, bug 1387948, bug 1387949, bug 1387951, bug 1387952, bug 1387973, bug 1387982, bug 1387986, bug 1387990 for bug 1387080 - stylo: Pass all test cases in test_transitions_per_property.html
  • bug 1387939 NEW:: stylo: Interpolation of "Integer or Auto" is incorrect in test_transitions_per_property.html
    • Filed bug 1292283, bug 1369614, bug 1387939, bug 1387941, bug 1387946, bug 1387948, bug 1387949, bug 1387951, bug 1387952, bug 1387973, bug 1387982, bug 1387986, bug 1387990 for bug 1387080 - stylo: Pass all test cases in test_transitions_per_property.html
  • bug 1387946 NEW:: stylo: The calc() computed values of {backgournd|mask}-{position|size} are not correct in test_transitions_per_property.html
    • Filed bug 1292283, bug 1369614, bug 1387939, bug 1387941, bug 1387946, bug 1387948, bug 1387949, bug 1387951, bug 1387952, bug 1387973, bug 1387982, bug 1387986, bug 1387990 for bug 1387080 - stylo: Pass all test cases in test_transitions_per_property.html
  • bug 1387080 NEW:: stylo: Pass all test cases in test_transitions_per_property.html
    • Filed bug 1292283, bug 1369614, bug 1387939, bug 1387941, bug 1387946, bug 1387948, bug 1387949, bug 1387951, bug 1387952, bug 1387973, bug 1387982, bug 1387986, bug 1387990 for bug 1387080 - stylo: Pass all test cases in test_transitions_per_property.html
  • bug 1387948 NEW:: stylo: Interpolation of font-weight is not correct in test_transitions_per_property.html
    • Filed bug 1292283, bug 1369614, bug 1387939, bug 1387941, bug 1387946, bug 1387948, bug 1387949, bug 1387951, bug 1387952, bug 1387973, bug 1387982, bug 1387986, bug 1387990 for bug 1387080 - stylo: Pass all test cases in test_transitions_per_property.html
  • bug 1387667 NEW:: stylo: Add the style backend flag to ProcessMatrixOperator
    • Filed bug 1387667 - stylo: Add the style backend flag to ProcessMatrixOperator
  • bug 1387949 NEW:: stylo: Interpolation of "vertical-align" is not correct in test_transitions_per_property.html
    • Filed bug 1292283, bug 1369614, bug 1387939, bug 1387941, bug 1387946, bug 1387948, bug 1387949, bug 1387951, bug 1387952, bug 1387973, bug 1387982, bug 1387986, bug 1387990 for bug 1387080 - stylo: Pass all test cases in test_transitions_per_property.html
  • bug 1387990 NEW:: stylo: Many interpolations of transform are failed in test_transitions_per_property.html
    • Filed bug 1292283, bug 1369614, bug 1387939, bug 1387941, bug 1387946, bug 1387948, bug 1387949, bug 1387951, bug 1387952, bug 1387973, bug 1387982, bug 1387986, bug 1387990 for bug 1387080 - stylo: Pass all test cases in test_transitions_per_property.html
  • bug 1383998 RESOLVED::FIXED stylo: dom/animation/test/css-transitions/test_element-get-animations.html fails
    • Landed bug 1383998 - stylo: dom/animation/test/css-transitions/test_element-get-animations.html fails
  • bug 1387951 NEW:: stylo: Interpolation of ClipRectOrAuto or ColorOrAuto is not correct in test_transitions_per_property.html
    • Filed bug 1292283, bug 1369614, bug 1387939, bug 1387941, bug 1387946, bug 1387948, bug 1387949, bug 1387951, bug 1387952, bug 1387973, bug 1387982, bug 1387986, bug 1387990 for bug 1387080 - stylo: Pass all test cases in test_transitions_per_property.html
  • bug 1385139 NEW:: stylo: CSS variables not resolved in values returned by getKeyframes()
    • Investigated bug 1385139 - stylo: CSS variables not resolved in values returned by getKeyframes()

Cameron McCormack

  • Re-prioritized stylo bugs opened by me and TY
  • bug 1387280 RESOLVED::INVALID stylo: Font metrics handling is killing us
    • Investigating bug 1387280
  • bug 1386602 RESOLVED::FIXED stylo: We rebuild stylesheets most of the time when we don't need to when media query affecting values change.
    • Reviewed bug 1386602
  • bug 1384824 RESOLVED::FIXED stylo: Crash in PLDHashTable::Add | mozilla::ArenaRefPtr<T>::assignFrom<T>
    • Uploaded WIP patch for bug 1384824
    • Uploaded patches for bug 1384824
    • Uploaded revised patches for bug 1384824
  • bug 1381851 RESOLVED::FIXED Stylo: Flash click to play doesn't display on video elements when Stylo is enabled
    • Reviewed remaining parts of bug 1381851
  • bug 1383998 RESOLVED::FIXED stylo: dom/animation/test/css-transitions/test_element-get-animations.html fails
    • Reviewed bug 1383998
  • bug 1384741 ASSIGNED:: Stylo: incorrect CSP reports when layout.css.servo.enabled is true
    • Uploaded patches for bug 1384741
  • bug 1386871 RESOLVED::FIXED stylo: May want to control font-display descriptor behind the pref
    • Reviewed bug 1386871
  • bug 1384216 NEW:: stylo: Error messages for invalid selectors are really bad
    • Reviewed bug 1384216
  • bug 1385896 RESOLVED::FIXED stylo: Move nsStyleContext::mParent to GeckoStyleContext.
    • Reviewed bug 1385896
  • bug 1382925 ASSIGNED:: stylo: Keep the UA parts of the stylist across changes to document sheets
  • bug 1387481 RESOLVED::FIXED stylo: Assertion failure: MayTraverseFrom(const_cast<Element*>(aRoot))
    • Landed bug 1387481
  • bug 1386865 RESOLVED::FIXED Stylo: DevTools test using XBL triggers "MOZ_ASSERT_UNREACHABLE: We should be able to map a raw rule to a rule"
    • Reviewed other patch in bug 1386865

CJ Ku

  • bug 1385861 RESOLVED::FIXED Fine tune nsDisplaySVGText::Paint
    • Bug 1385861 landed
    • Bug 1385861 r?
  • bug 1385929 NEW:: Find a way to prevent duplicate gfxContext::Save/Restore
    • Bug 1385929 WIP
    • Bug 1385929 r?
    • Bug 1385929 upload patches
  • bug 1386218 ASSIGNED:: Remove gfxContext::HasComplexClip.
    • Bug 1386218 filed
    • Bug 1386218 upload patches
  • bug 1386864 RESOLVED::FIXED stylo: Assertion failure: IsResolved() (Resolve() must be called first), at /home/ikezoe/autoland/layout/style/nsStyleStruct.h:355
    • Bug 1386864 landed
  • bug 1386986 RESOLVED::FIXED Move nsDisplaySVGEffect::mEffectsBounds to nsDisplayFilter
    • Bug 1386986 file and upload a patch.
    • Bug 1386986 landed

Eric Tsai

  • Issue triage & diagnosis
  • Implement webcompat to bugzilla extension
  • Modify webcompat.com front-end for milestone

Farmer Tseng

  • bug 1382512 ASSIGNED:: [PDF gtest] convert EMF to bitmap and compare the bitmap content with reference
    • Refactor patch, uploaded to Bug 1382512
    • Uploaded patches, asked for feedback on Bug 1382512
  • bug 1382509 ASSIGNED:: Pull out PDFium from xul
    • Uploaded patches, r? on Bug 1382509
  • bug 1358076 ASSIGNED:: [PDF gtest] Implement unit test for PDFViaEMFPrintHelper
    • Uploaded patches, r? on Bug 1358076

Jeremy Chen

  • bug 1367118 ASSIGNED:: stylo: investigate the disabled SVG related tests
    • Triaged, verified, and updated the status [Bug 1367118]
  • bug 1375332 ASSIGNED:: stylo: font-size doesn't always respond to language-specific minimum size when using charset-derived language
    • Investigated Bug 1375332
    • Updated WIP and request for review [Bug 1375332]
    • Uploaded WIP and request for feedback [Bug 1375332]

Louis Chang

  • bug 1385745 - Only first line is visible in inline element with filter and box-decoration-break:clone
    • WIP
  • bug 1352238 - Implement a native theme for checkbox/radio form controls on Android
    • r+snorp | r?mats

Shing Lyu

Ting-Yu Lin

  • bug 1381851 RESOLVED::FIXED Stylo: Flash click to play doesn't display on video elements when Stylo is enabled
    • Landed bug 1381851.
  • bug 1386865 RESOLVED::FIXED Stylo: DevTools test using XBL triggers "MOZ_ASSERT_UNREACHABLE: We should be able to map a raw rule to a rule"
    • Reply needinfo for bug 1386865.
    • Reviewed bug 1386865.
  • bug 1387926 RESOLVED::DUPLICATE The color refresh (loading) indicator is not obvious on light color theme (uses white instead of blue)
    • Filed bug 1387926.
  • bug 1374902 NEW:: stylo: Have unprefixed :fullscreen pseudo-class guarded behind full-screen-api.unprefix.enabled
    • Reviewed bug 1374902.

Tommy Kuo

  • bug 1377158 NEW:: stylo: Enable stylo for SVG-as-an-image.
    • Add new patch to reduce the cost of the assertion and ask to review (Bug 1377158)
    • Debug for the test failure of bc2 (bug 1377158)
    • Debug for the test failure of bc2 and update hash function and run a try (bug 1377158)
    • Test the patch from xidorn (bug 1377158)
    • Use hash table to store the change hint of content. (Bug 1377158)

Ya-Chieh Wu

  • bug 1379306 ASSIGNED:: Position absolute not work as expected in table with border collapse
    • Bug 1379306 r?

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