DevTools: introduce WI.SourceMap.Entry structure and cleanup WI.SourceMap API.

DevTools: introduce WI.SourceMap.Entry structure and cleanup WI.SourceMap API.

The patch:
- Use WI.SourceMap.Entry instead of an array tuples.
- Establish correct reversed mappings

BUG=527993, 463023
R=pfeldman

Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=201848

[lushnikov, 2015-09-03 20:47:30]

ptal

[pfeldman, 2015-09-04 01:38:36]

lgtm

https://codereview.chromium.org/1328843002/diff/20001/Source/devtools/front_e...
File Source/devtools/front_end/bindings/CompilerScriptMapping.js (right):

https://codereview.chromium.org/1328843002/diff/20001/Source/devtools/front_e...
Source/devtools/front_end/bindings/CompilerScriptMapping.js:128: var
mappingSearchLinesCount = 5;
Lets remove this.

https://codereview.chromium.org/1328843002/diff/20001/Source/devtools/front_e...
File Source/devtools/front_end/sdk/SourceMap.js (right):

https://codereview.chromium.org/1328843002/diff/20001/Source/devtools/front_e...
Source/devtools/front_end/sdk/SourceMap.js:346: reverseMappings.push(mapping);
reverseMapping.get(mapping.lineNumber).push(mapping);

[lushnikov, 2015-09-04 17:46:45]

https://codereview.chromium.org/1328843002/diff/20001/Source/devtools/front_e...
File Source/devtools/front_end/bindings/CompilerScriptMapping.js (right):

https://codereview.chromium.org/1328843002/diff/20001/Source/devtools/front_e...
Source/devtools/front_end/bindings/CompilerScriptMapping.js:128: var
mappingSearchLinesCount = 5;
On 2015/09/04 01:38:36, pfeldman wrote:
> Lets remove this.

Done.

https://codereview.chromium.org/1328843002/diff/20001/Source/devtools/front_e...
File Source/devtools/front_end/sdk/SourceMap.js (right):

https://codereview.chromium.org/1328843002/diff/20001/Source/devtools/front_e...
Source/devtools/front_end/sdk/SourceMap.js:346: reverseMappings.push(mapping);
This will render findEntriesReversed implementation work for linear time.

[lushnikov, 2015-09-04 17:46:51]

The CQ bit was checked by lushnikov@chromium.org

[lushnikov, 2015-09-04 17:46:51]

The patchset sent to the CQ was uploaded after l-g-t-m from
pfeldman@chromium.org
Link to the patchset: https://codereview.chromium.org/1328843002/#ps40001
(title: "fix comment")

[commit-bot: I haz the power, 2015-09-04 17:47:07]

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

[commit-bot: I haz the power, 2015-09-04 18:58:45]

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

[commit-bot: I haz the power, 2015-09-04 18:58:46]

Try jobs failed on following builders:
  win_chromium_rel_ng on tryserver.chromium.win (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.win/builders/win_chromium_rel_...)

[lushnikov, 2015-09-05 23:59:30]

The CQ bit was checked by lushnikov@chromium.org

[lushnikov, 2015-09-05 23:59:30]

The patchset sent to the CQ was uploaded after l-g-t-m from
pfeldman@chromium.org
Link to the patchset: https://codereview.chromium.org/1328843002/#ps60001
(title: "fix test")

[commit-bot: I haz the power, 2015-09-05 23:59:40]

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

[commit-bot: I haz the power, 2015-09-06 02:24:52]

Committed patchset #4 (id:60001) as
https://src.chromium.org/viewvc/blink?view=rev&revision=201848