Reflow comments in core/loader

Reflow comments in core/loader

BUG=563793
Committed: https://crrev.com/4013d22a7d6569f5761b437d88b9cf716909f223
Cr-Commit-Position: refs/heads/master@{#422419}

[Charlie Harrison, 2016-10-02 05:17:08]

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

[commit-bot: I haz the power, 2016-10-02 05:17:11]

Dry run: 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-02 07:57:17]

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

[commit-bot: I haz the power, 2016-10-02 07:57:18]

Dry run: This issue passed the CQ dry run.

[Charlie Harrison, 2016-10-03 02:12:09]

csharrison@chromium.org changed reviewers:
+ yhirano@chromium.org

[Charlie Harrison, 2016-10-03 02:12:10]

yhirano, ptal? Thanks!

[yhirano, 2016-10-03 08:37:34]

lgtm

All suggestions are optional.

https://codereview.chromium.org/2383403002/diff/1/third_party/WebKit/Source/c...
File third_party/WebKit/Source/core/loader/DocumentThreadableLoader.h (right):

https://codereview.chromium.org/2383403002/diff/1/third_party/WebKit/Source/c...
third_party/WebKit/Source/core/loader/DocumentThreadableLoader.h:237:
HTTPHeaderMap m_simpleRequestHeaders;  // stores simple request headers in
Placing the comment before the member variable might be good if it is long.

https://codereview.chromium.org/2383403002/diff/1/third_party/WebKit/Source/c...
File third_party/WebKit/Source/core/loader/HttpEquiv.h (right):

https://codereview.chromium.org/2383403002/diff/1/third_party/WebKit/Source/c...
third_party/WebKit/Source/core/loader/HttpEquiv.h:16: * Handles a HTTP header
equivalent set by a meta tag using <meta
Moving "<meta" to the next line would be good.

https://codereview.chromium.org/2383403002/diff/1/third_party/WebKit/Source/c...
File third_party/WebKit/Source/core/loader/ImageLoader.cpp (right):

https://codereview.chromium.org/2383403002/diff/1/third_party/WebKit/Source/c...
third_party/WebKit/Source/core/loader/ImageLoader.cpp:508: // immediately before
returning
Why don't you put more words on this line?

[Charlie Harrison, 2016-10-03 12:30:06]

https://codereview.chromium.org/2383403002/diff/1/third_party/WebKit/Source/c...
File third_party/WebKit/Source/core/loader/DocumentThreadableLoader.h (right):

https://codereview.chromium.org/2383403002/diff/1/third_party/WebKit/Source/c...
third_party/WebKit/Source/core/loader/DocumentThreadableLoader.h:237:
HTTPHeaderMap m_simpleRequestHeaders;  // stores simple request headers in
On 2016/10/03 08:37:34, yhirano wrote:
> Placing the comment before the member variable might be good if it is long.

Done. Yupp I tried to fix most of these cases.

https://codereview.chromium.org/2383403002/diff/1/third_party/WebKit/Source/c...
File third_party/WebKit/Source/core/loader/HttpEquiv.h (right):

https://codereview.chromium.org/2383403002/diff/1/third_party/WebKit/Source/c...
third_party/WebKit/Source/core/loader/HttpEquiv.h:16: * Handles a HTTP header
equivalent set by a meta tag using <meta
On 2016/10/03 08:37:34, yhirano wrote:
> Moving "<meta" to the next line would be good.

Done.

https://codereview.chromium.org/2383403002/diff/1/third_party/WebKit/Source/c...
File third_party/WebKit/Source/core/loader/ImageLoader.cpp (right):

https://codereview.chromium.org/2383403002/diff/1/third_party/WebKit/Source/c...
third_party/WebKit/Source/core/loader/ImageLoader.cpp:508: // immediately before
returning
On 2016/10/03 08:37:34, yhirano wrote:
> Why don't you put more words on this line?

Good catch, I must have missed this comment. Done.

[Charlie Harrison, 2016-10-03 12:30:12]

The CQ bit was checked by csharrison@chromium.org

[Charlie Harrison, 2016-10-03 12:30:12]

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

[commit-bot: I haz the power, 2016-10-03 12:30:22]

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 14:26:04]

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

[commit-bot: I haz the power, 2016-10-03 14:26:05]

Try jobs failed on following builders:
  android_n5x_swarming_rel on master.tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/android_n5x_...)

[Charlie Harrison, 2016-10-03 14:27:35]

The CQ bit was checked by csharrison@chromium.org

[commit-bot: I haz the power, 2016-10-03 14:27:47]

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 14:54:13]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-10-03 14:55:41]

Description was changed from

==========
Reflow comments in core/loader

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

to

==========
Reflow comments in core/loader

BUG=563793

Committed: https://crrev.com/4013d22a7d6569f5761b437d88b9cf716909f223
Cr-Commit-Position: refs/heads/master@{#422419}
==========

[commit-bot: I haz the power, 2016-10-03 14:55:42]

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