Fix ProcessOutputWatcherTest.OutputWatcher under ASAN

chrome/browser/chromeos/process_proxy/process_output_watcher_unittest.cc

Index: chrome/browser/chromeos/process_proxy/process_output_watcher_unittest.cc
diff --git a/chrome/browser/chromeos/process_proxy/process_output_watcher_unittest.cc b/chrome/browser/chromeos/process_proxy/process_output_watcher_unittest.cc
index 778e905bdc2c50bb0e25e4b83b3ed2ce455a4455..acb65173daf1a129964c044ff8f0fb0cd795f200 100644
--- a/chrome/browser/chromeos/process_proxy/process_output_watcher_unittest.cc
+++ b/chrome/browser/chromeos/process_proxy/process_output_watcher_unittest.cc
@@ -125,8 +125,10 @@ TEST_F(ProcessOutputWatcherTest, OutputWatcher) {
   test_cases.push_back(TestCase("testing output3\n", PROCESS_OUTPUT_TYPE_OUT));
   test_cases.push_back(TestCase(VeryLongString(), PROCESS_OUTPUT_TYPE_OUT));
   test_cases.push_back(TestCase("testing error2\n", PROCESS_OUTPUT_TYPE_OUT));
-  test_cases.push_back(TestCase("line with \0 in it\n",
-                                arraysize("line with \0 in it \n"),
+
+  char line_with_null_char[] = "line with \0 in it\n";
+  test_cases.push_back(TestCase(line_with_null_char,
+                                arraysize(line_with_null_char) - 1,

[oshima, 2012/04/10 00:58:12]

Maybe I'm dumb, but sorry I didn't understand why you need -1. Can you
elaborate?

[tonibarzic, 2012/04/10 01:16:59]

I don't necessary need -1, but it's more in line with the comment at line 139
(ProcessOutputWatcher will not generally be sent NULL terminated strings in
wild)

                                 PROCESS_OUTPUT_TYPE_OUT));
 
   output_watch_thread.message_loop()->PostTask(FROM_HERE,