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

Unified Diff: third_party/WebKit/Source/web/SharedWorkerRepositoryClientImpl.cpp

Issue 2389633002: reflow comments in web/ (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/web/SharedWorkerRepositoryClientImpl.cpp
diff --git a/third_party/WebKit/Source/web/SharedWorkerRepositoryClientImpl.cpp b/third_party/WebKit/Source/web/SharedWorkerRepositoryClientImpl.cpp
index b07f05c37d8187c95c6b227a1531569887a357d9..d25425496981b7c331b810b4cdcc003ced599ec6 100644
--- a/third_party/WebKit/Source/web/SharedWorkerRepositoryClientImpl.cpp
+++ b/third_party/WebKit/Source/web/SharedWorkerRepositoryClientImpl.cpp
@@ -55,7 +55,8 @@
namespace blink {
-// Callback class that keeps the SharedWorker and WebSharedWorker objects alive while connecting.
+// Callback class that keeps the SharedWorker and WebSharedWorker objects alive
+// while connecting.
class SharedWorkerConnector
: private WebSharedWorkerConnector::ConnectListener {
public:
@@ -96,13 +97,15 @@ void SharedWorkerConnector::connect() {
}
void SharedWorkerConnector::connected() {
- // Free ourselves (this releases the SharedWorker so it can be freed as well if unreferenced).
+ // Free ourselves (this releases the SharedWorker so it can be freed as well
+ // if unreferenced).
delete this;
}
void SharedWorkerConnector::scriptLoadFailed() {
m_worker->dispatchEvent(Event::createCancelable(EventTypeNames::error));
- // Free ourselves (this releases the SharedWorker so it can be freed as well if unreferenced).
+ // Free ourselves (this releases the SharedWorker so it can be freed as well
+ // if unreferenced).
delete this;
}
@@ -120,7 +123,8 @@ void SharedWorkerRepositoryClientImpl::connect(
ExceptionState& exceptionState) {
DCHECK(m_client);
- // No nested workers (for now) - connect() should only be called from document context.
+ // No nested workers (for now) - connect() should only be called from document
+ // context.
DCHECK(worker->getExecutionContext()->isDocument());
Document* document = toDocument(worker->getExecutionContext());
@@ -152,7 +156,8 @@ void SharedWorkerRepositoryClientImpl::connect(
&creationError));
if (creationError != WebWorkerCreationErrorNone) {
if (creationError == WebWorkerCreationErrorURLMismatch) {
- // Existing worker does not match this url, so return an error back to the caller.
+ // Existing worker does not match this url, so return an error back to the
+ // caller.
exceptionState.throwDOMException(
URLMismatchError, "The location of the SharedWorker named '" + name +
"' does not exactly match the provided URL ('" +
@@ -171,8 +176,8 @@ void SharedWorkerRepositoryClientImpl::connect(
}
}
- // The connector object manages its own lifecycle (and the lifecycles of the two worker objects).
- // It will free itself once connecting is completed.
+ // The connector object manages its own lifecycle (and the lifecycles of the
+ // two worker objects). It will free itself once connecting is completed.
SharedWorkerConnector* connector = new SharedWorkerConnector(
worker, url, name, std::move(port), std::move(webWorkerConnector));
connector->connect();
« no previous file with comments | « third_party/WebKit/Source/web/RotationViewportAnchor.cpp ('k') | third_party/WebKit/Source/web/TextFinder.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698