dart2js: Refactoring, documentation, and a few bugfixes in Namer class.

pkg/compiler/lib/src/js_backend/backend.dart

Index: pkg/compiler/lib/src/js_backend/backend.dart
diff --git a/pkg/compiler/lib/src/js_backend/backend.dart b/pkg/compiler/lib/src/js_backend/backend.dart
index 975581c922c175e6a839671ef8e12ffb6c41e3e3..de9e7ec25c25eeddcef6ebb0630bdda62d9ee093 100644
--- a/pkg/compiler/lib/src/js_backend/backend.dart
+++ b/pkg/compiler/lib/src/js_backend/backend.dart
@@ -738,7 +738,7 @@ class JavaScriptBackend extends Backend {
   }
 
   void registerSpecializedGetInterceptor(Set<ClassElement> classes) {
-    String name = namer.getInterceptorName(getInterceptorMethod, classes);
+    String name = namer.getInterceptorName(classes);

[floitsch, 2015/01/30 21:03:36]

We need to change this name.
nameForGetInterceptorFunction ?

otherwise it reads way too much like we are getting something.

[asgerf, 2015/02/03 17:39:11]

Yeah it was bothering me too. Renamed to nameForGetInterceptor.

     if (classes.contains(jsInterceptorClass)) {
       // We can't use a specialized [getInterceptorMethod], so we make
       // sure we emit the one with all checks.