Eliminate MediaPlayer & MediaPlayerClient abstractions (fullscreen apis)

Eliminate MediaPlayer & MediaPlayerClient abstractions

BUG=350571
Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=174940

[Srirama, 2014-05-21 06:39:17]

Removed fullscreen related APIs.
PTAL.

[acolwell GONE FROM CHROMIUM, 2014-05-22 01:08:28]

lgtm

[Srirama, 2014-05-22 05:14:54]

The CQ bit was checked by srirama.m@samsung.com

[commit-bot: I haz the power, 2014-05-22 05:16:01]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/srirama.m@samsung.com/291303003/1

[Srirama, 2014-05-22 07:50:06]

The CQ bit was unchecked by srirama.m@samsung.com

[Srirama, 2014-05-22 07:50:14]

The CQ bit was checked by srirama.m@samsung.com

[commit-bot: I haz the power, 2014-05-22 07:50:40]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/srirama.m@samsung.com/291303003/1

[commit-bot: I haz the power, 2014-05-22 17:17:21]

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:
  blink_presubmit on tryserver.blink
(http://build.chromium.org/p/tryserver.blink/builders/blink_presubmit/builds/5972)
  win_gpu_triggered_tests on tryserver.chromium.gpu
(http://build.chromium.org/p/tryserver.chromium.gpu/builders/win_gpu_triggered...)

[commit-bot: I haz the power, 2014-05-22 17:18:48]

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

[commit-bot: I haz the power, 2014-05-22 17:18:49]

Try jobs failed on following builders:
  win_gpu_triggered_tests on tryserver.chromium.gpu
(http://build.chromium.org/p/tryserver.chromium.gpu/builders/win_gpu_triggered...)

[acolwell GONE FROM CHROMIUM, 2014-05-22 18:00:47]

adamk@: Owners review for web/ and platform/ changes please.

[Srirama, 2014-05-26 13:21:04]

On 2014/05/22 18:00:47, acolwell_OOO_5-24_to_5-30 wrote:
> adamk@: Owners review for web/ and platform/ changes please.

Ping

[Srirama, 2014-05-27 14:15:22]

abarth@: Owners review for web/ and platform/ changes please.

[adamk, 2014-05-27 19:13:28]

lgtm for Source/{web,platform}

[adamk, 2014-05-27 19:13:40]

The CQ bit was checked by adamk@chromium.org

[commit-bot: I haz the power, 2014-05-27 19:14:45]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/srirama.m@samsung.com/291303003/1

[commit-bot: I haz the power, 2014-05-27 19:14:53]

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

[commit-bot: I haz the power, 2014-05-27 19:14:54]

Failed to apply patch for Source/web/WebMediaPlayerClientImpl.cpp:
While running patch -p1 --forward --force --no-backup-if-mismatch;
  patching file Source/web/WebMediaPlayerClientImpl.cpp
  Hunk #1 FAILED at 217.
  1 out of 1 hunk FAILED -- saving rejects to file
Source/web/WebMediaPlayerClientImpl.cpp.rej

Patch:       Source/web/WebMediaPlayerClientImpl.cpp
Index: Source/web/WebMediaPlayerClientImpl.cpp
diff --git a/Source/web/WebMediaPlayerClientImpl.cpp
b/Source/web/WebMediaPlayerClientImpl.cpp
index
09635ef423ac7da4eb5cd1becf68b75abdb9aa93..740da1ed0a51878e953ca2ca4ce5b3e2254faca8
100644
--- a/Source/web/WebMediaPlayerClientImpl.cpp
+++ b/Source/web/WebMediaPlayerClientImpl.cpp
@@ -217,23 +217,6 @@ void WebMediaPlayerClientImpl::pause()
         m_webMediaPlayer->pause();
 }
 
-void WebMediaPlayerClientImpl::showFullscreenOverlay()
-{
-    if (m_webMediaPlayer)
-        m_webMediaPlayer->enterFullscreen();
-}
-
-void WebMediaPlayerClientImpl::hideFullscreenOverlay()
-{
-    if (m_webMediaPlayer)
-        m_webMediaPlayer->exitFullscreen();
-}
-
-bool WebMediaPlayerClientImpl::canShowFullscreenOverlay() const
-{
-    return m_webMediaPlayer && m_webMediaPlayer->canEnterFullscreen();
-}
-
 IntSize WebMediaPlayerClientImpl::naturalSize() const
 {
     if (m_webMediaPlayer)

[Srirama, 2014-05-28 05:34:31]

Rebase with latest code

[Srirama, 2014-05-28 05:34:47]

The CQ bit was checked by srirama.m@samsung.com

[commit-bot: I haz the power, 2014-05-28 05:35:19]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/srirama.m@samsung.com/291303003/20001

[commit-bot: I haz the power, 2014-05-28 06:53:16]

Change committed as 174940