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

Issue 3915004: Convert LOG(INFO) to VLOG(1) - chrome/test/. (Closed)

Created:
10 years, 2 months ago by Peter Kasting
Modified:
9 years, 7 months ago
Reviewers:
Elliot Glaysher
CC:
chromium-reviews, Raghu Simha, ncarter (slow), idana, tim (not reviewing), Paweł Hajdan Jr.
Visibility:
Public.

Description

Convert LOG(INFO) to VLOG(1) - chrome/test/. Also fix indentation, remove a few unnecessary endls, collapse some append()s into +s, and remove a few extra {}s. BUG=none TEST=none Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=63253

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+115 lines, -143 lines) Patch
M chrome/test/live_sync/live_autofill_sync_test.h View 1 chunk +3 lines, -3 lines 0 comments Download
M chrome/test/live_sync/live_sync_test.cc View 3 chunks +5 lines, -5 lines 0 comments Download
M chrome/test/live_sync/profile_sync_service_test_harness.cc View 6 chunks +24 lines, -27 lines 0 comments Download
M chrome/test/live_sync/two_client_live_bookmarks_sync_test.cc View 1 chunk +1 line, -1 line 0 comments Download
M chrome/test/memory_test/memory_test.cc View 3 chunks +3 lines, -3 lines 0 comments Download
M chrome/test/mini_installer_test/mini_installer_test_util.cc View 5 chunks +6 lines, -6 lines 0 comments Download
M chrome/test/page_cycler/page_cycler_test.cc View 2 chunks +2 lines, -2 lines 0 comments Download
M chrome/test/plugin/plugin_test.cpp View 1 chunk +1 line, -1 line 0 comments Download
M chrome/test/startup/shutdown_test.cc View 1 chunk +5 lines, -6 lines 0 comments Download
M chrome/test/startup/startup_test.cc View 1 chunk +5 lines, -6 lines 0 comments Download
M chrome/test/test_launcher/test_runner.cc View 2 chunks +3 lines, -4 lines 0 comments Download
M chrome/test/ui/ui_layout_test.cc View 1 chunk +2 lines, -2 lines 0 comments Download
M chrome/test/ui/ui_test.cc View 2 chunks +3 lines, -4 lines 0 comments Download
M chrome/test/ui_test_utils.cc View 1 chunk +1 line, -1 line 0 comments Download
M chrome/test/unit/chrome_test_suite.cc View 1 chunk +2 lines, -2 lines 0 comments Download
M chrome/test/webdriver/commands/create_session.cc View 1 chunk +1 line, -1 line 0 comments Download
M chrome/test/webdriver/commands/webdriver_command.cc View 1 chunk +1 line, -1 line 0 comments Download
M chrome/test/webdriver/dispatch.h View 2 chunks +3 lines, -4 lines 0 comments Download
M chrome/test/webdriver/dispatch.cc View 7 chunks +15 lines, -24 lines 0 comments Download
M chrome/test/webdriver/keymap.cc View 1 chunk +1 line, -1 line 0 comments Download
M chrome/test/webdriver/server.cc View 2 chunks +2 lines, -3 lines 0 comments Download
M chrome/test/webdriver/session.cc View 6 chunks +16 lines, -23 lines 0 comments Download
M chrome/test/webdriver/session_manager.cc View 3 chunks +5 lines, -5 lines 0 comments Download
M chrome/test/webdriver/utility_functions.cc View 2 chunks +5 lines, -8 lines 0 comments Download

Messages

Total messages: 2 (0 generated)
Peter Kasting
10 years, 2 months ago (2010-10-20 00:33:16 UTC) #1
Elliot Glaysher
10 years, 2 months ago (2010-10-20 00:40:09 UTC) #2
LGTM

Powered by Google App Engine
This is Rietveld 408576698