reflow comments in core/{clipboard,streams,testing,timing}

third_party/WebKit/Source/core/clipboard/DataTransfer.cpp

 /*
  * Copyright (C) 2006, 2007, 2008 Apple 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:
  * 1. Redistributions of source code must retain the above copyright
  *    notice, this list of conditions and the following disclaimer.
  * 2. Redistributions in binary form must reproduce the above copyright
  *    notice, this list of conditions and the following disclaimer in the
@@ skipping 30 matching lines @@
 #include "core/layout/LayoutObject.h"
 #include "platform/DragImage.h"
 #include "platform/MIMETypeRegistry.h"
 #include "platform/clipboard/ClipboardMimeTypes.h"
 #include "platform/clipboard/ClipboardUtilities.h"
 #include <memory>
 
 namespace blink {
 
 static DragOperation convertEffectAllowedToDragOperation(const String& op) {
-  // Values specified in http://www.whatwg.org/specs/web-apps/current-work/multipage/dnd.html#dom-datatransfer-effectallowed
+  // Values specified in
+  // http://www.whatwg.org/specs/web-apps/current-work/multipage/dnd.html#dom-datatransfer-effectallowed
   if (op == "uninitialized")
     return DragOperationEvery;
   if (op == "none")
     return DragOperationNone;
   if (op == "copy")
     return DragOperationCopy;
   if (op == "link")
     return DragOperationLink;
   if (op == "move")
     return (DragOperation)(DragOperationGeneric | DragOperationMove);
@@ skipping 24 matching lines @@
     return "copyLink";
   if (moveSet)
     return "move";
   if (op & DragOperationCopy)
     return "copy";
   if (op & DragOperationLink)
     return "link";
   return "none";
 }
 
-// We provide the IE clipboard types (URL and Text), and the clipboard types specified in the WHATWG Web Applications 1.0 draft
-// see http://www.whatwg.org/specs/web-apps/current-work/ Section 6.3.5.3
+// We provide the IE clipboard types (URL and Text), and the clipboard types
+// specified in the WHATWG Web Applications 1.0 draft see
+// http://www.whatwg.org/specs/web-apps/current-work/ Section 6.3.5.3
 static String normalizeType(const String& type, bool* convertToURL = 0) {
   String cleanType = type.stripWhiteSpace().lower();
   if (cleanType == mimeTypeText || cleanType.startsWith(mimeTypeTextPlainEtc))
     return mimeTypeTextPlain;
   if (cleanType == mimeTypeURL) {
     if (convertToURL)
       *convertToURL = true;
     return mimeTypeTextURIList;
   }
   return cleanType;
 }
 
 DataTransfer* DataTransfer::create(DataTransferType type,
                                    DataTransferAccessPolicy policy,
                                    DataObject* dataObject) {
   return new DataTransfer(type, policy, dataObject);
 }
 
 DataTransfer::~DataTransfer() {}
 
 void DataTransfer::setDropEffect(const String& effect) {
   if (!isForDragAndDrop())
     return;
 
-  // The attribute must ignore any attempts to set it to a value other than none, copy, link, and move.
+  // The attribute must ignore any attempts to set it to a value other than
+  // none, copy, link, and move.
   if (effect != "none" && effect != "copy" && effect != "link" &&
       effect != "move")
     return;
 
-  // FIXME: The spec actually allows this in all circumstances, even though there's no point in
-  // setting the drop effect when this condition is not true.
+  // FIXME: The spec actually allows this in all circumstances, even though
+  // there's no point in setting the drop effect when this condition is not
+  // true.
   if (canReadTypes())
     m_dropEffect = effect;
 }
 
 void DataTransfer::setEffectAllowed(const String& effect) {
   if (!isForDragAndDrop())
     return;
 
   if (convertEffectAllowedToDragOperation(effect) == DragOperationPrivate) {
     // This means that there was no conversion, and the effectAllowed that
     // we are passed isn't a valid effectAllowed, so we should ignore it,
     // and not set m_effectAllowed.
 
     // The attribute must ignore any attempts to set it to a value other than
-    // none, copy, copyLink, copyMove, link, linkMove, move, all, and uninitialized.
+    // none, copy, copyLink, copyMove, link, linkMove, move, all, and
+    // uninitialized.
     return;
   }
 
   if (canWriteData())
     m_effectAllowed = effect;
 }
 
 void DataTransfer::clearData(const String& type) {
   if (!canWriteData())
     return;
@@ skipping 124 matching lines @@
 
   // Determine the filename for the file contents of the image.
   String filename = cachedImage->response().suggestedFilename();
   if (filename.isEmpty())
     filename = url.lastPathComponent();
 
   String fileExtension;
   if (filename.isEmpty()) {
     filename = element->getAttribute(HTMLNames::altAttr);
   } else {
-    // Strip any existing extension. Assume that alt text is usually not a filename.
+    // Strip any existing extension. Assume that alt text is usually not a
+    // filename.
     int extensionIndex = filename.reverseFind('.');
     if (extensionIndex != -1) {
       fileExtension = filename.substring(extensionIndex + 1);
       filename.truncate(extensionIndex);
     }
   }
 
   if (!fileExtension.isEmpty() && fileExtension != imageExtension) {
     String imageMimeType =
         MIMETypeRegistry::getMIMETypeForExtension(imageExtension);
     ASSERT(imageMimeType.startsWith("image/"));
-    // Use the file extension only if it has imageMimeType: it's untrustworthy otherwise.
+    // Use the file extension only if it has imageMimeType: it's untrustworthy
+    // otherwise.
     if (imageMimeType ==
         MIMETypeRegistry::getMIMETypeForExtension(fileExtension))
       imageExtension = fileExtension;
   }
 
   imageExtension = "." + imageExtension;
   validateFilename(filename, imageExtension);
 
   dataObject->addSharedBuffer(filename + imageExtension, imageBuffer);
 }
@@ skipping 104 matching lines @@
   if (keyword.startsWith("file:"))
     return hasFileOfType(keyword.substring(5));
 
   if (keyword.startsWith("string:"))
     return hasStringOfType(keyword.substring(7));
 
   return false;
 }
 
 DataTransferItemList* DataTransfer::items() {
-  // FIXME: According to the spec, we are supposed to return the same collection of items each
-  // time. We now return a wrapper that always wraps the *same* set of items, so JS shouldn't be
-  // able to tell, but we probably still want to fix this.
+  // FIXME: According to the spec, we are supposed to return the same collection
+  // of items each time. We now return a wrapper that always wraps the *same*
+  // set of items, so JS shouldn't be able to tell, but we probably still want
+  // to fix this.
   return DataTransferItemList::create(this, m_dataObject);
 }
 
 DataObject* DataTransfer::dataObject() const {
   return m_dataObject;
 }
 
 DataTransfer::DataTransfer(DataTransferType type,
                            DataTransferAccessPolicy policy,
                            DataObject* dataObject)
@@ skipping 60 matching lines @@
   }
 }
 
 DEFINE_TRACE(DataTransfer) {
   visitor->trace(m_dataObject);
   visitor->trace(m_dragImage);
   visitor->trace(m_dragImageElement);
 }
 
 }  // namespace blink