Don't call cancelAnimations() in RenderObject::willBeDestroyed() when no associate frame

This patch makes RenderObject::willBeDestroyed() not to do null dref for RenderObject::animation() call, which is frame()->animation(), by checking frame() returns non-null.

Note: Another patch will update src/tools/valgrind/tsan/suppressions.txt once this patch rolled into chromium DEPS; https://src.chromium.org/viewvc/chrome?view=rev&revision=225690

BUG=299647
TEST=fullscreen/fullscreen-child-not-allowed-crash.html; valgrind
R=morrita@chromium.org

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

git-svn-id: svn://svn.chromium.org/blink/trunk@159204 bbb929c8-8fbe-4397-9dbb-9b2b20218538
Avoid creating first-letter RenderTextFragments for unsuitable text nodes

First-letter code would formerly find the first RenderText in a RenderBlock
and pass it off to be split into RenderTextFragments. If no suitable letter
was found in that RenderText, we'd use its entire contents (be it whitespace
or punctuation) in the RenderTextFragment. When the RenderText is whitespace
only and was combined with column handling code that splits blocks, this
could lead to creating first-letter renderers out of pre-existing first-
letter renderers, which could cause crashes.

Adding some assertions and changing the search algorithm for the first
text node to also check for a valid first-letter character. This avoids
the crashes and leads to results that are closer to my interpretation of
what the spec suggests (i.e. we won't apply first-letter to only punctuation).

BUG=264574

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

git-svn-id: svn://svn.chromium.org/blink/trunk@159205 bbb929c8-8fbe-4397-9dbb-9b2b20218538
Blink gardening: dynamic-stylesheet-loading.html gets timeout

TBR=enne@chromium.org
BUG=N/A
NOTRY=true

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

git-svn-id: svn://svn.chromium.org/blink/trunk@159206 bbb929c8-8fbe-4397-9dbb-9b2b20218538
Blink gardening: image rebase on Linux

BUG=N/A
TBR=enne@chromium.org
NOTRY=true

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

git-svn-id: svn://svn.chromium.org/blink/trunk@159207 bbb929c8-8fbe-4397-9dbb-9b2b20218538
Make not to crash for mouse down event on pseudo element and removing associate node

This patch adds null check for event target node in shouldRefetchEventTarget(), which called by EventHandler::handleMousePressEvent(), rather than assuming target node isn't null.

In EventHandler::handleMousePressEvent(), we can have null target node when mouse down is on pseudo element, e.g. :before/:after CSS selector, and removing associated node during 'mousedown' handler.

BUG=304175
TEST=LayoutTests/fast/events/mouse-down-on-pseudo-element-remove-crash.html
R=tkent@chromium.org

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

git-svn-id: svn://svn.chromium.org/blink/trunk@159208 bbb929c8-8fbe-4397-9dbb-9b2b20218538
Test patches that rename binary files through the tryservers and CQ 

TBR=iannucci@chromium.org, chrisphan@chromium.org
BUG=23608

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

git-svn-id: svn://svn.chromium.org/blink/trunk@159209 bbb929c8-8fbe-4397-9dbb-9b2b20218538
Clean up EventHandler: slim down public API, remove dead code

R=eseidel

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

git-svn-id: svn://svn.chromium.org/blink/trunk@159210 bbb929c8-8fbe-4397-9dbb-9b2b20218538
Make stroke-dashoffset-interpolation results easier to eyeball

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

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