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

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

Issue 10222017: Make password generation switched by a preference in chrome settings rather than a command line fla… (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Change DCHECK_EQ to DCHECK to get around some type convertion issue. Created 8 years, 7 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
« no previous file with comments | « chrome/renderer/autofill/password_generation_manager_browsertest.cc ('k') | no next file » | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
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 8
9 #include "base/command_line.h" 9 #include "base/command_line.h"
10 #include "base/logging.h" 10 #include "base/logging.h"
(...skipping 237 matching lines...) Expand 10 before | Expand all | Expand 10 after
248 new SpellCheckProvider(render_view, this); 248 new SpellCheckProvider(render_view, this);
249 new prerender::PrerendererClient(render_view); 249 new prerender::PrerendererClient(render_view);
250 #if defined(ENABLE_SAFE_BROWSING) 250 #if defined(ENABLE_SAFE_BROWSING)
251 safe_browsing::MalwareDOMDetails::Create(render_view); 251 safe_browsing::MalwareDOMDetails::Create(render_view);
252 #endif 252 #endif
253 253
254 PasswordAutofillManager* password_autofill_manager = 254 PasswordAutofillManager* password_autofill_manager =
255 new PasswordAutofillManager(render_view); 255 new PasswordAutofillManager(render_view);
256 AutofillAgent* autofill_agent = new AutofillAgent(render_view, 256 AutofillAgent* autofill_agent = new AutofillAgent(render_view,
257 password_autofill_manager); 257 password_autofill_manager);
258 if (CommandLine::ForCurrentProcess()->HasSwitch(
259 switches::kEnablePasswordGeneration)) {
260 new PasswordGenerationManager(render_view);
261 }
258 PageClickTracker* page_click_tracker = new PageClickTracker(render_view); 262 PageClickTracker* page_click_tracker = new PageClickTracker(render_view);
259 // Note that the order of insertion of the listeners is important. 263 // Note that the order of insertion of the listeners is important.
260 // The password_autocomplete_manager takes the first shot at processing the 264 // The password_autocomplete_manager takes the first shot at processing the
261 // notification and can stop the propagation. 265 // notification and can stop the propagation.
262 page_click_tracker->AddListener(password_autofill_manager); 266 page_click_tracker->AddListener(password_autofill_manager);
263 page_click_tracker->AddListener(autofill_agent); 267 page_click_tracker->AddListener(autofill_agent);
264 268
265 TranslateHelper* translate = new TranslateHelper(render_view); 269 TranslateHelper* translate = new TranslateHelper(render_view);
266 new ChromeRenderViewObserver( 270 new ChromeRenderViewObserver(
267 render_view, content_settings, chrome_observer_.get(), 271 render_view, content_settings, chrome_observer_.get(),
268 extension_dispatcher_.get(), translate); 272 extension_dispatcher_.get(), translate);
269 273
270 // Used only for testing/automation. 274 // Used only for testing/automation.
271 if (CommandLine::ForCurrentProcess()->HasSwitch( 275 if (CommandLine::ForCurrentProcess()->HasSwitch(
272 switches::kDomAutomationController)) { 276 switches::kDomAutomationController)) {
273 new AutomationRendererHelper(render_view); 277 new AutomationRendererHelper(render_view);
274 } 278 }
275 if (CommandLine::ForCurrentProcess()->HasSwitch(
276 switches::kEnablePasswordGeneration)) {
277 new PasswordGenerationManager(render_view);
278 }
279 } 279 }
280 280
281 void ChromeContentRendererClient::SetNumberOfViews(int number_of_views) { 281 void ChromeContentRendererClient::SetNumberOfViews(int number_of_views) {
282 child_process_logging::SetNumberOfViews(number_of_views); 282 child_process_logging::SetNumberOfViews(number_of_views);
283 } 283 }
284 284
285 SkBitmap* ChromeContentRendererClient::GetSadPluginBitmap() { 285 SkBitmap* ChromeContentRendererClient::GetSadPluginBitmap() {
286 return ResourceBundle::GetSharedInstance().GetBitmapNamed(IDR_SAD_PLUGIN); 286 return ResourceBundle::GetSharedInstance().GetBitmapNamed(IDR_SAD_PLUGIN);
287 } 287 }
288 288
(...skipping 611 matching lines...) Expand 10 before | Expand all | Expand 10 after
900 bool ChromeContentRendererClient::IsOtherExtensionWithWebRequestInstalled() { 900 bool ChromeContentRendererClient::IsOtherExtensionWithWebRequestInstalled() {
901 return extension_dispatcher_->IsOtherExtensionWithWebRequestInstalled(); 901 return extension_dispatcher_->IsOtherExtensionWithWebRequestInstalled();
902 } 902 }
903 903
904 void ChromeContentRendererClient::RegisterPPAPIInterfaceFactories( 904 void ChromeContentRendererClient::RegisterPPAPIInterfaceFactories(
905 webkit::ppapi::PpapiInterfaceFactoryManager* factory_manager) { 905 webkit::ppapi::PpapiInterfaceFactoryManager* factory_manager) {
906 factory_manager->RegisterFactory(ChromePPAPIInterfaceFactory); 906 factory_manager->RegisterFactory(ChromePPAPIInterfaceFactory);
907 } 907 }
908 908
909 } // namespace chrome 909 } // namespace chrome
OLDNEW
« no previous file with comments | « chrome/renderer/autofill/password_generation_manager_browsertest.cc ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698