IDL compiler: [ActivityLogging] for methods

BUG=239771
NOTRY=true

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

git-svn-id: svn://svn.chromium.org/blink/trunk@161311 bbb929c8-8fbe-4397-9dbb-9b2b20218538
Clicking a non-focusable element in an inactive frame should clear focus in the frame.

Clicking a non-focusable element in an inactive frame did not remove focus
from the focused element in the frame because
 - FocusController::setFocusedElement incorrectly returned earlier if there
    was no focused element in the currently focused frame, and
 - FocusController::setFocusedElement cleared focus in the currently
    focused frame if new focused element is 0, but did not clear the focus in
    the new focused frame.

We shouldn't do early-return for a element==0 case. Also, we need to to
call clearSelectionIfNeeded() later than same-element check to avoid test
breakage.


BUG=304145
TEST=automated.

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

git-svn-id: svn://svn.chromium.org/blink/trunk@161312 bbb929c8-8fbe-4397-9dbb-9b2b20218538
Inline hot ElementData functions.

This is a merge of http://trac.webkit.org/changeset/156227 by
Andreas Kling <akling@apple.com>.

This patch inlines the following functions:
ElementData::deref
UniqueElementData::addAttribute
UniqueElementData::removeAttribute
UniqueElementData::attributeItem

In inlining these functions, two ASSERT_WITH_SECURITY_IMPLs have
been removed in attributeAt and removeAttribute. These functions
are accessing/removing items on Vector, we already have a
RELEASE_ASSERT for them in Vector.h.

This results in a 1.1% performance increase on
BlinkPerfMeasurement:Parser_html5-full-render

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

git-svn-id: svn://svn.chromium.org/blink/trunk@161313 bbb929c8-8fbe-4397-9dbb-9b2b20218538
PseudoElement::setNeedsStyleRecalc() should compute a new style

We need to clear the cached pseudo style whenever a PseudoElement specifically
requests a new style otherwise PseudoElement::customStyleForRenderer() will just
get the cached (old) style off the parent.

BUG=314968

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

git-svn-id: svn://svn.chromium.org/blink/trunk@161314 bbb929c8-8fbe-4397-9dbb-9b2b20218538
Move CustomFilterTransformParameter from core/platform/... to platform/...

BUG=303332

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

git-svn-id: svn://svn.chromium.org/blink/trunk@161315 bbb929c8-8fbe-4397-9dbb-9b2b20218538
Make run-bindings-test reset results for EventInterfaces.in quiet too

Missed one logging message ('Reset results: EventInterfaces.in')
in previous CL; this fixes it and makes it quiet by default.

Followup to:
Make run-bindings-tests quiet by default
https://codereview.chromium.org/55673002/

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

git-svn-id: svn://svn.chromium.org/blink/trunk@161316 bbb929c8-8fbe-4397-9dbb-9b2b20218538
IDL compiler: [CallWith] for methods (void, no optional arguments)

This provides basic [CallWith] support, with some refactoring.
(Shared attribute/method code is now v. small, since most isn't
 relevant for both.)

non-void methods and optional arguments both require significant
additional work, so will do in followup CLs.

BUG=239771
NOTRY=true

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

git-svn-id: svn://svn.chromium.org/blink/trunk@161317 bbb929c8-8fbe-4397-9dbb-9b2b20218538
NewWebSocketChannel should send remaining messages on quota update.

When the quota for sending messages is updated and there are
messages to be sent, NewWebSocketChannel should send them.

BUG=27459
R=tyoshino

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

git-svn-id: svn://svn.chromium.org/blink/trunk@161318 bbb929c8-8fbe-4397-9dbb-9b2b20218538
IDL compiler: [CheckSecurityForNode] for methods

BUG=239771
NOTRY=true

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

git-svn-id: svn://svn.chromium.org/blink/trunk@161319 bbb929c8-8fbe-4397-9dbb-9b2b20218538
Make StyleSharingList store raw ptrs

The list only makes sense in the inner block of Document::recalcStyle
and no DOM mutations can occur inside there so it's safe to store raw
pointers instead of RefPtrs.

I tried this before and it was reverted because we call into
styleForElement outside recalcStyle in some cases and if you did that
and then removed the Element from the page it'd leave a stale pointer
in the list.

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

git-svn-id: svn://svn.chromium.org/blink/trunk@161322 bbb929c8-8fbe-4397-9dbb-9b2b20218538
DevTools: Send returning value of a call frame.

Send the returning value to front-end if a call frame is at the return point.

BUG=155730
R=yurys@chromium.org, pfeldman@chromium.org

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

git-svn-id: svn://svn.chromium.org/blink/trunk@161323 bbb929c8-8fbe-4397-9dbb-9b2b20218538
DevTools: Totally avoid traversing CSSStyleSheet while inside RuleMutationScope

This resulted in various inspected page crashes during stylesheet text editing.

R=pfeldman, vsevik
BUG=312394

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

git-svn-id: svn://svn.chromium.org/blink/trunk@161324 bbb929c8-8fbe-4397-9dbb-9b2b20218538
Post cleanup after deprecating WorkerAllowMainThreadBridgeBase

This needs to land after chromium-side patch
(https://codereview.chromium.org/46583005/) lands.

BUG=none

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

git-svn-id: svn://svn.chromium.org/blink/trunk@161325 bbb929c8-8fbe-4397-9dbb-9b2b20218538
DevTools: Show displayName in popover while paused.

Remove name, inferredName and displayName from Debugger.FunctionDetails struct in favor of functionName calculated in the backend.

BUG=17356
R=yurys@chromium.org

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

git-svn-id: svn://svn.chromium.org/blink/trunk@161326 bbb929c8-8fbe-4397-9dbb-9b2b20218538
Add additional information to FAIL lines for animation interpolation tests

This change adds to the FAIL line the parsed value of the expected style, which
can help with debugging. For example ...

FAIL: border-image-width from [0] to [20] was [0] at -0.3, expected [0]

becomes ...

FAIL: border-image-width from [0] to [20] was [0] at -0.3, expected [0] (parsed as [0px])

R=dstockwell@chromium.org

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

git-svn-id: svn://svn.chromium.org/blink/trunk@161327 bbb929c8-8fbe-4397-9dbb-9b2b20218538
Add rtoy to watchlist for audio changes

R=rtoy@chromium.org,kbr@chromium.org
BUG=none

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

git-svn-id: svn://svn.chromium.org/blink/trunk@161328 bbb929c8-8fbe-4397-9dbb-9b2b20218538
<animate> not working on SVG Polygons.

The animation is taking place, but the rendering side didn't use the
computed values.

R=
BUG=196435

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

git-svn-id: svn://svn.chromium.org/blink/trunk@161329 bbb929c8-8fbe-4397-9dbb-9b2b20218538
Remove isActiveInsertionPoint check from recalcStyle

NodeRenderingTraversal::parent returns null for active insertion points
after r161137 which means parentRenderStyle() would be null too so
there's no reason to have this check.

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

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