Clean-up NativeBackendLibsecretTest

Clean-up NativeBackendLibsecretTest

This CL removes CheckPasswordChangesWithResult, because unlike in the other native backend tests, this method was not used inside base::Bind, and so is less useful than confusing. It also removes a global std::vector in favour of replacing it with a pointer to a test-owned instance, and turns that instance into a ScopedVector.

BUG=461385
Committed: https://crrev.com/be80bf74e784f8ad15402c7aecf593b384c5c33b
Cr-Commit-Position: refs/heads/master@{#318009}

[vabr (Chromium), 2015-02-24 17:25:30]

Patchset #1 (id:1) has been deleted

[vabr (Chromium), 2015-02-24 17:51:58]

vabr@chromium.org changed reviewers:
+ mkwst@chromium.org

[vabr (Chromium), 2015-02-24 17:51:59]

Hi Mike,

PTAL at this clean-up.

Cheers!
Vaclav

[Mike West, 2015-02-24 18:05:59]

LGTM.

[vabr (Chromium), 2015-02-25 08:09:52]

The CQ bit was checked by vabr@chromium.org

[commit-bot: I haz the power, 2015-02-25 08:10:41]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/951883005/20001

[commit-bot: I haz the power, 2015-02-25 09:44:30]

Committed patchset #1 (id:20001)

[commit-bot: I haz the power, 2015-02-25 09:45:56]

Patchset 1 (id:??) landed as
https://crrev.com/be80bf74e784f8ad15402c7aecf593b384c5c33b
Cr-Commit-Position: refs/heads/master@{#318009}