Add regression test for bug 7900 and necessary infrastructure....

chrome/app/chrome_main_uitest.cc

 // Copyright (c) 2006-2008 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.
 
-#include "chrome/common/chrome_constants.h"
 #include "chrome/test/ui/ui_test.h"
 
+#include "base/command_line.h"
+#include "base/file_path.h"
+#include "base/file_util.h"
+#include "net/base/net_util.h"
+
 typedef UITest ChromeMainTest;
 
 // Launch the app, then close the app.
 TEST_F(ChromeMainTest, AppLaunch) {
   // If we make it here at all, we've succeeded in retrieving the app window
   // in UITest::SetUp()--otherwise we'd fail with an exception in SetUp().
 
   if (UITest::in_process_renderer()) {
     EXPECT_EQ(1, UITest::GetBrowserProcessCount());
   } else {
@@ skipping 19 matching lines @@
   use_existing_browser_ = true;
 
   LaunchBrowser(CommandLine(L""), false);
 
   int window_count;
   ASSERT_TRUE(automation()->WaitForWindowCountToChange(1, &window_count,
                                                        action_timeout_ms()));
   ASSERT_EQ(2, window_count);
 }
 
+TEST_F(ChromeMainTest, ReuseBrowserInstanceWhenOpeningFile) {
+  include_testing_id_ = false;
+  use_existing_browser_ = true;
+
+  std::wstring test_file = test_data_directory_;
+  file_util::AppendToPath(&test_file, L"empty.html");
+
+  CommandLine command_line(L"");
+  command_line.AppendLooseValue(test_file);
+
+  LaunchBrowser(command_line, false);
+
+  FilePath test_file_path(FilePath::FromWStringHack(test_file));
+
+  ASSERT_TRUE(automation()->WaitForURLDisplayed(
+      net::FilePathToFileURL(test_file_path), action_timeout_ms()));

[Nicolas Sylvain, 2009/03/06 20:50:46]

did we understand why it's creating a window instead of a tab? My testing were
not really useful. I'd like to understand the problem.

And if you feel like it, the test just above (SecondLaunch) had the same problem
IIRC, so you might want to do the samething there and reenable it. (That might
need refactoring a little bit your code though). If you don't want it's ok too.

But I would still like to understand the problem..

Other than that the code LGTM.

thanks

Nicolas

+}