Index: chrome/browser/chromeos/login/default_images_view.cc |
diff --git a/chrome/browser/chromeos/login/default_images_view.cc b/chrome/browser/chromeos/login/default_images_view.cc |
index 8996d1038c48d51fa2475fb23a2f1384cdbfa417..3604f1d82d826fd777027d8d06c8966fb13bbc82 100644 |
--- a/chrome/browser/chromeos/login/default_images_view.cc |
+++ b/chrome/browser/chromeos/login/default_images_view.cc |
@@ -108,6 +108,12 @@ int DefaultImagesView::GetDefaultImageIndex() const { |
return selected_image_index_ - 1; |
} |
+void DefaultImagesView::SetDefaultImageIndex(int image_index) { |
+ selected_image_index_ = image_index + 1; |
Nikita (slow)
2011/05/16 16:45:16
nit: Please comment that we're skipping camera ima
whywhat
2011/05/17 12:25:43
Done.
I introduced a named constant and put a comm
|
+ if (delegate_) |
+ delegate_->OnImageSelected(image_index % kDefaultImagesCount); |
+} |
+ |
void DefaultImagesView::ClearSelection() { |
selected_image_index_ = -1; |
} |
@@ -141,9 +147,7 @@ void DefaultImagesView::ButtonPressed(views::Button* sender, |
NOTREACHED() << "Got ButtonPressed event from a view with wrong id."; |
return; |
} |
- selected_image_index_ = image_index + 1; |
- if (delegate_) |
- delegate_->OnImageSelected(image_index % kDefaultImagesCount); |
+ SetDefaultImageIndex(image_index); |
} |
} |