Remove PosterWaitingForVideo which is redundant

Source/core/html/HTMLVideoElement.cpp

Index: Source/core/html/HTMLVideoElement.cpp
diff --git a/Source/core/html/HTMLVideoElement.cpp b/Source/core/html/HTMLVideoElement.cpp
index d32d461fa08d360e8a7add657ae638a716ba8339..44adffe938dde5f46de4ec03ee61a5f79df6ca46 100644
--- a/Source/core/html/HTMLVideoElement.cpp
+++ b/Source/core/html/HTMLVideoElement.cpp
@@ -182,7 +182,7 @@ void HTMLVideoElement::setDisplayMode(DisplayMode mode)
         // Don't show the poster if there is a seek operation or
         // the video has restarted because of loop attribute
         if (mode == Video && oldMode == Poster && !hasAvailableVideoFrame())
-            mode = PosterWaitingForVideo;
+            mode = Poster;

[philipj_slow, 2014/09/01 11:09:17]

Since oldMode is Poster, can't we just return instead of setting mode = Poster?

[Srirama, 2014/09/01 13:27:55]

Acknowledged.

     }
 
     HTMLMediaElement::setDisplayMode(mode);