Remove VideoRenderer::Pause() as it serves no real purpose.

Remove VideoRenderer::Pause() as it serves no real purpose.

Flush() already accomplishes pausing by releasing all data and not
reading until Preroll() is called.

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

[scherkus (not reviewing), 2014-05-08 17:59:46]

working towards that pipeline-controlled rebuffer stuff we talked about
yesterday

first things first is removing code that doesn't accomplish anything

[acolwell GONE FROM CHROMIUM, 2014-05-08 18:43:01]

watch out folks. He's got the code flame thrower out. LGTM

[xhwang, 2014-05-08 20:02:37]

Good catch! Who doesn't like simpler code?


On Thu, May 8, 2014 at 11:43 AM, <acolwell@chromium.org> wrote:

> watch out folks. He's got the code flame thrower out. LGTM
>
> https://codereview.chromium.org/271923002/
>

To unsubscribe from this group and stop receiving emails from it, send an email
to chromium-reviews+unsubscribe@chromium.org.

[scherkus (not reviewing), 2014-05-08 20:04:16]

On 2014/05/08 20:02:37, xhwang wrote:
> Good catch! Who doesn't like simpler code?

Said nobody, ever. :)

[scherkus (not reviewing), 2014-05-10 07:19:08]

The CQ bit was checked by scherkus@chromium.org

[commit-bot: I haz the power, 2014-05-10 07:20:22]

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

[commit-bot: I haz the power, 2014-05-10 07:41:24]

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:
  android_clang_dbg on tryserver.chromium
(http://build.chromium.org/p/tryserver.chromium/builders/android_clang_dbg/bui...)
  android_dbg on tryserver.chromium
(http://build.chromium.org/p/tryserver.chromium/builders/android_dbg/builds/18...)
  chromium_presubmit on tryserver.chromium
(http://build.chromium.org/p/tryserver.chromium/builders/chromium_presubmit/bu...)
  ios_rel_device_ninja on tryserver.chromium
(http://build.chromium.org/p/tryserver.chromium/builders/ios_rel_device_ninja/...)
  linux_chromium_chromeos_clang_dbg on tryserver.chromium
(http://build.chromium.org/p/tryserver.chromium/builders/linux_chromium_chrome...)
  win_chromium_compile_dbg on tryserver.chromium
(http://build.chromium.org/p/tryserver.chromium/builders/win_chromium_compile_...)
  win_chromium_rel on tryserver.chromium
(http://build.chromium.org/p/tryserver.chromium/builders/win_chromium_rel/buil...)
  win_chromium_x64_rel on tryserver.chromium
(http://build.chromium.org/p/tryserver.chromium/builders/win_chromium_x64_rel/...)

[commit-bot: I haz the power, 2014-05-10 07:44:28]

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

[commit-bot: I haz the power, 2014-05-10 07:44:28]

Try jobs failed on following builders:
  chromium_presubmit on tryserver.chromium
(http://build.chromium.org/p/tryserver.chromium/builders/chromium_presubmit/bu...)

[scherkus (not reviewing), 2014-05-10 20:06:50]

The CQ bit was checked by scherkus@chromium.org

[commit-bot: I haz the power, 2014-05-10 20:07:49]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/scherkus@chromium.org/271923002/60001

[commit-bot: I haz the power, 2014-05-11 01:25:41]

Change committed as 269675