Delete dead code: OmxVideoDecodeAccelerator is unused.

Delete dead code: OmxVideoDecodeAccelerator is unused.

OVDA has been dead code in the tree since 3/13 and m27
https://gerrit.chromium.org/gerrit/gitweb?p=chromiumos/platform/login_manager.git;a=commit;h=934e55cd90c2fd7a5b89fe201b251d542e896f46

BUG=223194
NOTRY=true

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

[Ami GONE FROM CHROMIUM, 2013-08-01 17:07:47]

piman: since the changes are fairly trivial and I need your OWNERS review
anyway, please review this for realz.

sheu/posciak/scherkus: FYI.

[scherkus (not reviewing), 2013-08-01 18:15:56]

\o/

https://codereview.chromium.org/21584002/diff/1/content/public/common/content...
File content/public/common/content_switches.cc (left):

https://codereview.chromium.org/21584002/diff/1/content/public/common/content...
content/public/common/content_switches.cc:871: const char kUseExynosVda[]       
          = "use-exynos-vda";
is there a cros-side change to remove passing in this flag in
session_manager_setup.sh?

[sheu, 2013-08-01 18:18:11]

https://codereview.chromium.org/21584002/diff/1/content/public/common/content...
File content/public/common/content_switches.cc (left):

https://codereview.chromium.org/21584002/diff/1/content/public/common/content...
content/public/common/content_switches.cc:871: const char kUseExynosVda[]       
          = "use-exynos-vda";
On 2013/08/01 18:15:56, scherkus wrote:
> is there a cros-side change to remove passing in this flag in
> session_manager_setup.sh?

I can do that.  Does Chrome barf when starting with unrecognized flags, or can I
wait until this lands to do it?

[scherkus (not reviewing), 2013-08-01 18:19:38]

On 2013/08/01 18:18:11, sheu wrote:
>
https://codereview.chromium.org/21584002/diff/1/content/public/common/content...
> File content/public/common/content_switches.cc (left):
> 
>
https://codereview.chromium.org/21584002/diff/1/content/public/common/content...
> content/public/common/content_switches.cc:871: const char kUseExynosVda[]     
 
>           = "use-exynos-vda";
> On 2013/08/01 18:15:56, scherkus wrote:
> > is there a cros-side change to remove passing in this flag in
> > session_manager_setup.sh?
> 
> I can do that.  Does Chrome barf when starting with unrecognized flags, or can
I
> wait until this lands to do it?

fairly certain it doesn't barf seeing as I occasionally have typos in my command
line invocations of chrome

[Ami GONE FROM CHROMIUM, 2013-08-01 18:33:09]

No barf, need to wait for this to land and roll into chromeos before
removing the switch.


On Thu, Aug 1, 2013 at 11:18 AM, <sheu@chromium.org> wrote:

>
> https://codereview.chromium.**org/21584002/diff/1/content/**
>
public/common/content_**switches.cc<https://codereview.chromium.org/21584002/diff/1/content/public/common/content_switches.cc>
> File content/public/common/content_**switches.cc (left):
>
> https://codereview.chromium.**org/21584002/diff/1/content/**
>
public/common/content_**switches.cc#oldcode871<https://codereview.chromium.org/21584002/diff/1/content/public/common/content_switches.cc#oldcode871>
> content/public/common/content_**switches.cc:871: const char
> kUseExynosVda[]                  = "use-exynos-vda";
> On 2013/08/01 18:15:56, scherkus wrote:
>
>> is there a cros-side change to remove passing in this flag in
>> session_manager_setup.sh?
>>
>
> I can do that.  Does Chrome barf when starting with unrecognized flags,
> or can I wait until this lands to do it?
>
>
https://codereview.chromium.**org/21584002/<https://codereview.chromium.org/2...
>

[piman, 2013-08-02 19:11:40]

lgtm

[commit-bot: I haz the power, 2013-08-05 20:06:36]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/fischman@chromium.org/21584002/1

[commit-bot: I haz the power, 2013-08-05 20:24:25]

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

[commit-bot: I haz the power, 2013-08-05 20:29:13]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/fischman@chromium.org/21584002/22001

[Ami GONE FROM CHROMIUM, 2013-08-05 20:29:20]

Trying PS#2 whose only delta to PS#1 is that it does _not_ remove
third_party/openmax/OWNERS, because apparently presubmit first removes OWNERS
and then checks their content :(
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=chromium_p...

If CQ is happy with this then I'll file a bug about the above.

[Ami GONE FROM CHROMIUM, 2013-08-05 20:43:20]

> If CQ is happy with this then I'll file a bug about the above.

Looks like chromium_presubmit passed on PS#2, so filed
https://code.google.com/p/chromium/issues/detail?id=268528

[commit-bot: I haz the power, 2013-08-05 21:46:15]

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

[commit-bot: I haz the power, 2013-08-05 23:47:49]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/fischman@chromium.org/21584002/43001

[commit-bot: I haz the power, 2013-08-06 01:22:27]

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

[commit-bot: I haz the power, 2013-08-06 18:10:28]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/fischman@chromium.org/21584002/58001

[commit-bot: I haz the power, 2013-08-06 18:11:43]

Change committed as 215933