Reflow comments in core/fetch

third_party/WebKit/Source/core/fetch/ImageResourceTest.cpp

 /*
  * Copyright (c) 2013, Google Inc. All rights reserved.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions are
  * met:
  *
  *     * Redistributions of source code must retain the above copyright
  * notice, this list of conditions and the following disclaimer.
  *     * Redistributions in binary form must reproduce the above
@@ skipping 182 matching lines @@
   std::unique_ptr<scheduler::FakeWebTaskRunner> m_runner;
 };
 
 TEST(ImageResourceTest, MultipartImage) {
   ResourceFetcher* fetcher =
       ResourceFetcher::create(ImageResourceTestMockFetchContext::create());
   KURL testURL(ParsedURLString, "http://www.test.com/cancelTest.html");
   URLTestHelpers::registerMockedURLLoad(testURL, "cancelTest.html",
                                         "text/html");
 
-  // Emulate starting a real load, but don't expect any "real" WebURLLoaderClient callbacks.
+  // Emulate starting a real load, but don't expect any "real"
+  // WebURLLoaderClient callbacks.
   ImageResource* cachedImage = ImageResource::create(ResourceRequest(testURL));
   cachedImage->setIdentifier(createUniqueIdentifier());
   fetcher->startLoad(cachedImage);
   Platform::current()->getURLLoaderMockFactory()->unregisterURL(testURL);
 
   Persistent<MockImageResourceClient> client =
       new MockImageResourceClient(cachedImage);
   EXPECT_EQ(Resource::Pending, cachedImage->getStatus());
 
-  // Send the multipart response. No image or data buffer is created.
-  // Note that the response must be routed through ResourceLoader to
-  // ensure the load is flagged as multipart.
+  // Send the multipart response. No image or data buffer is created. Note that
+  // the response must be routed through ResourceLoader to ensure the load is
+  // flagged as multipart.
   ResourceResponse multipartResponse(KURL(), "multipart/x-mixed-replace", 0,
                                      nullAtom, String());
   multipartResponse.setMultipartBoundary("boundary", strlen("boundary"));
   cachedImage->loader()->didReceiveResponse(
       nullptr, WrappedResourceResponse(multipartResponse), nullptr);
   EXPECT_FALSE(cachedImage->resourceBuffer());
   EXPECT_FALSE(cachedImage->hasImage());
   EXPECT_EQ(0, client->imageChangedCount());
   EXPECT_FALSE(client->notifyFinishedCalled());
   EXPECT_EQ("multipart/x-mixed-replace", cachedImage->response().mimeType());
 
   const char firstPart[] =
       "--boundary\n"
       "Content-Type: image/svg+xml\n\n";
   cachedImage->appendData(firstPart, strlen(firstPart));
-  // Send the response for the first real part. No image or data buffer is created.
+  // Send the response for the first real part. No image or data buffer is
+  // created.
   EXPECT_FALSE(cachedImage->resourceBuffer());
   EXPECT_FALSE(cachedImage->hasImage());
   EXPECT_EQ(0, client->imageChangedCount());
   EXPECT_FALSE(client->notifyFinishedCalled());
   EXPECT_EQ("image/svg+xml", cachedImage->response().mimeType());
 
   const char secondPart[] =
       "<svg xmlns='http://www.w3.org/2000/svg' width='1' height='1'><rect "
       "width='1' height='1' fill='green'/></svg>\n";
-  // The first bytes arrive. The data buffer is created, but no image is created.
+  // The first bytes arrive. The data buffer is created, but no image is
+  // created.
   cachedImage->appendData(secondPart, strlen(secondPart));
   EXPECT_TRUE(cachedImage->resourceBuffer());
   EXPECT_FALSE(cachedImage->hasImage());
   EXPECT_EQ(0, client->imageChangedCount());
   EXPECT_FALSE(client->notifyFinishedCalled());
 
   // Add a client to check an assertion error doesn't happen
   // (crbug.com/630983).
   Persistent<MockImageResourceClient> client2 =
       new MockImageResourceClient(cachedImage);
   EXPECT_EQ(0, client2->imageChangedCount());
   EXPECT_FALSE(client2->notifyFinishedCalled());
 
   const char thirdPart[] = "--boundary";
   cachedImage->appendData(thirdPart, strlen(thirdPart));
   ASSERT_TRUE(cachedImage->resourceBuffer());
   EXPECT_EQ(strlen(secondPart) - 1, cachedImage->resourceBuffer()->size());
 
-  // This part finishes. The image is created, callbacks are sent, and the data buffer is cleared.
+  // This part finishes. The image is created, callbacks are sent, and the data
+  // buffer is cleared.
   cachedImage->loader()->didFinishLoading(nullptr, 0.0, 0);
   EXPECT_TRUE(cachedImage->resourceBuffer());
   EXPECT_FALSE(cachedImage->errorOccurred());
   ASSERT_TRUE(cachedImage->hasImage());
   EXPECT_FALSE(cachedImage->getImage()->isNull());
   EXPECT_EQ(1, cachedImage->getImage()->width());
   EXPECT_EQ(1, cachedImage->getImage()->height());
   EXPECT_EQ(1, client->imageChangedCount());
   EXPECT_TRUE(client->notifyFinishedCalled());
   EXPECT_EQ(1, client2->imageChangedCount());
@@ skipping 12 matching lines @@
   ImageResource* cachedImage = ImageResource::create(ResourceRequest(testURL));
   cachedImage->setIdentifier(createUniqueIdentifier());
 
   fetcher->startLoad(cachedImage);
   memoryCache()->add(cachedImage);
 
   Persistent<MockImageResourceClient> client =
       new MockImageResourceClient(cachedImage);
   EXPECT_EQ(Resource::Pending, cachedImage->getStatus());
 
-  // The load should still be alive, but a timer should be started to cancel the load inside removeClient().
+  // The load should still be alive, but a timer should be started to cancel the
+  // load inside removeClient().
   client->removeAsClient();
   EXPECT_EQ(Resource::Pending, cachedImage->getStatus());
   EXPECT_TRUE(memoryCache()->resourceForURL(testURL));
 
-  // Trigger the cancel timer, ensure the load was cancelled and the resource was evicted from the cache.
+  // Trigger the cancel timer, ensure the load was cancelled and the resource
+  // was evicted from the cache.
   blink::testing::runPendingTasks();
   EXPECT_EQ(Resource::LoadError, cachedImage->getStatus());
   EXPECT_FALSE(memoryCache()->resourceForURL(testURL));
 
   Platform::current()->getURLLoaderMockFactory()->unregisterURL(testURL);
 }
 
 TEST(ImageResourceTest, DecodedDataRemainsWhileHasClients) {
   ImageResource* cachedImage = ImageResource::create(ResourceRequest());
   cachedImage->setStatus(Resource::Pending);
@@ skipping 12 matching lines @@
       ResourceResponse(KURL(), "image/jpeg", jpeg.size(), nullAtom, String()),
       nullptr);
   cachedImage->appendData(reinterpret_cast<const char*>(jpeg.data()),
                           jpeg.size());
   cachedImage->finish();
   EXPECT_FALSE(cachedImage->errorOccurred());
   ASSERT_TRUE(cachedImage->hasImage());
   EXPECT_FALSE(cachedImage->getImage()->isNull());
   EXPECT_TRUE(client->notifyFinishedCalled());
 
-  // The prune comes when the ImageResource still has clients. The image should not be deleted.
+  // The prune comes when the ImageResource still has clients. The image should
+  // not be deleted.
   cachedImage->prune();
   EXPECT_TRUE(cachedImage->isAlive());
   ASSERT_TRUE(cachedImage->hasImage());
   EXPECT_FALSE(cachedImage->getImage()->isNull());
 
   // The ImageResource no longer has clients. The decoded image data should be
   // deleted by prune.
   client->removeAsClient();
   cachedImage->prune();
   EXPECT_FALSE(cachedImage->isAlive());
@@ skipping 336 matching lines @@
       nullptr, WrappedResourceResponse(ResourceResponse(
                    testURL, "image/jpeg", 18, nullAtom, String())),
       nullptr);
   cachedImage->loader()->didReceiveData(nullptr, "notactuallyanimage", 18, 18,
                                         18);
   EXPECT_EQ(Resource::DecodeError, cachedImage->getStatus());
   EXPECT_FALSE(cachedImage->isLoading());
 }
 
 }  // namespace blink