Cleanup now that WebHelperPlugin no longer needs to be a WebWidget.

BUG=346543
R=abarth@chromium.org, ddorwin@chromium.org

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

git-svn-id: svn://svn.chromium.org/blink/trunk@167936 bbb929c8-8fbe-4397-9dbb-9b2b20218538
Revert 167917 "Change FontFallbackWin to use FamilyNameIterator"

May have caused many crashes on Win layout test bots.

> Change FontFallbackWin to use FamilyNameIterator
> 
> Change the windows font fallback code to use SkTypeface and
> createFamilyNameIterator to check whether a font exists instead of
> calling GDI directly. This removes the last use of HFONT for DW builds
> and supports fonts with locale dependent names.
> 
> BUG=251355
> R=bungeman@chromium.org
> 
> Review URL: https://codereview.chromium.org/180143002

TBR=eae@chromium.org

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

git-svn-id: svn://svn.chromium.org/blink/trunk@167937 bbb929c8-8fbe-4397-9dbb-9b2b20218538
Have Document::timing() / Document::contextFeatures() return a reference

Have Document::timing() / Document::contextFeatures() return a reference
instead of a pointer as they can never return null. This makes the calling code
looks safer.

R=adamk

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

git-svn-id: svn://svn.chromium.org/blink/trunk@167938 bbb929c8-8fbe-4397-9dbb-9b2b20218538
Mark a few events as used from multiple threads.

R=haraken@chromium.org, vegorov@chromium.org
BUG=

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

git-svn-id: svn://svn.chromium.org/blink/trunk@167939 bbb929c8-8fbe-4397-9dbb-9b2b20218538
Have Document::timeline() / Document::transitionTimeline() return a reference

Have Document::timeline() / Document::transitionTimeline() return a reference
instead of a pointer as they can never return null. This makes the calling code
look safer and gets rid of a few unnecessary null checks.

R=adamk

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

git-svn-id: svn://svn.chromium.org/blink/trunk@167940 bbb929c8-8fbe-4397-9dbb-9b2b20218538
Fix ASSERT failure in RenderGeomtryMap for SVGs

SVGRenderSupport::mapLocalToContainer() was trying to apply transforms
in the incorrect order. Specifically, it would attempt to apply its
localToParentTransform before its localToBorderBoxTransform. This
was causing an ASSERT to fail when the computed transforms didn't
match up to those computed by RenderGeometryMap.

BUG=302719

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

git-svn-id: svn://svn.chromium.org/blink/trunk@167941 bbb929c8-8fbe-4397-9dbb-9b2b20218538
Avoid using transparency layer for fancy borders.

We paint transparent borders with radius on an offscreen layer. This is unnecessary and expensive especially for gpu rasterization. This patch simply removes the save-restore layer steps.

The border edges are now painted slightly differently. It is not wrong - just different. A few borders have color values that are off by one integral value, which was unexpected. This is a skia bug - skbug.com/2211.

BUG=343188

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

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