Create default implementations of ContentRendererClient, ContentUtilityClient, and ContentPluginCli…

content/public/plugin/content_plugin_client.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 CONTENT_PUBLIC_PLUGIN_CONTENT_PLUGIN_CLIENT_H_
 #define CONTENT_PUBLIC_PLUGIN_CONTENT_PLUGIN_CLIENT_H_
 #pragma once
 
 #include "base/string16.h"
 #include "content/public/common/content_client.h"

[tfarina, 2012/05/30 00:26:49]

looks like this include can be removed and content_export.h should be included
instead.

[jam, 2012/05/30 14:50:21]

by convention, all content_foo_client.h files include content_client.h so that
cc files don't have to include both of them

 
 namespace content {
 
 // Embedder API for participating in plugin logic.
-class ContentPluginClient {
+class CONTENT_EXPORT ContentPluginClient {

[tfarina, 2012/05/30 00:26:49]

does this need a virtual destructor?

[jam, 2012/05/30 14:50:21]

yep according to the style guide, i'll add one.

  public:
   // Notifies that a plugin process has started.
-  virtual void PluginProcessStarted(const string16& plugin_name) = 0;
+  virtual void PluginProcessStarted(const string16& plugin_name) {}
 };
 
 }  // namespace content
 
 #endif  // CONTENT_PUBLIC_PLUGIN_CONTENT_PLUGIN_CLIENT_H_