Fix crash when drag moving URLs in chrome://settings/startup.

chrome/browser/custom_home_pages_table_model.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/custom_home_pages_table_model.h"
 
 #include "base/bind.h"
 #include "base/bind_helpers.h"
 #include "base/i18n/rtl.h"
 #include "base/prefs/pref_service.h"
@@ skipping 78 matching lines @@
 
 /**
  * Move a number of existing entries to a new position, reordering the table.
  *
  * We determine the range of elements affected by the move, save the moved
  * elements, compact the remaining ones, and re-insert moved elements.
  * Expects |index_list| to be ordered ascending.
  */
 void CustomHomePagesTableModel::MoveURLs(int insert_before,
                                          const std::vector<int>& index_list) {
-  DCHECK(!index_list.empty());
   DCHECK(insert_before >= 0 && insert_before <= RowCount());
 
+  // This is a bug. This should be a DCHECK, but under circumstances, the JS can
+  // get confused and pass us an empty list of selected indices to drag move.

[Dan Beam, 2013/10/14 18:08:07]

Are you sure it's the WebUI's fault? This can only be called from C++ as it's
not a message handling method.

+  // See http://crbug.com/302284.
+  if (index_list.empty())

[Dan Beam, 2013/10/14 18:08:07]

NOTREACHED();

+    return;
+
   // The range of elements that needs to be reshuffled is [ |first|, |last| ).
   int first = std::min(insert_before, index_list.front());
   int last = std::max(insert_before, index_list.back() + 1);
 
   // Save the dragged elements. Also, adjust insertion point if it is before a
   // dragged element.
   std::vector<Entry> moved_entries;
   for (size_t i = 0; i < index_list.size(); ++i) {
     moved_entries.push_back(entries_[index_list[i]]);
     if (index_list[i] == insert_before)
@@ skipping 152 matching lines @@
   return NULL;
 }
 
 string16 CustomHomePagesTableModel::FormattedURL(int row) const {
   std::string languages =
       profile_->GetPrefs()->GetString(prefs::kAcceptLanguages);
   string16 url = net::FormatUrl(entries_[row].url, languages);
   url = base::i18n::GetDisplayStringInLTRDirectionality(url);
   return url;
 }