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

Side by Side Diff: webkit/glue/plugins/plugin_list.h

Issue 3522004: FBTF: Move ctors/dtors into implementation files. Adds ctors/dtors to non-POD structs. (Closed) Base URL: http://src.chromium.org/git/chromium.git
Patch Set: Created 10 years, 2 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 unified diff | Download patch
OLDNEW
1 // Copyright (c) 2010 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2010 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 WEBKIT_GLUE_PLUGINS_PLUGIN_LIST_H_ 5 #ifndef WEBKIT_GLUE_PLUGINS_PLUGIN_LIST_H_
6 #define WEBKIT_GLUE_PLUGINS_PLUGIN_LIST_H_ 6 #define WEBKIT_GLUE_PLUGINS_PLUGIN_LIST_H_
7 7
8 #include <set> 8 #include <set>
9 #include <string> 9 #include <string>
10 #include <vector> 10 #include <vector>
(...skipping 141 matching lines...) Expand 10 before | Expand all | Expand 10 after
152 // will be enabled. Note that plugins are enabled by default as far as 152 // will be enabled. Note that plugins are enabled by default as far as
153 // |PluginList| is concerned. 153 // |PluginList| is concerned.
154 bool EnablePlugin(const FilePath& filename); 154 bool EnablePlugin(const FilePath& filename);
155 155
156 // Disable a specific plugin, specified by path. Returns |true| iff a plugin 156 // Disable a specific plugin, specified by path. Returns |true| iff a plugin
157 // currently in the plugin list was actually disabled as a result; regardless 157 // currently in the plugin list was actually disabled as a result; regardless
158 // of return value, if a plugin is found in the future with the given name, it 158 // of return value, if a plugin is found in the future with the given name, it
159 // will be disabled. 159 // will be disabled.
160 bool DisablePlugin(const FilePath& filename); 160 bool DisablePlugin(const FilePath& filename);
161 161
162 ~PluginList();
163
162 private: 164 private:
163 // Constructors are private for singletons 165 // Constructors are private for singletons
164 PluginList(); 166 PluginList();
165 167
166 // Load all plugins from the default plugins directory 168 // Load all plugins from the default plugins directory
167 void LoadPlugins(bool refresh); 169 void LoadPlugins(bool refresh);
168 170
169 // Load all plugins from a specific directory. 171 // Load all plugins from a specific directory.
170 // |plugins| is updated with loaded plugin information. 172 // |plugins| is updated with loaded plugin information.
171 // |visited_plugins| is updated with paths to all plugins that were considered 173 // |visited_plugins| is updated with paths to all plugins that were considered
(...skipping 94 matching lines...) Expand 10 before | Expand all | Expand 10 after
266 Lock lock_; 268 Lock lock_;
267 269
268 friend struct base::DefaultLazyInstanceTraits<PluginList>; 270 friend struct base::DefaultLazyInstanceTraits<PluginList>;
269 271
270 DISALLOW_COPY_AND_ASSIGN(PluginList); 272 DISALLOW_COPY_AND_ASSIGN(PluginList);
271 }; 273 };
272 274
273 } // namespace NPAPI 275 } // namespace NPAPI
274 276
275 #endif // WEBKIT_GLUE_PLUGINS_PLUGIN_LIST_H_ 277 #endif // WEBKIT_GLUE_PLUGINS_PLUGIN_LIST_H_
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698