The ChromeFrame net tests would not work on IE6 and IE7. This is because the profile path used varies across these browsers.

This changed in the recent fixes for privacy where on IE6 and IE7 the profile is in temporary internet files. The ChromeFrame
net tests assumed the profile would always be in the default path which caused the tests to not work.

Added a helper function in chrome_Frame_test_utils.cc/.h to retrieve the correct profile path based on IE version and use this.

This should get the net tests up and running on the FYI IE6 builder.

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

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

2 files changed, 2 insertions, 16 deletions

diff --git a/chrome_frame/test/net/fake_external_tab.cc b/chrome_frame/test/net/fake_external_tab.cc
index 8db3b48..497ae88 100644
--- a/chrome_frame/test/net/fake_external_tab.cc
+++ b/chrome_frame/test/net/fake_external_tab.cc
@@ -12,6 +12,7 @@
 
 #include "base/command_line.h"
 #include "base/file_util.h"
+#include "base/file_version_info.h"
 #include "base/i18n/icu_util.h"
 #include "base/path_service.h"
 #include "base/scoped_bstr_win.h"
@@ -164,10 +165,9 @@ BOOL SupplyProxyCredentials::EnumChildren(HWND hwnd, LPARAM param) {
   return TRUE;
 }
 
-
 FakeExternalTab::FakeExternalTab() {
+  user_data_dir_ = chrome_frame_test::GetProfilePathForIE();
   PathService::Get(chrome::DIR_USER_DATA, &overridden_user_dir_);
-  GetProfilePath(&user_data_dir_);
   PathService::Override(chrome::DIR_USER_DATA, user_data_dir_);
   process_singleton_.reset(new ProcessSingleton(user_data_dir_));
 }
@@ -178,17 +178,6 @@ FakeExternalTab::~FakeExternalTab() {
   }
 }
 
-std::wstring FakeExternalTab::GetProfileName() {
-  return L"iexplore";
-}
-
-bool FakeExternalTab::GetProfilePath(FilePath* path) {
-  if (!chrome::GetChromeFrameUserDataDirectory(path))
-    return false;
-  *path = path->Append(GetProfileName());
-  return true;
-}
-
 void FakeExternalTab::Initialize() {
   DCHECK(g_browser_process == NULL);
   SystemMonitor system_monitor;
diff --git a/chrome_frame/test/net/fake_external_tab.h b/chrome_frame/test/net/fake_external_tab.h
index dd35756..5238277 100644
--- a/chrome_frame/test/net/fake_external_tab.h
+++ b/chrome_frame/test/net/fake_external_tab.h
@@ -26,9 +26,6 @@ class FakeExternalTab {
   FakeExternalTab();
   virtual ~FakeExternalTab();
 
-  virtual std::wstring GetProfileName();
-
-  virtual bool GetProfilePath(FilePath* path);
   virtual void Initialize();
   virtual void Shutdown();