Remove bogus ASSERT in EventHandler::handleGestureEvent

I wrote this in r177164 thinking just that I wanted to verify the hit
test traversed all iframe boundaries, but of course there are other
widgets that aren't iframes (eg. plugins).

I tried changing the ASSERT to verify explicitly that the target
element is not an HTMLIFrameElement, but this doesn't work
either because touch adjustment may adjust the target to be over
an iframe element without necessarily hit-testing down into the
iframe.  This seems like we might want to change this to hit-test
down through the frame after adjusting to it, but the behavior has
been like this for a long time and we haven't seen an issue.

So just remove the ASSERT completely.  I don't think there's
anything meaningful to verify here.

TBRing to get the bots green ASAP and since the fix is trivial.

TBR=zeeshanq
BUG=390032

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

git-svn-id: svn://svn.chromium.org/blink/trunk@177176 bbb929c8-8fbe-4397-9dbb-9b2b20218538
Move the table death tests to core/rendering/

Those tests were added to web/tests as this was the only way
to test them at the time. They can now use DummyPageHolder and
be moved to core/rendering where they belong.

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

git-svn-id: svn://svn.chromium.org/blink/trunk@177177 bbb929c8-8fbe-4397-9dbb-9b2b20218538
Avoid leaking a shared worker global scope on a failed script load.

The spec tells us,

 http://whatwg.org/specs/web-apps/current-work/#run-a-worker

that in case of failure to load the script of a (shared) worker, we
should report the failure and abort (step 4.) The global scope's
closing flag will be left as false, meaning that we can safely shut
down and destroy the shared worker global scope. Do so.

R=
BUG=364390

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

git-svn-id: svn://svn.chromium.org/blink/trunk@177178 bbb929c8-8fbe-4397-9dbb-9b2b20218538
Move implementation of WebFrameImpl::executeCommand to Editor

This patch moves |WebFrameImpl::executeComamnd()| to |Editor| class to make
"web" as thin API layer and ease of maintenance.

This patch also get rid of |WebViewImpl::bubblingScroll|, which
becomes unused function as result of moving |WebFrameImpl::executeCommand()|.

BUG=n/a
TEST=n/a; no behavior changes

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

git-svn-id: svn://svn.chromium.org/blink/trunk@177179 bbb929c8-8fbe-4397-9dbb-9b2b20218538
Simplify InlineFlowBox::paintBoxDecorations.

This change:
* simplifies the paintPhase checks in InlineFlowBox::paint
* clarifies and moves the "should paint box decorations" check above
  other work, including the roundedFrameRect computation
* makes some internal painting methods private
* fixes the name of "adjustedPaintOffset"
* resolves lint errors that arose during the above

3% reduction in record time on desktop Wikipedia.
Negligible impact on N7 key_silk_cases.

BUG=388946

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

git-svn-id: svn://svn.chromium.org/blink/trunk@177180 bbb929c8-8fbe-4397-9dbb-9b2b20218538
LeakExpectations: add more service worker tests

TBR=haraken
BUG=364377
NOTRY=true

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

git-svn-id: svn://svn.chromium.org/blink/trunk@177181 bbb929c8-8fbe-4397-9dbb-9b2b20218538
Refactor SelectorChecker::BehaviorAtBoundary

As of Blink r176304, SelectorCheckingContext::scope
for styles in shadow tree changed from its shadow host to
shadow root.

Thus the subtlety of scoping difference between TreeScope
and SelectorCheckingContext::scope is gone, and code for
handling the difference can be cleaned up.

This made possible by dropping support for <style scoped>,
which introduced the complexity at r175993.

With this change no functional difference is expected,
thus all layout tests should pass.

- Convert SelectorChecker::parentElement(context) to static helper function.
- Convert SelectorChecker::scopeContainsLastMatchedElement() to static
  helper function.
- s/{b,B}ehaviorAtBounary/{c,C}ontextFlags/g
- Removed the 2nd parameter in rulesApplicableInCurrentTreeScope()
- Removed the 2nd argument for isHostInItsShadowTree()
- Removed StaysWithinTreeScope
- Removed DoesNotCrossBoundary
- Removed ScopeIsShadowRoot.
- Removed BoundaryBehaviorMask
- Removed ScopeIsShadowHostInPseudoHostParameter
- Introduced DefaultBehavior (to be removed, though)

BUG=none
TEST=pass all layout tests.

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

git-svn-id: svn://svn.chromium.org/blink/trunk@177183 bbb929c8-8fbe-4397-9dbb-9b2b20218538
Remove duplicated provideMediaKeysTo() in MediaKeysClient.h

Though MediaKeysController has already provideMediaKeysTo() which is being used by WebViewImpl, MediaKeysClient.h also has provideMediaKeysTo() unnecessary.

BUG=none

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

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

We never use the system malloc anymore.

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

git-svn-id: svn://svn.chromium.org/blink/trunk@177185 bbb929c8-8fbe-4397-9dbb-9b2b20218538
Web Animations API: Add layout tests for element.animate() keyframes

Adds layout tests for keyframes with the following configurations:
 - 3 keyframes with and without offsets
 - Keyframes offsets specified out of order
 - Explicit null offset value
 - Shared offset keyframes with overlapping/disjoint properties
 - Invalid keyframe offsets
 - Per keyframe easing

Fixes flaky test issues due to bug 367903 by running the tests
inside requestAnimationFrame().

BUG=367644

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

git-svn-id: svn://svn.chromium.org/blink/trunk@177186 bbb929c8-8fbe-4397-9dbb-9b2b20218538
Eliminate preprocessor usage in html.css.

We move the content of |#if ENABLE_INPUT_MULTIPLE_FIELDS_UI| to
themeInputMultipleFields.css.

We can remove |--define| and |--preprocessor| options of
make-css-file-arrays.pl, and remove preprocessor.pm.

BUG=347863

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

git-svn-id: svn://svn.chromium.org/blink/trunk@177187 bbb929c8-8fbe-4397-9dbb-9b2b20218538
Remove usages of jquery and add sugar.js from garden-o-matic.

Use sugar.js to replace usages of jquery and replace any extraneous
uses as well. Didn't touch the custom event stuff (bind/trigger) or
the tabs UI code. Those will naturally go away as we port things to
polymer. Kept this patch for mechanical simple changes.

The only non-mechanical bits are $.param --> base.queryParam and
fadeIn/fadeOut --> element.animate.

NOTRY=true

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

git-svn-id: svn://svn.chromium.org/blink/trunk@177188 bbb929c8-8fbe-4397-9dbb-9b2b20218538
Polymer-based Garden-O-Matic should have a basic results panel

This CL makes it possible to examine failures on Sheriff-O-Matic (the Polymer
version of Garden-O-Matic). Currently, the results panel just lets you select
which test and builder you're interested in. It's not able to display the
actual test results yet.

NOTRY=true

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

git-svn-id: svn://svn.chromium.org/blink/trunk@177189 bbb929c8-8fbe-4397-9dbb-9b2b20218538
Remove PTHREAD_SETNAME_NP from config.h

This define is only used in one place.

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

git-svn-id: svn://svn.chromium.org/blink/trunk@177190 bbb929c8-8fbe-4397-9dbb-9b2b20218538
Consistently use WTF_ALLOW_INIT_WITH_MEM_FUNCTIONS for ALLOW_ONLY_INLINE_ALLOCATION objects

Currently we mix WTF_ALLOW_INIT_WITH_MEM_FUNCTIONS, SimpleClassVectorTraits and
hand-written vector traits for ALLOW_ONLY_INLINE_ALLOCATION objects.
This CL unifies them into WTF_ALLOW_INIT_WITH_MEM_FUNCTIONS macros.

BUG=340522

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

git-svn-id: svn://svn.chromium.org/blink/trunk@177191 bbb929c8-8fbe-4397-9dbb-9b2b20218538
Block mixed EventSource conections.

Similar to XMLHttpRequest and WebSockets, we should block mixed EventSource
connections by default.

BUG=389209

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

git-svn-id: svn://svn.chromium.org/blink/trunk@177192 bbb929c8-8fbe-4397-9dbb-9b2b20218538
Remove #if ENABLE(OILPAN) around DEFINE_STATIC_LOCAL

This CL removes unnecessary #if ENABLE(OILPAN)s by:

- using OwnPtrWillBePersistent for static on-heap objects
- using DEFIN_STATIC_REF_WILL_BE_PERSISTENT

BUG=340522

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

git-svn-id: svn://svn.chromium.org/blink/trunk@177193 bbb929c8-8fbe-4397-9dbb-9b2b20218538
Revert of Adds AL logging for Blink core events. (https://codereview.chromium.org/349263002/)

Reason for revert:
This is blocking the blink roll (https://codereview.chromium.org/346333005/) as it makes the test AdInjectionBrowserTest.DetectAdInjections in browser_tests fail

Original issue's description:
> Enables Activity Logger to log extension adding the following elements to the DOM:
> - script
> - link
> - anchor
> - iframe
> - form
> - input
> - button
> 
> BUG=388287
> 
> Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=177105

TBR=mvrable@chromium.org,haraken@chromium.org,pfeldman@chromium.org,pmarch@chromium.org
NOTREECHECKS=true
NOTRY=true
BUG=388287

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

git-svn-id: svn://svn.chromium.org/blink/trunk@177194 bbb929c8-8fbe-4397-9dbb-9b2b20218538
Disallow an odd number of strings when parsing the quotes property

The spec only allows an even number of strings in the quotes property,
but currently we will accept an odd number of strings as a valid value.

http://www.w3.org/TR/CSS2/generate.html#quotes

BUG=389724

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

git-svn-id: svn://svn.chromium.org/blink/trunk@177195 bbb929c8-8fbe-4397-9dbb-9b2b20218538
Harden a fullscreen transition test

Since the test uses only timeouts and no events, it passes even if the
webkitRequestFullScreen() call fails, as it would e.g. if the
webkitallowfullscreen attribute were simply removed.

BUG=390008

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

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