Make PromiseResolver's members read-only

chrome/test/data/webui/webui_resource_browsertest.cc

Index: chrome/test/data/webui/webui_resource_browsertest.cc
diff --git a/chrome/test/data/webui/webui_resource_browsertest.cc b/chrome/test/data/webui/webui_resource_browsertest.cc
index b1d55188fae0b77fd157fbbc9339da5521876918..5f966dc9ddb9a9a55e4f017a31770612f0e373b4 100644
--- a/chrome/test/data/webui/webui_resource_browsertest.cc
+++ b/chrome/test/data/webui/webui_resource_browsertest.cc
@@ -239,3 +239,9 @@ IN_PROC_BROWSER_TEST_F(WebUIResourceBrowserTest, UtilTest) {
   AddLibrary(IDR_WEBUI_JS_UTIL);
   LoadFile(base::FilePath(FILE_PATH_LITERAL("util_test.html")));
 }
+
+IN_PROC_BROWSER_TEST_F(WebUIResourceBrowserTest, PromiseResolverTest) {
+  AddLibrary(IDR_WEBUI_JS_ASSERT);
+  AddLibrary(IDR_WEBUI_JS_PROMISE_RESOLVER);
+  LoadFile(base::FilePath(FILE_PATH_LITERAL("promise_resolver_test.html")));

[dpapad, 2016/04/08 22:32:49]

Thanks for adding tests! Just one comment/question:

I was hoping we were moving away from putting JS test code in HTML files. Would
it make sense if you convert this test to look more like our more recent mocha
based tests, as I did with
https://code.google.com/p/chromium/codesearch#chromium/src/chrome/test/data/w...

[Dan Beam, 2016/04/08 23:11:21]

as per offline convo, i'll add a TODO to deprecate these tests and move them to
mocha (after this one)

+}