Chromium Code Reviews| Index: tools/clang/rewrite_to_chrome_style/tests/fields-original.cc |
| diff --git a/tools/clang/rewrite_to_chrome_style/tests/fields-original.cc b/tools/clang/rewrite_to_chrome_style/tests/fields-original.cc |
| index 0e1159b7ce64fd23a0d3fdab4b98adfeb805a58f..6985f4885fe72a436300510c906829259acdae1a 100644 |
| --- a/tools/clang/rewrite_to_chrome_style/tests/fields-original.cc |
| +++ b/tools/clang/rewrite_to_chrome_style/tests/fields-original.cc |
| @@ -14,9 +14,10 @@ class C { |
| return instanceCount + m_flagField + m_fieldMentioningHTTPAndHTTPS; |
| } |
| - private: |
| // The m_ prefix, if any, should be removed. |
|
danakj
2016/01/06 22:09:40
The comment is weird now, it really applies to the
dcheng
2016/01/08 01:41:11
I've added a comment on each one to make this clea
|
| static int instanceCount; |
| + |
| + private: |
| const int m_flagField; |
| // Statics should be named with s_, but make sure s_ and m_ are both correctly |
| // stripped. |
| @@ -44,3 +45,8 @@ union U { |
| }; |
| } // namespace blink |
| + |
| +void F() { |
| + // Test that references to a static field are correctly rewritten. |
| + blink::C::instanceCount++; |
| +} |