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

Unified Diff: chrome/browser/ui/views/chrome_to_mobile_bubble_view.cc

Issue 10837044: Correct const accessors in base/values.(h|cc), Part II (ListValue) (Closed) Base URL: http://git.chromium.org/chromium/src.git@master
Patch Set: Created 8 years, 5 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/ui/views/chrome_to_mobile_bubble_view.cc
diff --git a/chrome/browser/ui/views/chrome_to_mobile_bubble_view.cc b/chrome/browser/ui/views/chrome_to_mobile_bubble_view.cc
index cff2aebc7b90f6aa69458d81cfb8ea3031f17997..39aefd7ac5e397d0d0aa80ad85e657b3e6e54dc5 100644
--- a/chrome/browser/ui/views/chrome_to_mobile_bubble_view.cc
+++ b/chrome/browser/ui/views/chrome_to_mobile_bubble_view.cc
@@ -260,7 +260,7 @@ void ChromeToMobileBubbleView::Init() {
const ListValue* mobiles = service_->GetMobiles();
if (mobiles->GetSize() == 1) {
string16 name;
- DictionaryValue* mobile = NULL;
+ const DictionaryValue* mobile = NULL;
if (mobiles->GetDictionary(0, &mobile) &&
mobile->GetString("name", &name)) {
title_label->SetText(l10n_util::GetStringFUTF16(
@@ -273,7 +273,7 @@ void ChromeToMobileBubbleView::Init() {
IDS_CHROME_TO_MOBILE_BUBBLE_MULTI_TITLE));
string16 name;
- DictionaryValue* mobile = NULL;
+ const DictionaryValue* mobile = NULL;
views::RadioButton* radio = NULL;
layout->AddPaddingRow(0, views::kRelatedControlSmallVerticalSpacing);
for (size_t index = 0; index < mobiles->GetSize(); ++index) {
@@ -365,7 +365,7 @@ void ChromeToMobileBubbleView::Send() {
DCHECK(radio_buttons_.empty());
}
- DictionaryValue* mobile = NULL;
+ const DictionaryValue* mobile = NULL;
if (mobiles->GetDictionary(selected_index, &mobile)) {
FilePath snapshot = send_copy_->checked() ? snapshot_path_ : FilePath();
service_->SendToMobile(*mobile, snapshot, browser_,

Powered by Google App Engine
This is Rietveld 408576698