TPEGFX/2015-09-10

Weekly Report

Boris Chiou

• bug 1198522 - [Contacts] Long pressing on an image in edit mode will bring up a menu that does not function
  • r+

• bug 1202541 - Add more GIJ tests for copy selected "text and image"
  • WIP | OOAD
  • Add new tests for copying complicated rich text
  • Try to enable skipped tests

• bug 1201425 - Prevent the display of the context menu on any img tag without the src attribute when long press
  • file

• bug 904183 - Enable copypaste mochitests on b2g and Android
  • debug

Jeremy Chen

• bug 1201481 - WARNING: MORE THAN 10 OBSERVERS FOR copypaste.enabled: 101
  • debug | r+ | land
• bug 1202316 - Intermittent gfx/tests/reftest/1149923.html | image comparison (==), max difference: 63, number of differing pixels: 22
  • r+
• bug 1195653 - [LayerScope] Dump SecondaryEffects (EffectTypes::MASK) on Layerscope viewer
  • OOAD | WIP
    • TODO:
      • gedko: dump texture related info on viewer side
      • viewer: display mask/texture attributes properly so developer can easily take advantage of it
• C++11/14 study group sharing

Jerry Shih

• bug 1166173 - off-main thread painting
  • OOAD, WIP
  • https://bugzilla.mozilla.org/attachment.cgi?id=8653877

• bug 1201777 - Make TextureClient::BorrowDrawTarget() work for off-main thread painting
  • file
  • We need this for off-main thread painting

Tingyu Lin

• bug 1202320 - PresShell::AsyncPanZoomEnabled() returns false in B2G root process
  • file | land
• bug 1200194 - [Caret] Caret behavior is wrong when the cursor is at the end of the input
  • r+

Vincent Liu

• 2D Canvas Worker
  • Assert happens under stress test. Assertion on nsCSSParser because of thread competing.
  • Fail to draw canvas with text. mCanvasElement gets nullprt and fails to return in CanvasRenderingContext2D::SetFont()
• bug 1186826 - Implement an abstraction object for NativeWindow to hide hardware dependent operations
  • WIP

CJ

• Canvas2D on worker thread.
  • nsCSSParser/ nsCSSParserImp relationship.
• bug 1198626 - (LayerScope) Change the position of zoom controls
  • r?
• GDB command
  • V老神推薦
  • You don't need printf anymore.

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)