Closed GoogleCodeExporter closed 9 years ago
I have tried to modify the original soruce code
if (work_bound_->TryToWork()) {
//ignore
} else {
image_rewrites_dropped_due_to_load_->IncBy(1);
message_handler->Message(kInfo, "%s: Too busy to rewrite image.",
input_resource->url().c_str());
}
to:
if (work_bound_->TryToWork()) {
//ignore
} else {
CachedResult* cached = result->EnsureCachedResultCreated();
cached->set_url(input_resource->url().c_str());
image_rewrites_dropped_due_to_load_->IncBy(1);
message_handler->Message(kInfo, "%s: Too busy to rewrite image.",
input_resource->url().c_str());
}
and test ok,but i am not sure whether it's the best implementation or not.
Original comment by yanzhenl...@gmail.com
on 24 May 2013 at 9:35
[deleted comment]
Issue 706 has been merged into this issue.
Original comment by sligocki@google.com
on 24 May 2013 at 5:51
Yikes, reproduced this locally, looking into it.
Original comment by sligocki@google.com
on 24 May 2013 at 10:48
Original comment by sligocki@google.com
on 28 May 2013 at 8:30
Issue 598 has been merged into this issue.
Original comment by jmaes...@google.com
on 30 May 2013 at 9:31
Yes! This is the issue that I've been experiencing but couldn't figure out
exactly why.
Original comment by mwhe...@360psg.com
on 31 May 2013 at 8:22
This has been fixed in trunk. The problem was that rewrites that were
abandoned due to load still had their URLs rewritten.
Original comment by jkar...@google.com
on 10 Jun 2013 at 4:07
This has been fixed in trunk. The problem was that rewrites that were
abandoned due to load still had their URLs rewritten.
Original comment by jkar...@google.com
on 10 Jun 2013 at 4:07
Original comment by jkar...@google.com
on 13 Jun 2013 at 1:33
Original issue reported on code.google.com by
yanzhenl...@gmail.com
on 24 May 2013 at 9:09