reflow comments in Source/bindings/core/v8

third_party/WebKit/Source/bindings/core/v8/PrivateScriptRunner.cpp

 // Copyright 2014 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "bindings/core/v8/PrivateScriptRunner.h"
 
 #include "bindings/core/v8/DOMWrapperWorld.h"
 #include "bindings/core/v8/ExceptionState.h"
 #include "bindings/core/v8/V8Binding.h"
 #include "bindings/core/v8/V8PerContextData.h"
@@ skipping 92 matching lines @@
               result, block)) {
     fprintf(stderr,
             "Private script error: installClass() failed. (Class name = %s)\n",
             scriptClassName.utf8().data());
     dumpV8Message(context, block.Message());
     RELEASE_NOTREACHED();
   }
   return result;
 }
 
-// Private scripts can use privateScriptController.import(bundledResource, compileAndRunScript) to import dependent resources.
+// Private scripts can use privateScriptController.import(bundledResource,
+// compileAndRunScript) to import dependent resources.
 // |bundledResource| is a string resource name.
-// |compileAndRunScript| optional boolean representing if the javascript should be executed. Default: true.
+// |compileAndRunScript| optional boolean representing if the javascript should
+// be executed. Default: true.
 void importFunction(const v8::FunctionCallbackInfo<v8::Value>& args) {
   v8::Isolate* isolate = args.GetIsolate();
   RELEASE_ASSERT(isolate && (args.Length() >= 1));
   String resourceFileName = toCoreString(
       args[0]->ToString(isolate->GetCurrentContext()).ToLocalChecked());
   String resourceData =
       loadResourceAsASCIIString(resourceFileName.utf8().data());
   RELEASE_ASSERT(resourceData.length());
   bool compileAndRunScript = true;
   if (args.Length() == 2) {
     RELEASE_ASSERT(args[1]->IsBoolean());
     compileAndRunScript = args[1].As<v8::Boolean>()->Value();
   }
 
   if (resourceFileName.endsWith(".js") && compileAndRunScript)
     compileAndRunPrivateScript(
         ScriptState::current(isolate), resourceFileName.replace(".js", ""),
         resourceData.utf8().data(), resourceData.length());
   args.GetReturnValue().Set(v8String(isolate, resourceData));
 }
 
-// FIXME: If we have X.js, XPartial-1.js and XPartial-2.js, currently all of the JS files
-// are compiled when any of the JS files is requested. Ideally we should avoid compiling
-// unrelated JS files. For example, if a method in XPartial-1.js is requested, we just
-// need to compile X.js and XPartial-1.js, and don't need to compile XPartial-2.js.
+// FIXME: If we have X.js, XPartial-1.js and XPartial-2.js, currently all of the
+// JS files are compiled when any of the JS files is requested. Ideally we
+// should avoid compiling unrelated JS files. For example, if a method in
+// XPartial-1.js is requested, we just need to compile X.js and XPartial-1.js,
+// and don't need to compile XPartial-2.js.
 static void installPrivateScript(v8::Isolate* isolate, String className) {
   ScriptState* scriptState = ScriptState::current(isolate);
   int compiledScriptCount = 0;
-// |kPrivateScriptSourcesForTesting| is defined in V8PrivateScriptSources.h, which is auto-generated
-// by make_private_script_source.py.
+// |kPrivateScriptSourcesForTesting| is defined in V8PrivateScriptSources.h,
+// which is auto-generated by make_private_script_source.py.
 #ifndef NDEBUG
   for (size_t index = 0;
        index < WTF_ARRAY_LENGTH(kPrivateScriptSourcesForTesting); index++) {
     if (className == kPrivateScriptSourcesForTesting[index].className) {
       compileAndRunPrivateScript(
           scriptState, kPrivateScriptSourcesForTesting[index].scriptClassName,
           kPrivateScriptSourcesForTesting[index].source,
           kPrivateScriptSourcesForTesting[index].size);
       compiledScriptCount++;
     }
   }
 #endif
 
-  // |kPrivateScriptSources| is defined in V8PrivateScriptSources.h, which is auto-generated
-  // by make_private_script_source.py.
+  // |kPrivateScriptSources| is defined in V8PrivateScriptSources.h, which is
+  // auto-generated by make_private_script_source.py.
   for (size_t index = 0; index < WTF_ARRAY_LENGTH(kPrivateScriptSources);
        index++) {
     if (className == kPrivateScriptSources[index].className) {
       String resourceData =
           loadResourceAsASCIIString(kPrivateScriptSources[index].resourceFile);
       compileAndRunPrivateScript(
           scriptState, kPrivateScriptSources[index].scriptClassName,
           resourceData.utf8().data(), resourceData.length());
       compiledScriptCount++;
     }
   }
 
   if (!compiledScriptCount) {
     fprintf(stderr,
             "Private script error: Target source code was not found. (Class "
             "name = %s)\n",
             className.utf8().data());
     RELEASE_NOTREACHED();
   }
 }
 
 static v8::Local<v8::Value> installPrivateScriptRunner(v8::Isolate* isolate) {
   const String className = "PrivateScriptRunner";
   size_t index;
-  // |kPrivateScriptSources| is defined in V8PrivateScriptSources.h, which is auto-generated
-  // by make_private_script_source.py.
+  // |kPrivateScriptSources| is defined in V8PrivateScriptSources.h, which is
+  // auto-generated by make_private_script_source.py.
   for (index = 0; index < WTF_ARRAY_LENGTH(kPrivateScriptSources); index++) {
     if (className == kPrivateScriptSources[index].className)
       break;
   }
   if (index == WTF_ARRAY_LENGTH(kPrivateScriptSources)) {
     fprintf(stderr,
             "Private script error: Target source code was not found. (Class "
             "name = %s)\n",
             className.utf8().data());
     RELEASE_NOTREACHED();
@@ skipping 61 matching lines @@
              v8::Local<v8::Function>::Cast(initializeFunction), holder, 0, 0,
              isolate)
              .ToLocal(&result)) {
       fprintf(stderr,
               "Private script error: Object constructor threw an exception.\n");
       dumpV8Message(context, block.Message());
       RELEASE_NOTREACHED();
     }
   }
 
-  // Inject the prototype object of the private script into the prototype chain of the holder object.
-  // This is necessary to let the holder object use properties defined on the prototype object
-  // of the private script. (e.g., if the prototype object has |foo|, the holder object should be able
-  // to use it with |this.foo|.)
+  // Inject the prototype object of the private script into the prototype chain
+  // of the holder object.  This is necessary to let the holder object use
+  // properties defined on the prototype object of the private script. (e.g., if
+  // the prototype object has |foo|, the holder object should be able to use it
+  // with |this.foo|.)
   if (classObject->GetPrototype() != holderObject->GetPrototype()) {
     if (!v8CallBoolean(
             classObject->SetPrototype(context, holderObject->GetPrototype()))) {
       fprintf(stderr, "Private script error: SetPrototype failed.\n");
       dumpV8Message(context, block.Message());
       RELEASE_NOTREACHED();
     }
   }
   if (!v8CallBoolean(holderObject->SetPrototype(context, classObject))) {
     fprintf(stderr, "Private script error: SetPrototype failed.\n");
@@ skipping 213 matching lines @@
     rethrowExceptionInPrivateScript(
         scriptState->isolate(), block, scriptStateInUserScript,
         ExceptionState::ExecutionContext, methodName, className);
     block.ReThrow();
     return v8::Local<v8::Value>();
   }
   return result;
 }
 
 }  // namespace blink