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

Unified Diff: chrome/browser/chromeos/cros/brightness_library.cc

Issue 7891021: Use stub impl when libcros fails to load (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Fix power manager stub impl Created 9 years, 3 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 side-by-side diff with in-line comments
Download patch
Index: chrome/browser/chromeos/cros/brightness_library.cc
diff --git a/chrome/browser/chromeos/cros/brightness_library.cc b/chrome/browser/chromeos/cros/brightness_library.cc
index 22553200fdc157a8b36c62e173dbe808a7e3f361..1205608f053a43447a88adc8656adc22b80b994b 100644
--- a/chrome/browser/chromeos/cros/brightness_library.cc
+++ b/chrome/browser/chromeos/cros/brightness_library.cc
@@ -28,11 +28,10 @@ class BrightnessLibraryImpl : public BrightnessLibrary {
// Begin BrightnessLibrary implementation.
virtual void Init() OVERRIDE {
- if (CrosLibrary::Get()->EnsureLoaded()) {
- CHECK(!brightness_connection_) << "Already intialized";
- brightness_connection_ =
- chromeos::MonitorBrightnessV2(&BrightnessChangedHandler, this);
- }
+ DCHECK(CrosLibrary::Get()->libcros_loaded());
+ CHECK(!brightness_connection_) << "Already intialized";
+ brightness_connection_ =
+ chromeos::MonitorBrightnessV2(&BrightnessChangedHandler, this);
}
virtual void AddObserver(Observer* observer) OVERRIDE {
@@ -44,12 +43,10 @@ class BrightnessLibraryImpl : public BrightnessLibrary {
}
virtual void DecreaseScreenBrightness(bool allow_off) OVERRIDE {
- if (chromeos::DecreaseScreenBrightness)
- chromeos::DecreaseScreenBrightness(allow_off);
+ chromeos::DecreaseScreenBrightness(allow_off);
}
virtual void IncreaseScreenBrightness() OVERRIDE {
- if (chromeos::IncreaseScreenBrightness)
chromeos::IncreaseScreenBrightness();
satorux1 2011/09/16 20:05:55 Indentation is off?
stevenjb 2011/09/16 21:44:24 Done.
}
// End BrightnessLibrary implementation.
« no previous file with comments | « no previous file | chrome/browser/chromeos/cros/burn_library.h » ('j') | chrome/browser/chromeos/cros/cros_library.cc » ('J')

Powered by Google App Engine
This is Rietveld 408576698