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

Side by Side Diff: chrome/renderer/plugins/chrome_plugin_placeholder.h

Issue 104833006: Switch ContentSettingsObserver to be a RenderFrameObserver instead of a RenderViewObserver (Closed) Base URL: svn://chrome-svn/chrome/trunk/src/
Patch Set: sync Created 7 years 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 unified diff | Download patch | Annotate | Revision Log
OLDNEW
1 // Copyright 2013 The Chromium Authors. All rights reserved. 1 // Copyright 2013 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #ifndef CHROME_RENDERER_PLUGINS_CHROME_PLUGIN_PLACEHOLDER_H_ 5 #ifndef CHROME_RENDERER_PLUGINS_CHROME_PLUGIN_PLACEHOLDER_H_
6 #define CHROME_RENDERER_PLUGINS_CHROME_PLUGIN_PLACEHOLDER_H_ 6 #define CHROME_RENDERER_PLUGINS_CHROME_PLUGIN_PLACEHOLDER_H_
7 7
8 #include "components/plugins/renderer/plugin_placeholder.h" 8 #include "components/plugins/renderer/plugin_placeholder.h"
9 9
10 struct ChromeViewHostMsg_GetPluginInfo_Status; 10 struct ChromeViewHostMsg_GetPluginInfo_Status;
(...skipping 79 matching lines...) Expand 10 before | Expand all | Expand 10 after
90 #if defined(ENABLE_PLUGIN_INSTALLATION) 90 #if defined(ENABLE_PLUGIN_INSTALLATION)
91 // |routing_id()| is the routing ID of our associated RenderView, but we have 91 // |routing_id()| is the routing ID of our associated RenderView, but we have
92 // a separate routing ID for messages specific to this placeholder. 92 // a separate routing ID for messages specific to this placeholder.
93 int32 placeholder_routing_id_; 93 int32 placeholder_routing_id_;
94 #endif 94 #endif
95 95
96 bool has_host_; 96 bool has_host_;
97 int context_menu_request_id_; // Nonzero when request pending. 97 int context_menu_request_id_; // Nonzero when request pending.
98 base::string16 plugin_name_; 98 base::string16 plugin_name_;
99 99
100 // TODO(jam): remove this class, it's temporary until ContentSettingObserver
101 // is a RenderFrameObserver.
102 class RenderViewObserver;
103 scoped_ptr<RenderViewObserver> view_observer_;
104
105 DISALLOW_COPY_AND_ASSIGN(ChromePluginPlaceholder); 100 DISALLOW_COPY_AND_ASSIGN(ChromePluginPlaceholder);
106 }; 101 };
107 102
108 #endif // CHROME_RENDERER_PLUGINS_CHROME_PLUGIN_PLACEHOLDER_H_ 103 #endif // CHROME_RENDERER_PLUGINS_CHROME_PLUGIN_PLACEHOLDER_H_
OLDNEW
« no previous file with comments | « chrome/renderer/content_settings_observer_unittest.cc ('k') | chrome/renderer/plugins/chrome_plugin_placeholder.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698