[Devtools] Fix multi-line curl copy response when multi-part-form

third_party/WebKit/Source/devtools/front_end/network/NetworkLogView.js

 /*
  * Copyright (C) 2007, 2008 Apple Inc.  All rights reserved.
  * Copyright (C) 2008, 2009 Anthony Ricaud <rik@webkit.org>
  * Copyright (C) 2011 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:
  *
  * 1.  Redistributions of source code must retain the above copyright
@@ skipping 1346 matching lines @@
      */
     _generateCurlCommand: function(request, platform)
     {
         var command = ["curl"];
         // These headers are derived from URL (except "version") and would be added by cURL anyway.
         var ignoredHeaders = {"host": 1, "method": 1, "path": 1, "scheme": 1, "version": 1};
 
         function escapeStringWin(str)
         {
             /* Replace quote by double quote (but not by \") because it is
                recognized by both cmd.exe and MS Crt arguments parser.

[lushnikov, 2016/07/27 20:16:18]

Can we update this comment?

[allada, 2016/07/27 22:27:59]

Done.

 
                Replace % by "%" because it could be expanded to an environment
                variable value. So %% becomes "%""%". Even if an env variable ""
                (2 doublequotes) is declared, the cmd.exe will not
                substitute it with its value.
 
                Replace each backslash with double backslash to make sure
                MS Crt arguments parser won't collapse them.
 
                Replace new line outside of quotes since cmd.exe doesn't let
                to do it inside.
             */
-            return "\"" + str.replace(/"/g, "\"\"")
-                             .replace(/%/g, "\"%\"")
-                             .replace(/\\/g, "\\\\")
-                             .replace(/[\r\n]+/g, "\"^$&\"") + "\"";
+            var encapsChars = /[\r\n]/.test(str) ? "^\"" : "\"";

[lushnikov, 2016/07/27 20:16:18]

are you sure you want to have /[\r\n]/ and not /\r\n/?

[allada, 2016/07/27 22:27:59]

This is simply checking to see if there are any new line characters; so any \r
or \n characters.

+            return encapsChars + str.replace(/\\/g, "\\\\")
+                             .replace(/"/g, "\\\"")
+                             .replace(/\^/g, "^^")
+                             .replace(/[^a-zA-Z0-9\s_\-:=+~'^\/.',?;()*`]/g, "^$&")
+                             .replace(/%(?=[a-zA-Z0-9_])/g, "%^")
+                             .replace(/\r\n|[\n\r]/g, "^\n\n") + encapsChars;
         }
 
         function escapeStringPosix(str)
         {
             function escapeCharacter(x)
             {
                 var code = x.charCodeAt(0);
                 if (code < 256) {
                     // Add leading zero when needed to not care about the next character.
                     return code < 16 ? "\\x0" + code.toString(16) : "\\x" + code.toString(16);
@@ skipping 294 matching lines @@
         return false;
     return true;
 }
 
 WebInspector.NetworkLogView.EventTypes = {
     RequestSelected: "RequestSelected",
     SearchCountUpdated: "SearchCountUpdated",
     SearchIndexUpdated: "SearchIndexUpdated",
     UpdateRequest: "UpdateRequest"
 };