SetPrinterInfo

chrome/common/child_process_logging_posix.cc

 // 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.
 
 #include "chrome/common/child_process_logging.h"
 
 #include "base/command_line.h"
 #include "base/format_macros.h"
 #include "base/string_number_conversions.h"
 #include "base/string_util.h"
@@ skipping 11 matching lines @@
 
 // We use static strings to hold the most recent active url and the client
 // identifier. If we crash, the crash handler code will send the contents of
 // these strings to the browser.
 char g_active_url[kMaxActiveURLSize];
 char g_client_id[kClientIdSize];
 
 char g_channel[kChannelSize] = "";
 
 static const size_t kGpuStringSize = 32;
-
 char g_gpu_vendor_id[kGpuStringSize] = "";
 char g_gpu_device_id[kGpuStringSize] = "";
 char g_gpu_driver_ver[kGpuStringSize] = "";
 char g_gpu_ps_ver[kGpuStringSize] = "";
 char g_gpu_vs_ver[kGpuStringSize] = "";
 
+char g_printer_info[256] = "";

[Lei Zhang, 2012/03/14 04:20:05]

This is unused. You need to put this into the crash report in breakpad_linux.cc.

[Vitaly Buka (NO REVIEWS), 2012/03/14 20:37:42]

Done.

+
 static const size_t kNumSize = 32;
 char g_num_extensions[kNumSize] = "";
 char g_num_switches[kNumSize] = "";
 char g_num_views[kNumSize] = "";
 
 static const size_t kMaxExtensionSize =
     kExtensionLen * kMaxReportedActiveExtensions + 1;
 char g_extension_ids[kMaxExtensionSize] = "";
 
 // Assume command line switches are less than 64 chars.
@@ skipping 47 matching lines @@
   strncpy(g_gpu_ps_ver,
           gpu_info.pixel_shader_version.c_str(),
           kGpuStringSize - 1);
   g_gpu_ps_ver[kGpuStringSize - 1] = '\0';
   strncpy(g_gpu_vs_ver,
           gpu_info.vertex_shader_version.c_str(),
           kGpuStringSize - 1);
   g_gpu_vs_ver[kGpuStringSize - 1] = '\0';
 }
 
+void SetPrinterInfo(const char* printer_info) {

[eroman, 2012/03/14 02:12:49]

This is using a different format than the mac/windows one. Why?

Will be easier for searching in dremel if they use the same format.

[Vitaly Buka (NO REVIEWS), 2012/03/14 20:37:42]

Actually it's like URL and extensions here. It's divided on Win and MAC and as
single string here.


On 2012/03/14 02:12:49, eroman wrote:

+  strncpy(g_printer_info, printer_info, arraysize(g_printer_info) - 1);
+  g_printer_info[arraysize(g_printer_info) - 1] = '\0';
+}
+
 void SetNumberOfViews(int number_of_views) {
   snprintf(g_num_views, kNumSize - 1, "%d", number_of_views);
   g_num_views[kNumSize - 1] = '\0';
 }
 
 void SetCommandLine(const CommandLine* command_line) {
   const CommandLine::StringVector& argv = command_line->argv();
 
   snprintf(g_num_switches, kNumSize - 1, "%" PRIuS, argv.size() - 1);
   g_num_switches[kNumSize - 1] = '\0';
 
   std::string command_line_str;
   for (size_t argv_i = 1;
        argv_i < argv.size() && argv_i <= kMaxSwitches;
        ++argv_i) {
     command_line_str += argv[argv_i];
     // Truncate long switches, align short ones with spaces to be trimmed later.
     command_line_str.resize(argv_i * kSwitchLen, ' ');
   }
   strncpy(g_switches, command_line_str.c_str(), kMaxSwitchesSize - 1);
   g_switches[kMaxSwitchesSize - 1] = '\0';
 }
 
 void SetChannel(const std::string& channel) {
   strncpy(g_channel, channel.c_str(), kChannelSize - 1);
   g_channel[kChannelSize - 1] = '\0';
 }
 
 }  // namespace child_process_logging