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

Unified Diff: third_party/WebKit/Source/modules/cachestorage/CacheTest.cpp

Issue 2392443007: reflow comments in modules/[app_banner,encoding] (Closed)
Patch Set: 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/modules/cachestorage/CacheTest.cpp
diff --git a/third_party/WebKit/Source/modules/cachestorage/CacheTest.cpp b/third_party/WebKit/Source/modules/cachestorage/CacheTest.cpp
index 4b0c918b22c2c473f2d826525c25948892e46d12..03235b76a0555930acdac908fe3a9977908506b8 100644
--- a/third_party/WebKit/Source/modules/cachestorage/CacheTest.cpp
+++ b/third_party/WebKit/Source/modules/cachestorage/CacheTest.cpp
@@ -71,7 +71,8 @@ class ScopedFetcherForTests final
"Unexpected call to fetch, no response available."));
}
- // This does not take ownership of its parameter. The provided sample object is used to check the parameter when called.
+ // This does not take ownership of its parameter. The provided sample object
+ // is used to check the parameter when called.
void setExpectedFetchUrl(const String* expectedUrl) {
m_expectedUrl = expectedUrl;
}
@@ -92,8 +93,10 @@ class ScopedFetcherForTests final
Member<Response> m_response;
};
-// A test implementation of the WebServiceWorkerCache interface which returns a (provided) error for every operation, and optionally checks arguments
-// to methods against provided arguments. Also used as a base class for test specific caches.
+// A test implementation of the WebServiceWorkerCache interface which returns a
+// (provided) error for every operation, and optionally checks arguments to
+// methods against provided arguments. Also used as a base class for test
+// specific caches.
class ErrorWebCacheForTests : public WebServiceWorkerCache {
public:
ErrorWebCacheForTests(const WebServiceWorkerCacheError error)
@@ -108,7 +111,8 @@ class ErrorWebCacheForTests : public WebServiceWorkerCache {
return old;
}
- // These methods do not take ownership of their parameter. They provide an optional sample object to check parameters against.
+ // These methods do not take ownership of their parameter. They provide an
+ // optional sample object to check parameters against.
void setExpectedUrl(const String* expectedUrl) {
m_expectedUrl = expectedUrl;
}
@@ -305,8 +309,8 @@ class CacheStorageTest : public ::testing::Test {
: ScriptFunction(scriptState) {}
};
- // A ScriptFunction that saves its parameter; used by tests to assert on correct
- // values being passed.
+ // A ScriptFunction that saves its parameter; used by tests to assert on
+ // correct values being passed.
class TestFunction : public ScriptFunction {
public:
static v8::Local<v8::Function> create(ScriptState* scriptState,
@@ -375,8 +379,8 @@ TEST_F(CacheStorageTest, Basics) {
getRejectString(matchPromise));
}
-// Tests that arguments are faithfully passed on calls to Cache methods, except for methods which use batch operations,
-// which are tested later.
+// Tests that arguments are faithfully passed on calls to Cache methods, except
+// for methods which use batch operations, which are tested later.
TEST_F(CacheStorageTest, BasicArguments) {
ScriptState::Scope scope(getScriptState());
NonThrowableExceptionState exceptionState;
@@ -446,7 +450,8 @@ TEST_F(CacheStorageTest, BasicArguments) {
EXPECT_EQ(kNotImplementedString, getRejectString(stringKeysResult2));
}
-// Tests that arguments are faithfully passed to API calls that degrade to batch operations.
+// Tests that arguments are faithfully passed to API calls that degrade to batch
+// operations.
TEST_F(CacheStorageTest, BatchOperationArguments) {
ScriptState::Scope scope(getScriptState());
NonThrowableExceptionState exceptionState;

Powered by Google App Engine
This is Rietveld 408576698