Store ChromeToMobile device list and timestamp prefs.

chrome/browser/ui/views/chrome_to_mobile_bubble_view.cc

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "chrome/browser/ui/views/chrome_to_mobile_bubble_view.h"
 
 #include "base/bind.h"
 #include "base/file_util.h"
 #include "base/message_loop.h"
 #include "base/string16.h"
@@ skipping 207 matching lines @@
     layout->AddPaddingRow(0, views::kRelatedControlSmallVerticalSpacing);
     layout->StartRow(0, 0 /*single_column_set_id*/);
     layout->AddView(error_label);
     SizeToContents();
   }
 
   Layout();
 }
 
 void ChromeToMobileBubbleView::Init() {
+  DCHECK(service_->HasMobiles());
+  service_->LogMetric(ChromeToMobileService::BUBBLE_SHOWN);
+
   GridLayout* layout = new GridLayout(this);
   SetLayoutManager(layout);
 
   const size_t single_column_set_id = 0;
   views::ColumnSet* cs = layout->AddColumnSet(single_column_set_id);
   cs->AddColumn(GridLayout::LEADING, GridLayout::LEADING, 0,
                 GridLayout::USE_PREF, 0, 0);
   cs->AddPaddingColumn(1, 0);
 
   const size_t button_column_set_id = 1;
   cs = layout->AddColumnSet(button_column_set_id);
   cs->AddColumn(GridLayout::LEADING, GridLayout::TRAILING, 0,
                 GridLayout::USE_PREF, 0, 0);
   cs->AddPaddingColumn(1, 0);
   cs->AddColumn(GridLayout::LEADING, GridLayout::TRAILING, 0,
                 GridLayout::USE_PREF, 0, 0);
   // Subtract 2px for the natural button padding and to correspond with row
   // separation height; like BookmarkBubbleView.
   cs->AddPaddingColumn(0, views::kRelatedButtonHSpacing - 2);
   cs->AddColumn(GridLayout::LEADING, GridLayout::TRAILING, 0,
                 GridLayout::USE_PREF, 0, 0);
 
-  // Get the list of mobile devices.
-  const ListValue& mobiles = service_->mobiles();
-  DCHECK(!mobiles.empty());
-
   ui::ResourceBundle& rb = ui::ResourceBundle::GetSharedInstance();
   views::Label* title_label = new views::Label();
   title_label->SetFont(rb.GetFont(ui::ResourceBundle::MediumFont));
   title_label->SetEnabledColor(kTitleColor);
   layout->StartRow(0, single_column_set_id);
   layout->AddView(title_label);
 
-  if (mobiles.GetSize() == 1) {
+  const ListValue* mobiles = service_->GetMobiles();
+  if (mobiles->GetSize() == 1) {
     string16 name;
     DictionaryValue* mobile = NULL;
-    if (mobiles.GetDictionary(0, &mobile) && mobile->GetString("name", &name)) {
+    if (mobiles->GetDictionary(0, &mobile) &&
+        mobile->GetString("name", &name)) {
       title_label->SetText(l10n_util::GetStringFUTF16(
           IDS_CHROME_TO_MOBILE_BUBBLE_SINGLE_TITLE, name));
     } else {
       NOTREACHED();
     }
   } else {
     title_label->SetText(l10n_util::GetStringUTF16(
         IDS_CHROME_TO_MOBILE_BUBBLE_MULTI_TITLE));
 
     string16 name;
     DictionaryValue* mobile = NULL;
     views::RadioButton* radio = NULL;
     layout->AddPaddingRow(0, views::kRelatedControlSmallVerticalSpacing);
-    for (size_t index = 0; index < mobiles.GetSize(); ++index) {
-      if (mobiles.GetDictionary(index, &mobile) &&
+    for (size_t index = 0; index < mobiles->GetSize(); ++index) {
+      if (mobiles->GetDictionary(index, &mobile) &&
           mobile->GetString("name", &name)) {
         radio = new views::RadioButton(name, 0);
         radio->SetEnabledColor(SK_ColorBLACK);
         radio->SetHoverColor(SK_ColorBLACK);
         radio_buttons_.push_back(radio);
         layout->StartRow(0, single_column_set_id);
         layout->AddView(radio);
       } else {
         NOTREACHED();
       }
@@ skipping 33 matching lines @@
 
 ChromeToMobileBubbleView::ChromeToMobileBubbleView(views::View* anchor_view,
                                                    Browser* browser)
     : BubbleDelegateView(anchor_view, views::BubbleBorder::TOP_RIGHT),
       ALLOW_THIS_IN_INITIALIZER_LIST(weak_ptr_factory_(this)),
       browser_(browser),
       service_(ChromeToMobileServiceFactory::GetForProfile(browser->profile())),
       send_copy_(NULL),
       send_(NULL),
       cancel_(NULL) {
-  service_->LogMetric(ChromeToMobileService::BUBBLE_SHOWN);
-
   // Generate the MHTML snapshot now to report its size in the bubble.
   service_->GenerateSnapshot(browser_, weak_ptr_factory_.GetWeakPtr());
 
   // Request a mobile device list update.
   service_->RequestMobileListUpdate();
 }
 
 void ChromeToMobileBubbleView::LinkClicked(views::Link* source,
                                            int event_flags) {
   service_->LearnMore(browser_);
   GetWidget()->Close();
 }
 
 void ChromeToMobileBubbleView::HandleButtonPressed(views::Button* sender) {
   if (sender == send_)
     Send();
   else if (sender == cancel_)
     GetWidget()->Close();
 }
 
 void ChromeToMobileBubbleView::Send() {
   // TODO(msw): Handle updates to the mobile list while the bubble is open.
-  const ListValue& mobiles = service_->mobiles();
+  const ListValue* mobiles = service_->GetMobiles();
   size_t selected_index = 0;
-  if (mobiles.GetSize() > 1) {
-    DCHECK_EQ(mobiles.GetSize(), radio_buttons_.size());
+  if (mobiles->GetSize() > 1) {
+    DCHECK_EQ(mobiles->GetSize(), radio_buttons_.size());
     for (; selected_index < radio_buttons_.size(); ++selected_index) {
       if (radio_buttons_[selected_index]->checked())
         break;
     }
   } else {
     DCHECK(radio_buttons_.empty());
   }
 
   DictionaryValue* mobile = NULL;
-  if (mobiles.GetDictionary(selected_index, &mobile)) {
+  if (mobiles->GetDictionary(selected_index, &mobile)) {
     FilePath snapshot = send_copy_->checked() ? snapshot_path_ : FilePath();
     service_->SendToMobile(*mobile, snapshot, browser_,
                            weak_ptr_factory_.GetWeakPtr());
   } else {
     NOTREACHED();
   }
 
   // Update the view's contents to show the "Sending..." progress animation.
   cancel_->SetEnabled(false);
   send_->SetEnabled(false);
   send_->set_alignment(views::TextButtonBase::ALIGN_LEFT);
   progress_animation_.reset(new ui::ThrobAnimation(this));
   progress_animation_->SetDuration(kProgressThrobDurationMS);
   progress_animation_->StartThrobbing(-1);
 }