reflow comments in Source/bindings/core/v8

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

 /*
  * Copyright (C) 2006, 2007, 2008, 2009 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 534 matching lines @@
     v8::TryCatch block(isolate);
     if (!v8Call(value->ToString(isolate->GetCurrentContext()), stringObject,
                 block)) {
       exceptionState.rethrowV8Exception(block.Exception());
       return String();
     }
   }
 
   String x = toCoreString(stringObject);
 
-  // 2. If the value of any element of x is greater than 255, then throw a TypeError.
+  // 2. If the value of any element of x is greater than 255, then throw a
+  //    TypeError.
   if (!x.containsOnlyLatin1()) {
     exceptionState.throwTypeError("Value is not a valid ByteString.");
     return String();
   }
 
-  // 3. Return an IDL ByteString value whose length is the length of x, and where the
-  // value of each element is the value of the corresponding element of x.
-  // Blink: A ByteString is simply a String with a range constrained per the above, so
-  // this is the identity operation.
+  // 3. Return an IDL ByteString value whose length is the length of x, and
+  //    where the value of each element is the value of the corresponding
+  //    element of x.
+  //    Blink: A ByteString is simply a String with a range constrained per the
+  //    above, so this is the identity operation.
   return x;
 }
 
 static bool hasUnmatchedSurrogates(const String& string) {
   // By definition, 8-bit strings are confined to the Latin-1 code page and
   // have no surrogates, matched or otherwise.
   if (string.is8Bit())
     return false;
 
   const UChar* characters = string.characters16();
@@ skipping 64 matching lines @@
         u.append(replacementCharacter);
       } else {
         // 2. Otherwise, i < n-1:
         ASSERT(i < n - 1);
         // ....1. Let d be the code unit in S at index i+1.
         UChar d = s[i + 1];
         if (U16_IS_TRAIL(d)) {
           // 2. If 0xDC00 <= d <= 0xDFFF, then:
           // ..1. Let a be c & 0x3FF.
           // ..2. Let b be d & 0x3FF.
-          // ..3. Append to U the Unicode character with code point 2^16+2^10*a+b.
+          // ..3. Append to U the Unicode character with code point
+          //      2^16+2^10*a+b.
           u.append(U16_GET_SUPPLEMENTARY(c, d));
           // Blink: This is equivalent to u.append(c); u.append(d);
           ++i;
         } else {
-          // 3. Otherwise, d < 0xDC00 or d > 0xDFFF. Append to U a U+FFFD REPLACEMENT CHARACTER.
+          // 3. Otherwise, d < 0xDC00 or d > 0xDFFF. Append to U a U+FFFD
+          //    REPLACEMENT CHARACTER.
           u.append(replacementCharacter);
         }
       }
     }
     // 3. Set i to i+1.
     ++i;
   }
 
   // 6. Return U.
   ASSERT(u.length() == string.length());
@@ skipping 51 matching lines @@
 DOMWindow* toDOMWindow(v8::Local<v8::Context> context) {
   if (context.IsEmpty())
     return 0;
   return toDOMWindow(context->GetIsolate(), context->Global());
 }
 
 LocalDOMWindow* enteredDOMWindow(v8::Isolate* isolate) {
   LocalDOMWindow* window =
       toLocalDOMWindow(toDOMWindow(isolate->GetEnteredContext()));
   if (!window) {
-    // We don't always have an entered DOM window, for example during microtask callbacks from V8
-    // (where the entered context may be the DOM-in-JS context). In that case, we fall back
-    // to the current context.
+    // We don't always have an entered DOM window, for example during microtask
+    // callbacks from V8 (where the entered context may be the DOM-in-JS
+    // context). In that case, we fall back to the current context.
     //
-    // TODO(haraken): It's nasty to return a current window from enteredDOMWindow.
-    // All call sites should be updated so that it works even if it doesn't have
-    // an entered window.
+    // TODO(haraken): It's nasty to return a current window from
+    // enteredDOMWindow.  All call sites should be updated so that it works even
+    // if it doesn't have an entered window.
     window = currentDOMWindow(isolate);
     ASSERT(window);
   }
   return window;
 }
 
 LocalDOMWindow* currentDOMWindow(v8::Isolate* isolate) {
   return toLocalDOMWindow(toDOMWindow(isolate->GetCurrentContext()));
 }
 
@@ skipping 19 matching lines @@
   return nullptr;
 }
 
 ExecutionContext* currentExecutionContext(v8::Isolate* isolate) {
   return toExecutionContext(isolate->GetCurrentContext());
 }
 
 ExecutionContext* enteredExecutionContext(v8::Isolate* isolate) {
   ExecutionContext* context = toExecutionContext(isolate->GetEnteredContext());
   if (!context) {
-    // We don't always have an entered execution context, for example during microtask callbacks from V8
-    // (where the entered context may be the DOM-in-JS context). In that case, we fall back
-    // to the current context.
+    // We don't always have an entered execution context, for example during
+    // microtask callbacks from V8 (where the entered context may be the
+    // DOM-in-JS context). In that case, we fall back to the current context.
     context = currentExecutionContext(isolate);
     ASSERT(context);
   }
   return context;
 }
 
 Frame* toFrameIfNotDetached(v8::Local<v8::Context> context) {
   DOMWindow* window = toDOMWindow(context);
   if (window && window->isCurrentlyDisplayedInFrame())
     return window->frame();
@@ skipping 204 matching lines @@
 v8::Local<v8::Value> freezeV8Object(v8::Local<v8::Value> value,
                                     v8::Isolate* isolate) {
   value.As<v8::Object>()
       ->SetIntegrityLevel(isolate->GetCurrentContext(),
                           v8::IntegrityLevel::kFrozen)
       .ToChecked();
   return value;
 }
 
 }  // namespace blink