Gardening: Widen flaky expectations for timeline-event-dispatch.html

BUG=407915
TBR=eustas@chromium.org

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

git-svn-id: svn://svn.chromium.org/blink/trunk@183155 bbb929c8-8fbe-4397-9dbb-9b2b20218538
Gardening: Widen flaky expectation for fast/text/aat-morx.html

BUG=418091
TBR=eustas@chromium.org,eae@chromium.org

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

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

Since the rest of the OS X drawing code moved to FontHarfBuzz
and mostly the Skia based code, we can remove the special cased
metrics callback implementation here.

BUG=419386
R=eae, behdad

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

git-svn-id: svn://svn.chromium.org/blink/trunk@183157 bbb929c8-8fbe-4397-9dbb-9b2b20218538
Oilpan: reliably track listening PlatformEventControllers.

Turn PlatformEventController into a GC mixin with Oilpan enabled, and
have the PlatformEventDispatcher instance track those via a Persistent
reference.

It seems precarious to rely on the controller lifetimes be maintained by
way of other references 'long enough' for the embedder's platform event
handling layer.

R=mlamouri,mkwst,haraken,timvolodine
BUG=419515

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

git-svn-id: svn://svn.chromium.org/blink/trunk@183158 bbb929c8-8fbe-4397-9dbb-9b2b20218538
Modify ALLOW_UNUSED to make it effective on MSVC as well.

This requires a syntax change.  After feedback from various people, it seems
like perhaps the least offensive transformation is

int x ALLOW_UNUSED = 0;

-->

int x = 0;
ALLOW_UNUSED_LOCAL(x);

The name changed for a couple of reasons:
(1) Various files #include both the wtf headers and the Chromium base headers
which both define this macro.  To avoid compile breaks when changing one
definition before the other, either both places have to gain #ifndef guards, or
the names have to (at least temporarily) be changed to not match.
(2) On the Chromium side, the macro is going to be split into ALLOW_UNUSED_LOCAL
and ALLOW_UNUSED_TYPE, with the latter used for e.g. possibly-unused typedefs.
These must be annotated on gcc but not on MSVC, and can't use the same
implementation as ALLOW_UNUSED_LOCAL() anyway, so ALLOW_UNUSED_TYPE will retain
the attribute-based implementation.  This is a long-winded way of saying that
the name change here is chosen to match an eventual name change there.

BUG=81439
TEST=none

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

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