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

Side by Side Diff: chrome/renderer/chrome_content_renderer_client.cc

Issue 12091115: Allow manifest handlers to declare keys they depend on that must be parsed before them. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: dcronin Created 7 years, 10 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 | Annotate | Revision Log
OLDNEW
1 // Copyright (c) 2012 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2012 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 #include "chrome/renderer/chrome_content_renderer_client.h" 5 #include "chrome/renderer/chrome_content_renderer_client.h"
6 6
7 #include <string> 7 #include <string>
8 #include <vector> 8 #include <vector>
9 9
10 #include "base/command_line.h" 10 #include "base/command_line.h"
(...skipping 111 matching lines...) Expand 10 before | Expand all | Expand 10 after
122 using WebKit::WebURLResponse; 122 using WebKit::WebURLResponse;
123 using WebKit::WebVector; 123 using WebKit::WebVector;
124 124
125 namespace { 125 namespace {
126 126
127 // Explicitly register all extension ManifestHandlers needed to parse 127 // Explicitly register all extension ManifestHandlers needed to parse
128 // fields used in the renderer. 128 // fields used in the renderer.
129 void RegisterExtensionManifestHandlers() { 129 void RegisterExtensionManifestHandlers() {
130 extensions::ManifestHandler::Register( 130 extensions::ManifestHandler::Register(
131 extension_manifest_keys::kDevToolsPage, 131 extension_manifest_keys::kDevToolsPage,
132 new extensions::DevToolsPageHandler); 132 make_linked_ptr(new extensions::DevToolsPageHandler));
133 extensions::ManifestHandler::Register( 133 extensions::ManifestHandler::Register(
134 extension_manifest_keys::kWebAccessibleResources, 134 extension_manifest_keys::kWebAccessibleResources,
135 new extensions::WebAccessibleResourcesHandler); 135 make_linked_ptr(new extensions::WebAccessibleResourcesHandler));
136 } 136 }
137 137
138 static void AppendParams(const std::vector<string16>& additional_names, 138 static void AppendParams(const std::vector<string16>& additional_names,
139 const std::vector<string16>& additional_values, 139 const std::vector<string16>& additional_values,
140 WebVector<WebString>* existing_names, 140 WebVector<WebString>* existing_names,
141 WebVector<WebString>* existing_values) { 141 WebVector<WebString>* existing_values) {
142 DCHECK(additional_names.size() == additional_values.size()); 142 DCHECK(additional_names.size() == additional_values.size());
143 DCHECK(existing_names->size() == existing_values->size()); 143 DCHECK(existing_names->size() == existing_values->size());
144 144
145 size_t existing_size = existing_names->size(); 145 size_t existing_size = existing_names->size();
(...skipping 914 matching lines...) Expand 10 before | Expand all | Expand 10 after
1060 } 1060 }
1061 1061
1062 void ChromeContentRendererClient::RegisterPPAPIInterfaceFactories( 1062 void ChromeContentRendererClient::RegisterPPAPIInterfaceFactories(
1063 webkit::ppapi::PpapiInterfaceFactoryManager* factory_manager) { 1063 webkit::ppapi::PpapiInterfaceFactoryManager* factory_manager) {
1064 #if defined(ENABLE_PLUGINS) 1064 #if defined(ENABLE_PLUGINS)
1065 factory_manager->RegisterFactory(ChromePPAPIInterfaceFactory); 1065 factory_manager->RegisterFactory(ChromePPAPIInterfaceFactory);
1066 #endif 1066 #endif
1067 } 1067 }
1068 1068
1069 } // namespace chrome 1069 } // namespace chrome
OLDNEW
« no previous file with comments | « chrome/common/extensions/unpacker_unittest.cc ('k') | chrome/utility/chrome_content_utility_client.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698