Left arrow on NTP is corrupted

Now that detecting meta@charset doesn't block processing the resource, we need to be more careful about how we prioritize encodings from meta@charset relative to other mechanisms. Specifically, we meta@charset is supposed to be less important than autodetection from the "<?xml" preamble but more important than encodings detected from sniffing the content itself.

BUG=244049
NOTRY=true

Review URL: https://chromiumcodereview.appspot.com/16113002

git-svn-id: svn://svn.chromium.org/blink/trunk@151300 bbb929c8-8fbe-4397-9dbb-9b2b20218538
Switch box-shadow code to use WebCore::DrawLooper directly, allowing it to
draw shadows without having to draw the background outside the clip as well.

There is a modest (record-time, in the case of impl-side painting) performance
benefit to doing this.

BUG=244419
NOTRY=true

Review URL: https://chromiumcodereview.appspot.com/15914006

git-svn-id: svn://svn.chromium.org/blink/trunk@151303 bbb929c8-8fbe-4397-9dbb-9b2b20218538
Add wtf_unittests target that links unit tests outside DLL

This moves the wtf unittests out of the webkit_unit_tests target and into
a dedicated gtest executable. This target builds and links much faster since
it's tiny and moves the tests outside the DLL. This target is in all_webkit,
but annoyingly we'll have to configure the bots to run this step.

R=abarth@chromium.org, darin@chromium.org

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

git-svn-id: svn://svn.chromium.org/blink/trunk@151305 bbb929c8-8fbe-4397-9dbb-9b2b20218538
Simplify MediaPlayer plumbing and state

This patch moves most MediaPlayerClient callback invocations into
WebMediaPlayerClientImpl, removing boilerplate code in MediaPlayer.
The only remaining callbacks invoked from MediaPlayer are those which
store state in MediaPlayer.

As part of the above, removed unused state from MediaPlayer and clarified
memory ownership model. In particular, MediaPlayer (and
WebMediaPlayerClientImpl) now assume that MediaPlayer::m_mediaPlayerClient
cannot be NULL, and MediaPlayer asserts such in its constructor.

Other cleanup also included in this patch:
- Removed unused MediaPlayer* argument from MediaPlayerClient callbacks.
- Removed various unused methods of MediaPlayer.
- Remove an MediaPlayerClient -> HTMLMediaElement cast.
- Fix lifetime management of MediaPlayerPrivateInterface to avoid calling
back into a destroyed MediaPlayer.

R=abarth@chromium.org, acolwell@chromium.org
BUG=243568

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

git-svn-id: svn://svn.chromium.org/blink/trunk@151306 bbb929c8-8fbe-4397-9dbb-9b2b20218538
Don't stretch margin: auto items in a vertical flexbox

Even with align-self: stretch (the default), we shouldn't stretch items
that have margin: auto;

BUG=242654
R=ojan@chromium.org

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

git-svn-id: svn://svn.chromium.org/blink/trunk@151314 bbb929c8-8fbe-4397-9dbb-9b2b20218538
Apply binding integrity to several remaining items.

BUG=236671
R=abarth@chromium.org

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

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