DevTools: [SASS] introduce sass module, implement SCSS AST-builder.

DevTools: [SASS] introduce sass module, implement SCSS AST-builder.

The patch introduces "sass" module which will be responsible for all
SASS-processing in DevTools.

The patch also adds a simple SCSS AST-builder, which is capable of
parsing variables, mixins and properties.

BUG=527993
R=pfeldman, dgozman

Committed: https://crrev.com/c85e962a63be9b75b51eb71218725fbf21d2b7c6
Cr-Commit-Position: refs/heads/master@{#364206}

[lushnikov, 2015-12-09 03:30:25]

ptal

[pfeldman, 2015-12-09 03:50:59]

https://codereview.chromium.org/1511073002/diff/20001/third_party/WebKit/Sour...
File third_party/WebKit/Source/devtools/front_end/sass/SASSSupport.js (right):

https://codereview.chromium.org/1511073002/diff/20001/third_party/WebKit/Sour...
third_party/WebKit/Source/devtools/front_end/sass/SASSSupport.js:78: // The
|processToken| is called per-line, so no token spans more then one line.
huh?

https://codereview.chromium.org/1511073002/diff/20001/third_party/WebKit/Sour...
third_party/WebKit/Source/devtools/front_end/sass/SASSSupport.js:205:
this.parent = null;
Why is the AST tree not reflective? I.e. why can't I visit it as a tree?

[lushnikov, 2015-12-09 07:04:57]

https://codereview.chromium.org/1511073002/diff/20001/third_party/WebKit/Sour...
File third_party/WebKit/Source/devtools/front_end/sass/SASSSupport.js (right):

https://codereview.chromium.org/1511073002/diff/20001/third_party/WebKit/Sour...
third_party/WebKit/Source/devtools/front_end/sass/SASSSupport.js:205:
this.parent = null;
On 2015/12/09 03:50:59, pfeldman wrote:
> Why is the AST tree not reflective? I.e. why can't I visit it as a tree?

Because it's structure is simple enough to avoid reflective overhead in the
code.

[lushnikov, 2015-12-09 18:40:19]

addressed comments, added faster root() calculation

[lushnikov, 2015-12-09 19:58:11]

introduced ASTDocument instead of root()

[pfeldman, 2015-12-09 20:13:43]

lgtm

https://codereview.chromium.org/1511073002/diff/60001/third_party/WebKit/Sour...
File third_party/WebKit/Source/devtools/front_end/sass/SASSSupport.js (right):

https://codereview.chromium.org/1511073002/diff/60001/third_party/WebKit/Sour...
third_party/WebKit/Source/devtools/front_end/sass/SASSSupport.js:228: this.text
= text;
Why is text necessary? You can fetch it at any moment dynamically.

[lushnikov, 2015-12-09 21:41:54]

https://codereview.chromium.org/1511073002/diff/60001/third_party/WebKit/Sour...
File third_party/WebKit/Source/devtools/front_end/sass/SASSSupport.js (right):

https://codereview.chromium.org/1511073002/diff/60001/third_party/WebKit/Sour...
third_party/WebKit/Source/devtools/front_end/sass/SASSSupport.js:228: this.text
= text;
On 2015/12/09 20:13:43, pfeldman wrote:
> Why is text necessary? You can fetch it at any moment dynamically.

I want to modify AST tree later, adding properties in it. These properties will
be unrelated to
 document.

[lushnikov, 2015-12-09 21:42:03]

The CQ bit was checked by lushnikov@chromium.org

[commit-bot: I haz the power, 2015-12-09 21:46:18]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1511073002/60001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1511073002/60001

[commit-bot: I haz the power, 2015-12-09 23:19:23]

Committed patchset #4 (id:60001)

[commit-bot: I haz the power, 2015-12-09 23:20:56]

Description was changed from

==========
DevTools: [SASS] introduce sass module, implement SCSS AST-builder.

The patch introduces "sass" module which will be responsible for all
SASS-processing in DevTools.

The patch also adds a simple SCSS AST-builder, which is capable of
parsing variables, mixins and properties.

BUG=527993
R=pfeldman, dgozman
==========

to

==========
DevTools: [SASS] introduce sass module, implement SCSS AST-builder.

The patch introduces "sass" module which will be responsible for all
SASS-processing in DevTools.

The patch also adds a simple SCSS AST-builder, which is capable of
parsing variables, mixins and properties.

BUG=527993
R=pfeldman, dgozman

Committed: https://crrev.com/c85e962a63be9b75b51eb71218725fbf21d2b7c6
Cr-Commit-Position: refs/heads/master@{#364206}
==========

[commit-bot: I haz the power, 2015-12-09 23:20:57]

Patchset 4 (id:??) landed as
https://crrev.com/c85e962a63be9b75b51eb71218725fbf21d2b7c6
Cr-Commit-Position: refs/heads/master@{#364206}