Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(45)

Issue 113229: Add new tab startup test to the linux & mac build. (Closed)

Created:
11 years, 7 months ago by tony
Modified:
9 years, 7 months ago
Reviewers:
Evan Martin
CC:
chromium-reviews_googlegroups.com
Visibility:
Public.

Description

Add new tab startup test to the linux & mac build.

Patch Set 1 #

Total comments: 1

Patch Set 2 : make the test actually run #

Total comments: 1
Unified diffs Side-by-side diffs Delta from patch set Stats (+7 lines, -16 lines) Patch
M base/test_file_util_posix.cc View 1 chunk +1 line, -2 lines 0 comments Download
M chrome/browser/automation/automation_provider.cc View 1 chunk +1 line, -1 line 1 comment Download
M chrome/chrome.gyp View 1 chunk +0 lines, -5 lines 0 comments Download
M chrome/test/startup/feature_startup_test.cc View 1 3 chunks +5 lines, -8 lines 0 comments Download

Messages

Total messages: 4 (0 generated)
tony
11 years, 7 months ago (2009-05-11 22:23:54 UTC) #1
Evan Martin
LGTM http://codereview.chromium.org/113229/diff/1/3 File chrome/test/startup/feature_startup_test.cc (right): http://codereview.chromium.org/113229/diff/1/3#newcode25 Line 25: source_history_file = source_history_file.AppendASCII("profiles"); source_history_file = source_history_file.AppendASCII("profiles") .AppendASCII("typical_history") ...
11 years, 7 months ago (2009-05-11 22:25:32 UTC) #2
tony
Fixed. Also, I ran the new tab startup test and it was hanging. This new ...
11 years, 7 months ago (2009-05-11 23:08:24 UTC) #3
Evan Martin
11 years, 7 months ago (2009-05-11 23:25:40 UTC) #4
LGTM if it doesnt' break windows

http://codereview.chromium.org/113229/diff/1003/8
File chrome/browser/automation/automation_provider.cc (right):

http://codereview.chromium.org/113229/diff/1003/8#newcode868
Line 868: #endif  // defined(OS_WIN)
Put a NOTIMPLEMENTED() in here?

Powered by Google App Engine
This is Rietveld 408576698