Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(734)

Unified Diff: chrome/renderer/render_view.cc

Issue 100118: plugins: Some obvious ifdef removals (code that previously wouldn't link). (Closed)
Patch Set: better patch Created 11 years, 8 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « chrome/browser/plugin_process_host.cc ('k') | webkit/glue/webplugin_impl.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/renderer/render_view.cc
diff --git a/chrome/renderer/render_view.cc b/chrome/renderer/render_view.cc
index bce6d4c975e51e764c74f3f2764ddffb94c8f076..0f085dc7b35ddd2e6494fce29c71345b4197d20e 100644
--- a/chrome/renderer/render_view.cc
+++ b/chrome/renderer/render_view.cc
@@ -257,16 +257,8 @@ void RenderView::PluginDestroyed(WebPluginDelegateProxy* proxy) {
// clicks the info bar to install. Unfortunately we are getting
// PluginDestroyed in single process mode. However, that is not a huge
// concern.
-#if defined(OS_WIN)
if (proxy == first_default_plugin_)
first_default_plugin_ = NULL;
-#else
- // TODO(port): because of the headers that we aren't including, the compiler
- // has only seen a forward decl, not the subclass relation. Thus it doesn't
- // know that the two pointer types compared above are comparable. Once we
- // port and include the headers this problem should go away.
- NOTIMPLEMENTED();
-#endif
}
void RenderView::PluginCrashed(const FilePath& plugin_path) {
« no previous file with comments | « chrome/browser/plugin_process_host.cc ('k') | webkit/glue/webplugin_impl.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698