TPEPlatform/2014-06-05: Difference between revisions

From MozillaWiki
Jump to navigation Jump to search
 
(35 intermediate revisions by 15 users not shown)
Line 31: Line 31:
If your patch is target for specific branch, instead of central, postfix branch version. For exp
If your patch is target for specific branch, instead of central, postfix branch version. For exp
  ** land(1.4)
  ** land(1.4)
==Summary==
Status changed by this week
<bugzilla>
    {
        "status": ["NEW", "ASSIGNED", "UNCONFIRMED"],
        "changed_after": "2014-05-30",
        "changed_before": "2014-06-06",
        "assigned_to": ["jolin@mozilla.com", "pchang@mozilla.com", "cku@mozilla.com", "ayang@mozilla.com", "jwwang@mozilla.com", "rlin@mozilla.com", "slee@mozilla.com", "boris.chiou@gmail.com", "bechen@mozilla.com", "vlin@mozilla.com", "schiou@mozilla.com", "hshih@mozilla.com", "mtseng@mozilla.com", "chung@mozilla.com", "ctai@mozilla.com",  "tlin@mozilla.com"],
        "include_fields": "id, summary, status, resolution, assigned_to"
    }
</bugzilla>
Bugs which are fixed by this week
<bugzilla>
    {
        "changed_after": "2014-05-30",
        "changed_before": "2014-06-06",
        "assigned_to": ["jolin@mozilla.com", "pchang@mozilla.com", "cku@mozilla.com", "ayang@mozilla.com", "jwwang@mozilla.com", "rlin@mozilla.com", "slee@mozilla.com", "boris.chiou@gmail.com", "bechen@mozilla.com", "vlin@mozilla.com", "schiou@mozilla.com", "hshih@mozilla.com", "mtseng@mozilla.com", "chung@mozilla.com", "ctai@mozilla.com",  "tlin@mozilla.com"],
        "include_fields": "id, summary, status, resolution, assigned_to",
        "changed_field": "status",
        "changed_field_to": "resolved"
    }
</bugzilla>


==Media==
==Media==
====Randy Lin====
====Randy Lin====
*{{Bug|1011149}} - Video freezes when audio track is disabled on remote stream
** debug
** It's regression at Bug 998711. Finding the cause if there is no pcm data feed in audio chunk, the MediaStreamGraph would block the video playing.
* {{Bug|1008797}} - Test case for bug 993598.
** review
** Review the test case and gave some feedbacks.
*PTO
====Alfredo Yang====
====Alfredo Yang====
*{{Bug|984274}} - Intermittent test_sandbox_permission.html | Test timed out.
** debug
** found the root cause of the rapid increased failed ratio
*{{Bug|916643}} - ImageCapture - Implement WebIDL and takePhoto()
** WIP
** Implementing the error event webidl
====John Lin====
====John Lin====
* {{Bug|1003712}} [B2G][WebRTC] Fall back to VP8 when H.264 HW codec is in use.
** develop patches to extend MediaResourceManagerService for WebRTC to request/reserve HW codec.
** r+ & landed
====JW Wang====
====JW Wang====
*{{Bug|1020227}} - Deadlock in opensl_stream_destroy
** debug
** race in destroying player object while buffer callback undergoing.
** try to delay destruction until all buffers are consumed.
*{{Bug|1008079}} - Assertion failure: !mAudioStream->IsPaused() (Don't play when paused)
** modify patch for coding style
*PTO
====Benjamin Chen====
====Benjamin Chen====
*{{Bug|969372}} - Intermittent | test_mediarecorder_record_no_timeslice.html | Test timed out.
** f+ | r- | review?
** Refine the MediaRecorder and Session's life cycle
*{{Bug|1015856}} - [MediaEncoder::GTest] Testcase for bug 970774, WebM aspect ratio.
** r?
* {{Bug|1018402}} - VP8VideoTrackEncoder.FrameEncode | SEH exception with code 0xc0000005 thrown in the test body. @ (null):-1 when GTest is enabled on a VS2013 Debug build
** debug
** Gtest TestVideoTrackEncoder crash only on vs2012 build.
==Platform==
==Platform==
====Steven Lee====
====Steven Lee====
* {{Bug|1016184}} - [Touch Caret]enable touch caret on B2G
** debug
** Cannot run mochitest on emulator, still trying
* {{Bug|1015570}} - "ERROR PROCESSING MOZBUILD FILE [...] The error appears to be part of the mozbuild.frontend.reader Python module itself!" --> mozilla/media/libyuv/moz.build
** feebback
* {{Bug|932562}} - Allow running gtests with a b2g build
** debug
** Still trying to find out how to enable gtests in build system
* Touch caret/Text selection project meetings
** meeting
** Sync status with PM
====Chiahung Tai====
====Chiahung Tai====
====Vincent Lin====
*{{Bug|1015860}} - [B2G] We can't use systrace style Gecko Profiler in content process.
** land
** ^o^
*{{Bug|938034}} - [B2G] GetUserMedia can provide recording callback in media stream
** WIP
** Rebased to latest codes.
*{{Bug|969312}} - [MediaEncoder] Support data from GonkCameraSource in HW video recording path.
** WIP
** Rebased to latest codes. Now we can use camera recorder for media recording. But need to find a way for suspending video and rotation.
*{{Bug|651060}} - Decode JPEGs to YCbCr and do RGB conversion on the GPU
** OOAD
** Hold a meeting with CJay, Steven Lee, Peter Chang, Chiajung Hung to discuss the value and potential risk for this bug.
 
====Tingyu Lin====
====Tingyu Lin====
* {{Bug|924692}} - [Touch Caret] Display a touch caret according to caret postion in input element
** land
** Help rebase the patches.
* {{Bug|1016147}} - marionette HTMLElement.send_keys() doesn't insert text after the caret position
** file | land
* {{Bug|960897}} - [Touch Caret] Touch caret sanity test
** R?
* {{Bug|1020261}} - [Touch Caret] Enable touch caret sanity test on B2G
** file
** Touch caret cannot recognize touch event id synthesized by marionette.
* {{Bug|1019441}} - [Text Selection] Text Selection sanity test
** OOAD
====Boris Chiou====
* {{Bug|1003893}} - crash in imgFrame::~imgFrame()
** debug | study
** Check the mRefCnt of RasterImage.
** Trace code to understand the relationship among RasterImage, DiscardTracker, and DecodePool. Try to understand who should owns the RasterImage.
* {{Bug|1015863}} - [dolphin][monkey test] monkey test crash at libxul.so!imgFrame::~imgFrame() [imgFrame.cpp : 169 + 0x0]
** debug
** Check the call stack and crash address. (Is it duplicated?)
** This bug also happened in Gallery and the call stack was the same as Bug 1003893. However, the version is different from Bug 1003893.
* {{Bug|999935}} - [Leo][MMS] Screenshot taken (2D Canvas) while attaching an image (drawn on 3D Canvas) from Gallery cropper is shown black
** debug
** Test on flame and open c (Reproduction rate: 0%)


==Graphics==
==Graphics==
====Peter Chang====
====Peter Chang====
* {{Bug|846421}} support snapshot for nested remote mozbrowser content
** WIP
** Add OOP support for taking snapshot by compositor
** Measure the performance between taking snapshot by content and compositor
** Consider security for compositor way
====Morris Tseng====
====Morris Tseng====
* {{Bug|987718}} - [Text Selection] Display selection caret based on touch caret
** r+
** Separate perference from touch caret
** All r+, wait for landing
* {{Bug|987040}} - Implement mozbrowserSelection
** WIP
** Discussed with Ehsan for implementation detail.
** Implemented using Ehsan's suggestion.
====Jerry Shih====
====Jerry Shih====
* {{Bug|987529}} - Implement Vsync dispatch pipeline
** OOAD | WIP
** read vsync related bug
** test auto-generating input event patch from Vincent
* {{Bug|998916}} - Defer the webgl context restore until the app becomes foreground
** r-
** reviewer suggest to remove some if condition, use MOZ_ASSERT instead.
* {{Bug|1015852}} - crash at GLContextProviderEGL::CreateForWindow in dolphin monkey test
** debug
** provide a debug patch to vender and wait for adb log.
* {{Bug|1008026}} - wallpaper app will flicker when we scroll
** r?
** show thumbnail instead of full size wallpaper
* {{Bug|1012551}} - picture preview in edit mode sometimes appear abnormal after unlock screen(tarako)
** debug
** add more log to debug
====Chiajung Hung====
====Chiajung Hung====
* {{bug|814524}} - Make WebGLContext::TexImage2D avoid readback for video elements
**r-
**fix based on feedback
* {{bug|880114}} - Enhance render video-to-SkiaGL performance by GPU-based color space conversion
**WIP
* {{Bug|918984}} - Direct video stream texturing support on WebGL
**WIP
**fixing webidl related build problem
====Solomon Chiou====
====Solomon Chiou====
* PTO: 2 days
* {{Bug|1020745}} - [Flame] Unable to use full HW composition in video player
** debug
** This bug seems only happen in 2.0. When playing the video in full screen, an special CanvasLayer(its width=480, height=854, however contains a 0x0 ImageHost) seems prevent the composition be performed by HwcComposer2D.
** Another strange thing is: some layers of background was also can be observed in LayerScope, which is different from 1.4
* {{Bug|1020750}} - [Flame] Unable to use full HW composition in Camera
** debug
** There is also a Canvas layer stop the composition be performed by HwcComposer2D. The canvas layers seems generated by a thumbnail icon which should not appear then. Still checking with gaia team to clarify the root cause.
* {{Bug|1017389}} - [Flame] Unable to use full HW composition in most of use case
** debug
** Add some debug functions to help debugging, will upload those function for review later.


==Manager==
==Manager==
====CJ Ku====
====CJ Ku====
* {{Bug|960897}} - [Touch Caret] Touch caret sanity test
** feedback
** don't hard code string content at py file
* {{Bug|969372}} - Intermittent | test_mediarecorder_record_no_timeslice.html | Test timed out.
** feedback
* {{Bug|1019441}} - [Text Selection] Text Selection sanity test
** file
** The test case need for Text selection landing.
* {{Bug|1012551}} - [tarako] picture preview in edit mode sometimes appear abnormal after unlock screen
** debug
** By tracing libEGL log, we do use a texture after it had been destroyed... Jerry have found the root cause and upload a patch
* {{Bug|1020884}} - Have a mechanism inside GLContext to detect redundant gl call
** file
** outcome of 1012551 debugging.
*{{Bug|651060}} - Decode JPEGs to YCbCr and do RGB conversion on the GPU
** OOAD
** It's a good perf enhancement topic. Discuss with CTai and gfx team

Latest revision as of 07:29, 6 June 2014

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)

Summary

Status changed by this week

No results.

0 Total; 0 Open (0%); 0 Resolved (0%); 0 Verified (0%);

Bugs which are fixed by this week

Full Query
ID Summary Status Resolution Assigned to
1015860 [B2G] We can't use systrace style Gecko Profiler in content process. RESOLVED FIXED Chia-hung Tai [:ctai :ctai_mozilla :cht]
951040 [MediaEncoder::GTest] Implement unit test of WebM 1.0 container writer RESOLVED FIXED Benjamin Chen [:bechen]

2 Total; 0 Open (0%); 2 Resolved (100%); 0 Verified (0%);


Media

Randy Lin

  • bug 1011149 - Video freezes when audio track is disabled on remote stream
    • debug
    • It's regression at Bug 998711. Finding the cause if there is no pcm data feed in audio chunk, the MediaStreamGraph would block the video playing.
  • bug 1008797 - Test case for bug 993598.
    • review
    • Review the test case and gave some feedbacks.
  • PTO

Alfredo Yang

  • bug 984274 - Intermittent test_sandbox_permission.html | Test timed out.
    • debug
    • found the root cause of the rapid increased failed ratio
  • bug 916643 - ImageCapture - Implement WebIDL and takePhoto()
    • WIP
    • Implementing the error event webidl

John Lin

  • bug 1003712 [B2G][WebRTC] Fall back to VP8 when H.264 HW codec is in use.
    • develop patches to extend MediaResourceManagerService for WebRTC to request/reserve HW codec.
    • r+ & landed

JW Wang

  • bug 1020227 - Deadlock in opensl_stream_destroy
    • debug
    • race in destroying player object while buffer callback undergoing.
    • try to delay destruction until all buffers are consumed.
  • bug 1008079 - Assertion failure: !mAudioStream->IsPaused() (Don't play when paused)
    • modify patch for coding style
  • PTO

Benjamin Chen

  • bug 969372 - Intermittent | test_mediarecorder_record_no_timeslice.html | Test timed out.
    • f+ | r- | review?
    • Refine the MediaRecorder and Session's life cycle
  • bug 1015856 - [MediaEncoder::GTest] Testcase for bug 970774, WebM aspect ratio.
    • r?
  • bug 1018402 - VP8VideoTrackEncoder.FrameEncode | SEH exception with code 0xc0000005 thrown in the test body. @ (null):-1 when GTest is enabled on a VS2013 Debug build
    • debug
    • Gtest TestVideoTrackEncoder crash only on vs2012 build.

Platform

Steven Lee

  • bug 1016184 - [Touch Caret]enable touch caret on B2G
    • debug
    • Cannot run mochitest on emulator, still trying
  • bug 1015570 - "ERROR PROCESSING MOZBUILD FILE [...] The error appears to be part of the mozbuild.frontend.reader Python module itself!" --> mozilla/media/libyuv/moz.build
    • feebback
  • bug 932562 - Allow running gtests with a b2g build
    • debug
    • Still trying to find out how to enable gtests in build system
  • Touch caret/Text selection project meetings
    • meeting
    • Sync status with PM

Chiahung Tai

  • bug 1015860 - [B2G] We can't use systrace style Gecko Profiler in content process.
    • land
    • ^o^
  • bug 938034 - [B2G] GetUserMedia can provide recording callback in media stream
    • WIP
    • Rebased to latest codes.
  • bug 969312 - [MediaEncoder] Support data from GonkCameraSource in HW video recording path.
    • WIP
    • Rebased to latest codes. Now we can use camera recorder for media recording. But need to find a way for suspending video and rotation.
  • bug 651060 - Decode JPEGs to YCbCr and do RGB conversion on the GPU
    • OOAD
    • Hold a meeting with CJay, Steven Lee, Peter Chang, Chiajung Hung to discuss the value and potential risk for this bug.

Tingyu Lin

  • bug 924692 - [Touch Caret] Display a touch caret according to caret postion in input element
    • land
    • Help rebase the patches.
  • bug 1016147 - marionette HTMLElement.send_keys() doesn't insert text after the caret position
    • file | land
  • bug 960897 - [Touch Caret] Touch caret sanity test
    • R?
  • bug 1020261 - [Touch Caret] Enable touch caret sanity test on B2G
    • file
    • Touch caret cannot recognize touch event id synthesized by marionette.
  • bug 1019441 - [Text Selection] Text Selection sanity test
    • OOAD

Boris Chiou

  • bug 1003893 - crash in imgFrame::~imgFrame()
    • debug | study
    • Check the mRefCnt of RasterImage.
    • Trace code to understand the relationship among RasterImage, DiscardTracker, and DecodePool. Try to understand who should owns the RasterImage.
  • bug 1015863 - [dolphin][monkey test] monkey test crash at libxul.so!imgFrame::~imgFrame() [imgFrame.cpp : 169 + 0x0]
    • debug
    • Check the call stack and crash address. (Is it duplicated?)
    • This bug also happened in Gallery and the call stack was the same as Bug 1003893. However, the version is different from Bug 1003893.
  • bug 999935 - [Leo][MMS] Screenshot taken (2D Canvas) while attaching an image (drawn on 3D Canvas) from Gallery cropper is shown black
    • debug
    • Test on flame and open c (Reproduction rate: 0%)

Graphics

Peter Chang

  • bug 846421 support snapshot for nested remote mozbrowser content
    • WIP
    • Add OOP support for taking snapshot by compositor
    • Measure the performance between taking snapshot by content and compositor
    • Consider security for compositor way

Morris Tseng

  • bug 987718 - [Text Selection] Display selection caret based on touch caret
    • r+
    • Separate perference from touch caret
    • All r+, wait for landing
  • bug 987040 - Implement mozbrowserSelection
    • WIP
    • Discussed with Ehsan for implementation detail.
    • Implemented using Ehsan's suggestion.

Jerry Shih

  • bug 987529 - Implement Vsync dispatch pipeline
    • OOAD | WIP
    • read vsync related bug
    • test auto-generating input event patch from Vincent
  • bug 998916 - Defer the webgl context restore until the app becomes foreground
    • r-
    • reviewer suggest to remove some if condition, use MOZ_ASSERT instead.
  • bug 1015852 - crash at GLContextProviderEGL::CreateForWindow in dolphin monkey test
    • debug
    • provide a debug patch to vender and wait for adb log.
  • bug 1008026 - wallpaper app will flicker when we scroll
    • r?
    • show thumbnail instead of full size wallpaper
  • bug 1012551 - picture preview in edit mode sometimes appear abnormal after unlock screen(tarako)
    • debug
    • add more log to debug

Chiajung Hung

  • bug 814524 - Make WebGLContext::TexImage2D avoid readback for video elements
    • r-
    • fix based on feedback
  • bug 880114 - Enhance render video-to-SkiaGL performance by GPU-based color space conversion
    • WIP
  • bug 918984 - Direct video stream texturing support on WebGL
    • WIP
    • fixing webidl related build problem

Solomon Chiou

  • PTO: 2 days
  • bug 1020745 - [Flame] Unable to use full HW composition in video player
    • debug
    • This bug seems only happen in 2.0. When playing the video in full screen, an special CanvasLayer(its width=480, height=854, however contains a 0x0 ImageHost) seems prevent the composition be performed by HwcComposer2D.
    • Another strange thing is: some layers of background was also can be observed in LayerScope, which is different from 1.4
  • bug 1020750 - [Flame] Unable to use full HW composition in Camera
    • debug
    • There is also a Canvas layer stop the composition be performed by HwcComposer2D. The canvas layers seems generated by a thumbnail icon which should not appear then. Still checking with gaia team to clarify the root cause.
  • bug 1017389 - [Flame] Unable to use full HW composition in most of use case
    • debug
    • Add some debug functions to help debugging, will upload those function for review later.

Manager

CJ Ku

  • bug 960897 - [Touch Caret] Touch caret sanity test
    • feedback
    • don't hard code string content at py file
  • bug 969372 - Intermittent | test_mediarecorder_record_no_timeslice.html | Test timed out.
    • feedback
  • bug 1019441 - [Text Selection] Text Selection sanity test
    • file
    • The test case need for Text selection landing.
  • bug 1012551 - [tarako] picture preview in edit mode sometimes appear abnormal after unlock screen
    • debug
    • By tracing libEGL log, we do use a texture after it had been destroyed... Jerry have found the root cause and upload a patch
  • bug 1020884 - Have a mechanism inside GLContext to detect redundant gl call
    • file
    • outcome of 1012551 debugging.
  • bug 651060 - Decode JPEGs to YCbCr and do RGB conversion on the GPU
    • OOAD
    • It's a good perf enhancement topic. Discuss with CTai and gfx team