diff options
author | ben@chromium.org <ben@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2010-11-02 15:15:33 +0000 |
---|---|---|
committer | ben@chromium.org <ben@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2010-11-02 15:15:33 +0000 |
commit | 616381f0a55322de447465573e39612e5af18b31 (patch) | |
tree | 4cbf687bb163f3dcd95e55e7be7fc9197e0dd8fd /chrome/browser/browser.h | |
parent | e5925962c61dbb9c8f5a5425ca2dc2953fb6a3e1 (diff) | |
download | chromium_src-616381f0a55322de447465573e39612e5af18b31.zip chromium_src-616381f0a55322de447465573e39612e5af18b31.tar.gz chromium_src-616381f0a55322de447465573e39612e5af18b31.tar.bz2 |
Make all browser code use browser::Navigate to open tabs.
BUG=none
TEST=existing unittests. Also, test all places where new tabs and windows are opened from UI, e.g. links in options, new tab button, Ctrl+T, popup windows, etc.
Review URL: http://codereview.chromium.org/3834002
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@64745 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'chrome/browser/browser.h')
-rw-r--r-- | chrome/browser/browser.h | 48 |
1 files changed, 0 insertions, 48 deletions
diff --git a/chrome/browser/browser.h b/chrome/browser/browser.h index 5654106..7d89334 100644 --- a/chrome/browser/browser.h +++ b/chrome/browser/browser.h @@ -343,40 +343,6 @@ class Browser : public TabHandlerDelegate, TabContents* AddSelectedTabWithURL(const GURL& url, PageTransition::Type transition); - // Parameters for AddTabWithURL. - struct AddTabWithURLParams { - AddTabWithURLParams(const GURL& a_url, PageTransition::Type a_transition); - ~AddTabWithURLParams(); - - // Basic configuration. - GURL url; - GURL referrer; - PageTransition::Type transition; - - // The index to open the tab at. This could be ignored by the tabstrip - // depending on the combination of |add_types| specified. - int index; - - // A bitmask of values defined in TabStripModel::AddTabTypes. - // The default is ADD_SELECTED. - int add_types; - - // If non-NULL, used to render the tab. - SiteInstance* instance; - - // If non-empty, the new tab is an app tab. - std::string extension_app_id; - - // The browser where the tab was added. - Browser* target; - - private: - AddTabWithURLParams(); - }; - - // Adds a tab to the browser (or another) and returns the created TabContents. - TabContents* AddTabWithURL(AddTabWithURLParams* params); - // Add a new tab, given a TabContents. A TabContents appropriate to // display the last committed entry is created and returned. TabContents* AddTab(TabContents* tab_contents, PageTransition::Type type); @@ -928,20 +894,6 @@ class Browser : public TabHandlerDelegate, WindowOpenDisposition *disposition, PageTransition::Type transition); - // The low-level function that other OpenURL...() functions call. This - // determines the appropriate SiteInstance to pass to AddTabWithURL(), focuses - // the newly created tab as needed, and does other miscellaneous housekeeping. - // |add_types| is a bitmask of TabStripModel::AddTabTypes and may be used to - // pass specific flags to the TabStripModel. |add_types| is only used if - // the disposition is NEW_WINDOW or SUPPRESS_OPEN. - void OpenURLAtIndex(TabContents* source, - const GURL& url, - const GURL& referrer, - WindowOpenDisposition disposition, - PageTransition::Type transition, - int index, - int add_types); - // Shows the Find Bar, optionally selecting the next entry that matches the // existing search string for that Tab. |forward_direction| controls the // search direction. |