Revert "Revert "Create TestExpectationLines directly instead of creating a dummy expectations string""

This relands r150926. The early return in base.py was incorrectly checking for
None instead of 'default'. So, I had accidentally made this on by default.

TBR=mkwst@chromium.org

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

git-svn-id: svn://svn.chromium.org/blink/trunk@151010 bbb929c8-8fbe-4397-9dbb-9b2b20218538
Another round of rebaselining after content shell switch

Those were pulled from the try jobs that are red because of these
failures.

TBR=mkwst@chromium.org

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

git-svn-id: svn://svn.chromium.org/blink/trunk@151011 bbb929c8-8fbe-4397-9dbb-9b2b20218538
Don't buffer data while scanning for meta charset

Previously we'd buffer an arbitrarily large amount of data while scanning for a
meta charset declaration. Normally, we'd end our scan when we left the <head>
section, but when the document consists mostly of script tags, we can get a lot
of data before leaving the head section.

After this CL, we no longer buffer the response looking for a meta charset tag.
Instead, we go ahead and parse the data if we can't find a meta charset directive.
If we do end up finding one, we switch the encoding mid-parse. This approach
matches the approach in the HTML5 specification.

Even after this CL, we still "look ahead" into the network data to try to find
the declared encoding as early as possible. To match the HTML5 model better, we'd
need to remove the look-ahead. We should consider removing the look-ahead in
a future CL.

R=eseidel@chromium.org
BUG=223682

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

git-svn-id: svn://svn.chromium.org/blink/trunk@151012 bbb929c8-8fbe-4397-9dbb-9b2b20218538
Remove usage of uint8_t in WebMediaPlayer.h.

BUG=243347
TEST=Everything still compiles.
R=abarth@chromium.org, acolwell@chromium.org

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

git-svn-id: svn://svn.chromium.org/blink/trunk@151013 bbb929c8-8fbe-4397-9dbb-9b2b20218538
Fix webkitpy unittests.

-Fix all the things that change now that content shell is the default driver.
-Have the default value for --ignore-flaky-tests be None instead of 'default'.
Having it be 'default' doesn't gain anything and would require updating a bunch
of unittests to set the ignore_flaky_tests to 'default' in their options.

TBR=dpranke@chromium.org,jochen@chromium.org

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

git-svn-id: svn://svn.chromium.org/blink/trunk@151014 bbb929c8-8fbe-4397-9dbb-9b2b20218538
Updating expectations after https://src.chromium.org/viewvc/blink?view=rev&revision=150456

TBR=pdr
BUG=233848

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

git-svn-id: svn://svn.chromium.org/blink/trunk@151015 bbb929c8-8fbe-4397-9dbb-9b2b20218538
Fixing several incorrect assumptions with handling isolated inlines.

First, when an isolated inline spans multiple lines, we aren't guaranteed
to exit BidiResolver with a nested inline count of zero. Removing the
assert that says otherwise.

Previously in constructBidiRunsForSegment, we called bidiFirst in
an attempt to properly setup the isolatedResolver for any dom/style
that applied, but this only worked on the first line the isolated
inline appeared in. Adding a function that approaches this properly
by recursing through the parents of the starting object for the line
and post-fixing direction attributes to the resolver.

Finally, addressing an issue where the line following a removed isolated
inline (with a continuation) failed to be marked dirty.

BUG=178266
R=eseidel@chromium.org

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

git-svn-id: svn://svn.chromium.org/blink/trunk@151016 bbb929c8-8fbe-4397-9dbb-9b2b20218538
More rebaselining / skipping after switching to content_shell.

TBR=mkwst@chromium.org,jochen@chromium.org

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

git-svn-id: svn://svn.chromium.org/blink/trunk@151018 bbb929c8-8fbe-4397-9dbb-9b2b20218538
Clean up WebKitPlatformSupport references and redundant copies

This tidies up references to the class formerly known as WebKitPlatformSupport
and removes a redundant copy to the Platform*.

R=abarth
NOTRY=true

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

git-svn-id: svn://svn.chromium.org/blink/trunk@151019 bbb929c8-8fbe-4397-9dbb-9b2b20218538
Even more rebaselining / skipping.

Hopefully that should take care of the failures on the trybots.

TBR=mkwst@chromium.org,jochen@chromium.org

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

git-svn-id: svn://svn.chromium.org/blink/trunk@151020 bbb929c8-8fbe-4397-9dbb-9b2b20218538
Send TestExpectations through the wash (read it with webkitpy and write it out again)

This is in preparation for splitting FlakyTests out of TestExpectations.

I removed support for serializing the old "Skipped" file syntax.  Now when we write out Skip expectations we're always explicit about it.

Otherwise all of the changes to the file should be printing out the expectations in alphabetical sort order.

BUG=242366

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

git-svn-id: svn://svn.chromium.org/blink/trunk@151021 bbb929c8-8fbe-4397-9dbb-9b2b20218538
Treat unexpected SKIP like NOTRUN.

When the bots exit early due to too many crashes/timeouts, right now
we litter the results json with skip entries for all the tests that weren't run.
While this is probably right in principle, in practice, it makes the results
JSON enormous and everything falls over.

Instead, treat them as NOTRUNs for the purposes of their entires in
the results json file. We'll still include them as SKIP when tracking
number of each type of failure.

I waffled on whether this change should instead be made in run-webkit-tests
directly and decided on this because it keeps full_results.json being an
accurate reflection of what actually happened during the test run.

R=dpranke@chromium.org

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

git-svn-id: svn://svn.chromium.org/blink/trunk@151022 bbb929c8-8fbe-4397-9dbb-9b2b20218538
Rebaselined oes-vertex-array-object.html and more skipping.

Most of the tests are just flaky and the expectations had to be
updated to reflect this.

TBR=mkwst@chromium.org,jochen@chromium.org

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

git-svn-id: svn://svn.chromium.org/blink/trunk@151023 bbb929c8-8fbe-4397-9dbb-9b2b20218538
Rebaseline the remaining tests unskipped in r149700.

BUG=237679
TBR=adamk@chromium.org

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

git-svn-id: svn://svn.chromium.org/blink/trunk@151024 bbb929c8-8fbe-4397-9dbb-9b2b20218538
Cleanup jsonresults to minimize copy-paste of strings.

-Move all duplicated strings into variables and use them everywhere.
-Change the unittest to do dict equality instead of string equality
for better error messages.
-Use CHAR_TO_FAILURE in the unittests instead of copy-pasting the
stringified version of it.
-Put spaces after commas and colons to make check-webkit-style
less upset.

No functional changes. Just cleanups.

R=dpranke@chromium.org

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

git-svn-id: svn://svn.chromium.org/blink/trunk@151025 bbb929c8-8fbe-4397-9dbb-9b2b20218538
HTMLSelectElement should use lazyReattach

When changing the size attribute on <select> we shouldn't do a sync reattach().

R=ojan@chromium.org

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

git-svn-id: svn://svn.chromium.org/blink/trunk@151026 bbb929c8-8fbe-4397-9dbb-9b2b20218538
Even more rebaselining and skipping after switching to content shell.

This is based on what the try jobs are seeing.

TBR=mkwst@chromium.org,jochen@chromium.org

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

git-svn-id: svn://svn.chromium.org/blink/trunk@151027 bbb929c8-8fbe-4397-9dbb-9b2b20218538
Update run-binding-tests baselines.

R=abarth@chromium.org

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

git-svn-id: svn://svn.chromium.org/blink/trunk@151028 bbb929c8-8fbe-4397-9dbb-9b2b20218538
Move TRACE_EVENT macros for V8 compilation and execution to V8ScriptRunner

Now call sites of Script::Compile() and Script::Run() are aggregated to V8ScriptRunner.
Thus we should put TRACE_EVENT macros in V8ScriptRunner.

BUG=241743
R=abarth,eseidel
NOTRY=true

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

git-svn-id: svn://svn.chromium.org/blink/trunk@151029 bbb929c8-8fbe-4397-9dbb-9b2b20218538
Remove custom binding for Location::toString()

The generated code is exactly the same as the existing custom binding.

R=abarth,eseidel
NOTRY=true

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

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