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

Unified Diff: remoting/host/setup/me2me_native_messaging_host.cc

Issue 281223002: Removed LOG_GETLASTERROR and LOG_ERRNO macros. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Wed 05/14/2014 11:20:03.21 Created 6 years, 7 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
Index: remoting/host/setup/me2me_native_messaging_host.cc
diff --git a/remoting/host/setup/me2me_native_messaging_host.cc b/remoting/host/setup/me2me_native_messaging_host.cc
index 72076db769febe6409d3307a08dd61c8294911e1..339c64547924a7f3290784bbc277fe31cc2bb7ae 100644
--- a/remoting/host/setup/me2me_native_messaging_host.cc
+++ b/remoting/host/setup/me2me_native_messaging_host.cc
@@ -541,8 +541,8 @@ void Me2MeNativeMessagingHost::EnsureElevatedHostCreated() {
ScopedSd sd = ConvertSddlToSd(security_descriptor);
if (!sd) {
- LOG_GETLASTERROR(ERROR) << "Failed to create a security descriptor for the"
- << "Chromoting Me2Me native messaging host.";
+ PLOG(ERROR) << "Failed to create a security descriptor for the"
+ << "Chromoting Me2Me native messaging host.";
OnError();
return;
}
@@ -567,8 +567,7 @@ void Me2MeNativeMessagingHost::EnsureElevatedHostCreated() {
&security_attributes));
if (!delegate_write_handle.IsValid()) {
- LOG_GETLASTERROR(ERROR) <<
- "Failed to create named pipe '" << input_pipe_name << "'";
+ PLOG(ERROR) << "Failed to create named pipe '" << input_pipe_name << "'";
OnError();
return;
}
@@ -588,8 +587,7 @@ void Me2MeNativeMessagingHost::EnsureElevatedHostCreated() {
&security_attributes));
if (!delegate_read_handle.IsValid()) {
- LOG_GETLASTERROR(ERROR) <<
- "Failed to create named pipe '" << output_pipe_name << "'";
+ PLOG(ERROR) << "Failed to create named pipe '" << output_pipe_name << "'";
OnError();
return;
}
@@ -631,7 +629,7 @@ void Me2MeNativeMessagingHost::EnsureElevatedHostCreated() {
if (!ShellExecuteEx(&info)) {
DWORD error = ::GetLastError();
- LOG_GETLASTERROR(ERROR) << "Unable to launch '" << binary.value() << "'";
+ PLOG(ERROR) << "Unable to launch '" << binary.value() << "'";
if (error != ERROR_CANCELLED) {
OnError();
}
@@ -641,8 +639,7 @@ void Me2MeNativeMessagingHost::EnsureElevatedHostCreated() {
if (!::ConnectNamedPipe(delegate_write_handle.Get(), NULL)) {
DWORD error = ::GetLastError();
if (error != ERROR_PIPE_CONNECTED) {
- LOG_GETLASTERROR(ERROR) << "Unable to connect '"
- << input_pipe_name << "'";
+ PLOG(ERROR) << "Unable to connect '" << input_pipe_name << "'";
OnError();
return;
}
@@ -651,8 +648,7 @@ void Me2MeNativeMessagingHost::EnsureElevatedHostCreated() {
if (!::ConnectNamedPipe(delegate_read_handle.Get(), NULL)) {
DWORD error = ::GetLastError();
if (error != ERROR_PIPE_CONNECTED) {
- LOG_GETLASTERROR(ERROR) << "Unable to connect '"
- << output_pipe_name << "'";
+ PLOG(ERROR) << "Unable to connect '" << output_pipe_name << "'";
OnError();
return;
}
« no previous file with comments | « remoting/host/setup/daemon_controller_delegate_win.cc ('k') | remoting/host/setup/me2me_native_messaging_host_main.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698