Use isSpaceOrNewline rather than its implementation

These two if statements use the isSpaceOrNewline implementation but it is clearer and safer to call the inlined method instead.

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

git-svn-id: svn://svn.chromium.org/blink/trunk@165283 bbb929c8-8fbe-4397-9dbb-9b2b20218538
Perform SVGUseElement instance tree updates at the start of recalcStyle

Previously SVGUseElement would modify the DOM inside ::attach or
::willRecalcStyle. That required a terrible hack in the style machinery
since the <use> could add a new <style> to the DOM in the middle of a
recalcStyle. This patch moves that logic to a pre-recalcStyle phase
so we can remove that hack and add an ASSERT that the DOM is never
modified from inside recalcStyle.

I also noticed that the updateDistributionIfNeeded() step inside
Document::recalcStyle was not before the inspector timeline logic. This
patch moves that step and the new one below the inspector timeline logic
so that the time spent in "recalcStyle" in the timeline is reported
properly.

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

git-svn-id: svn://svn.chromium.org/blink/trunk@165284 bbb929c8-8fbe-4397-9dbb-9b2b20218538
Mark inspector/profiler/heap-snapshot-loader.html as flaky

The test fails time to time after heap profiler tests have switched from fake worker to real one.

BUG=330617
TBR=loislo@chromium.org

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

git-svn-id: svn://svn.chromium.org/blink/trunk@165285 bbb929c8-8fbe-4397-9dbb-9b2b20218538
Remove custom V8 bindings for MIDI

We can remove the custom bindings using [SetReference].
[SetReference] generates code that is semantically the same as the custom bindings.

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

git-svn-id: svn://svn.chromium.org/blink/trunk@165286 bbb929c8-8fbe-4397-9dbb-9b2b20218538
Rename bindingFoo to fooForBindings

Switch to the majority convention so that no one needs to investigate
which to use, as in https://codereview.chromium.org/141363002/

BUG=none

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

git-svn-id: svn://svn.chromium.org/blink/trunk@165287 bbb929c8-8fbe-4397-9dbb-9b2b20218538
Deprecate HTMLVideoElement-specific prefixed fullscreen API

Intent to Deprecate:
https://groups.google.com/a/chromium.org/d/msg/blink-dev/DULRMEUkeJw/9HXn55X1gj4J

Unfortunately Blink doesn't support the unprefixed fullscreen API yet,
so advise authors to use both the unprefixed and prefixed APIs.

webkitSupportsFullscreen was found to be needlessly complicated for what
it does, so a bug to simplify it was filed: http://crbug.com/334985

BUG=334987

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

git-svn-id: svn://svn.chromium.org/blink/trunk@165288 bbb929c8-8fbe-4397-9dbb-9b2b20218538
Split dependencies from built IDL files in GYP

This statically separates:
* actual target IDL files (for which .h/.cpp files are generated)
from:
* dependency IDL files (for which dummy files are generated)
...which significantly reduces the work done in generating bindings and
allows us to remove lots of code and complexity (b/c these aren't distinguished).

Improvements:
* Reduces generated bindings by 9% (1348 => 1236)
* Reduces build edges by 9% (678 => 618)
* Reduces tests/results by 10% (61 => 55)
* Removes --additionalIdlFiles parameter and code
* Removes dummy file generation code

Further simplification is possible:
* compute_dependencies
  (currently has to remove dependencies, and treat main and support separately)
* derived_sources_all_in_one
  Possibly can be eliminated or significantly simplified, as we now have
  the full list (it's just main_interface_idl_files)
* idl_compiler.py & code_generator_v8.py, since guaranteed to have real IDL file
* Perhaps remove the 'support_*' variables (used by testing)

I'll do these in followup CLs.

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

git-svn-id: svn://svn.chromium.org/blink/trunk@165289 bbb929c8-8fbe-4397-9dbb-9b2b20218538
Measure usage of the prefixed encrypted media APIs

The unprefixed replacements for these aren't implemented yet, but it's
just as well to start collecting data early.

PrefixedMediaGenerateKeyRequest and PrefixedMediaAddKey were already in
UseCounter.h, but unused. Add the missing PrefixedMediaCancelKeyRequest
and put them all to use.

Remove the deprecation message for PrefixedMediaGenerateKeyRequest,
which was left behind in r150962. Otherwise, the ASSERTs in
UseCounter::count would start failing.

BUG=334983

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

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