Fix parameters order.

Fix parameters order.

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

[bartfab (slow), 2014-06-02 16:33:50]

LGTM. Thanks for catching this.

[bartfab (slow), 2014-06-02 16:34:04]

The CQ bit was checked by bartfab@chromium.org

[commit-bot: I haz the power, 2014-06-02 16:34:28]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/antrim@chromium.org/301353008/1

[bartfab (slow), 2014-06-02 16:34:33]

The CQ bit was unchecked by bartfab@chromium.org

[bartfab (slow), 2014-06-02 16:35:19]

https://codereview.chromium.org/301353008/diff/1/chrome/browser/chromeos/logi...
File
chrome/browser/chromeos/login/managed/managed_user_creation_controller_new.cc
(right):

https://codereview.chromium.org/301353008/diff/1/chrome/browser/chromeos/logi...
chrome/browser/chromeos/login/managed/managed_user_creation_controller_new.cc:275:
std::string(),  // No need for salt, key's already hashed.
Actually, the original comment was correct: The Key class is meant to store the
salt, even if the key is hashed already. Since you store the salt elsewhere,
std::string() is passed here as an exception.

[Denis Kuznetsov (DE-MUC), 2014-06-02 16:39:28]

https://chromiumcodereview.appspot.com/301353008/diff/1/chrome/browser/chrome...
File
chrome/browser/chromeos/login/managed/managed_user_creation_controller_new.cc
(right):

https://chromiumcodereview.appspot.com/301353008/diff/1/chrome/browser/chrome...
chrome/browser/chromeos/login/managed/managed_user_creation_controller_new.cc:275:
std::string(),  // No need for salt, key's already hashed.
On 2014/06/02 16:35:20, bartfab wrote:
> Actually, the original comment was correct: The Key class is meant to store
the
> salt, even if the key is hashed already. Since you store the salt elsewhere,
> std::string() is passed here as an exception.

Done.

[Denis Kuznetsov (DE-MUC), 2014-06-02 16:39:37]

The CQ bit was checked by antrim@chromium.org

[commit-bot: I haz the power, 2014-06-02 16:40:24]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/antrim@chromium.org/301353008/20001

[commit-bot: I haz the power, 2014-06-02 21:55:27]

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

[commit-bot: I haz the power, 2014-06-02 22:57:24]

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

[commit-bot: I haz the power, 2014-06-02 22:57:24]

Try jobs failed on following builders:
  win_chromium_rel on tryserver.chromium
(http://build.chromium.org/p/tryserver.chromium/builders/win_chromium_rel/buil...)

[Nikita (slow), 2014-06-03 10:12:31]

The CQ bit was checked by nkostylev@chromium.org

[commit-bot: I haz the power, 2014-06-03 10:13:28]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/antrim@chromium.org/301353008/20001

[commit-bot: I haz the power, 2014-06-03 12:11:58]

Change committed as 274476