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

Unified Diff: chrome_frame/test/chrome_frame_ui_test_utils.cc

Issue 9838058: More non-debug logging in Chrome Frame tests. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: copyright and formatting tweaks Created 8 years, 9 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « chrome_frame/test/chrome_frame_test_utils.cc ('k') | chrome_frame/test/ie_event_sink.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome_frame/test/chrome_frame_ui_test_utils.cc
diff --git a/chrome_frame/test/chrome_frame_ui_test_utils.cc b/chrome_frame/test/chrome_frame_ui_test_utils.cc
index ffec43305756c6d1269ff5d777e11bcee59117ba..0f557b3abd0bbe8b85490af01010bb2e1e731a0e 100644
--- a/chrome_frame/test/chrome_frame_ui_test_utils.cc
+++ b/chrome_frame/test/chrome_frame_ui_test_utils.cc
@@ -1,4 +1,4 @@
-// Copyright (c) 2011 The Chromium Authors. All rights reserved.
+// Copyright (c) 2012 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.
@@ -186,15 +186,14 @@ bool AccObject::GetRoleText(std::wstring* role_text) {
*role_text = role_text_array;
return true;
} else {
- DLOG(ERROR) << "GetRoleText failed for role: "
- << V_I4(&role_variant);
+ LOG(ERROR) << "GetRoleText failed for role: " << V_I4(&role_variant);
}
} else if (role_variant.type() == VT_BSTR) {
*role_text = V_BSTR(&role_variant);
return true;
} else {
- DLOG(ERROR) << "Role was unexpected variant type: "
- << role_variant.type();
+ LOG(ERROR) << "Role was unexpected variant type: "
+ << role_variant.type();
}
}
return false;
@@ -242,8 +241,9 @@ bool AccObject::GetLocationInClient(gfx::Rect* client_location) {
return false;
POINT offset = {0, 0};
if (!::ScreenToClient(container_window, &offset)) {
- DLOG(ERROR) << "Could not convert from screen to client coordinates for "
- << "window containing accessibility object: " << GetDescription();
+ LOG(ERROR) << "Could not convert from screen to client coordinates for "
+ "window containing accessibility object: "
+ << GetDescription();
return false;
}
location.Offset(offset.x, offset.y);
@@ -525,8 +525,8 @@ AccObject* AccObject::CreateFromVariant(AccObject* object,
bool AccObject::PostMouseClickAtCenter(int button_down, int button_up) {
std::wstring class_name;
if (!GetWindowClassName(&class_name)) {
- DLOG(ERROR) << "Could not get class name of window for accessibility "
- << "object: " << GetDescription();
+ LOG(ERROR) << "Could not get class name of window for accessibility "
+ << "object: " << GetDescription();
return false;
}
gfx::Rect location;
« no previous file with comments | « chrome_frame/test/chrome_frame_test_utils.cc ('k') | chrome_frame/test/ie_event_sink.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698