Clean ups for trailing whitespace and block style

Clean ups for trailing whitespace and block style

Also M1 migration for 'operator negate()'.


R=messick@google.com
BUG=

Committed: https://code.google.com/p/dart/source/detail?r=11957

[messick, 2012-09-06 14:50:45]

lgtm

https://chromiumcodereview.appspot.com/10913112/diff/1/editor/tools/plugins/c...
File
editor/tools/plugins/com.google.dart.tools.ui/src/com/google/dart/tools/ui/internal/cleanup/style/Style_useBlocks_CleanUp.java
(right):

https://chromiumcodereview.appspot.com/10913112/diff/1/editor/tools/plugins/c...
editor/tools/plugins/com.google.dart.tools.ui/src/com/google/dart/tools/ui/internal/cleanup/style/Style_useBlocks_CleanUp.java:36:
public class Style_useBlocks_CleanUp extends AbstractMigrateCleanUp {
I'm going to leave it to you, but I would have created constants to hold values
of "somestring".length(), especially since some of them are repeated.

https://chromiumcodereview.appspot.com/10913112/diff/1/editor/tools/plugins/c...
editor/tools/plugins/com.google.dart.tools.ui/src/com/google/dart/tools/ui/internal/cleanup/style/Style_useBlocks_CleanUp.java:72:
SourceRangeFactory.forStartLength(node.getCloseParenOffset() + 1, 0),
I believe the style guide encourages single-line tests:
  if (!guardClause()) return;
You should add a newline in these cases. (Yes, a problem better solved by a
formatter, but we're not there yet.)