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

Unified Diff: chrome/browser/ui/gtk/chrome_to_mobile_bubble_gtk.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, 4 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/gtk/chrome_to_mobile_bubble_gtk.cc
diff --git a/chrome/browser/ui/gtk/chrome_to_mobile_bubble_gtk.cc b/chrome/browser/ui/gtk/chrome_to_mobile_bubble_gtk.cc
index 8f868f00fcd08b9ad9bc0dd17cd27a71e25fe97c..92a678b95fd338b6819615f3101c9e7620dfbc16 100644
--- a/chrome/browser/ui/gtk/chrome_to_mobile_bubble_gtk.cc
+++ b/chrome/browser/ui/gtk/chrome_to_mobile_bubble_gtk.cc
@@ -172,7 +172,7 @@ ChromeToMobileBubbleGtk::ChromeToMobileBubbleGtk(GtkImage* anchor_image,
GtkWidget* title = NULL;
if (mobiles->GetSize() == 1) {
string16 name;
- DictionaryValue* mobile = NULL;
+ const DictionaryValue* mobile = NULL;
if (mobiles->GetDictionary(0, &mobile) &&
mobile->GetString("name", &name)) {
title = gtk_label_new(l10n_util::GetStringFUTF8(
@@ -191,7 +191,7 @@ ChromeToMobileBubbleGtk::ChromeToMobileBubbleGtk(GtkImage* anchor_image,
// Create and pack the device radio group; init the selected mobile device.
if (mobiles->GetSize() > 1) {
std::string name;
- DictionaryValue* mobile = NULL;
+ const DictionaryValue* mobile = NULL;
GtkWidget* radio = NULL;
GtkWidget* row = NULL;
for (size_t index = 0; index < mobiles->GetSize(); ++index) {
@@ -321,7 +321,7 @@ void ChromeToMobileBubbleGtk::OnSendClicked(GtkWidget* widget) {
DCHECK(radio_buttons_.empty());
}
- DictionaryValue* mobile = NULL;
+ const DictionaryValue* mobile = NULL;
if (mobiles->GetDictionary(selected_index, &mobile)) {
bool snapshot = gtk_toggle_button_get_active(GTK_TOGGLE_BUTTON(send_copy_));
service_->SendToMobile(*mobile, snapshot ? snapshot_path_ : FilePath(),
« no previous file with comments | « chrome/browser/ui/cocoa/chrome_to_mobile_bubble_controller.mm ('k') | chrome/browser/ui/tabs/pinned_tab_codec.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698