Reflow comments in core/html/forms/.

Reflow comments in core/html/forms/.

This CL has no behavior changes.

BUG=563793
Committed: https://crrev.com/35c6377052771b615c563e1306141d1082c25e71
Cr-Commit-Position: refs/heads/master@{#422372}

[tkent, 2016-10-03 03:44:58]

The CQ bit was checked by tkent@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-10-03 03:45:06]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[tkent, 2016-10-03 03:46:02]

Description was changed from

==========
Reflow comments in core/html/forms/.

This CL has no behavior changes.
==========

to

==========
Reflow comments in core/html/forms/.

This CL has no behavior changes.

BUG=563793
==========

[tkent, 2016-10-03 03:47:05]

tkent@chromium.org changed reviewers:
+ yosin@chromium.org

[tkent, 2016-10-03 03:47:06]

yosin@, would you stamp this please?

[yosin_UTC9, 2016-10-03 04:00:24]

lgtm w/ suggestions

https://codereview.chromium.org/2390563002/diff/1/third_party/WebKit/Source/c...
File third_party/WebKit/Source/core/html/forms/ImageInputType.h (right):

https://codereview.chromium.org/2390563002/diff/1/third_party/WebKit/Source/c...
third_party/WebKit/Source/core/html/forms/ImageInputType.h:76: // Valid only
during HTMLFormElement::prepareForSubmission().
I think this comment is for |m_clickLocation|, so it should be above
|m_clickLocation| rather than below.

Note: 2390563002: Reflow comments in core/html/forms/ doesn't mention about
location of comment for variable.
But, for function, there is:
Almost every function declaration should have comments immediately preceding it
that describe what the function does and how to use it.

It is better to follow this.

https://codereview.chromium.org/2390563002/diff/1/third_party/WebKit/Source/c...
File third_party/WebKit/Source/core/html/forms/InputType.h (right):

https://codereview.chromium.org/2390563002/diff/1/third_party/WebKit/Source/c...
third_party/WebKit/Source/core/html/forms/InputType.h:90: // Checked first,
before internal storage or the value attribute.
It seems function comment and function declaration separated by blank line, e.g.
https://cs.chromium.org/chromium/src/components/bookmarks/browser/bookmark_ma...
Note: Google C++ style guide doesn't mention.

[tkent, 2016-10-03 04:14:14]

https://codereview.chromium.org/2390563002/diff/1/third_party/WebKit/Source/c...
File third_party/WebKit/Source/core/html/forms/ImageInputType.h (right):

https://codereview.chromium.org/2390563002/diff/1/third_party/WebKit/Source/c...
third_party/WebKit/Source/core/html/forms/ImageInputType.h:76: // Valid only
during HTMLFormElement::prepareForSubmission().
On 2016/10/03 at 04:00:24, Yosi_UTC9 wrote:
> I think this comment is for |m_clickLocation|, so it should be above
|m_clickLocation| rather than below.

I agree with it.  The comment is followed by the variable declaration in Patch
Set 1 actually.

https://codereview.chromium.org/2390563002/diff/1/third_party/WebKit/Source/c...
File third_party/WebKit/Source/core/html/forms/InputType.h (right):

https://codereview.chromium.org/2390563002/diff/1/third_party/WebKit/Source/c...
third_party/WebKit/Source/core/html/forms/InputType.h:90: // Checked first,
before internal storage or the value attribute.
On 2016/10/03 at 04:00:24, Yosi_UTC9 wrote:
> It seems function comment and function declaration separated by blank line,
e.g.
https://cs.chromium.org/chromium/src/components/bookmarks/browser/bookmark_ma...
> Note: Google C++ style guide doesn't mention.

IMO, it's out of scope of this CL, and should be addressed in another CL.
I move these comments just because it's hard to wrap comments at the original
positions.  I don't intent to resolve all formatting issues about comments.

[commit-bot: I haz the power, 2016-10-03 04:44:46]

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

[commit-bot: I haz the power, 2016-10-03 04:44:47]

Dry run: This issue passed the CQ dry run.

[tkent, 2016-10-03 04:45:42]

The CQ bit was checked by tkent@chromium.org

[commit-bot: I haz the power, 2016-10-03 04:45:56]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-10-03 04:51:26]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-10-03 04:53:30]

Description was changed from

==========
Reflow comments in core/html/forms/.

This CL has no behavior changes.

BUG=563793
==========

to

==========
Reflow comments in core/html/forms/.

This CL has no behavior changes.

BUG=563793

Committed: https://crrev.com/35c6377052771b615c563e1306141d1082c25e71
Cr-Commit-Position: refs/heads/master@{#422372}
==========

[commit-bot: I haz the power, 2016-10-03 04:53:31]

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