Remove accelerated hardware decode blacklist entry for Mac OS X.

Remove accelerated hardware decode blacklist entry for Mac OS X.

Now that VTVideoDecodeAccelerator exists, this entry is not necessary.

BUG=133828
Committed: https://crrev.com/c0e69446b02848fce2e311662a43b8e782304ae3
Cr-Commit-Position: refs/heads/master@{#311622}

[sandersd (OOO until July 31), 2015-01-14 23:14:04]

sandersd@chromium.org changed reviewers:
+ scherkus@chromium.org

[scherkus (not reviewing), 2015-01-14 23:18:06]

lgtm

[sandersd (OOO until July 31), 2015-01-14 23:22:06]

sandersd@chromium.org changed reviewers:
+ sievers@chromium.org

[sandersd (OOO until July 31), 2015-01-14 23:22:06]

sievers@chromium.org: Please review changes in gpu/config.

[no sievers, 2015-01-15 00:31:39]

https://codereview.chromium.org/849143002/diff/1/gpu/config/software_renderin...
File gpu/config/software_rendering_list_json.cc (right):

https://codereview.chromium.org/849143002/diff/1/gpu/config/software_renderin...
gpu/config/software_rendering_list_json.cc:21: "version": "9.15",
bump this number

https://codereview.chromium.org/849143002/diff/1/gpu/config/software_renderin...
gpu/config/software_rendering_list_json.cc:525: "os": {
can this simply specify os: "linux" now instead of an exception list with the
sum of the opposite (cros, mac, win, android)?

[sandersd (OOO until July 31), 2015-01-15 00:47:51]

https://codereview.chromium.org/849143002/diff/1/gpu/config/software_renderin...
File gpu/config/software_rendering_list_json.cc (right):

https://codereview.chromium.org/849143002/diff/1/gpu/config/software_renderin...
gpu/config/software_rendering_list_json.cc:21: "version": "9.15",
On 2015/01/15 00:31:38, sievers wrote:
> bump this number

Done.

https://codereview.chromium.org/849143002/diff/1/gpu/config/software_renderin...
gpu/config/software_rendering_list_json.cc:525: "os": {
On 2015/01/15 00:31:38, sievers wrote:
> can this simply specify os: "linux" now instead of an exception list with the
> sum of the opposite (cros, mac, win, android)?

That is much better, thanks! (Done.)

[no sievers, 2015-01-15 00:50:54]

lgtm

[sandersd (OOO until July 31), 2015-01-15 02:19:19]

The CQ bit was checked by sandersd@chromium.org

[commit-bot: I haz the power, 2015-01-15 02:19:59]

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

[commit-bot: I haz the power, 2015-01-15 03:17:44]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2015-01-15 03:18:36]

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

[ricow1, 2015-01-15 13:55:42]

A revert of this CL (patchset #2 id:20001) has been created in
https://codereview.chromium.org/853563003/ by ricow@google.com.

The reason for reverting is: This seems related to the regression here:
https://build.chromium.org/p/chromium.gpu/builders/Mac%20Release%20%28Intel%2....

[acumoncxm_gmail.com, 2015-02-10 06:42:04]

https://build.chromium.org/p/chromium.gpu/builders/Mac%20Release%20%28Intel%2....
 
this returns 404 Not found.

On Thursday, January 15, 2015 at 5:55:43 AM UTC-8, ri...@google.com wrote:
>
> A revert of this CL (patchset #2 id:20001) has been created in 
> https://codereview.chromium.org/853563003/ by ri...@google.com 
> <javascript:>. 
>
> The reason for reverting is: This seems related to the regression here: 
>
>
https://build.chromium.org/p/chromium.gpu/builders/Mac%20Release%20%28Intel%2....

>
>
> https://codereview.chromium.org/849143002/ 
>

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