DevTools: remove indirection between CSSWorkspaceBindings and CSS mappings

DevTools: remove indirection between CSSWorkspaceBindings and CSS mappings

This patch:
- makes StylesSourceMapping listen to styleSheetAdded/styleSheetRemoved
  events by itself. SASSSourceMapping doesn't need to - it relies on
  SourceMapAttached event.
- disposes CSSWorkspaceBindings.TargetInfo properly, by disposing
  CSS mappings.

BUG=none
R=dgozman

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

[lushnikov, 2016-08-31 19:41:14]

please, take a look

[lushnikov, 2016-08-31 19:41:20]

The CQ bit was checked by lushnikov@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-08-31 19:41:45]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-08-31 21:14:44]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-08-31 21:14:44]

Dry run: This issue passed the CQ dry run.

[dgozman, 2016-09-01 00:49:25]

https://codereview.chromium.org/2298373002/diff/1/third_party/WebKit/Source/d...
File third_party/WebKit/Source/devtools/front_end/bindings/SASSSourceMapping.js
(left):

https://codereview.chromium.org/2298373002/diff/1/third_party/WebKit/Source/d...
third_party/WebKit/Source/devtools/front_end/bindings/SASSSourceMapping.js:116:
WebInspector.cssWorkspaceBinding.updateLocations(header);
Who calls this now?

[lushnikov, 2016-09-01 01:34:34]

https://codereview.chromium.org/2298373002/diff/1/third_party/WebKit/Source/d...
File third_party/WebKit/Source/devtools/front_end/bindings/SASSSourceMapping.js
(left):

https://codereview.chromium.org/2298373002/diff/1/third_party/WebKit/Source/d...
third_party/WebKit/Source/devtools/front_end/bindings/SASSSourceMapping.js:116:
WebInspector.cssWorkspaceBinding.updateLocations(header);
On 2016/09/01 00:49:25, dgozman wrote:
> Who calls this now?

SourceMapDetached will call this (if there was a sourceMap attached)

[dgozman, 2016-09-01 01:42:34]

lgtm

[lushnikov, 2016-09-08 03:40:09]

The CQ bit was checked by lushnikov@chromium.org

[commit-bot: I haz the power, 2016-09-08 03:40:29]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-09-08 05:40:08]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-09-08 05:41:49]

Description was changed from

==========
DevTools: remove indirection between CSSWorkspaceBindings and CSS mappings

This patch:
- makes StylesSourceMapping listen to styleSheetAdded/styleSheetRemoved
  events by itself. SASSSourceMapping doesn't need to - it relies on
  SourceMapAttached event.
- disposes CSSWorkspaceBindings.TargetInfo properly, by disposing
  CSS mappings.

BUG=none
R=dgozman
==========

to

==========
DevTools: remove indirection between CSSWorkspaceBindings and CSS mappings

This patch:
- makes StylesSourceMapping listen to styleSheetAdded/styleSheetRemoved
  events by itself. SASSSourceMapping doesn't need to - it relies on
  SourceMapAttached event.
- disposes CSSWorkspaceBindings.TargetInfo properly, by disposing
  CSS mappings.

BUG=none
R=dgozman

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

[commit-bot: I haz the power, 2016-09-08 05:41:50]

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