Enable warning 4389 as an error on windows builds. This will make...

views/controls/table/table_view.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 "views/controls/table/table_view.h"
 
 #include <commctrl.h>
 #include <windowsx.h>
 
 #include <algorithm>
@@ skipping 352 matching lines @@
   //
   // When the model is set to NULL all the rows are removed. We don't notify
   // the delegate in this case as setting the model to NULL is usually done as
   // the last step before being deleted and callers shouldn't have to deal with
   // getting a selection change when the model is being reset.
   if (model_ && table_view_observer_ && had_selection && RowCount() == 0)
     table_view_observer_->OnSelectionChanged();
 }
 
 void TableView::AddColumn(const TableColumn& col) {
-  DCHECK_EQ(0, all_columns_.count(col.id));
+  DCHECK_EQ(0u, all_columns_.count(col.id));
   all_columns_[col.id] = col;
 }
 
 void TableView::SetColumns(const std::vector<TableColumn>& columns) {
   // Remove the currently visible columns.
   while (!visible_columns_.empty())
     SetColumnVisibility(visible_columns_.front(), false);
 
   all_columns_.clear();
   for (std::vector<TableColumn>::const_iterator i = columns.begin();
@@ skipping 1186 matching lines @@
 }
 
 void TableSelectionIterator::UpdateModelIndexFromViewIndex() {
   if (view_index_ == -1)
     model_index_ = -1;
   else
     model_index_ = table_view_->view_to_model(view_index_);
 }
 
 }  // namespace views