DevTools: fix jsdoc validator to recognize `this' usage within arrow functions

DevTools: fix jsdoc validator to recognize `this' usage within arrow functions

Arrow functions implicitly bind to this, but the validator was ignoring the
fact and warned on unused this in the outer function when its nested arrow
function was actually binding to this.

Committed: https://crrev.com/1b47d6a1606ab62ce03a09a4f7746d16900edd87
Cr-Commit-Position: refs/heads/master@{#427435}

[caseq, 2016-10-25 18:00:36]

caseq@chromium.org changed reviewers:
+ dgozman@chromium.org, lushnikov@chromium.org

[dgozman, 2016-10-25 18:08:27]

lgtm

https://codereview.chromium.org/2449173002/diff/1/third_party/WebKit/Source/d...
File third_party/WebKit/Source/devtools/scripts/jsdoc-validator/tests/this.js
(right):

https://codereview.chromium.org/2449173002/diff/1/third_party/WebKit/Source/d...
third_party/WebKit/Source/devtools/scripts/jsdoc-validator/tests/this.js:302: *
@this {!C}
@this annotations don't need "!"

[caseq, 2016-10-25 18:12:30]

The CQ bit was checked by caseq@chromium.org

[caseq, 2016-10-25 18:12:31]

The patchset sent to the CQ was uploaded after l-g-t-m from dgozman@chromium.org
Link to the patchset: https://codereview.chromium.org/2449173002/#ps20001
(title: "review comments + moar tests")

[commit-bot: I haz the power, 2016-10-25 18:13:26]

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-25 19:08:10]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-10-25 19:35:48]

Description was changed from

==========
DevTools: fix jsdoc validator to recognize `this' usage within arrow functions

Arrow functions implicitly bind to this, but the validator was ignoring the
fact and warned on unused this in the outer function when its nested arrow
function was actually binding to this.
==========

to

==========
DevTools: fix jsdoc validator to recognize `this' usage within arrow functions

Arrow functions implicitly bind to this, but the validator was ignoring the
fact and warned on unused this in the outer function when its nested arrow
function was actually binding to this.

Committed: https://crrev.com/1b47d6a1606ab62ce03a09a4f7746d16900edd87
Cr-Commit-Position: refs/heads/master@{#427435}
==========

[commit-bot: I haz the power, 2016-10-25 19:35:49]

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