Use MAKEPOINTS instead of GET_X/Y_LPARAM to reduce a dependency on ATL....

chrome/browser/views/keyword_editor_view.cc

 // Copyright (c) 2009 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/views/keyword_editor_view.h"
 
-#include <atlbase.h>
-#include <atlapp.h>
 #include <vector>
 
 #include "app/l10n_util.h"
+#include "base/gfx/point.h"
 #include "base/stl_util-inl.h"
 #include "base/string_util.h"
 #include "chrome/browser/profile.h"
 #include "chrome/browser/search_engines/template_url.h"
 #include "chrome/browser/search_engines/template_url_model.h"
 #include "chrome/browser/search_engines/template_url_table_model.h"
 #include "chrome/browser/views/browser_dialogs.h"
 #include "chrome/common/pref_names.h"
 #include "chrome/common/pref_service.h"
 #include "googleurl/src/gurl.h"
@@ skipping 190 matching lines @@
     can_make_default = controller_->CanMakeDefault(selected_url);
     can_remove = controller_->CanRemove(selected_url);
   }
   remove_button_->SetEnabled(can_remove);
   make_default_button_->SetEnabled(can_make_default);
 }
 
 void KeywordEditorView::OnDoubleClick() {
   if (edit_button_->IsEnabled()) {
     DWORD pos = GetMessagePos();
-    POINT cursor_point = { GET_X_LPARAM(pos), GET_Y_LPARAM(pos) };
+    gfx::Point cursor_point(pos);
     views::MouseEvent event(views::Event::ET_MOUSE_RELEASED,
-                            cursor_point.x, cursor_point.y,
+                            cursor_point.x(), cursor_point.y(),
                             views::Event::EF_LEFT_BUTTON_DOWN);
     ButtonPressed(edit_button_, event);
   }
 }
 
 void KeywordEditorView::ButtonPressed(
     views::Button* sender, const views::Event& event) {
   if (sender == add_button_) {
     browser::EditSearchEngine(GetWindow()->GetNativeWindow(), NULL, this,
                               profile_);
   } else if (sender == remove_button_) {
-    DCHECK(table_view_->SelectedRowCount() == 1);
+    DCHECK_EQ(1, table_view_->SelectedRowCount());
     int last_view_row = -1;
     for (views::TableView::iterator i = table_view_->SelectionBegin();
          i != table_view_->SelectionEnd(); ++i) {
       last_view_row = table_view_->model_to_view(*i);
       controller_->RemoveTemplateURL(*i);
     }
     if (last_view_row >= controller_->table_model()->RowCount())
       last_view_row = controller_->table_model()->RowCount() - 1;
     if (last_view_row >= 0)
       table_view_->Select(table_view_->view_to_model(last_view_row));
@@ skipping 13 matching lines @@
 void KeywordEditorView::OnTemplateURLModelChanged() {
   add_button_->SetEnabled(controller_->loaded());
 }
 
 void KeywordEditorView::MakeDefaultTemplateURL() {
   int new_index =
       controller_->MakeDefaultTemplateURL(table_view_->FirstSelectedRow());
   if (new_index >= 0)
     table_view_->Select(new_index);
 }