CPP bindings: DCHECK when a Callback is destructed without being invoked

mojo/public/tools/bindings/generators/cpp_templates/interface_stub_declaration.tmpl

Index: mojo/public/tools/bindings/generators/cpp_templates/interface_stub_declaration.tmpl
diff --git a/mojo/public/tools/bindings/generators/cpp_templates/interface_stub_declaration.tmpl b/mojo/public/tools/bindings/generators/cpp_templates/interface_stub_declaration.tmpl
index afc6504dc81f33bf535546964eafcd06a29d266b..37f9e070ab3f5cf36a9a8485cd8fef84d0925df0 100644
--- a/mojo/public/tools/bindings/generators/cpp_templates/interface_stub_declaration.tmpl
+++ b/mojo/public/tools/bindings/generators/cpp_templates/interface_stub_declaration.tmpl
@@ -1,13 +1,14 @@
-class {{interface.name}}Stub : public mojo::MessageReceiverWithResponder {
+class {{interface.name}}Stub : public mojo::MessageReceiverWithResponderStatus {
  public:
   {{interface.name}}Stub();
+  ~{{interface.name}}Stub() override;
   void set_sink({{interface.name}}* sink) { sink_ = sink; }
   {{interface.name}}* sink() { return sink_; }
 
   virtual bool Accept(mojo::Message* message) override;
   virtual bool AcceptWithResponder(mojo::Message* message,
-                                   mojo::MessageReceiver* responder) override;
-
+      mojo::MessageReceiverWithStatus* responder) override;

[jamesr, 2015/03/23 18:16:33]

nit: since you're here (hah!) generating both 'virtual' and 'override' is
unnecessary, this should generate only the 'override' keyword

[rudominer, 2015/03/30 21:00:50]

Done.

+  
  private:
   {{interface.name}}* sink_;
 };