[Cast,Android] Remove TransportXX dependency from ExpandedControllerActivity

[Cast,Android] Remove TransportXX dependency from ExpandedControllerActivity

Since using the default andorid.widget.MediaController causes hard to fix crashes and using MediaSessionCompat required a huge code change, this seems to be the
simplest solution, mergeable to 59 if needed.

Note that I uploaded this separately from the revert to ease reviewing.

BUG=725028
TEST=manual, cast a video and open the activity via the notification.

[whywhat, 2017-05-22 19:17:27]

avayvod@chromium.org changed reviewers:
+ dgn@chromium.org

[whywhat, 2017-05-22 19:17:27]

Nicolas, PTaL
Alex, FYI

[whywhat, 2017-05-22 19:29:21]

Fix lint warnings.

[whywhat, 2017-05-24 00:58:47]

Updated README.chromium

[whywhat, 2017-05-24 01:04:07]

Description was changed from

==========
[Cast,Android] Check window attachment status for MC

android.widget.MediaController show() crashes if the view is not attached
to the Window yet. So exit early if shown before this happened and
show/hide on window being attached/detached.

BUG=725028
TEST=manual, cast a video and open the activity via the notification.
==========

to

==========
[Cast,Android] Remove TransportXX dependency from ExpandedControllerActivity

Since using the default andorid.widget.MediaController causes hard to fix
crashes and using MediaSessionCompat required a huge code change, this seems to
be the
simplest solution, mergeable to 59 if needed.

Note that I uploaded this separately from the revert to ease reviewing.

BUG=725028
TEST=manual, cast a video and open the activity via the notification.
==========

[whywhat, 2017-05-24 01:27:56]

The CQ bit was checked by avayvod@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2017-05-24 01:29:34]

Dry run: CQ is trying da patch.

Follow status at:
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-05-24 04:21:23]

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

[commit-bot: I haz the power, 2017-05-24 04:21:24]

Dry run: This issue passed the CQ dry run.

[dgn, 2017-05-24 14:13:52]

dgn@chromium.org changed reviewers:
+ mlamouri@chromium.org

[dgn, 2017-05-24 14:13:52]

mlamouri@: would you mind having a look? I'm not familiar with the cast logic
anymore. Happy to RS if that looks good to you. You should get into the owners
too!

[whywhat, 2017-05-26 01:24:09]

On 2017/05/24 at 14:13:52, dgn wrote:
> mlamouri@: would you mind having a look? I'm not familiar with the cast logic
anymore. Happy to RS if that looks good to you. You should get into the owners
too!

Ping. Mounir probably knows this part of the Cast code no better than you,
Nicolas :) This is a M60 crash fixer. M60 branches today...

[dgn, 2017-05-26 09:58:40]

lgtm

[mlamouri (slow - plz ping), 2017-05-26 13:27:54]

lgtm