From 36be2f94e6b1d6ef3b1f8ee6c638a4f929f506d0 Mon Sep 17 00:00:00 2001 From: "brettw@google.com" Date: Sun, 14 Sep 2008 21:29:30 +0000 Subject: Fix a debug assertin when running the web contents unit test. The error is in the test since it would give the same page ID to two different pages in the same site instance, which should not happen. Review URL: http://codereview.chromium.org/2842 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@2206 0039d316-1c4b-4281-b951-d872f2087c98 --- chrome/browser/web_contents_unittest.cc | 3 +++ 1 file changed, 3 insertions(+) (limited to 'chrome/browser') diff --git a/chrome/browser/web_contents_unittest.cc b/chrome/browser/web_contents_unittest.cc index 5d386c5..dde6ba1 100644 --- a/chrome/browser/web_contents_unittest.cc +++ b/chrome/browser/web_contents_unittest.cc @@ -994,6 +994,9 @@ TEST_F(WebContentsTest, NavigateTwoTabsCrossSite) { // Open a new tab with the same SiteInstance, navigated to the same site. TestWebContents* contents2 = new TestWebContents(profile.get(), instance1); + params1.page_id = 2; // Need this since the site instance is the same (which + // is the scope of page IDs) and we want to consider + // this a new page. contents2->transition_cross_site = true; contents2->SetupController(profile.get()); contents2->controller()->LoadURL(url, PageTransition::TYPED); -- cgit v1.1