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) {
-    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;
-    }
-  }

[Avi (use Gerrit), 2012/10/14 18:26:53]

This code made no sense. Shell windows have TabContents, so if you call
TabContents::FromWebContents on them, you will get non-NULL back. From what I
see, this if() block was NEVER true...

[jennb, 2012/10/15 16:42:33]

ShellWindows aren't supposed to have TabContents. Ben Wells has a TODO to undo
that change.  http://crbug.com/151493

[Avi (use Gerrit), 2012/10/15 16:45:06]

Oh, so this code predates that.

Does this look reasonable to you, then, to drop this entire section?

[jennb, 2012/10/15 17:08:17]

Looks reasonable to me as no TabContents are needed here after this change, so
it won't matter whether ShellWindow has tabcontents or not.

[Avi (use Gerrit), 2012/10/15 17:39:48]

That's exactly the point ;)

 
   // 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 -------------------------------------