Split out of reverted change http://src.chromium.org/viewvc/chrome?view=rev&r...

chrome/browser/geolocation/network_location_provider.cc

 // Copyright (c) 2010 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/geolocation/network_location_provider.h"
 
 #include "base/time.h"
 
 namespace {
 // The maximum period of time we'll wait for a complete set of device data
@@ skipping 62 matching lines @@
                       string16* key) {
     // Currently we use only the WiFi data, and base the key only on the MAC
     // addresses.
     // TODO(steveblock): Make use of radio_data.
     DCHECK(key);
     key->clear();
     key->reserve(wifi_data.access_point_data.size() * 19);
     const string16 separator(ASCIIToUTF16("|"));
     for (WifiData::AccessPointDataSet::const_iterator iter =
          wifi_data.access_point_data.begin();
-         iter != wifi_data.access_point_data.begin();
+         iter != wifi_data.access_point_data.end();
          iter++) {
       *key += separator;
       *key += iter->mac_address;
       *key += separator;
     }
     // If the key is the empty string, return false, as we don't want to cache a
     // position for such a set of device data.
     return !key->empty();
   }
 
@@ skipping 190 matching lines @@
     return;
   }
   device_data_updated_timestamp_ = GetCurrentTimeMillis();
 
   is_new_data_available_ = is_radio_data_complete_ || is_radio_data_complete_;
   if (delayed_start_task_.empty() ||
       (is_radio_data_complete_ && is_radio_data_complete_)) {
     UpdatePosition();
   }
 }