rewrite_to_chrome_style: skip nodes in synthesized functions.

tools/clang/rewrite_to_chrome_style/tests/fields-original.cc

 // Copyright 2015 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.
 
 namespace blink {
 
 class C {

[danakj, 2016/01/28 23:08:20]

Can you add a comment saying that adding a copy/move constructor of any sort
will break the test?

[dcheng, 2016/01/28 23:15:43]

Done.

  public:
   // Make sure initializers are updated to use the new names.
   C() : m_flagField(~0), m_fieldMentioningHTTPAndHTTPS(1) {}
 
   int method() {
     // Test that references to fields are updated correctly.
     return instanceCount + m_flagField + m_fieldMentioningHTTPAndHTTPS;
   }
 
   // Test that a field without a m_ prefix is correctly renamed.
@@ skipping 25 matching lines @@
   char fourChars[4];
   short twoShorts[2];
   int one_hopefully_four_byte_int;
 };
 
 }  // namespace blink
 
 void F() {
   // Test that references to a static field are correctly rewritten.
   blink::C::instanceCount++;
+  // Force instantiation of a copy constructor for blink::C to make sure field

[dcheng, 2016/01/28 06:07:32]

I couldn't repro this with an explicitly defaulted copy ctor.

+  // initializers for synthesized functions don't cause weird rewrites.
+  blink::C c;
+  blink::C c2 = c;
 }