Create the correct SiteInstance when restoring tabs that belong to apps.

chrome/browser/tab_contents/tab_util.h

Index: chrome/browser/tab_contents/tab_util.h
diff --git a/chrome/browser/tab_contents/tab_util.h b/chrome/browser/tab_contents/tab_util.h
index 896be7cf81a102796e1fd557ea451d840b1cf20f..b04cb804bb6417153343c0031b79a5db979120e8 100644
--- a/chrome/browser/tab_contents/tab_util.h
+++ b/chrome/browser/tab_contents/tab_util.h
@@ -1,4 +1,4 @@
-// Copyright (c) 2010 The Chromium Authors. All rights reserved.
+// Copyright (c) 2011 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.
 
@@ -6,6 +6,9 @@
 #define CHROME_BROWSER_TAB_CONTENTS_TAB_UTIL_H_
 #pragma once
 
+class GURL;
+class Profile;
+class SiteInstance;
 class TabContents;
 
 namespace tab_util {
@@ -15,6 +18,12 @@ namespace tab_util {
 // Should only be called from the UI thread, since it accesses TabContent.
 TabContents* GetTabContentsByID(int render_process_host_id, int routing_id);
 
+// Returns a new SiteInstance for WebUI and app URLs. Returns the SiteInstance
+// for |source_contents| if it represents the same website as |url|. Returns
+// NULL otherwise.
+SiteInstance* GetSiteInstanceForNewTab(
+    TabContents* source_contents, Profile* profile, const GURL& url);

[Charlie Reis, 2011/07/20 20:41:23]

I think the arguments are supposed to each be on their own line if they're not
all on the first line, right?

+
 }  // namespace tab_util
 
 #endif  // CHROME_BROWSER_TAB_CONTENTS_TAB_UTIL_H_