diff options
author | cbentzel@chromium.org <cbentzel@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2010-10-25 13:59:59 +0000 |
---|---|---|
committer | cbentzel@chromium.org <cbentzel@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2010-10-25 13:59:59 +0000 |
commit | 543bcda1772cdd04e39fc8d912f61155ff570b9f (patch) | |
tree | 9fa0adc649d9c6f1b70e97f9bab91230fd6ef04c /chrome_frame/utils.h | |
parent | bcad9546fcb5da9d35173dbb91deadaf27899d0d (diff) | |
download | chromium_src-543bcda1772cdd04e39fc8d912f61155ff570b9f.zip chromium_src-543bcda1772cdd04e39fc8d912f61155ff570b9f.tar.gz chromium_src-543bcda1772cdd04e39fc8d912f61155ff570b9f.tar.bz2 |
This change moves the test_server_.Start from the test body to the test constructor. This needs to be done so the browser launch arguments can include the correct port in the mapping rules.
I considered adding an ASSERT in the constructor, but this warns against it:
http://code.google.com/p/googletest/wiki/AdvancedGuide#Assertion_Placement
BUG=56814
TEST=ui_tests --gtest_filter="*TestIsSearchProviderInstalled*"
Review URL: http://codereview.chromium.org/4007011
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@63727 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'chrome_frame/utils.h')
0 files changed, 0 insertions, 0 deletions