Disable playback controls if audio/video element has errored.

webkit/glue/context_menu_client_impl.cc

Index: webkit/glue/context_menu_client_impl.cc
diff --git a/webkit/glue/context_menu_client_impl.cc b/webkit/glue/context_menu_client_impl.cc
index 5ad232d403eafe5c9277e9eed571e68f9f43975e..b32b572cf68cccdf6abdf7e37a83fc6626417856 100644
--- a/webkit/glue/context_menu_client_impl.cc
+++ b/webkit/glue/context_menu_client_impl.cc
@@ -18,6 +18,7 @@ MSVC_PUSH_WARNING_LEVEL(0);
 #include "HTMLMediaElement.h"
 #include "HTMLNames.h"
 #include "KURL.h"
+#include "MediaError.h"

[awong, 2009/08/20 18:33:54]

Do you  need to include MediaError?

[scherkus (not reviewing), 2009/08/20 18:50:24]

HTMLMediaElement::error() returns PassRefPtr<MediaError>.  The compiler dies
when attempting to resolve that class.

[awong, 2009/08/20 18:57:52]

Ah...makes sense.  <insert grumbling about making a template that requires a
fully defined type to do something that's not necessary>

 #include "Widget.h"
 MSVC_POP_WARNING();
 #undef LOG
@@ -191,6 +192,9 @@ WebCore::PlatformMenuDescription
       node_type.type |= ContextNodeType::AUDIO;
     }
 
+    if (media_element->error()) {
+      media_params.player_state |= ContextMenuMediaParams::IN_ERROR;
+    }
     if (media_element->paused()) {
       media_params.player_state |= ContextMenuMediaParams::PAUSED;
     }
@@ -203,7 +207,6 @@ WebCore::PlatformMenuDescription
     if (media_element->supportsSave()) {
       media_params.player_state |= ContextMenuMediaParams::CAN_SAVE;
     }
-    // TODO(ajwong): Report error states in the media player.
   }
 
   // If it's not a link, an image, a media element, or an image/media link,