Video Player: Disables cast feature by default

Video Player: Disables cast feature by default

The feature will work only when the flag is on.

BUG=305511
TEST=Confirms the cast icon doesn't appear when the flag is off.

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

[yoshiki, 2014-08-14 09:22:47]

@hirono, PTAL. Thanks.

[hirono, 2014-08-14 09:39:19]

lgtm with a nit?

https://codereview.chromium.org/471903002/diff/20001/chrome/browser/chromeos/...
File chrome/browser/chromeos/extensions/file_manager/private_api_drive.cc
(right):

https://codereview.chromium.org/471903002/diff/20001/chrome/browser/chromeos/...
chrome/browser/chromeos/extensions/file_manager/private_api_drive.cc:905:
base::CommandLine* command_line = base::CommandLine::ForCurrentProcess();
nit: const base::CommandLine* const command_line

[hirono, 2014-08-14 09:40:27]

On 2014/08/14 09:39:19, hirono wrote:
> lgtm with a nit?
> 
>
https://codereview.chromium.org/471903002/diff/20001/chrome/browser/chromeos/...
> File chrome/browser/chromeos/extensions/file_manager/private_api_drive.cc
> (right):
> 
>
https://codereview.chromium.org/471903002/diff/20001/chrome/browser/chromeos/...
> chrome/browser/chromeos/extensions/file_manager/private_api_drive.cc:905:
> base::CommandLine* command_line = base::CommandLine::ForCurrentProcess();
> nit: const base::CommandLine* const command_line

Sorry. I mistook the termination symbol. lgtm with a nit? -> lgtm with a nit.

[yoshiki, 2014-08-14 10:21:14]

Thanks.

https://codereview.chromium.org/471903002/diff/20001/chrome/browser/chromeos/...
File chrome/browser/chromeos/extensions/file_manager/private_api_drive.cc
(right):

https://codereview.chromium.org/471903002/diff/20001/chrome/browser/chromeos/...
chrome/browser/chromeos/extensions/file_manager/private_api_drive.cc:905:
base::CommandLine* command_line = base::CommandLine::ForCurrentProcess();
On 2014/08/14 09:39:18, hirono wrote:
> nit: const base::CommandLine* const command_line

Done.

[yoshiki, 2014-08-14 10:21:17]

The CQ bit was checked by yoshiki@chromium.org

[commit-bot: I haz the power, 2014-08-14 10:24:41]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/yoshiki@chromium.org/471903002/40001

[commit-bot: I haz the power, 2014-08-14 15:47:33]

FYI, CQ is re-trying this CL (attempt #1).
The failing builders are:
  linux_gpu on tryserver.chromium.gpu
(http://build.chromium.org/p/tryserver.chromium.gpu/builders/linux_gpu/builds/...)

[commit-bot: I haz the power, 2014-08-14 17:03:58]

Committed patchset #2 (40001) as 289621