reflow comments in core/xml*

reflow comments in core/xml*

BUG=563793
Committed: https://crrev.com/56d5d7aa7ad96ad3c853cc128a769404f6e0294c
Cr-Commit-Position: refs/heads/master@{#422990}

[Nico, 2016-10-04 19:53:19]

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

[Nico, 2016-10-04 19:53:34]

thakis@chromium.org changed reviewers:
+ jbroman@chromium.org

[commit-bot: I haz the power, 2016-10-04 19:53:51]

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

[jbroman, 2016-10-04 19:56:52]

lgtm; two-space nit is up to you (not a big deal, but slightly weird)

https://codereview.chromium.org/2389883005/diff/1/third_party/WebKit/Source/c...
File third_party/WebKit/Source/core/xml/XPathFunctions.cpp (right):

https://codereview.chromium.org/2389883005/diff/1/third_party/WebKit/Source/c...
third_party/WebKit/Source/core/xml/XPathFunctions.cpp:363: // one.  In WebKit,
getElementById behaves so, too, although its behavior in
ditto

https://codereview.chromium.org/2389883005/diff/1/third_party/WebKit/Source/c...
File third_party/WebKit/Source/core/xml/parser/XMLDocumentParser.cpp (right):

https://codereview.chromium.org/2389883005/diff/1/third_party/WebKit/Source/c...
third_party/WebKit/Source/core/xml/parser/XMLDocumentParser.cpp:392: // called. 
I don't believe XMLDocumentParserQt needs doEnd called in the
super-duper-nit: vim likes to put two spaces between sentences, but we don't
generally do that. Maybe fix?

https://codereview.chromium.org/2389883005/diff/1/third_party/WebKit/Source/c...
File third_party/WebKit/Source/core/xmlhttprequest/XMLHttpRequest.cpp (right):

https://codereview.chromium.org/2389883005/diff/1/third_party/WebKit/Source/c...
third_party/WebKit/Source/core/xmlhttprequest/XMLHttpRequest.cpp:1254: //
false|, when |handleRequestError| is called after |internalAbort()|.  This
ditto

https://codereview.chromium.org/2389883005/diff/1/third_party/WebKit/Source/c...
third_party/WebKit/Source/core/xmlhttprequest/XMLHttpRequest.cpp:1522: // the
data.  When using |HTMLDocumentParser|, which works asynchronously,
ditto

[Nico, 2016-10-04 20:17:12]

thanks!

https://codereview.chromium.org/2389883005/diff/1/third_party/WebKit/Source/c...
File third_party/WebKit/Source/core/xml/parser/XMLDocumentParser.cpp (right):

https://codereview.chromium.org/2389883005/diff/1/third_party/WebKit/Source/c...
third_party/WebKit/Source/core/xml/parser/XMLDocumentParser.cpp:392: // called. 
I don't believe XMLDocumentParserQt needs doEnd called in the
I use vim's gq to join lines manually, and it defaults to two spaces
after periods.

Looks like we deliberately decided that we don't care about this a while
ago: https://codereview.chromium.org/609633004

[Nico, 2016-10-04 20:17:19]

The CQ bit was unchecked by thakis@chromium.org

[Nico, 2016-10-04 20:17:22]

The CQ bit was checked by thakis@chromium.org

[commit-bot: I haz the power, 2016-10-04 20:17:53]

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-04 23:15:16]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-10-04 23:17:07]

Description was changed from

==========
reflow comments in core/xml*

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

to

==========
reflow comments in core/xml*

BUG=563793

Committed: https://crrev.com/56d5d7aa7ad96ad3c853cc128a769404f6e0294c
Cr-Commit-Position: refs/heads/master@{#422990}
==========

[commit-bot: I haz the power, 2016-10-04 23:17:09]

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