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

ppapi/cpp/dev/video_decoder_client_dev.h

 // Copyright (c) 2011 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #ifndef PPAPI_CPP_DEV_VIDEO_DECODER_CLIENT_DEV_H_
 #define PPAPI_CPP_DEV_VIDEO_DECODER_CLIENT_DEV_H_
 
 #include "ppapi/c/pp_stdint.h"
 #include "ppapi/c/dev/pp_video_dev.h"
 
@@ skipping 18 matching lines @@
 
   // Callback for decoder to deliver unneeded picture buffers back to the
   // plugin.
   virtual void DismissPictureBuffer(PP_Resource decoder,
                                     int32_t picture_buffer_id) = 0;
 
   // Callback to deliver decoded pictures ready to be displayed.
   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;
 
  private:
   Instance* associated_instance_;
 };
 
 }  // namespace pp
 
 #endif  // PPAPI_CPP_DEV_VIDEO_DECODER_CLIENT_DEV_H_