AsanCoverage: make the filenames of zygote/gpu coverage files more unique.

AsanCoverage: make the filenames of zygote/gpu coverage files more unique.

BUG=336212
R=jln@chromium.org
TBR=kbr@chromium.org
NOTRY=true
TEST=ASan

Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=274644

[earthdok, 2014-06-03 21:27:12]

ptal

[jln (very slow on Chromium), 2014-06-03 21:43:40]

lgtm with nit.

Let's not use stringstream. ASAN will add .sancov to the name automatically I
presume?

https://codereview.chromium.org/317503002/diff/1/content/gpu/gpu_main.cc
File content/gpu/gpu_main.cc (right):

https://codereview.chromium.org/317503002/diff/1/content/gpu/gpu_main.cc#newc...
content/gpu/gpu_main.cc:455: std::string sancov_file_name = ss.str();
please do:

const std::string sancov_file_name = "gpu." +
base::Uint64ToString(base::RandUint64());

instead.

https://codereview.chromium.org/317503002/diff/1/content/zygote/zygote_main_l...
File content/zygote/zygote_main_linux.cc (right):

https://codereview.chromium.org/317503002/diff/1/content/zygote/zygote_main_l...
content/zygote/zygote_main_linux.cc:499: std::stringstream ss;
please do:

const std::string sancov_file_name = "zygote." +
base::Uint64ToString(base::RandUint64());

instead.

[earthdok, 2014-06-03 21:49:26]

https://codereview.chromium.org/317503002/diff/1/content/gpu/gpu_main.cc
File content/gpu/gpu_main.cc (right):

https://codereview.chromium.org/317503002/diff/1/content/gpu/gpu_main.cc#newc...
content/gpu/gpu_main.cc:455: std::string sancov_file_name = ss.str();
On 2014/06/03 21:43:40, jln wrote:
> please do:
> 
> const std::string sancov_file_name = "gpu." +
> base::Uint64ToString(base::RandUint64());
> 
> instead.

Done.

https://codereview.chromium.org/317503002/diff/1/content/zygote/zygote_main_l...
File content/zygote/zygote_main_linux.cc (right):

https://codereview.chromium.org/317503002/diff/1/content/zygote/zygote_main_l...
content/zygote/zygote_main_linux.cc:499: std::stringstream ss;
On 2014/06/03 21:43:40, jln wrote:
> please do:
> 
> const std::string sancov_file_name = "zygote." +
> base::Uint64ToString(base::RandUint64());
> 
> instead.

Done.

[earthdok, 2014-06-03 21:52:34]

The CQ bit was checked by earthdok@chromium.org

[commit-bot: I haz the power, 2014-06-03 21:53:55]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/earthdok@chromium.org/317503002/20001

[Ken Russell (switch to Gerrit), 2014-06-03 21:55:56]

lgtm

[commit-bot: I haz the power, 2014-06-03 21:57:40]

Change committed as 274644