TPE-Layout/2016-09-06

From MozillaWiki
Jump to: navigation, search

Weekly Report

Astley Chen

CSSOM

  • bug 1292447 - Shrink the number of properties which return used value as resolved value for getComputedStyle
    • r+
  • bug 1207900 - Intermittent Windows xpcshell command timed out: 7200 seconds elapsed, attempting to kill
    • WIP

Boris Chiou

  • Web Animations API
    • bug 1274944 - Implement writable keyframeEffect spacing
      • land
    • bug 1298742 - Don't call CanThrottle if the effect is not included in EffectSet
      • land
    • bug 1272549 - Support paced spacing for transform
      • OOAD
  • SVG
    • bug 1245751 - Allow href attribute without xlink on SVG elements
      • land

Cameron McCormack

CJ Ku

  • bug 1190881 - SVG CSS animation not working through img tag
    • landed
  • bug 1275826 - Remove composition adjustment code, base on layer-index, in nsCSSRendering::PaintBackgroundWithSC
    • landed
  • bug 1300401 - 2.87 - 4.15% cart (linux64) regression on push 958201c5ca69 (Fri Sep 2 2016)
    • WIP

Jeremy Chen

layout code refactoring

  • bug 1297306 - Convert NS_STYLE_CLEAR_* to an enum class
    • r+ | r?
    • add tests, wait for review results from mfbt peer
   int64_t a = -1;
   uint8_t b = 2;
   cout << (a < b) << endl;
   cout << (a < (int64_t)b) << endl;
   cout << ((uint8_t)a < b) << endl;

   int c = -1;
   unsigned int d = 2;
   cout << (c < d) << endl;
   cout << (c < (int)d) << endl;
   cout << ((unsigned int)c < d) << endl;

Shing Lyu

Tingyu Lin

  • bug 1291110 - The inline element overlaps with the float elements
    • r?
  • bug 1299753 - <details> with float: left has an incorrect computed height when preceded with other blocks with float: left
    • r?
  • bug 1098939 - Implement CSS shape-outside property
    • Writing test cases for shape-box only and shape-box with border-radius.

Team

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)
Retrieved from "https://wiki.mozilla.org/index.php?title=TPE-Layout/2016-09-06&oldid=1147088"