Unit tests for attempt manager

chrome/test/data/webui/test_api.js

Index: chrome/test/data/webui/test_api.js
diff --git a/chrome/test/data/webui/test_api.js b/chrome/test/data/webui/test_api.js
index 4974933c31d45cf6eb8906262d74704e9809a409..3db396e786668c66d30fa3195c1753c450589a05 100644
--- a/chrome/test/data/webui/test_api.js
+++ b/chrome/test/data/webui/test_api.js
@@ -572,7 +572,6 @@ var testing = {};
     }
 
     var fieldName = path[path.length-1];
-    assertEquals(undefined, namespace[fieldName]);

[robliao, 2013/09/04 20:17:25]

Why is this getting removed?

[vadimt, 2013/09/04 22:34:40]

It became too restrictive. Sometimes (I have 2 cases), the API already exists,
and you really want to override the existing call. Removing this assert allows
to avoid writing something like "instrumented.api.function = undefined;" before
mocking.

     namespace[fieldName] = theFunction;
   }