Reland of rotate frames correctly for back camera

Reland of ate frames correctly for back camera (patchset #1 id:1 of https://codereview.chromium.org/2554903002/ )

Reason for revert:
try to reland https://codereview.chromium.org/2508803002/#

Original issue's description:
> Revert of Rotate frames correctly for back camera (patchset #17 id:320001 of https://codereview.chromium.org/2508803002/ )
>
> Reason for revert:
> Causes compile failures:
>
> https://build.chromium.org/p/chromium.chromiumos/builders/Linux%20ChromiumOS%20Builder%20%28dbg%29/builds/81371/steps/compile/logs/stdio
>
> Original issue's description:
> > Rotate frames correctly for back camera
> >
> > In 90 and 270 degree tablet mode, back camera used to show upside-down
> > frames. We need to rotate the frames correctly for back cameras.
> >
> > BUG=chrome-os-partner:56762
> > TEST=Verify that back camera shows correct preview in all rotations.
> >
> > Committed: https://crrev.com/fd4e73efe59269463f10743694597074ef2ceefb
> > Cr-Commit-Position: refs/heads/master@{#436594}
>
> TBR=wuchengli@chromium.org,posciak@chromium.org,henryhsu@chromium.org,mcasas@chromium.org,kcwu@chromium.org,shenghao@google.com,phistuck@gmail.com,shenghao@chromium.org
> # Skipping CQ checks because original CL landed less than 1 days ago.
> NOPRESUBMIT=true
> NOTREECHECKS=true
> NOTRY=true
> BUG=chrome-os-partner:56762
>
> Committed: https://crrev.com/78ba7d636db7e127c8587977482639c387644e6f
> Cr-Commit-Position: refs/heads/master@{#436598}

TBR=wuchengli@chromium.org,posciak@chromium.org,henryhsu@chromium.org,mcasas@chromium.org,kcwu@chromium.org,shenghao@google.com,phistuck@gmail.com,hbos@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=chrome-os-partner:56762
Committed: https://crrev.com/ec8cf0e66777785a7685629ffb045016e0f12d74
Cr-Commit-Position: refs/heads/master@{#437206}

[shenghao, 2016-12-07 06:11:09]

Created Reland of ate frames correctly for back camera

[shenghao, 2016-12-07 06:12:01]

The CQ bit was checked by shenghao@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-12-07 06:12:11]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-12-07 06:12:12]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-12-07 06:12:13]

Dry run: This issue passed the CQ dry run.

[shenghao, 2016-12-07 06:24:14]

Description was changed from

==========
Reland of ate frames correctly for back camera (patchset #1 id:1 of
https://codereview.chromium.org/2554903002/ )

Reason for revert:
try to reland https://codereview.chromium.org/2508803002/#

Original issue's description:
> Revert of Rotate frames correctly for back camera (patchset #17 id:320001 of
https://codereview.chromium.org/2508803002/ )
> 
> Reason for revert:
> Causes compile failures:
> 
>
https://build.chromium.org/p/chromium.chromiumos/builders/Linux%20ChromiumOS%...
> 
> Original issue's description:
> > Rotate frames correctly for back camera
> >
> > In 90 and 270 degree tablet mode, back camera used to show upside-down
> > frames. We need to rotate the frames correctly for back cameras.
> >
> > BUG=56762
> > TEST=Verify that back camera shows correct preview in all rotations.
> >
> > Committed: https://crrev.com/fd4e73efe59269463f10743694597074ef2ceefb
> > Cr-Commit-Position: refs/heads/master@{#436594}
> 
>
TBR=wuchengli@chromium.org,posciak@chromium.org,henryhsu@chromium.org,mcasas@...
> # Skipping CQ checks because original CL landed less than 1 days ago.
> NOPRESUBMIT=true
> NOTREECHECKS=true
> NOTRY=true
> BUG=56762
> 
> Committed: https://crrev.com/78ba7d636db7e127c8587977482639c387644e6f
> Cr-Commit-Position: refs/heads/master@{#436598}

TBR=wuchengli@chromium.org,posciak@chromium.org,henryhsu@chromium.org,mcasas@...
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=56762
==========

to

==========
Reland of ate frames correctly for back camera (patchset #1 id:1 of
https://codereview.chromium.org/2554903002/ )

Reason for revert:
try to reland https://codereview.chromium.org/2508803002/#

Original issue's description:
> Revert of Rotate frames correctly for back camera (patchset #17 id:320001 of
https://codereview.chromium.org/2508803002/ )
> 
> Reason for revert:
> Causes compile failures:
> 
>
https://build.chromium.org/p/chromium.chromiumos/builders/Linux%20ChromiumOS%...
> 
> Original issue's description:
> > Rotate frames correctly for back camera
> >
> > In 90 and 270 degree tablet mode, back camera used to show upside-down
> > frames. We need to rotate the frames correctly for back cameras.
> >
> > BUG=56762
> > TEST=Verify that back camera shows correct preview in all rotations.
> >
> > Committed: https://crrev.com/fd4e73efe59269463f10743694597074ef2ceefb
> > Cr-Commit-Position: refs/heads/master@{#436594}
> 
>
TBR=wuchengli@chromium.org,posciak@chromium.org,henryhsu@chromium.org,mcasas@...
> # Skipping CQ checks because original CL landed less than 1 days ago.
> NOPRESUBMIT=true
> NOTREECHECKS=true
> NOTRY=true
> BUG=56762
> 
> Committed: https://crrev.com/78ba7d636db7e127c8587977482639c387644e6f
> Cr-Commit-Position: refs/heads/master@{#436598}

TBR=wuchengli@chromium.org,posciak@chromium.org,henryhsu@chromium.org,mcasas@...
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=56762
==========

[shenghao, 2016-12-07 06:30:44]

Description was changed from

==========
Reland of ate frames correctly for back camera (patchset #1 id:1 of
https://codereview.chromium.org/2554903002/ )

Reason for revert:
try to reland https://codereview.chromium.org/2508803002/#

Original issue's description:
> Revert of Rotate frames correctly for back camera (patchset #17 id:320001 of
https://codereview.chromium.org/2508803002/ )
> 
> Reason for revert:
> Causes compile failures:
> 
>
https://build.chromium.org/p/chromium.chromiumos/builders/Linux%20ChromiumOS%...
> 
> Original issue's description:
> > Rotate frames correctly for back camera
> >
> > In 90 and 270 degree tablet mode, back camera used to show upside-down
> > frames. We need to rotate the frames correctly for back cameras.
> >
> > BUG=56762
> > TEST=Verify that back camera shows correct preview in all rotations.
> >
> > Committed: https://crrev.com/fd4e73efe59269463f10743694597074ef2ceefb
> > Cr-Commit-Position: refs/heads/master@{#436594}
> 
>
TBR=wuchengli@chromium.org,posciak@chromium.org,henryhsu@chromium.org,mcasas@...
> # Skipping CQ checks because original CL landed less than 1 days ago.
> NOPRESUBMIT=true
> NOTREECHECKS=true
> NOTRY=true
> BUG=56762
> 
> Committed: https://crrev.com/78ba7d636db7e127c8587977482639c387644e6f
> Cr-Commit-Position: refs/heads/master@{#436598}

TBR=wuchengli@chromium.org,posciak@chromium.org,henryhsu@chromium.org,mcasas@...
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=56762
==========

to

==========
Reland of ate frames correctly for back camera (patchset #1 id:1 of
https://codereview.chromium.org/2554903002/ )

Reason for revert:
try to reland https://codereview.chromium.org/2508803002/#

Original issue's description:
> Revert of Rotate frames correctly for back camera (patchset #17 id:320001 of
https://codereview.chromium.org/2508803002/ )
> 
> Reason for revert:
> Causes compile failures:
> 
>
https://build.chromium.org/p/chromium.chromiumos/builders/Linux%20ChromiumOS%...
> 
> Original issue's description:
> > Rotate frames correctly for back camera
> >
> > In 90 and 270 degree tablet mode, back camera used to show upside-down
> > frames. We need to rotate the frames correctly for back cameras.
> >
> > BUG=56762
> > TEST=Verify that back camera shows correct preview in all rotations.
> >
> > Committed: https://crrev.com/fd4e73efe59269463f10743694597074ef2ceefb
> > Cr-Commit-Position: refs/heads/master@{#436594}
> 
>
TBR=wuchengli@chromium.org,posciak@chromium.org,henryhsu@chromium.org,mcasas@...
> # Skipping CQ checks because original CL landed less than 1 days ago.
> NOPRESUBMIT=true
> NOTREECHECKS=true
> NOTRY=true
> BUG=56762
> 
> Committed: https://crrev.com/78ba7d636db7e127c8587977482639c387644e6f
> Cr-Commit-Position: refs/heads/master@{#436598}

TBR=wuchengli@chromium.org,posciak@chromium.org,henryhsu@chromium.org,mcasas@...
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=chrome-os-partner:56762
==========

[mcasas, 2016-12-07 06:42:52]

On 2016/12/07 06:11:09, shenghao wrote:
> Created Reland of ate frames correctly for back camera

When you reland, please keep PS1 as a verbatim copy 
of the landed-and-reverted crrev.com, and then add the
extra deltas as successive patchsets, so it's easier for
reviewers and QA to follow what happens.

[PhistucK, 2016-12-07 06:57:51]

And if you can just somehow change the issue number to have a prepended
chrome-os-partner:, that would be great.

[shenghao, 2016-12-07 08:14:23]

On 2016/12/07 06:57:51, PhistucK wrote:
> And if you can just somehow change the issue number to have a prepended
> chrome-os-partner:, that would be great.

The correct issue number has been entered.

[shenghao, 2016-12-07 08:14:56]

Description was changed from

==========
Reland of ate frames correctly for back camera (patchset #1 id:1 of
https://codereview.chromium.org/2554903002/ )

Reason for revert:
try to reland https://codereview.chromium.org/2508803002/#

Original issue's description:
> Revert of Rotate frames correctly for back camera (patchset #17 id:320001 of
https://codereview.chromium.org/2508803002/ )
> 
> Reason for revert:
> Causes compile failures:
> 
>
https://build.chromium.org/p/chromium.chromiumos/builders/Linux%20ChromiumOS%...
> 
> Original issue's description:
> > Rotate frames correctly for back camera
> >
> > In 90 and 270 degree tablet mode, back camera used to show upside-down
> > frames. We need to rotate the frames correctly for back cameras.
> >
> > BUG=56762
> > TEST=Verify that back camera shows correct preview in all rotations.
> >
> > Committed: https://crrev.com/fd4e73efe59269463f10743694597074ef2ceefb
> > Cr-Commit-Position: refs/heads/master@{#436594}
> 
>
TBR=wuchengli@chromium.org,posciak@chromium.org,henryhsu@chromium.org,mcasas@...
> # Skipping CQ checks because original CL landed less than 1 days ago.
> NOPRESUBMIT=true
> NOTREECHECKS=true
> NOTRY=true
> BUG=56762
> 
> Committed: https://crrev.com/78ba7d636db7e127c8587977482639c387644e6f
> Cr-Commit-Position: refs/heads/master@{#436598}

TBR=wuchengli@chromium.org,posciak@chromium.org,henryhsu@chromium.org,mcasas@...
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=chrome-os-partner:56762
==========

to

==========
Reland of ate frames correctly for back camera (patchset #1 id:1 of
https://codereview.chromium.org/2554903002/ )

Reason for revert:
try to reland https://codereview.chromium.org/2508803002/#

Original issue's description:
> Revert of Rotate frames correctly for back camera (patchset #17 id:320001 of
https://codereview.chromium.org/2508803002/ )
> 
> Reason for revert:
> Causes compile failures:
> 
>
https://build.chromium.org/p/chromium.chromiumos/builders/Linux%20ChromiumOS%...
> 
> Original issue's description:
> > Rotate frames correctly for back camera
> >
> > In 90 and 270 degree tablet mode, back camera used to show upside-down
> > frames. We need to rotate the frames correctly for back cameras.
> >
> > BUG=chrome-os-partner:56762
> > TEST=Verify that back camera shows correct preview in all rotations.
> >
> > Committed: https://crrev.com/fd4e73efe59269463f10743694597074ef2ceefb
> > Cr-Commit-Position: refs/heads/master@{#436594}
> 
>
TBR=wuchengli@chromium.org,posciak@chromium.org,henryhsu@chromium.org,mcasas@...
> # Skipping CQ checks because original CL landed less than 1 days ago.
> NOPRESUBMIT=true
> NOTREECHECKS=true
> NOTRY=true
> BUG=chrome-os-partner:56762
> 
> Committed: https://crrev.com/78ba7d636db7e127c8587977482639c387644e6f
> Cr-Commit-Position: refs/heads/master@{#436598}

TBR=wuchengli@chromium.org,posciak@chromium.org,henryhsu@chromium.org,mcasas@...
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=chrome-os-partner:56762
==========

[PhistucK, 2016-12-07 08:17:02]

On 2016/12/07 08:14:23, shenghao wrote:
> On 2016/12/07 06:57:51, PhistucK wrote:
> > And if you can just somehow change the issue number to have a prepended
> > chrome-os-partner:, that would be great.
> 
> The correct issue number has been entered.

Yes, but the original description still has the bare issue number... I wonder
whether Bugdroid would pick it up and post a commit comment there as well. I
hope not.

[shenghao, 2016-12-07 09:41:07]

On 2016/12/07 08:17:02, PhistucK wrote:
> On 2016/12/07 08:14:23, shenghao wrote:
> > On 2016/12/07 06:57:51, PhistucK wrote:
> > > And if you can just somehow change the issue number to have a prepended
> > > chrome-os-partner:, that would be great.
> > 
> > The correct issue number has been entered.
> 
> Yes, but the original description still has the bare issue number... I wonder
> whether Bugdroid would pick it up and post a commit comment there as well. I
> hope not.

Ok. I also changed the original issue numbers.

[wuchengli, 2016-12-08 07:18:55]

lgtm

[shenghao, 2016-12-08 07:58:17]

The CQ bit was checked by shenghao@chromium.org

[commit-bot: I haz the power, 2016-12-08 07:58:37]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-12-08 07:59:17]

CQ is committing da patch.

Bot data: {"patchset_id": 200001, "attempt_start_ts": 1481183897375580,
"parent_rev": "f667f8b06e322a1565db31b2606087b36872d75d", "commit_rev":
"152bf2abd7da05c8f86307e47221967e3ad3c1b9"}

[commit-bot: I haz the power, 2016-12-08 07:59:45]

Description was changed from

==========
Reland of ate frames correctly for back camera (patchset #1 id:1 of
https://codereview.chromium.org/2554903002/ )

Reason for revert:
try to reland https://codereview.chromium.org/2508803002/#

Original issue's description:
> Revert of Rotate frames correctly for back camera (patchset #17 id:320001 of
https://codereview.chromium.org/2508803002/ )
> 
> Reason for revert:
> Causes compile failures:
> 
>
https://build.chromium.org/p/chromium.chromiumos/builders/Linux%20ChromiumOS%...
> 
> Original issue's description:
> > Rotate frames correctly for back camera
> >
> > In 90 and 270 degree tablet mode, back camera used to show upside-down
> > frames. We need to rotate the frames correctly for back cameras.
> >
> > BUG=chrome-os-partner:56762
> > TEST=Verify that back camera shows correct preview in all rotations.
> >
> > Committed: https://crrev.com/fd4e73efe59269463f10743694597074ef2ceefb
> > Cr-Commit-Position: refs/heads/master@{#436594}
> 
>
TBR=wuchengli@chromium.org,posciak@chromium.org,henryhsu@chromium.org,mcasas@...
> # Skipping CQ checks because original CL landed less than 1 days ago.
> NOPRESUBMIT=true
> NOTREECHECKS=true
> NOTRY=true
> BUG=chrome-os-partner:56762
> 
> Committed: https://crrev.com/78ba7d636db7e127c8587977482639c387644e6f
> Cr-Commit-Position: refs/heads/master@{#436598}

TBR=wuchengli@chromium.org,posciak@chromium.org,henryhsu@chromium.org,mcasas@...
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=chrome-os-partner:56762
==========

to

==========
Reland of ate frames correctly for back camera (patchset #1 id:1 of
https://codereview.chromium.org/2554903002/ )

Reason for revert:
try to reland https://codereview.chromium.org/2508803002/#

Original issue's description:
> Revert of Rotate frames correctly for back camera (patchset #17 id:320001 of
https://codereview.chromium.org/2508803002/ )
> 
> Reason for revert:
> Causes compile failures:
> 
>
https://build.chromium.org/p/chromium.chromiumos/builders/Linux%20ChromiumOS%...
> 
> Original issue's description:
> > Rotate frames correctly for back camera
> >
> > In 90 and 270 degree tablet mode, back camera used to show upside-down
> > frames. We need to rotate the frames correctly for back cameras.
> >
> > BUG=chrome-os-partner:56762
> > TEST=Verify that back camera shows correct preview in all rotations.
> >
> > Committed: https://crrev.com/fd4e73efe59269463f10743694597074ef2ceefb
> > Cr-Commit-Position: refs/heads/master@{#436594}
> 
>
TBR=wuchengli@chromium.org,posciak@chromium.org,henryhsu@chromium.org,mcasas@...
> # Skipping CQ checks because original CL landed less than 1 days ago.
> NOPRESUBMIT=true
> NOTREECHECKS=true
> NOTRY=true
> BUG=chrome-os-partner:56762
> 
> Committed: https://crrev.com/78ba7d636db7e127c8587977482639c387644e6f
> Cr-Commit-Position: refs/heads/master@{#436598}

TBR=wuchengli@chromium.org,posciak@chromium.org,henryhsu@chromium.org,mcasas@...
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=chrome-os-partner:56762
==========

[commit-bot: I haz the power, 2016-12-08 07:59:46]

Committed patchset #3 (id:200001)

[commit-bot: I haz the power, 2016-12-08 08:02:12]

Description was changed from

==========
Reland of ate frames correctly for back camera (patchset #1 id:1 of
https://codereview.chromium.org/2554903002/ )

Reason for revert:
try to reland https://codereview.chromium.org/2508803002/#

Original issue's description:
> Revert of Rotate frames correctly for back camera (patchset #17 id:320001 of
https://codereview.chromium.org/2508803002/ )
> 
> Reason for revert:
> Causes compile failures:
> 
>
https://build.chromium.org/p/chromium.chromiumos/builders/Linux%20ChromiumOS%...
> 
> Original issue's description:
> > Rotate frames correctly for back camera
> >
> > In 90 and 270 degree tablet mode, back camera used to show upside-down
> > frames. We need to rotate the frames correctly for back cameras.
> >
> > BUG=chrome-os-partner:56762
> > TEST=Verify that back camera shows correct preview in all rotations.
> >
> > Committed: https://crrev.com/fd4e73efe59269463f10743694597074ef2ceefb
> > Cr-Commit-Position: refs/heads/master@{#436594}
> 
>
TBR=wuchengli@chromium.org,posciak@chromium.org,henryhsu@chromium.org,mcasas@...
> # Skipping CQ checks because original CL landed less than 1 days ago.
> NOPRESUBMIT=true
> NOTREECHECKS=true
> NOTRY=true
> BUG=chrome-os-partner:56762
> 
> Committed: https://crrev.com/78ba7d636db7e127c8587977482639c387644e6f
> Cr-Commit-Position: refs/heads/master@{#436598}

TBR=wuchengli@chromium.org,posciak@chromium.org,henryhsu@chromium.org,mcasas@...
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=chrome-os-partner:56762
==========

to

==========
Reland of ate frames correctly for back camera (patchset #1 id:1 of
https://codereview.chromium.org/2554903002/ )

Reason for revert:
try to reland https://codereview.chromium.org/2508803002/#

Original issue's description:
> Revert of Rotate frames correctly for back camera (patchset #17 id:320001 of
https://codereview.chromium.org/2508803002/ )
>
> Reason for revert:
> Causes compile failures:
>
>
https://build.chromium.org/p/chromium.chromiumos/builders/Linux%20ChromiumOS%...
>
> Original issue's description:
> > Rotate frames correctly for back camera
> >
> > In 90 and 270 degree tablet mode, back camera used to show upside-down
> > frames. We need to rotate the frames correctly for back cameras.
> >
> > BUG=chrome-os-partner:56762
> > TEST=Verify that back camera shows correct preview in all rotations.
> >
> > Committed: https://crrev.com/fd4e73efe59269463f10743694597074ef2ceefb
> > Cr-Commit-Position: refs/heads/master@{#436594}
>
>
TBR=wuchengli@chromium.org,posciak@chromium.org,henryhsu@chromium.org,mcasas@...
> # Skipping CQ checks because original CL landed less than 1 days ago.
> NOPRESUBMIT=true
> NOTREECHECKS=true
> NOTRY=true
> BUG=chrome-os-partner:56762
>
> Committed: https://crrev.com/78ba7d636db7e127c8587977482639c387644e6f
> Cr-Commit-Position: refs/heads/master@{#436598}

TBR=wuchengli@chromium.org,posciak@chromium.org,henryhsu@chromium.org,mcasas@...
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=chrome-os-partner:56762

Committed: https://crrev.com/ec8cf0e66777785a7685629ffb045016e0f12d74
Cr-Commit-Position: refs/heads/master@{#437206}
==========

[commit-bot: I haz the power, 2016-12-08 08:02:13]

Patchset 3 (id:??) landed as
https://crrev.com/ec8cf0e66777785a7685629ffb045016e0f12d74
Cr-Commit-Position: refs/heads/master@{#437206}