Check for NULL PasswordFormManager* in ManagePasswordsUIController::SavePasswordInternal().

Check for NULL PasswordFormManager* in ManagePasswordsUIController::SavePasswordInternal().

BUG=468474
Committed: https://crrev.com/363750d17c11bf953e19b4da4aeb1c12f30f997f
Cr-Commit-Position: refs/heads/master@{#321408}

[vasilii, 2015-03-19 18:10:23]

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

[vasilii, 2015-03-19 18:10:23]

Hi Vaclav,

please review the CL.

[vabr (Chromium), 2015-03-19 18:19:10]

LGTM with a simple nit.
Thanks!
Vaclav

https://codereview.chromium.org/1019943002/diff/1/chrome/browser/ui/passwords...
File chrome/browser/ui/passwords/manage_passwords_ui_controller.cc (right):

https://codereview.chromium.org/1019943002/diff/1/chrome/browser/ui/passwords...
chrome/browser/ui/passwords/manage_passwords_ui_controller.cc:222: // If this is
a cause of crbug/468474 then we should hide the bubble when
nit: crbug/468474 -> http://crbug.com/468474
(For convenience when clicking through the link in CodeSearch.)

[vasilii, 2015-03-19 18:23:11]

https://codereview.chromium.org/1019943002/diff/1/chrome/browser/ui/passwords...
File chrome/browser/ui/passwords/manage_passwords_ui_controller.cc (right):

https://codereview.chromium.org/1019943002/diff/1/chrome/browser/ui/passwords...
chrome/browser/ui/passwords/manage_passwords_ui_controller.cc:222: // If this is
a cause of crbug/468474 then we should hide the bubble when
On 2015/03/19 18:19:10, vabr (Chromium) wrote:
> nit: crbug/468474 -> http://crbug.com/468474
> (For convenience when clicking through the link in CodeSearch.)

Done.

[vasilii, 2015-03-19 18:23:19]

The CQ bit was checked by vasilii@chromium.org

[vasilii, 2015-03-19 18:23:19]

The patchset sent to the CQ was uploaded after l-g-t-m from vabr@chromium.org
Link to the patchset: https://codereview.chromium.org/1019943002/#ps20001
(title: "addressed the nit")

[commit-bot: I haz the power, 2015-03-19 18:23:47]

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

[commit-bot: I haz the power, 2015-03-19 19:34:15]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2015-03-19 19:34:49]

Patchset 2 (id:??) landed as
https://crrev.com/363750d17c11bf953e19b4da4aeb1c12f30f997f
Cr-Commit-Position: refs/heads/master@{#321408}