DevTools: [JsDocValidator] Avoid false-positive function receiver-related errors

The RequiredThisAnnotationChecker has been merged into FunctionReceiverChecker,
so it can now make use of the knowledge whether a nested function
references |this| or not. As such, the receiver checks now rely
on the fact of the |this| usage rather than the presence of the @this annotation.

R=aandrey@chromium.org, aandrey, sergeyv, vsevik

Review URL: https://codereview.chromium.org/199733011

git-svn-id: svn://svn.chromium.org/blink/trunk@169643 bbb929c8-8fbe-4397-9dbb-9b2b20218538
Move element flags from ElementRareData to NodeRareData

By doing this, they can be packed together with the existing bit-field
member NodeRareData::m_connectedFrameCount, and save some space.

Impact according to sizeof(ElementRareData), on 64-bit Linux (GCC and
Clang alike,) is a modest 8 byte saving, from 144 bytes to 136 bytes.

BUG=

Review URL: https://codereview.chromium.org/201153006

git-svn-id: svn://svn.chromium.org/blink/trunk@169644 bbb929c8-8fbe-4397-9dbb-9b2b20218538
Use monotonic time for SVGImage's serviceScriptedAnimations()

Breakage from r168748 which used the wrong time-source.

BUG=231576

Review URL: https://codereview.chromium.org/204673004

git-svn-id: svn://svn.chromium.org/blink/trunk@169645 bbb929c8-8fbe-4397-9dbb-9b2b20218538
[DevTools] Remove unnecessary scrollbar in settings.

BUG=353961
TBR=vsevik@chromium.org

Review URL: https://codereview.chromium.org/206373002

git-svn-id: svn://svn.chromium.org/blink/trunk@169646 bbb929c8-8fbe-4397-9dbb-9b2b20218538
DevTools: Remove unneeded willReceiveResourceData instrumentation and reuse didReceiveData instead of didReceiveResourceData.

R=eustas@chromium.org, pfeldman@chromium.org, caseq, pfeldman

Review URL: https://codereview.chromium.org/108103002

git-svn-id: svn://svn.chromium.org/blink/trunk@169647 bbb929c8-8fbe-4397-9dbb-9b2b20218538
Remove 3-args |SVGAnimatedLength::baseValueToString()| impl.

This CL is to prepare for auto-dispatching baseValueToString in SVG*Element::parseAttribute.
SVGAnimatedLength required {Forbid,Allow}NegativeValues setting to be passed in as an argument of baseValueToString.
This was making auto-dispatch for baseValueToString calls difficult.

This CL removes the SVGLengthNegativeValuesMode argument from |SVGAnimatedLength::baseValueToString()|. The setting is passed in by |SVGAnimatedLength| ctor.

BUG=349370

Review URL: https://codereview.chromium.org/206133005

git-svn-id: svn://svn.chromium.org/blink/trunk@169648 bbb929c8-8fbe-4397-9dbb-9b2b20218538