SVGCSSParser: m_implicitShorthand value is not restored.

Use the RAII class used in CSSParser.

BUG=242023

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

git-svn-id: svn://svn.chromium.org/blink/trunk@150669 bbb929c8-8fbe-4397-9dbb-9b2b20218538
Add checkdeps to PRESUMBIT.py

Add include_rules to Blink DEPS files and enforce them in presubmit.

BUG=238992

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

git-svn-id: svn://svn.chromium.org/blink/trunk@150670 bbb929c8-8fbe-4397-9dbb-9b2b20218538
Adding suppressed svg/foreignObject/filter-repaint.svg

Parent CL: https://codereview.chromium.org/15341004/

Since the Blink infrastructure cannot handle SVG files at the moment, I'm landing this test manually and un-suppressing it in the real CL.

TBR=schenney@chromium.org

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

git-svn-id: svn://svn.chromium.org/blink/trunk@150671 bbb929c8-8fbe-4397-9dbb-9b2b20218538
DevTools: [CodeMirror] update js/css/html modes

Pull codemirror modes from the upstream. The intent for
this change is the codemirror's comment.js plugin which
requires modes to have "lineComment" property.

BUG=167284

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

git-svn-id: svn://svn.chromium.org/blink/trunk@150672 bbb929c8-8fbe-4397-9dbb-9b2b20218538
DevTools: Remove file systems from workspace if they are not registered in browser settings.

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

git-svn-id: svn://svn.chromium.org/blink/trunk@150673 bbb929c8-8fbe-4397-9dbb-9b2b20218538
Inspector instrumentation: rearranging methods and renaming variables

Rearranged methods in .cpp to match the order of .h. Normalized whitespace. 
Used the uniform way of agents access everywhere.

BUG=232160

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

git-svn-id: svn://svn.chromium.org/blink/trunk@150674 bbb929c8-8fbe-4397-9dbb-9b2b20218538
Invalidate SVG filter results in SVGResourcesCache::clientLayoutChanged().

This is a regression introduced by http://trac.webkit.org/changeset/103539.

For the SVG tree, filter result invalidation is handled by
RenderSVGResource::markForLayoutAndParentResourceInvalidation. But non-SVG content (such as
a <foreignObject> HTML subtree) does not call markForLayoutAndParentResourceInvalidation and
instead uses the regular setNeedsLayout() invalidation path.

Hence, changes in the HTML subtree do not invalidate cached results for filters applied on
ancestor SVG elements. In order to cover this case, clientLayoutChanged() needs to invalidate
filter results even if the SVG element itself does not need a re-layout. Note that prior to
http://trac.webkit.org/changeset/103539 the method used to do exactly that, but the branch was
removed on the assumption that markForLayoutAndParentResourceInvalidation() is already taking care
of it. Obviously, the assumption doesn't hold for non-SVG content.

BUG=165180
R=dschulze@chromium.org,schenney@chromium.org,pdr@chromium.org

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

git-svn-id: svn://svn.chromium.org/blink/trunk@150675 bbb929c8-8fbe-4397-9dbb-9b2b20218538
DevTools: remove erroneous line from inspector.css

It did not have effect anyway.

TBR=apavlov

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

git-svn-id: svn://svn.chromium.org/blink/trunk@150676 bbb929c8-8fbe-4397-9dbb-9b2b20218538
Timeline: emit native heap & DOM counters data based on record depth, not type

I'd like to remove late checks against timeline record type, so we can stop
passing it around to a bunch of methods and set early in createGenricRecord.
Given that we currently always emit Program record first, this shouldn't
change the output.

R=yurys@chromium.org

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

git-svn-id: svn://svn.chromium.org/blink/trunk@150677 bbb929c8-8fbe-4397-9dbb-9b2b20218538
Revert 2 commits to fix an exception on the Android bots

This change reverts:
* "Generate a failing_results.json file." (commit 648ba37d33802486dea28cbad36d5b20765dfa70)
* "Stop generating and uploading incremental_results.json." (commit 68559d2c7e207041b0ac4bcc8e8636f647b6b00c)

which should fix the following error:

AttributeError: 'module' object has no attribute 'JSONResultsGeneratorBase'
(which was removed in commit 68559d2c7e207041b0ac4bcc8e8636f647b6b00c)

BUG=242206
TBR=ojan

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

git-svn-id: svn://svn.chromium.org/blink/trunk@150678 bbb929c8-8fbe-4397-9dbb-9b2b20218538
Make track objects ScriptWrappable

BUG=236671
R=abarth@chromium.org

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

git-svn-id: svn://svn.chromium.org/blink/trunk@150679 bbb929c8-8fbe-4397-9dbb-9b2b20218538
Replace Script::New() with Script::Compile()

This is a preparation patch to aggregate caller sites of Script::Compile()
into one place.

The difference between Script::New() and Script::Compile() is whether the script
is compiled bound to the current context or not. (Script::New() is context-independent
and Script::Compile() is context-bounded.) Given that ScriptDebugServer is intending
to compile a script bound to a context, it is safe to replace Script::New() with Script::Compile().

BUG=241743
R=abarth,eseidel

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

git-svn-id: svn://svn.chromium.org/blink/trunk@150680 bbb929c8-8fbe-4397-9dbb-9b2b20218538
Move compileScript() and precompileScript() to V8ScriptRunner

{compile,precompile,run}Script() should be put in V8ScriptRunner.
I'll share the code with other places in follow-up patches to aggregate
call sites of Script::XXX() into one place.

BUG=241743
R=abarth,eseidel

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

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