Rename AppListMainViewTest::ContentsView to AppListMainViewTest::GetContentsView to fix build break

ui/app_list/views/app_list_main_view_unittest.cc

Index: ui/app_list/views/app_list_main_view_unittest.cc
diff --git a/ui/app_list/views/app_list_main_view_unittest.cc b/ui/app_list/views/app_list_main_view_unittest.cc
index b8307618d0d4919af2e5e9b1628245fb9afbf1cd..e523fd6778839a623b80bbc692f68fcd57fc46f3 100644
--- a/ui/app_list/views/app_list_main_view_unittest.cc
+++ b/ui/app_list/views/app_list_main_view_unittest.cc
@@ -154,7 +154,7 @@ class AppListMainViewTest : public views::ViewsTestBase {
     grid_view->UpdateDragFromItem(pointer, drag_event);
   }
 
-  ContentsView* ContentsView() { return main_view_->contents_view(); }
+  app_list::ContentsView* ContentsView() { return main_view_->contents_view(); }

[Matt Giuca, 2014/12/02 04:33:37]

I'm not too sure about this, but I asked around and was told that the root cause
of this issue is not the namespacing, but the fact that ContentsView is the name
of both the method and the class.

We should *not* have to explicitly qualify this type. The correct fix (I think)
is to just rename the method. The correct name for this method is
contents_view() or GetContentsView(). I prefer contents_view() since the
accessor is reasonably simple. Would you be able to rename it and try again?

(Don't worry about renaming these other methods; there would be too many places
to update.)

 
   AppsGridView* RootGridView() {
     return ContentsView()->apps_container_view()->apps_grid_view();