Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(845)

Unified Diff: third_party/WebKit/Source/core/fetch/ResourceLoaderOptionsTest.cpp

Issue 2389643002: Reflow comments in core/fetch (Closed)
Patch Set: yoavs comments Created 4 years, 2 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: third_party/WebKit/Source/core/fetch/ResourceLoaderOptionsTest.cpp
diff --git a/third_party/WebKit/Source/core/fetch/ResourceLoaderOptionsTest.cpp b/third_party/WebKit/Source/core/fetch/ResourceLoaderOptionsTest.cpp
index fd8e41095e3199579c9386114130e898bbbcd484..1ad930ed7d7d5471441b0ee0db5057e7b243b588 100644
--- a/third_party/WebKit/Source/core/fetch/ResourceLoaderOptionsTest.cpp
+++ b/third_party/WebKit/Source/core/fetch/ResourceLoaderOptionsTest.cpp
@@ -12,8 +12,8 @@ namespace blink {
namespace {
TEST(ResourceLoaderOptionsTest, DeepCopy) {
- // Check that the fields of ResourceLoaderOptions are enums,
- // except for initiatorInfo and securityOrigin.
+ // Check that the fields of ResourceLoaderOptions are enums, except for
+ // initiatorInfo and securityOrigin.
static_assert(std::is_enum<DataBufferingPolicy>::value,
"DataBufferingPolicy should be an enum");
static_assert(std::is_enum<StoredCredentials>::value,
@@ -87,7 +87,8 @@ TEST(ResourceLoaderOptionsTest, DeepCopy) {
// Check that pointers are different between |original| and |copy|
// FIXME: When |original| and |copy| are in different threads, then
- // EXPECT_NE(original.initiatorInfo.name.impl(), copy.initiatorInfo.name.impl());
+ // EXPECT_NE(original.initiatorInfo.name.impl(),
+ // copy.initiatorInfo.name.impl());
// should pass. However, in the unit test here, these two pointers are the
// same, because initiatorInfo.name is AtomicString.
EXPECT_NE(original.securityOrigin.get(), copy.securityOrigin.get());
@@ -99,8 +100,8 @@ TEST(ResourceLoaderOptionsTest, DeepCopy) {
copy.securityOrigin->domain().impl());
// FIXME: The checks for content equality/pointer inequality for
- // securityOrigin here is not complete (i.e. m_filePath is not checked).
- // A unit test for SecurityOrigin::isolatedCopy() that covers these checks
+ // securityOrigin here is not complete (i.e. m_filePath is not checked). A
+ // unit test for SecurityOrigin::isolatedCopy() that covers these checks
// should be added.
}
« no previous file with comments | « third_party/WebKit/Source/core/fetch/ResourceLoaderOptions.h ('k') | third_party/WebKit/Source/core/fetch/ScriptResource.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698