Revert "Change fontContainsCharacter to use SkTypeface::charsToGlyphs"

r160727 ended up causing a 14% performance regression on the html5 spec
test case on windows 7. As that change was intended to improve
performance by using a lighter weight API it clearly failed.

This change reverts r160727.

BUG=312945
R=tonyg@chromium.org

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

git-svn-id: svn://svn.chromium.org/blink/trunk@160967 bbb929c8-8fbe-4397-9dbb-9b2b20218538
Get rid of the virtual/crash test suite.

Making this virtual without a command-line flag doesn't actually
enforce that these tests will get run in their own process.
Having them in the "crash" directory is enough to make it clear
that these tests are expected to crash.

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

git-svn-id: svn://svn.chromium.org/blink/trunk@160968 bbb929c8-8fbe-4397-9dbb-9b2b20218538
Move RefCountedSupplement to platform/

BUG=297477

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

git-svn-id: svn://svn.chromium.org/blink/trunk@160969 bbb929c8-8fbe-4397-9dbb-9b2b20218538
Use a TextTrackLoaderClient reference (not pointer)

It's never null, so remove all doubt for future readers of the code.

BUG=313287

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

git-svn-id: svn://svn.chromium.org/blink/trunk@160970 bbb929c8-8fbe-4397-9dbb-9b2b20218538
Simplify HTMLTrackElement::didCompleteLoad

It doesn't need to be virtual and didn't use the first argument.

BUG=313287

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

git-svn-id: svn://svn.chromium.org/blink/trunk@160971 bbb929c8-8fbe-4397-9dbb-9b2b20218538
Have HTMLFrameOwnerElement / SubframeLoadingDisabler deal with references

Have HTMLFrameOwnerElement / SubframeLoadingDisabler deal with references
instead of pointers when possible (i.e. when the pointers are expected to
be non-null).

R=adamk

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

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