Cleanup: Remove redundant MockGetAuthTokenFunction, add missing cases

Cleanup: Remove redundant MockGetAuthTokenFunction, add missing cases

In a previous change, an improved test subclass of
GetAuthTokenFunction was created for multi-login tests. All
chrome.identity.getAuthToken browser tests now use this class, and the
original test class has been removed.

The new class does not have any mock methods since the originals were
not testing anything interesting anyway, and does not need to be a
friend of its base class.

This change also adds some coverage for error cases in getAuthToken,
for requests specifying a non-primary account ID.

BUG=364203
Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=285020

[fgorski, 2014-07-22 20:09:25]

On 2014/07/22 18:21:20, Michael Courage wrote:

lgtm

[Michael Courage, 2014-07-22 20:16:10]

The CQ bit was checked by courage@chromium.org

[commit-bot: I haz the power, 2014-07-22 20:18:09]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/courage@chromium.org/410553002/40001

[commit-bot: I haz the power, 2014-07-22 23:25:28]

FYI, CQ is re-trying this CL (attempt #1).
The failing builders are:
  win_gpu_triggered_tests on tryserver.chromium.gpu
(http://build.chromium.org/p/tryserver.chromium.gpu/builders/win_gpu_triggered...)

[commit-bot: I haz the power, 2014-07-23 01:20:38]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2014-07-23 01:20:39]

Try jobs failed on following builders:
  win_gpu_triggered_tests on tryserver.chromium.gpu
(http://build.chromium.org/p/tryserver.chromium.gpu/builders/win_gpu_triggered...)

[Michael Courage, 2014-07-23 19:48:08]

The CQ bit was checked by courage@chromium.org

[commit-bot: I haz the power, 2014-07-23 19:50:22]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/courage@chromium.org/410553002/40001

[commit-bot: I haz the power, 2014-07-23 19:52:56]

Change committed as 285020