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

Side by Side Diff: chrome/browser/automation/automation_provider.h

Issue 209029: Fix order of parameters for AutomationProvider::GetBooleanPreference(). (Closed) Base URL: http://src.chromium.org/svn/trunk/src/
Patch Set: '' Created 11 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 | Annotate | Revision Log
« no previous file with comments | « no previous file | chrome/browser/automation/automation_provider.cc » ('j') | 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) 2006-2008 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2006-2008 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 // This implements a browser-side endpoint for UI automation activity. 5 // This implements a browser-side endpoint for UI automation activity.
6 // The client-side endpoint is implemented by AutomationProxy. 6 // The client-side endpoint is implemented by AutomationProxy.
7 // The entire lifetime of this object should be contained within that of 7 // The entire lifetime of this object should be contained within that of
8 // the BrowserProcess, and in particular the NotificationService that's 8 // the BrowserProcess, and in particular the NotificationService that's
9 // hung off of it. 9 // hung off of it.
10 10
(...skipping 405 matching lines...) Expand 10 before | Expand all | Expand 10 after
416 416
417 // Sets the string value for preference with name |name|. 417 // Sets the string value for preference with name |name|.
418 void SetStringPreference(int handle, 418 void SetStringPreference(int handle,
419 const std::wstring& name, 419 const std::wstring& name,
420 const std::wstring& value, 420 const std::wstring& value,
421 bool* success); 421 bool* success);
422 422
423 // Gets the bool value for preference with name |name|. 423 // Gets the bool value for preference with name |name|.
424 void GetBooleanPreference(int handle, 424 void GetBooleanPreference(int handle,
425 const std::wstring& name, 425 const std::wstring& name,
426 bool* success, 426 bool* value,
427 bool* value); 427 bool* success);
428 428
429 // Sets the bool value for preference with name |name|. 429 // Sets the bool value for preference with name |name|.
430 void SetBooleanPreference(int handle, 430 void SetBooleanPreference(int handle,
431 const std::wstring& name, 431 const std::wstring& name,
432 bool value, 432 bool value,
433 bool* success); 433 bool* success);
434 434
435 // Gets the current used encoding name of the page in the specified tab. 435 // Gets the current used encoding name of the page in the specified tab.
436 void GetPageCurrentEncoding(int tab_handle, std::string* current_encoding); 436 void GetPageCurrentEncoding(int tab_handle, std::string* current_encoding);
437 437
(...skipping 124 matching lines...) Expand 10 before | Expand all | Expand 10 after
562 virtual void Observe(NotificationType type, 562 virtual void Observe(NotificationType type,
563 const NotificationSource& source, 563 const NotificationSource& source,
564 const NotificationDetails& details); 564 const NotificationDetails& details);
565 565
566 void OnRemoveProvider(); // Called via PostTask 566 void OnRemoveProvider(); // Called via PostTask
567 567
568 NotificationRegistrar registrar_; 568 NotificationRegistrar registrar_;
569 }; 569 };
570 570
571 #endif // CHROME_BROWSER_AUTOMATION_AUTOMATION_PROVIDER_H_ 571 #endif // CHROME_BROWSER_AUTOMATION_AUTOMATION_PROVIDER_H_
OLDNEW
« no previous file with comments | « no previous file | chrome/browser/automation/automation_provider.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698