From 7dd06a0131c05212100c81ae88ae07f0757ff4db Mon Sep 17 00:00:00 2001 From: "tommi@chromium.org" Date: Thu, 9 Dec 2010 02:56:24 +0000 Subject: Temporary workaround to get tests that reference singletons either directly or indirectly to pass. With the recent Singleton refactoring, these tests started exposing some inherent problems with singletons, namely that one test can reference a state set by a different test, which can cause all sorts of flakiness. Furthermore, some of these tests could reference the registry which may have some user specific settings that could cause the tests to fail To work around these issues, I'm adding a few utility classes for tests (registry virtualization) and exposing a method to reset singletons before a test is run. TEST=Fixes flakiness and a few tests that could fail on the waterfall. BUG=none Review URL: http://codereview.chromium.org/5564009 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@68701 0039d316-1c4b-4281-b951-d872f2087c98 --- chrome_frame/urlmon_bind_status_callback.cc | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) (limited to 'chrome_frame/urlmon_bind_status_callback.cc') diff --git a/chrome_frame/urlmon_bind_status_callback.cc b/chrome_frame/urlmon_bind_status_callback.cc index 3d03542..474f8f3 100644 --- a/chrome_frame/urlmon_bind_status_callback.cc +++ b/chrome_frame/urlmon_bind_status_callback.cc @@ -171,7 +171,7 @@ void SniffData::DetermineRendererType(bool last_chance) { if (is_undetermined()) { if (last_chance) renderer_type_ = OTHER; - if (IsChrome(RendererTypeForUrl(url_.c_str()))) { + if (IsChrome(RendererTypeForUrl(url_))) { renderer_type_ = CHROME; } else { if (is_cache_valid() && cache_) { @@ -428,4 +428,3 @@ bool BSCBStorageBind::ShouldCacheProgress(unsigned long status_code) const { return false; } - -- cgit v1.1