Refactor GN source expansions.

tools/gn/escape.cc

 // Copyright (c) 2013 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 "tools/gn/escape.h"
 
 #include "base/containers/stack_container.h"
 #include "base/logging.h"
 
 namespace {
@@ skipping 28 matching lines @@
   dest->push_back(ch);
 }
 
 template<typename DestString>
 void EscapeStringToString_Ninja(const base::StringPiece& str,
                                 const EscapeOptions& options,
                                 DestString* dest,
                                 bool* needed_quoting) {
   for (size_t i = 0; i < str.size(); i++)
     NinjaEscapeChar(str[i], dest);
-  if (needed_quoting)
-    *needed_quoting = false;
 }
 
 // Escape for CommandLineToArgvW and additionally escape Ninja characters.
 //
 // The basic algorithm is if the string doesn't contain any parse-affecting
 // characters, don't do anything (other than the Ninja processing). If it does,
 // quote the string, and backslash-escape all quotes and backslashes.
 // See:
 //   http://blogs.msdn.com/b/twistylittlepassagesallalike/archive/2011/04/23/everyone-quotes-arguments-the-wrong-way.aspx
 //   http://blogs.msdn.com/b/oldnewthing/archive/2010/09/17/10063629.aspx
@@ skipping 124 matching lines @@
 }
 
 void EscapeStringToStream(std::ostream& out,
                           const base::StringPiece& str,
                           const EscapeOptions& options) {
   base::StackString<256> escaped;
   EscapeStringToString(str, options, &escaped.container(), NULL);
   if (!escaped->empty())
     out.write(escaped->data(), escaped->size());
 }