New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 710068 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Last visit > 30 days ago
Closed: Apr 2017
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: All
Pri: 2
Type: Bug



Sign in to add a comment

Use incoming frame's rotation while converting it to I420

Project Member Reported by c.pa...@samsung.com, Apr 10 2017

Issue description

UserAgent: Mozilla/5.0 (Windows NT 6.1; WOW64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/57.0.2987.133 Safari/537.36

Steps to reproduce the problem:
Rotation is not considered while converting incoming frame to I420.

What is the expected behavior?

What went wrong?
NA

Did this work before? N/A 

Does this work in other browsers? N/A

Chrome version: 57.0.2987.133  Channel: stable
OS Version: 6.1 (Windows 7, Windows Server 2008 R2)
Flash Version: Shockwave Flash 25.0 r0
 
Components: -Blink>Media Blink>MediaRecording
Labels: M-59
Project Member

Comment 2 by bugdroid1@chromium.org, Apr 11 2017

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/b06713810a8546bf4a5a135fe1fa2ef66d778e92

commit b06713810a8546bf4a5a135fe1fa2ef66d778e92
Author: c.padhi <c.padhi@samsung.com>
Date: Tue Apr 11 03:51:01 2017

Use incoming frame's rotation while converting it to I420

Rotation was not considered while converting incoming frame to I420.
This CL uses incoming frame's rotation for the conversion.

BUG= 710068 
TEST=H264 connection where recording the rotated remote stream on a
Mac results in a properly rotated stream.

Review-Url: https://codereview.chromium.org/2798863005
Cr-Commit-Position: refs/heads/master@{#463521}

[modify] https://crrev.com/b06713810a8546bf4a5a135fe1fa2ef66d778e92/content/renderer/media_recorder/video_track_recorder.cc

Cc: kkaluri@chromium.org
Labels: Needs-Feedback
c.padhi@ could you please let us know steps to verify this issue from TE-End.

Thank You...

Comment 4 by mcasas@chromium.org, Apr 28 2017

Labels: -OS-Windows OS-All
Owner: emir...@chromium.org
Status: Fixed (was: Unconfirmed)

Comment 5 by c.pa...@samsung.com, May 19 2017

Cc: emir...@chromium.org
Owner: c.pa...@samsung.com

Sign in to add a comment