From 698d13fbcbedca85bfd84ae88571b804920f556e Mon Sep 17 00:00:00 2001 From: "davidben@chromium.org" <davidben@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> Date: Tue, 13 May 2014 19:30:52 +0000 Subject: Clarify ResourceHandler::OnWillRead's documentation. Document the min_size argument and clarify buffer lifetimes to match how BufferedResourceHandler uses it. BUG=none Review URL: https://codereview.chromium.org/273683007 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@270163 0039d316-1c4b-4281-b951-d872f2087c98 --- content/browser/loader/resource_handler.h | 7 +++++-- 1 file changed, 5 insertions(+), 2 deletions(-) diff --git a/content/browser/loader/resource_handler.h b/content/browser/loader/resource_handler.h index ded3a35..e47aa81 100644 --- a/content/browser/loader/resource_handler.h +++ b/content/browser/loader/resource_handler.h @@ -84,8 +84,11 @@ class CONTENT_EXPORT ResourceHandler // Data will be read for the response. Upon success, this method places the // size and address of the buffer where the data is to be written in its - // out-params. This call will be followed by either OnReadCompleted or - // OnResponseCompleted, at which point the buffer may be recycled. + // out-params. This call will be followed by either OnReadCompleted (on + // successful read or EOF) or OnResponseCompleted (on error). If + // OnReadCompleted is called, the buffer may be recycled. Otherwise, it may + // not be recycled and may potentially outlive the handler. If |min_size| is + // not -1, it is the minimum size of the returned buffer. // // If the handler returns false, then the request is cancelled. Otherwise, // once data is available, OnReadCompleted will be called. -- cgit v1.1