Remove unnecessary uses of |this->| in core/loader

The Blink Rename has made most uses of this-> unnecessary. The
relevant variable names no longer clash with the getters.

Bug: 789653
Change-Id: I016817495e3192c6bf2d226afb2901df4b4c90c7
Reviewed-on: https://chromium-review.googlesource.com/805255Reviewed-by: Christian Biesinger <cbiesinger@chromium.org>
Commit-Queue: Robert Hogan <robhogan@gmail.com>
Cr-Commit-Position: refs/heads/master@{#521404}

third_party/WebKit/Source/core/loader/DocumentThreadableLoader.cpp

@@ -622,7 +622,7 @@ bool DocumentThreadableLoader::RedirectReceivedBlinkCORS(
     const ResourceRequest& new_request,
     const ResourceResponse& redirect_response) {
   DCHECK(client_);
-  DCHECK_EQ(resource, this->GetResource());
+  DCHECK_EQ(resource, GetResource());
   DCHECK(async_);
 
   suborigin_force_credentials_ = false;
@@ -797,7 +797,7 @@ void DocumentThreadableLoader::DataSent(
     unsigned long long bytes_sent,
     unsigned long long total_bytes_to_be_sent) {
   DCHECK(client_);
-  DCHECK_EQ(resource, this->GetResource());
+  DCHECK_EQ(resource, GetResource());
   DCHECK(async_);
 
   checker_.DataSent();
@@ -807,7 +807,7 @@ void DocumentThreadableLoader::DataSent(
 void DocumentThreadableLoader::DataDownloaded(Resource* resource,
                                               int data_length) {
   DCHECK(client_);
-  DCHECK_EQ(resource, this->GetResource());
+  DCHECK_EQ(resource, GetResource());
   DCHECK(actual_request_.IsNull());
   DCHECK(async_);
 
@@ -819,7 +819,7 @@ void DocumentThreadableLoader::DidReceiveResourceTiming(
     Resource* resource,
     const ResourceTimingInfo& info) {
   DCHECK(client_);
-  DCHECK_EQ(resource, this->GetResource());
+  DCHECK_EQ(resource, GetResource());
   DCHECK(async_);
 
   client_->DidReceiveResourceTiming(info);
@@ -829,7 +829,7 @@ void DocumentThreadableLoader::ResponseReceived(
     Resource* resource,
     const ResourceResponse& response,
     std::unique_ptr<WebDataConsumerHandle> handle) {
-  DCHECK_EQ(resource, this->GetResource());
+  DCHECK_EQ(resource, GetResource());
   DCHECK(async_);
 
   checker_.ResponseReceived();
@@ -1032,7 +1032,7 @@ void DocumentThreadableLoader::SetSerializedCachedMetadata(Resource*,
 void DocumentThreadableLoader::DataReceived(Resource* resource,
                                             const char* data,
                                             size_t data_length) {
-  DCHECK_EQ(resource, this->GetResource());
+  DCHECK_EQ(resource, GetResource());
   DCHECK(async_);
 
   checker_.DataReceived();
@@ -1060,7 +1060,7 @@ void DocumentThreadableLoader::HandleReceivedData(const char* data,
 
 void DocumentThreadableLoader::NotifyFinished(Resource* resource) {
   DCHECK(client_);
-  DCHECK_EQ(resource, this->GetResource());
+  DCHECK_EQ(resource, GetResource());
   DCHECK(async_);
 
   checker_.NotifyFinished(resource);

third_party/WebKit/Source/core/loader/TextTrackLoader.cpp

@@ -69,7 +69,7 @@ void TextTrackLoader::CancelLoad() {
 bool TextTrackLoader::RedirectReceived(Resource* resource,
                                        const ResourceRequest& request,
                                        const ResourceResponse&) {
-  DCHECK_EQ(this->GetResource(), resource);
+  DCHECK_EQ(GetResource(), resource);
   if (resource->GetResourceRequest().GetFetchRequestMode() ==
           network::mojom::FetchRequestMode::kCORS ||
       GetDocument().GetSecurityOrigin()->CanRequestNoSuborigin(request.Url()))
@@ -85,7 +85,7 @@ bool TextTrackLoader::RedirectReceived(Resource* resource,
 void TextTrackLoader::DataReceived(Resource* resource,
                                    const char* data,
                                    size_t length) {
-  DCHECK_EQ(this->GetResource(), resource);
+  DCHECK_EQ(GetResource(), resource);
 
   if (state_ == kFailed)
     return;
@@ -110,7 +110,7 @@ void TextTrackLoader::CorsPolicyPreventedLoad(SecurityOrigin* security_origin,
 }
 
 void TextTrackLoader::NotifyFinished(Resource* resource) {
-  DCHECK_EQ(this->GetResource(), resource);
+  DCHECK_EQ(GetResource(), resource);
   if (cue_parser_)
     cue_parser_->Flush();
 

third_party/WebKit/Source/core/loader/resource/ImageResource.cpp

@@ -329,7 +329,7 @@ void ImageResource::AppendData(const char* data, size_t length) {
 
     // Update the image immediately if needed.
     if (GetContent()->ShouldUpdateImageImmediately()) {
-      UpdateImage(this->Data(), ImageResourceContent::kUpdateImage, false);
+      UpdateImage(Data(), ImageResourceContent::kUpdateImage, false);
       return;
     }
 
@@ -357,7 +357,7 @@ void ImageResource::FlushImageIfNeeded(TimerBase*) {
   // to call |updateImage()|.
   if (IsLoading()) {
     last_flush_time_ = WTF::MonotonicallyIncreasingTime();
-    UpdateImage(this->Data(), ImageResourceContent::kUpdateImage, false);
+    UpdateImage(Data(), ImageResourceContent::kUpdateImage, false);
   }
 }