Remove VideoDecodeAccelerator::Client::NotifyEndOfStream / PPP_VideoDecoder_Dev::EndOfStream

ppapi/cpp/dev/video_decoder_client_dev.h

Index: ppapi/cpp/dev/video_decoder_client_dev.h
diff --git a/ppapi/cpp/dev/video_decoder_client_dev.h b/ppapi/cpp/dev/video_decoder_client_dev.h
index 5510b4c8834c41b4e6b9206c6c5e073e0f971e54..6af4f04a63a93d66a701fc2f43e3b7035d57589a 100644
--- a/ppapi/cpp/dev/video_decoder_client_dev.h
+++ b/ppapi/cpp/dev/video_decoder_client_dev.h
@@ -36,10 +36,6 @@ class VideoDecoderClient_Dev {
   virtual void PictureReady(PP_Resource decoder,
                             const PP_Picture_Dev& picture) = 0;
 
-  // Callback to notify that decoder has decoded end of stream marker and has
-  // outputted all displayable pictures.
-  virtual void EndOfStream(PP_Resource decoder) = 0;

[Ami GONE FROM CHROMIUM, 2012/01/11 23:50:05]

piman: is this legit to do?  IOW does the cpp/ version of the c/ interface
always track the latest API, or what?

[piman, 2012/01/12 19:44:07]

It's fine.
1- yes, we're trying to keep the CPP in sync with the C interface - and we don't
guarantee compatibility at that level, we just try.
2- In this case, you won't directly break anything (except OVERRIDE users). 

-
   // Callback to notify about decoding errors.
   virtual void NotifyError(PP_Resource decoder,
                            PP_VideoDecodeError_Dev error) = 0;