Drag and drop between app list and launcher - First patch

ui/app_list/views/app_list_view.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 "ui/app_list/views/app_list_view.h"
 
 #include "base/string_util.h"
 #include "ui/app_list/app_list_constants.h"
 #include "ui/app_list/app_list_model.h"
 #include "ui/app_list/app_list_view_delegate.h"
@@ skipping 103 matching lines @@
   GetBubbleFrameView()->bubble_border()->set_arrow(arrow);
   SizeToContents();  // Recalcuates with new border.
   GetBubbleFrameView()->SchedulePaint();
 }
 
 void AppListView::SetAnchorPoint(const gfx::Point& anchor_point) {
   set_anchor_rect(gfx::Rect(anchor_point, gfx::Size()));
   SizeToContents();  // Repositions view relative to the anchor.
 }
 
+void AppListView::SetAppListDnDHost(app_list::ApplicationDnDHost* dnd_host) {
+  DCHECK(app_list_main_view_);

[James Cook, 2013/04/29 22:08:29]

I like asserts!

[Mr4D (OOO till 08-26), 2013/04/30 16:59:01]

Okay, converted into a CHECK since this should not happen - debug or not.

+  app_list_main_view_->SetAppListDnDHost(dnd_host);
+}
+
 void AppListView::ShowWhenReady() {
   app_list_main_view_->ShowAppListWhenReady();
 }
 
 void AppListView::Close() {
   app_list_main_view_->Close();
-
+  app_list_main_view_ = NULL;
   if (delegate_.get())
     delegate_->Dismiss();
   else
     GetWidget()->Close();
 }
 
 void AppListView::UpdateBounds() {
   SizeToContents();
 }
 
@@ skipping 82 matching lines @@
 
 void AppListView::OnSigninSuccess() {
   OnSigninStatusChanged();
 }
 
 SigninDelegate* AppListView::GetSigninDelegate() {
   return delegate_ ? delegate_->GetSigninDelegate() : NULL;
 }
 
 }  // namespace app_list