Show more different NaCl loading errors.

ppapi/native_client/src/trusted/plugin/plugin_error.h

 /*
  * Copyright (c) 2011 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.
  */
 
 /*
  * Error codes and data structures used to report errors when loading a nexe.
  */
 
@@ skipping 97 matching lines @@
     Reset();
   }
 
   void Reset() {
     SetReport(ERROR_UNKNOWN, std::string());
   }
 
   void SetReport(PluginErrorCode error_code, const std::string& message) {
     error_code_ = error_code;
     message_ = message;
+    console_message_ = message;
+  }
+
+  void SetReport(PluginErrorCode error_code, const std::string& message,

[Mark Seaborn, 2013/07/17 18:21:49]

Please avoid function overloading when possible: it makes it hard to find call
sites by grepping.  Can you give this variant a different name?  e.g.
SetReportWithTrustedError() or SetReportWithConsoleOnlyError().

There should also be a comment explaining why some errors are only logged to the
console.

[halyavin, 2013/07/18 09:32:22]

Done.

+                 const std::string& console_message) {
+    error_code_ = error_code;
+    message_ = message;
+    console_message_ = console_message;

[Mark Seaborn, 2013/07/17 18:21:49]

Are you going to concatenate the messages like we discussed on IM?

[halyavin, 2013/07/18 09:32:22]

Done.

   }
 
   PluginErrorCode error_code() const {
     return error_code_;
   }
 
   void PrependMessage(const std::string& prefix) {
     message_ = prefix + message_;
+    console_message_ = prefix + console_message_;
   }
 
   const std::string& message() const {
     return message_;
   }
 
+  const std::string& console_message() const {
+    return console_message_;
+  }
+
  private:
   PluginErrorCode error_code_;
   std::string message_;
+  std::string console_message_;
   NACL_DISALLOW_COPY_AND_ASSIGN(ErrorInfo);
 };
 
 }  // namespace plugin
 
 #endif  // NATIVE_CLIENT_SRC_TRUSTED_PLUGIN_PLUGIN_ERROR_H