[Password Manager] Correct the documentation within PasswordFormFillData struct for form origin and…

[Password Manager] Correct the documentation within PasswordFormFillData struct for form origin and action.

It's a follow-up CL to: https://codereview.chromium.org/614023002/

BUG=417295
Committed: https://crrev.com/6c48e440419da5a73344d1010cf33029e173dd1d
Cr-Commit-Position: refs/heads/master@{#303635}

[Pritam Nikam, 2014-11-11 10:20:02]

pritam.nikam@samsung.com changed reviewers:
+ vabr@chromium.org

[Pritam Nikam, 2014-11-11 10:20:41]

Hi Vaclav, PTAL!

Thanks!

[vabr (Chromium), 2014-11-11 10:35:03]

Thanks, Pritam Nikam.
Some comments:
(1) Please include bug 417295 in the CL description.
(2) Please also correct the already existing comment for PasswordForm::origin

Thanks!
Vaclav

https://codereview.chromium.org/711333002/diff/1/components/autofill/core/com...
File components/autofill/core/common/password_form_fill_data.h (right):

https://codereview.chromium.org/711333002/diff/1/components/autofill/core/com...
components/autofill/core/common/password_form_fill_data.h:45: // An origin is
(scheme, host, port and path) only; keeps the path but strips
optional nit: The use of parentheses is a bit strange here. An alternative
suggestion for the whole comment would be: "An origin URL consists of the
scheme, host, port and path; the rest is stripped."

[Pritam Nikam, 2014-11-11 10:47:04]

Thanks Vaclav, I've incorporated nits & modified documentation for
PasswordForm::origin & action. PTAL!

Thanks!

https://codereview.chromium.org/711333002/diff/1/components/autofill/core/com...
File components/autofill/core/common/password_form_fill_data.h (right):

https://codereview.chromium.org/711333002/diff/1/components/autofill/core/com...
components/autofill/core/common/password_form_fill_data.h:45: // An origin is
(scheme, host, port and path) only; keeps the path but strips
On 2014/11/11 10:35:03, vabr (Chromium) wrote:
> optional nit: The use of parentheses is a bit strange here. An alternative
> suggestion for the whole comment would be: "An origin URL consists of the
> scheme, host, port and path; the rest is stripped."

Done.

[vabr (Chromium), 2014-11-11 12:25:59]

LGTM, thank you!
Vaclav

[Pritam Nikam, 2014-11-11 12:26:11]

The CQ bit was checked by pritam.nikam@samsung.com

[commit-bot: I haz the power, 2014-11-11 12:27:00]

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

[commit-bot: I haz the power, 2014-11-11 13:15:36]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2014-11-11 13:16:51]

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