Refactoring: Changing variable type in MediaCanPlayTypeTest

Refactoring: Changing variable type in MediaCanPlayTypeTest

Changing const char* kFoo to const char kFoo[]

Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=269772

[amogh.bihani, 2014-04-11 10:40:34]

PTAL.
This is in accordance to what we discussed in the other CL
https://codereview.chromium.org/230413003/

[xhwang, 2014-04-11 17:03:14]

LGTM, thanks for cleaning this up!

[amogh.bihani, 2014-04-12 02:13:29]

On 2014/04/11 17:03:14, xhwang wrote:
> LGTM, thanks for cleaning this up!

Thanks for the review :)

[amogh.bihani, 2014-04-12 02:13:34]

The CQ bit was checked by amogh.bihani@samsung.com

[commit-bot: I haz the power, 2014-04-12 02:13:51]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/amogh.bihani@samsung.com/233713005/1

[commit-bot: I haz the power, 2014-04-12 03:09:06]

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

[commit-bot: I haz the power, 2014-04-12 03:09:06]

Try jobs failed on following builders:
  tryserver.chromium on mac_chromium_rel

[amogh.bihani, 2014-04-12 09:53:52]

The CQ bit was checked by amogh.bihani@samsung.com

[commit-bot: I haz the power, 2014-04-12 09:54:08]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/amogh.bihani@samsung.com/233713005/20001

[commit-bot: I haz the power, 2014-04-12 10:52:54]

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

[commit-bot: I haz the power, 2014-04-12 10:52:55]

Retried try job too often on android_clang_dbg for step(s) slave_steps
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=android_cl...

[amogh.bihani, 2014-04-14 04:13:33]

PTAL. The android_clang_dbg was failing because these tests are currently
disabled for android (crbug 357665).
So the compiler was showing warning because of unused variables (and all
warnings are treated as errors)
So I have disabled everything till I fix that bug.

[amogh.bihani, 2014-04-15 04:08:40]

The CQ bit was checked by amogh.bihani@samsung.com

[commit-bot: I haz the power, 2014-04-15 04:09:14]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/amogh.bihani@samsung.com/233713005/40001

[amogh.bihani, 2014-04-15 06:08:05]

The CQ bit was unchecked by amogh.bihani@samsung.com

[amogh.bihani, 2014-04-16 08:36:41]

I think we won't be able to merge this before the tests are re-enabled for
android as the compiler does not allow unused variables.

[amogh.bihani, 2014-05-12 09:37:06]

The CQ bit was checked by amogh.bihani@samsung.com

[commit-bot: I haz the power, 2014-05-12 09:37:19]

CQ is trying da patch. Follow status at

https://chromium-status.appspot.com/cq/amogh.bihani@samsung.com/233713005/100001

[commit-bot: I haz the power, 2014-05-12 11:12:51]

FYI, CQ is re-trying this CL (attempt #1).
Please consider checking whether the failures are real,
and report flakes to chrome-troopers@google.com.
The failing builders are:
  mac_chromium_rel on tryserver.chromium
(http://build.chromium.org/p/tryserver.chromium/builders/mac_chromium_rel/buil...)

[commit-bot: I haz the power, 2014-05-12 12:42:45]

Change committed as 269772