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

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: David's comments 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 b140c36beabfb3127e86779e845393139a9b7267..f881633c3a442b34294514218eeb3b0de896b2ba 100644
--- a/chrome/browser/ui/views/chrome_to_mobile_bubble_view.cc
+++ b/chrome/browser/ui/views/chrome_to_mobile_bubble_view.cc
@@ -261,7 +261,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(
@@ -274,7 +274,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) {
@@ -366,7 +366,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_,
« no previous file with comments | « chrome/browser/ui/tabs/pinned_tab_codec.cc ('k') | chrome/browser/ui/webui/chromeos/imageburner/imageburner_ui.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698