[iOS] Cleanup comments in find_in_page.js

[iOS] Cleanup comments in find_in_page.js

BUG=None

Committed: https://crrev.com/a0933169507826cbbad85333d70312b0e02d3d45
Cr-Commit-Position: refs/heads/master@{#330714}

[sdefresne, 2015-05-18 14:29:17]

PTAL

[stuartmorgan, 2015-05-19 18:26:47]

LGTM with caveats

https://codereview.chromium.org/1123383006/diff/40001/ios/chrome/browser/find...
File ios/chrome/browser/find_in_page/resources/find_in_page.js (right):

https://codereview.chromium.org/1123383006/diff/40001/ios/chrome/browser/find...
ios/chrome/browser/find_in_page/resources/find_in_page.js:218: // TODO(ios): It
would be better if this DCHECKed.
We should *never* be adding TODO(ios); it's a bad habit left over from early
porting. Put the name of the person who knows the most about this TODO, which is
still justincohen. Please put an actual person below as well.

https://codereview.chromium.org/1123383006/diff/40001/ios/chrome/browser/find...
ios/chrome/browser/find_in_page/resources/find_in_page.js:867: // TODO (ios):
handle scrolling within the DOM.
TODO(

[sdefresne, 2015-05-20 08:05:32]

Thank you for the review.

https://codereview.chromium.org/1123383006/diff/40001/ios/chrome/browser/find...
File ios/chrome/browser/find_in_page/resources/find_in_page.js (right):

https://codereview.chromium.org/1123383006/diff/40001/ios/chrome/browser/find...
ios/chrome/browser/find_in_page/resources/find_in_page.js:218: // TODO(ios): It
would be better if this DCHECKed.
On 2015/05/19 18:26:47, stuartmorgan wrote:
> We should *never* be adding TODO(ios); it's a bad habit left over from early
> porting. Put the name of the person who knows the most about this TODO, which
is
> still justincohen. Please put an actual person below as well.

Acknowledged.

https://codereview.chromium.org/1123383006/diff/40001/ios/chrome/browser/find...
ios/chrome/browser/find_in_page/resources/find_in_page.js:867: // TODO (ios):
handle scrolling within the DOM.
On 2015/05/19 18:26:47, stuartmorgan wrote:
> TODO(

Acknowledged.

[sdefresne, 2015-05-20 08:05:38]

The CQ bit was checked by sdefresne@chromium.org

[sdefresne, 2015-05-20 08:05:38]

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

[commit-bot: I haz the power, 2015-05-20 08:05:51]

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

[commit-bot: I haz the power, 2015-05-20 08:14:09]

Committed patchset #4 (id:60001)

[commit-bot: I haz the power, 2015-05-20 08:14:56]

Patchset 4 (id:??) landed as
https://crrev.com/a0933169507826cbbad85333d70312b0e02d3d45
Cr-Commit-Position: refs/heads/master@{#330714}