Roll libyuv 455c66b:9107460

Roll libyuv 455c66b:9107460

(libyuv r1125)

BUG=400684
Committed: https://crrev.com/20429ab66f5eee7af8df407bb2cab4af624f650b
Cr-Commit-Position: refs/heads/master@{#300098}

[gordanac, 2014-10-10 14:51:37]

gordana.cmiljanovic@imgtec.com changed reviewers:
+ benm@google.com, fbarchard@chromium.org, fbarchard@google.com

[gordanac, 2014-10-10 14:51:50]

PTAL

[gordanac, 2014-10-10 18:40:55]

This is continued from:
https://codereview.chromium.org/544253002/

[gordanac, 2014-10-15 10:57:41]

gordana.cmiljanovic@imgtec.com changed reviewers:
+ amineer@chromium.org, amineer@google.com

[gordanac, 2014-10-15 10:57:42]

Updated to r1120. 
Please take a look.

[amineer_google, 2014-10-15 17:07:59]

On 2014/10/15 10:57:42, gordanac wrote:
> Updated to r1120. 
> Please take a look.

lgtm

[gordanac, 2014-10-15 17:15:05]

The CQ bit was checked by gordana.cmiljanovic@imgtec.com

[commit-bot: I haz the power, 2014-10-15 17:16:13]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/643123002/50001

[commit-bot: I haz the power, 2014-10-15 17:16:14]

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

[commit-bot: I haz the power, 2014-10-15 17:16:16]

No LGTM from a valid reviewer yet. Only full committers are accepted.
Even if an LGTM may have been provided, it was from a non-committer or
a provisional committer, _not_ a full super star committer.
See http://www.chromium.org/getting-involved/become-a-committer
Note that this has nothing to do with OWNERS files.

[amineer, 2014-10-15 18:27:49]

On 2014/10/15 17:16:16, I haz the power (commit-bot) wrote:
> No LGTM from a valid reviewer yet. Only full committers are accepted.
> Even if an LGTM may have been provided, it was from a non-committer or
> a provisional committer, _not_ a full super star committer.
> See http://www.chromium.org/getting-involved/become-a-committer
> Note that this has nothing to do with OWNERS files.

lgtm rd 2

[gordanac, 2014-10-15 19:03:08]

The CQ bit was checked by gordana.cmiljanovic@imgtec.com

[commit-bot: I haz the power, 2014-10-15 19:05:17]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/643123002/50001

[commit-bot: I haz the power, 2014-10-15 19:05:19]

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

[commit-bot: I haz the power, 2014-10-15 19:05:20]

No LGTM from a valid reviewer yet. Only full committers are accepted.
Even if an LGTM may have been provided, it was from a non-committer or
a provisional committer, _not_ a full super star committer.
See http://www.chromium.org/getting-involved/become-a-committer
Note that this has nothing to do with OWNERS files.

[dxie1, 2014-10-15 22:56:11]

dxie@google.com changed reviewers:
+ dxie@google.com, gordana.cmiljanovic@imgtec.com
- Gordana.Cmiljanovic@imgtec.com

[dxie1, 2014-10-15 22:56:11]

lgtm

[gordanac, 2014-10-15 23:23:38]

The CQ bit was checked by gordana.cmiljanovic@imgtec.com

[commit-bot: I haz the power, 2014-10-15 23:25:10]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/643123002/50001

[commit-bot: I haz the power, 2014-10-15 23:25:14]

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

[commit-bot: I haz the power, 2014-10-15 23:25:15]

No LGTM from a valid reviewer yet. Only full committers are accepted.
Even if an LGTM may have been provided, it was from a non-committer or
a provisional committer, _not_ a full super star committer.
See http://www.chromium.org/getting-involved/become-a-committer
Note that this has nothing to do with OWNERS files.

[fbarchard, 2014-10-16 01:12:49]

Do try bots pass?

[fbarchard, 2014-10-16 02:15:48]

lgtm

Looking thru the revisions
http://git.chromium.org/gitweb/?p=external/libyuv.git;a=summary
this is the head code revision, which is 1120, and try bots appear to pass on
arm platforms and copyrow_avx, so we should be good.
Note there was also a small mips change - on of the unaligned functions was
removed.

[fbarchard, 2014-10-16 02:16:01]

The CQ bit was checked by fbarchard@chromium.org

[commit-bot: I haz the power, 2014-10-16 02:19:22]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/643123002/50001

[commit-bot: I haz the power, 2014-10-16 04:21:46]

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

[commit-bot: I haz the power, 2014-10-16 04:21:47]

Try jobs failed on following builders:
  win_chromium_rel_swarming on tryserver.chromium.win
(http://build.chromium.org/p/tryserver.chromium.win/builders/win_chromium_rel_...)

[gordanac, 2014-10-16 08:24:57]

The CQ bit was checked by gordana.cmiljanovic@imgtec.com

[commit-bot: I haz the power, 2014-10-16 08:25:18]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/643123002/50001

[commit-bot: I haz the power, 2014-10-16 10:28:19]

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

[commit-bot: I haz the power, 2014-10-16 10:28:20]

Try jobs failed on following builders:
  win_chromium_x64_rel_swarming on tryserver.chromium.win
(http://build.chromium.org/p/tryserver.chromium.win/builders/win_chromium_x64_...)

[Ben Murdoch, 2014-10-16 10:30:50]

On 2014/10/16 10:28:20, I haz the power (commit-bot) wrote:
> Try jobs failed on following builders:
>   win_chromium_x64_rel_swarming on tryserver.chromium.win
>
(http://build.chromium.org/p/tryserver.chromium.win/builders/win_chromium_x64_...)

Looks like this could be a legit failure?

YuvToRgbTest.Clipping

[chromium-reviews, 2014-10-16 23:33:18]

ya, I think I see the problem... unaligned memory.
filed bug, fix coming.  Could also turn off assembly for win64 with a macro


On Thu, Oct 16, 2014 at 3:30 AM, <benm@google.com> wrote:

> On 2014/10/16 10:28:20, I haz the power (commit-bot) wrote:
>
>> Try jobs failed on following builders:
>>    win_chromium_x64_rel_swarming on tryserver.chromium.win
>>
>
> (http://build.chromium.org/p/tryserver.chromium.win/
> builders/win_chromium_x64_rel_swarming/builds/19666)
>
> Looks like this could be a legit failure?
>
> YuvToRgbTest.Clipping
>
> https://codereview.chromium.org/643123002/
>

To unsubscribe from this group and stop receiving emails from it, send an email
to chromium-reviews+unsubscribe@chromium.org.

[gordanac, 2014-10-17 12:24:31]

Giving a try to r1125
(https://chromium.googlesource.com/external/libyuv/+/9107460c7f76a10cc4122d91e...)

[gordanac, 2014-10-17 12:25:06]

The CQ bit was checked by gordana.cmiljanovic@imgtec.com

[commit-bot: I haz the power, 2014-10-17 12:25:22]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/643123002/70001

[commit-bot: I haz the power, 2014-10-17 13:34:30]

Committed patchset #3 (id:70001)

[commit-bot: I haz the power, 2014-10-17 13:35:38]

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