Commit 230430ab authored by flackr's avatar flackr Committed by Commit bot

DCHECK written counts from fwrite to fix unused return result.

These checks were lost in refs/heads/master@{#294801}: https://codereview.chromium.org/541763002.

BUG=382975
TEST=Compiling does not warn about an unused result.

Review URL: https://codereview.chromium.org/573963007

Cr-Commit-Position: refs/heads/master@{#295195}
parent ffbd8b95
......@@ -69,7 +69,9 @@ class FileTraceDataSink : public TracingController::TraceDataSink {
const scoped_refptr<base::RefCountedString> chunk) {
if (!OpenFileIfNeededOnFileThread())
return;
fwrite(chunk->data().c_str(), strlen(chunk->data().c_str()), 1, file_);
size_t written = fwrite(chunk->data().c_str(),
strlen(chunk->data().c_str()), 1, file_);
DCHECK_EQ(1u, written);
}
bool OpenFileIfNeededOnFileThread() {
......@@ -81,7 +83,8 @@ class FileTraceDataSink : public TracingController::TraceDataSink {
return false;
}
const char preamble[] = "{\"traceEvents\": [";
fwrite(preamble, strlen(preamble), 1, file_);
size_t written = fwrite(preamble, strlen(preamble), 1, file_);
DCHECK_EQ(1u, written);
return true;
}
......@@ -90,8 +93,12 @@ class FileTraceDataSink : public TracingController::TraceDataSink {
fputc(']', file_);
if (!system_trace_.empty()) {
const char systemTraceEvents[] = ",\"systemTraceEvents\": ";
fwrite(systemTraceEvents, strlen(systemTraceEvents), 1, file_);
fwrite(system_trace_.c_str(), strlen(system_trace_.c_str()), 1, file_);
size_t written = fwrite(systemTraceEvents, strlen(systemTraceEvents),
1, file_);
DCHECK_EQ(1u, written);
written = fwrite(system_trace_.c_str(),
strlen(system_trace_.c_str()), 1, file_);
DCHECK_EQ(1u, written);
}
fputc('}', file_);
base::CloseFile(file_);
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment