SetPrinterInfo

chrome/common/child_process_logging_posix.cc

Index: chrome/common/child_process_logging_posix.cc
diff --git a/chrome/common/child_process_logging_posix.cc b/chrome/common/child_process_logging_posix.cc
index 2015256fb7529aecd6dbf917e36d46f683499a32..4e2128b56bccb9a77665962b16c7d67a7bf08dd7 100644
--- a/chrome/common/child_process_logging_posix.cc
+++ b/chrome/common/child_process_logging_posix.cc
@@ -29,13 +29,14 @@ 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] = "";
@@ -103,6 +104,11 @@ void SetGpuInfo(const content::GPUInfo& gpu_info) {
   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';