Revert of Show the Cast toolbar icon ephemerally when Cast is in use

Revert of Show the Cast toolbar icon ephemerally when Cast is in use (patchset #15 id:440001 of https://codereview.chromium.org/2155293002/ )

Reason for revert:
Reverting this CL since it is causing crashes and needs a redesign. Also reverting a related CL: https://codereview.chromium.org/2260873003/

Original issue's description:
> Show the Cast toolbar icon ephemerally when Cast is in use
>
> This CL changes the behavior of the Cast/Media Router toolbar component
> action icon. Currently, the only way to add the icon is by installing
> the Cast extension from the Web Store. After this change, the icon will
> be displayed temporarily whenever the Media Router dialog is open, or there
> is an active local media route.
>
> This CL also replaces the "Remove icon" option in the action context menu
> with "Always show icon" which can be checked. When this option is unchecked,
> the icon is neither on the toolbar nor in the overflow menu, and is only shown
> ephemerally in the situations mentioned above.
>
> We replace the word "popup" with "dialog" wherever possible in
> MediaRouterAction, to stay consistent with the rest of Media Router files.
>
> We also change the default position of the component actions on the toolbar
> from the leftmost to the rightmost (same behavior as extension actions).
>
> TBR=grt@chromium.org
> BUG=594577
>
> Committed: https://crrev.com/de716ca8b6cffb4c7ae30990b3288b3499d28a4c
> Cr-Commit-Position: refs/heads/master@{#412280}

TBR=anthonyvd@chromium.org,apacible@chromium.org,msw@chromium.org,rdevlin.cronin@chromium.org,mfoltz@chromium.org,grt@chromium.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=594577
Committed: https://crrev.com/3b3841e82bec9ffa674ba6596834ef4db2c95481
Cr-Commit-Position: refs/heads/master@{#413338}

[takumif, 2016-08-19 22:15:26]

The CQ bit was checked by takumif@chromium.org

[takumif, 2016-08-19 22:15:26]

Created Revert of Show the Cast toolbar icon ephemerally when Cast is in use

[commit-bot: I haz the power, 2016-08-19 22:16:24]

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, 2016-08-19 22:16:24]

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

[commit-bot: I haz the power, 2016-08-19 22:16:26]

No L-G-T-M from a valid reviewer yet. 
CQ run can only be started by full committers or once the patch has
received an L-G-T-M from a full committer.
Even if an L-G-T-M may have been provided, it was from a non-committer,
_not_ a full super star committer.
See http://www.chromium.org/getting-involved/become-a-committer
Note that this has nothing to do with OWNERS files.

[mark a. foltz, 2016-08-19 22:31:54]

lgtm

[takumif, 2016-08-19 22:33:03]

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

[commit-bot: I haz the power, 2016-08-19 22:33:49]

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, 2016-08-20 00:49:28]

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

[commit-bot: I haz the power, 2016-08-20 00:49:29]

Dry run: Try jobs failed on following builders:
  win_chromium_x64_rel_ng on master.tryserver.chromium.win (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.win/builders/win_chromium_x64_...)

[takumif, 2016-08-20 04:52:28]

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

[commit-bot: I haz the power, 2016-08-20 04:52:39]

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, 2016-08-20 06:40:41]

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

[commit-bot: I haz the power, 2016-08-20 06:40:42]

Dry run: This issue passed the CQ dry run.

[takumif, 2016-08-20 15:25:07]

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

[commit-bot: I haz the power, 2016-08-20 15:25:16]

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, 2016-08-20 15:28:02]

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

[commit-bot: I haz the power, 2016-08-20 15:28:03]

Dry run: This issue passed the CQ dry run.

[takumif, 2016-08-20 15:29:06]

The CQ bit was checked by takumif@chromium.org

[commit-bot: I haz the power, 2016-08-20 15:29:12]

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, 2016-08-20 15:34:22]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-08-20 15:35:56]

Description was changed from

==========
Revert of Show the Cast toolbar icon ephemerally when Cast is in use (patchset
#15 id:440001 of https://codereview.chromium.org/2155293002/ )

Reason for revert:
Reverting this CL since it is causing crashes and needs a redesign. Also
reverting a related CL: https://codereview.chromium.org/2260873003/

Original issue's description:
> Show the Cast toolbar icon ephemerally when Cast is in use
> 
> This CL changes the behavior of the Cast/Media Router toolbar component
> action icon. Currently, the only way to add the icon is by installing
> the Cast extension from the Web Store. After this change, the icon will
> be displayed temporarily whenever the Media Router dialog is open, or there
> is an active local media route.
> 
> This CL also replaces the "Remove icon" option in the action context menu
> with "Always show icon" which can be checked. When this option is unchecked,
> the icon is neither on the toolbar nor in the overflow menu, and is only shown
> ephemerally in the situations mentioned above.
> 
> We replace the word "popup" with "dialog" wherever possible in
> MediaRouterAction, to stay consistent with the rest of Media Router files.
> 
> We also change the default position of the component actions on the toolbar
> from the leftmost to the rightmost (same behavior as extension actions).
> 
> TBR=grt@chromium.org
> BUG=594577
> 
> Committed: https://crrev.com/de716ca8b6cffb4c7ae30990b3288b3499d28a4c
> Cr-Commit-Position: refs/heads/master@{#412280}

TBR=anthonyvd@chromium.org,apacible@chromium.org,msw@chromium.org,rdevlin.cro...
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=594577
==========

to

==========
Revert of Show the Cast toolbar icon ephemerally when Cast is in use (patchset
#15 id:440001 of https://codereview.chromium.org/2155293002/ )

Reason for revert:
Reverting this CL since it is causing crashes and needs a redesign. Also
reverting a related CL: https://codereview.chromium.org/2260873003/

Original issue's description:
> Show the Cast toolbar icon ephemerally when Cast is in use
>
> This CL changes the behavior of the Cast/Media Router toolbar component
> action icon. Currently, the only way to add the icon is by installing
> the Cast extension from the Web Store. After this change, the icon will
> be displayed temporarily whenever the Media Router dialog is open, or there
> is an active local media route.
>
> This CL also replaces the "Remove icon" option in the action context menu
> with "Always show icon" which can be checked. When this option is unchecked,
> the icon is neither on the toolbar nor in the overflow menu, and is only shown
> ephemerally in the situations mentioned above.
>
> We replace the word "popup" with "dialog" wherever possible in
> MediaRouterAction, to stay consistent with the rest of Media Router files.
>
> We also change the default position of the component actions on the toolbar
> from the leftmost to the rightmost (same behavior as extension actions).
>
> TBR=grt@chromium.org
> BUG=594577
>
> Committed: https://crrev.com/de716ca8b6cffb4c7ae30990b3288b3499d28a4c
> Cr-Commit-Position: refs/heads/master@{#412280}

TBR=anthonyvd@chromium.org,apacible@chromium.org,msw@chromium.org,rdevlin.cro...
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=594577

Committed: https://crrev.com/3b3841e82bec9ffa674ba6596834ef4db2c95481
Cr-Commit-Position: refs/heads/master@{#413338}
==========

[commit-bot: I haz the power, 2016-08-20 15:35:57]

Patchset 1 (id:??) landed as
https://crrev.com/3b3841e82bec9ffa674ba6596834ef4db2c95481
Cr-Commit-Position: refs/heads/master@{#413338}