Remove TabContents from media galleries.

chrome/browser/extensions/api/media_galleries/media_galleries_api.cc

Index: chrome/browser/extensions/api/media_galleries/media_galleries_api.cc
diff --git a/chrome/browser/extensions/api/media_galleries/media_galleries_api.cc b/chrome/browser/extensions/api/media_galleries/media_galleries_api.cc
index 143096434049f2dd0f0a93cd83661e98d0f8056c..85e840ca5a051024ca5bd56b57e51ce41a8d9c79 100644
--- a/chrome/browser/extensions/api/media_galleries/media_galleries_api.cc
+++ b/chrome/browser/extensions/api/media_galleries/media_galleries_api.cc
@@ -14,12 +14,9 @@
 #include "base/platform_file.h"
 #include "base/stl_util.h"
 #include "base/values.h"
-#include "chrome/browser/extensions/shell_window_registry.h"
 #include "chrome/browser/media_gallery/media_file_system_registry.h"
 #include "chrome/browser/media_gallery/media_galleries_dialog_controller.h"
 #include "chrome/browser/ui/chrome_select_file_policy.h"
-#include "chrome/browser/ui/extensions/shell_window.h"
-#include "chrome/browser/ui/tab_contents/tab_contents.h"
 #include "chrome/common/extensions/api/experimental_media_galleries.h"
 #include "chrome/common/extensions/api/media_galleries.h"
 #include "chrome/common/extensions/permissions/media_galleries_permission.h"
@@ -161,24 +158,11 @@ void MediaGalleriesGetMediaFileSystemsFunction::ReturnGalleries(
 
 void MediaGalleriesGetMediaFileSystemsFunction::ShowDialog() {
   WebContents* contents = WebContents::FromRenderViewHost(render_view_host());
-  TabContents* tab_contents =
-      contents ? TabContents::FromWebContents(contents) : NULL;
-  if (!tab_contents) {

[Evan Stade, 2012/10/15 21:34:06]

this code was here in case the API was called from the background page. You
still need to do something similar, but I don't see that happening anywhere.

[Avi (use Gerrit), 2012/10/15 22:10:01]

Question: Suppose that the background page doesn't have a TabContents. Why does
the source page need a TabContents for this to work? What about a TabContents is
needed?

[Evan Stade, 2012/10/15 22:51:33]

the background page does not have a tab contents. That is why we have to find a
different one (i.e. the tab contents for the last-used shell window for the
app). This tab contents is necessary as the contents the constrained window sits
on top of/within.

[Avi (use Gerrit), 2012/10/17 22:15:42]

I have a fix, but found a related crash at the same time. If the background page
loads a constrained window on the shell window (as this code below does), and
then kicks off navigation on that window, the navigation completing cancels the
constrained window but doesn't hide it. When the user clicks a button it
crashes.

I can't commit a fix yet because I've got a dependency to fix. Working on this,
though.

[Evan Stade, 2012/10/17 22:18:59]

might that be fixed by r162402?

[Avi (use Gerrit), 2012/10/17 22:28:48]

This appears to be different. I'm verifying with a trunk build that doesn't have
this patch :) and I'll send you a bug.

[Evan Stade, 2012/10/17 22:35:11]

thanks.

-    ShellWindow* window = ShellWindowRegistry::Get(profile())->
-        GetCurrentShellWindowForApp(GetExtension()->id());
-    if (window) {
-      tab_contents = window->tab_contents();
-    } else {
-      // Abort showing the dialog. TODO(estade) Perhaps return an error instead.
-      GetAndReturnGalleries();
-      return;
-    }
-  }
 
   // Controller will delete itself.
   base::Closure cb = base::Bind(
       &MediaGalleriesGetMediaFileSystemsFunction::GetAndReturnGalleries, this);
-  new chrome::MediaGalleriesDialogController(tab_contents, *GetExtension(), cb);
+  new chrome::MediaGalleriesDialogController(contents, *GetExtension(), cb);
 }
 
 // MediaGalleriesAssembleMediaFileFunction -------------------------------------