Port identity_internals to mojo

chrome/browser/ui/webui/identity_internals/identity_internals_ui_browsertest.js

 // Copyright 2013 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
-GEN('#include "chrome/browser/ui/webui/identity_internals_ui_browsertest.h"');
+GEN('#include "chrome/browser/ui/webui/identity_internals/identity_internals_ui_browsertest.h"');
 
 /**
  * Test C++ fixture for downloads WebUI testing.
  * @constructor
  * @extends {testing.Test}
  */
 function IdentityInternalsUIBrowserTest() {}
 
 /**
  * Base fixture for Downloads WebUI testing.
  * @extends {testing.Test}
  * @constructor
  */
 function BaseIdentityInternalsWebUITest() {}
 
 BaseIdentityInternalsWebUITest.prototype = {
   __proto__: testing.Test.prototype,
 
   /**
    * Browse to the downloads page & call our preLoad().
    */
-  browsePreload: 'chrome://identity-internals',
+  browsePreloadAndWaitForMain: 'chrome://identity-internals',
 
   /** @override */
   typedefCppFixture: 'IdentityInternalsUIBrowserTest',
 
   /**
    * Gets all of the token entries on the page.
    * @return {!NodeList} Elements displaying token information.
    */
   getTokens: function() {
     return document.querySelectorAll('#token-list > div');
@@ skipping 181 matching lines @@
   expectEquals('token1', this.getAccessToken(tokenListEntries[1]));
   expectEquals('Token Present', this.getTokenStatus(tokenListEntries[1]));
   expectLT(this.getExpirationTime(tokenListEntries[1]) - new Date(),
            3600 * 1000);
   var scopes = this.getScopes(tokenListEntries[1]);
   expectEquals(3, scopes.length);
   expectEquals('scope_1_1', scopes[0]);
   expectEquals('scope_2_1', scopes[1]);
   expectEquals('', scopes[2]);
 });
-
-/**
- * Fixture for asynchronous testing of Identity Internals Web UI with multiple
- * tokens in Identity API token cache.
- * @extends {IdentityInternalsMultipleTokensWebUITest}
- * @constructor
- */
-function IdentityInternalsWebUITestAsync() {}
-
-IdentityInternalsWebUITestAsync.prototype = {
-  __proto__: IdentityInternalsMultipleTokensWebUITest.prototype,
-
-  /** @override */
-  isAsync: true,
-};
-
-TEST_F('IdentityInternalsWebUITestAsync', 'revokeToken', function() {
-  var tokenListBefore = this.getTokens();
-  expectEquals(2, tokenListBefore.length);
-  var tokenRevokeDone = identity_internals.tokenRevokeDone;
-  identity_internals.tokenRevokeDone = this.continueTest(

[Elliot Glaysher, 2014/07/21 21:56:03]

This test was removed because I don't believe we can hot patch this in a mojo
world.

[fgorski, 2014/07/23 19:02:09]

Is there a long term solution to a problem. I was a noogler when writing this
piece of code, so it is hard to believe (and remember) that I didn't see this
approach used someplace else. That means this you may have to delete a few more
tests at least. I don't think that's reasonable and there should be an approach
worked out for this problem.

Would you consider disabling the test instead of deleting it? Or is testing
user's actions on WebUI never meant to work?

[Elliot Glaysher, 2014/07/23 20:20:04]

There isn't a generic way to do this (and long term, one will have to exist for
testing). I've readded this test and modified it to use a bit of a hack.

-      WhenTestDone.ALWAYS, function(accessTokens) {
-        tokenRevokeDone.call(identity_internals, accessTokens);
-        identity_internals.tokenRevokeDone = tokenRevokeDone;
-        var tokenListAfter = this.getTokens();
-        expectEquals(1, tokenListAfter.length);
-        expectEquals(this.getAccessToken(tokenListBefore[0]),
-                     this.getAccessToken(tokenListAfter[0]));
-      }.bind(this));
-  this.getRevokeButton(tokenListBefore[1]).click();
-});
-