DevTools: [JSDoc] Avoid partial arg list annotations in code except "profiler" module

This patch also fixes some ?Event -> !Event cases.

R=vsevik@chromium.org, lushnikov, vsevik

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

git-svn-id: svn://svn.chromium.org/blink/trunk@175278 bbb929c8-8fbe-4397-9dbb-9b2b20218538
Removed v8::Isolate::GetCurrent() and fixed toV8 with empty context.

BUG=369451

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

git-svn-id: svn://svn.chromium.org/blink/trunk@175279 bbb929c8-8fbe-4397-9dbb-9b2b20218538
DevTools: [JsDocValidator] Check that param lists are completely annotated

Having the @param annotation for only a part of function's params is both
confusing (makes people think the function is properly annotated) and
error-prone (missing a @param can result in subtle coding errors).

R=sergeyv
NOTRY=true

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

git-svn-id: svn://svn.chromium.org/blink/trunk@175280 bbb929c8-8fbe-4397-9dbb-9b2b20218538
Adds WebWorker support to NetInfo v3.

Design doc: https://docs.google.com/a/chromium.org/document/d/1LTk9uVMGi4kurzcF5ellsAJReTF31fFJMHrQwSVtBjc/

BUG=368358

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

git-svn-id: svn://svn.chromium.org/blink/trunk@175281 bbb929c8-8fbe-4397-9dbb-9b2b20218538
Move NoEventDispatchAssertion class to its own file

Move NoEventDispatchAssertion class to its own file instead of having it in
ContainerNode.h.

R=eseidel@chromium.org

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

git-svn-id: svn://svn.chromium.org/blink/trunk@175282 bbb929c8-8fbe-4397-9dbb-9b2b20218538
Refactor ThreadableLoaderOptions for readability

ThreadableLoaderOptions shouldn't derive from ResourceLoaderOptions.
ResourceLoaderOptions members are basically just passed through to
FetchRequest while items added by ThreadableLoaderOptions definition
are configuring how CORS, etc. are handled in ThreadableLoader.
Inheritance looks making things less readable to me.

Also, I'd like to make DocumentThreadbleLoader more readable by making
constant variables held by const members.

Items in ThreadableLoaderOptions that determines ThreadableLoader's
behavior could be held by DocumentThreadableLoader as a const member.
Items defined in ResourceLoaderOptions can be altered inside
DocumentThreadableLoader, but can still be held as a const member by
having members to hold overridden items.

BUG=377541

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

git-svn-id: svn://svn.chromium.org/blink/trunk@175283 bbb929c8-8fbe-4397-9dbb-9b2b20218538
DevTools: added support for image url & node in tracing-based timeline

- add trace event for RenderImage::paintIntoRect;
- fix name of decode & resize image events for tracing vs. old timeline;
- add processing of decode/resize/paint/imageref events to TimelineTraceEventBindings;
- generalize handling of additional event properties when building details, so that
    we avoid extra switches over event type by extracting properties relevant for
    details into TraceEvent properties;
- temporary workaround for crash on record stop in tests by cloning an AtomicString
    (see crbug.com/375242 for more details).

BUG=361045,375242

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

git-svn-id: svn://svn.chromium.org/blink/trunk@175284 bbb929c8-8fbe-4397-9dbb-9b2b20218538
Convert SVGSMILElement::m_interval{Begin,End} into a SMILInterval

This is a mostly mechanical change that makes use of the SMILInterval
type added in https://codereview.chromium.org/298043005.

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

git-svn-id: svn://svn.chromium.org/blink/trunk@175285 bbb929c8-8fbe-4397-9dbb-9b2b20218538
DevTools: [CodeMirror] show native copy/paste/cut items in editor context menu

Currently, if you select some text in text editor and then do a right-click
inside selection, the just-summoned context menu won't have a copy/paste/cut
items, as well as other native items natural for text area context menu.

This happens due to the way CodeMirror handles contextmenu event. It
focuses a small text area during the contextmenu bubbling phase, so the
upcoming ContextMenu gets populated with appropriate items. Unfortunately,
codemirror does all this textarea mess *after* firing its own "contextmenu"
event, and as we create menu in the listener, it doesn't have correct active
element.

This patch goes back to listening contextmenu event on element, which due to
bubbling nature happens after textarea was focused.

R=vsevik

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

git-svn-id: svn://svn.chromium.org/blink/trunk@175286 bbb929c8-8fbe-4397-9dbb-9b2b20218538
Update http/resources/teshharnessreport.js to match CL 109763007.

testharness.js is used for http tests but the changes made to the report
file were not done to the one in the http/resources/ directory.

BUG=268711

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

git-svn-id: svn://svn.chromium.org/blink/trunk@175288 bbb929c8-8fbe-4397-9dbb-9b2b20218538
Add initial implementation of TimelineModel.Record based on trace event

BUG=361045
R=alph@chromium.org

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

git-svn-id: svn://svn.chromium.org/blink/trunk@175289 bbb929c8-8fbe-4397-9dbb-9b2b20218538
Remove ScriptState::current from modules/serviceworker/

ScriptState::current should be called only in the binding layer.

BUG=357144

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

git-svn-id: svn://svn.chromium.org/blink/trunk@175290 bbb929c8-8fbe-4397-9dbb-9b2b20218538
Rename repaint to invalidation Part 1

Doing the rename from repaint to invalidation in one CL proved to be unwieldy.
This CL is the first step at the renaming. This moves a lot of pices that have
been added due to repaint-after-layout (although I'm leaving the flag named
as is to avoid Chromium side changes).

BUG=338691

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

git-svn-id: svn://svn.chromium.org/blink/trunk@175291 bbb929c8-8fbe-4397-9dbb-9b2b20218538
Remove isolate->GetCurrentContext from V8ErrorHandler

- This CL replaces isolate->GetCurrentContext in V8ErrorHandler::callListenerFunction with scriptState()->context(). This doesn't change any behavior because we've already entered the ScriptState before calling V8ErrorHandler::callListenerFunction.

- This CL drops an ExecutionContext* parameter from callListenerFunction because it's unused.

BUG=357144

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

git-svn-id: svn://svn.chromium.org/blink/trunk@175292 bbb929c8-8fbe-4397-9dbb-9b2b20218538
Fix a srcset fractional density bug

Since https://codereview.chromium.org/293423002/ fractional density descriptors
(e.g. "1.5x.gif 1.5x") get rounded down to an int for intrinsic size calculations.

This CL fixes that, and adds tests to prevent such regressions in the future.

BUG=379651

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

git-svn-id: svn://svn.chromium.org/blink/trunk@175293 bbb929c8-8fbe-4397-9dbb-9b2b20218538
Make sure we never pass a null Frame to Document::canNavigate()

Make sure we never pass a null Frame to Document::canNavigate() and update it
to take the Frame by reference to const. It makes little sense to pass null to
a canNavigate() function, especially considering the implementation was
returning true in this case (there was a FIXME about it).

R=adamk@chromium.org, eseidel@chromium.org

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

git-svn-id: svn://svn.chromium.org/blink/trunk@175295 bbb929c8-8fbe-4397-9dbb-9b2b20218538
Move bindings_modules_generated from modules to bindings/modules

There's a missing dependency b/c core_generated (webcore_generated) depends on modules_generated, due to partial interface definitions in modules extending classes in core.
This will require significant work to fix (changing how we handle bindings for partial interface definitions), so for now move these files to bindings_modules_generated.
This means we have a core -> bindings_modules dependency (which is allowed), instead of a core -> modules dependency (which is not allowed).

This CL is a work-around until we split the files in bindings for core and modules eventually.

BUG=378808

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

git-svn-id: svn://svn.chromium.org/blink/trunk@175296 bbb929c8-8fbe-4397-9dbb-9b2b20218538
Remove RenderEmbeddedObject::layerTypeRequired().

This method does not seem to do anything useful. It overrides
RenderPart::layerTypeRequired(), which does exactly the same thing.

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

git-svn-id: svn://svn.chromium.org/blink/trunk@175297 bbb929c8-8fbe-4397-9dbb-9b2b20218538
Drop unnecessary NodeWithIndex::m_haveIndex data member

Drop unnecessary NodeWithIndex::m_haveIndex data member. Since
Node::nodeIndex() returns an unsigned integer but NodeWithIndex::m_index is
signed, we can simply set m_index to -1 until it is lazily set.

R=eseidel@chromium.org

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

git-svn-id: svn://svn.chromium.org/blink/trunk@175299 bbb929c8-8fbe-4397-9dbb-9b2b20218538
DevTools: [CSS] implement hidden CSS.getMediaQueries method in CSS domain

This patch implements CSS.getMediaQueries method in CSS domain for the
needs of responsive design.

R=pfeldman

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

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