From fa0afad01fb77371f1f3addd5973f078919440dc Mon Sep 17 00:00:00 2001
From: "scherkus@chromium.org"
 <scherkus@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98>
Date: Tue, 21 Dec 2010 15:17:44 +0000
Subject: Clean up comments in BufferedDataSource and BufferedResourceLoader.

No code change.

BUG=none
TEST=none
TBR=hclam

Review URL: http://codereview.chromium.org/6081002

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@69829 0039d316-1c4b-4281-b951-d872f2087c98
---
 webkit/glue/media/buffered_data_source.cc     | 12 +++++-------
 webkit/glue/media/buffered_resource_loader.cc |  5 ++++-
 webkit/glue/media/buffered_resource_loader.h  |  5 ++---
 3 files changed, 11 insertions(+), 11 deletions(-)

(limited to 'webkit')

diff --git a/webkit/glue/media/buffered_data_source.cc b/webkit/glue/media/buffered_data_source.cc
index c1aed27..8ddd5fa 100644
--- a/webkit/glue/media/buffered_data_source.cc
+++ b/webkit/glue/media/buffered_data_source.cc
@@ -79,7 +79,7 @@ base::TimeDelta BufferedDataSource::GetTimeoutMilliseconds() {
 }
 
 /////////////////////////////////////////////////////////////////////////////
-// BufferedDataSource, media::Filter implementation
+// media::Filter implementation.
 void BufferedDataSource::Initialize(const std::string& url,
                                     media::FilterCallback* callback) {
   // Saves the url.
@@ -134,7 +134,7 @@ void BufferedDataSource::SetPlaybackRate(float playback_rate) {
 }
 
 /////////////////////////////////////////////////////////////////////////////
-// BufferedDataSource, media::DataSource implementation
+// media::DataSource implementation.
 void BufferedDataSource::Read(int64 position, size_t size, uint8* data,
                               media::DataSource::ReadCallback* read_callback) {
   render_loop_->PostTask(FROM_HERE,
@@ -175,7 +175,7 @@ void BufferedDataSource::Abort() {
 }
 
 /////////////////////////////////////////////////////////////////////////////
-// BufferedDataSource, render thread tasks
+// Render thread tasks.
 void BufferedDataSource::InitializeTask() {
   DCHECK(MessageLoop::current() == render_loop_);
   DCHECK(!loader_.get());
@@ -373,9 +373,7 @@ void BufferedDataSource::DoneInitialization_Locked() {
 }
 
 /////////////////////////////////////////////////////////////////////////////
-// BufferedDataSource, callback methods.
-// These methods are called on the render thread for the events reported by
-// BufferedResourceLoader.
+// BufferedResourceLoader callback methods.
 void BufferedDataSource::HttpInitialStartCallback(int error) {
   DCHECK(MessageLoop::current() == render_loop_);
   DCHECK(loader_.get());
@@ -402,7 +400,7 @@ void BufferedDataSource::HttpInitialStartCallback(int error) {
     // Assuming that the Range header was causing the problem. Retry without
     // the Range header.
     using_range_request_ = false;
-    loader_ = CreateResourceLoader(kPositionNotSpecified, 
+    loader_ = CreateResourceLoader(kPositionNotSpecified,
                                    kPositionNotSpecified);
     loader_->Start(
         NewCallback(this, &BufferedDataSource::HttpInitialStartCallback),
diff --git a/webkit/glue/media/buffered_resource_loader.cc b/webkit/glue/media/buffered_resource_loader.cc
index 8045d07..61aac72 100644
--- a/webkit/glue/media/buffered_resource_loader.cc
+++ b/webkit/glue/media/buffered_resource_loader.cc
@@ -232,7 +232,8 @@ void BufferedResourceLoader::SetURLLoaderForTest(WebURLLoader* mock_loader) {
   keep_test_loader_ = true;
 }
 
-// WebKit::WebURLLoaderClient implementations.
+/////////////////////////////////////////////////////////////////////////////
+// WebKit::WebURLLoaderClient implementation.
 void BufferedResourceLoader::willSendRequest(
     WebURLLoader* loader,
     WebURLRequest& newRequest,
@@ -424,6 +425,8 @@ void BufferedResourceLoader::didFail(
   Release();
 }
 
+/////////////////////////////////////////////////////////////////////////////
+// Helper methods.
 void BufferedResourceLoader::EnableDeferIfNeeded() {
   if (!defer_allowed_)
     return;
diff --git a/webkit/glue/media/buffered_resource_loader.h b/webkit/glue/media/buffered_resource_loader.h
index 8b5939e..09b05e6 100644
--- a/webkit/glue/media/buffered_resource_loader.h
+++ b/webkit/glue/media/buffered_resource_loader.h
@@ -111,8 +111,7 @@ class BufferedResourceLoader :
   // Used to inject a mock used for unittests.
   virtual void SetURLLoaderForTest(WebKit::WebURLLoader* mock_loader);
 
-  /////////////////////////////////////////////////////////////////////////////
-  // WebKit::WebURLLoaderClient implementations.
+  // WebKit::WebURLLoaderClient implementation.
   virtual void willSendRequest(
       WebKit::WebURLLoader* loader,
       WebKit::WebURLRequest& newRequest,
@@ -232,7 +231,7 @@ class BufferedResourceLoader :
   uint8* read_buffer_;
 
   // Offsets of the requested first byte and last byte in |buffer_|. They are
-  // written by VerifyRead().
+  // written by Read().
   int first_offset_;
   int last_offset_;
 
-- 
cgit v1.1