Move GetTabContentsID out of tab util because it has nothing to do with tabs or

TabContents. I put a more accurately named static function in
ResourceDispatcherHost which is what really controls this request data.

I also moved a couple of static functions from the header to the cc in
resource_dispatcher.

Review URL: http://codereview.chromium.org/150124

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@19675 0039d316-1c4b-4281-b951-d872f2087c98

1 files changed, 4 insertions, 2 deletions

diff --git a/chrome/browser/login_prompt_win.cc b/chrome/browser/login_prompt_win.cc
index 4437737..b74024b 100644
--- a/chrome/browser/login_prompt_win.cc
+++ b/chrome/browser/login_prompt_win.cc
@@ -7,6 +7,7 @@
 #include "app/l10n_util.h"

 #include "base/message_loop.h"

 #include "chrome/browser/password_manager/password_manager.h"

+#include "chrome/browser/renderer_host/resource_dispatcher_host.h"
 #include "chrome/browser/tab_contents/navigation_controller.h"

 #include "chrome/browser/tab_contents/tab_contents.h"

 #include "chrome/browser/tab_contents/tab_util.h"

@@ -40,8 +41,9 @@ class LoginHandlerWin : public LoginHandler,
     DCHECK(request_) << "LoginHandler constructed with NULL request";

 

     AddRef();  // matched by ReleaseLater.

-    if (!tab_util::GetTabContentsID(request_, &render_process_host_id_,

-                                    &tab_contents_id_)) {

+    if (!ResourceDispatcherHost::RenderViewForRequest(request_,

+                                                      &render_process_host_id_,

+                                                      &tab_contents_id_)) {

       NOTREACHED();

     }

   }