[tools/perf] Enable many more lint checks for perf directory.

tools/perf/pylintrc

 [MESSAGES CONTROL]
 
 # Disable the message, report, category or checker with the given id(s).
 # TODO(perf-owners): Reduce this list to as small as possible.
-disable=I0010,I0011,abstract-class-little-used,abstract-class-not-used,anomalous-backslash-in-string,bad-builtin,bad-context-manager,bad-continuation,bad-indentation,bad-str-strip-call,bad-whitespace,broad-except,cell-var-from-loop,deprecated-lambda,deprecated-module,duplicate-code,eval-used,exec-used,fixme,function-redefined,global-statement,interface-not-implemented,invalid-name,locally-enabled,logging-not-lazy,missing-docstring,missing-final-newline,no-init,no-member,no-name-in-module,no-self-use,no-self-use,not-callable,reimported,star-args,super-on-old-class,too-few-public-methods,too-many-ancestors,too-many-arguments,too-many-branches,too-many-function-args,too-many-instance-attributes,too-many-lines,too-many-locals,too-many-public-methods,too-many-return-statements,too-many-statements,trailing-whitespace,useless-else-on-loop
+disable=

[petrcermak, 2016/08/11 08:51:13]

I went through the list and it seems that while most items were removed, the
following were ADDED:

design
similarities
locally-disabled
protected-access

Just wanted to check that this is intended ;-)

[petrcermak, 2016/08/11 08:51:13]

nit: Sort the items alphabetically? It's mostly there with a few exceptions.

[nednguyen, 2016/08/11 12:30:30]

Done.

[nednguyen, 2016/08/11 12:30:30]

Done.

[nednguyen, 2016/08/11 12:32:07]

Note that locally-disabled only got triggered in this CL. I think probably due
to this presubmit check only goes over python files that are touched :-/

+  design,
+  similarities,
+

[petrcermak, 2016/08/11 08:51:13]

why the blank line?

[nednguyen, 2016/08/11 12:30:30]

Done.

+  abstract-class-little-used,
+  abstract-class-not-used,
+  bad-builtin,
+  bad-continuation,
+  bad-whitespace,
+  bad-indentation,
+  broad-except,
+  eval-used,
+  fixme,
+  global-statement,
+  interface-not-implemented,
+  invalid-name,
+  locally-disabled,
+  locally-enabled,
+  logging-not-lazy,
+  missing-docstring,
+  no-member,
+  no-self-use,
+  protected-access,
+  star-args,
 
 
 [REPORTS]
 
 # Don't write out full reports, just messages.
 reports=no
 
 
 [VARIABLES]
 
 # Google style
 dummy-variables-rgx=_$|unused_
 
 
 [FORMAT]
 
 # We use two spaces for indents, instead of the usual four spaces or tab.
 indent-string='  '
 
 # We suppress long line check for lines that contain only the URL (with or
 # without quote).
 ignore-long-lines=^\s*'?https?://\S+'?$