Commit 9070b9c5 authored by Shivani Sharma's avatar Shivani Sharma Committed by Commit Bot

Getting more debug info for the crash.

Bug: 750725
Change-Id: I9a70b4fd03aec9a74bd76f3988442cd6e9f908d3
Reviewed-on: https://chromium-review.googlesource.com/596151
Commit-Queue: Shivani Sharma <shivanisha@chromium.org>
Reviewed-by: default avatarJosh Karlin <jkarlin@chromium.org>
Cr-Commit-Position: refs/heads/master@{#491188}
parent 4accbc94
......@@ -838,7 +838,20 @@ int HttpCache::DoneWithResponseHeaders(ActiveEntry* entry,
// Partial requests may have write mode even when there is a writer present
// since they may be reader for a particular range and writer for another
// range.
DCHECK(is_partial || (!entry->writer && entry->done_headers_queue.empty()));
if (!is_partial) {
// TODO(shivanisha): Convert these to DCHECKs after crbug.com/750725 is
// fixed.
CHECK(!entry->writer)
<< "Writer's mode: " << entry->writer->mode() << " entry: " << entry
<< " writer's entry: " << entry->writer->entry();
CHECK(entry->done_headers_queue.empty())
<< "done_headers_queue size: " << entry->done_headers_queue.size()
<< " first element's mode: "
<< (*(entry->done_headers_queue.begin()))->mode()
<< " first element's entry: "
<< (*(entry->done_headers_queue.begin()))->entry()
<< " entry: " << entry;
}
if (!entry->writer) {
entry->writer = transaction;
......
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