[devtools] _parseMap in SourceMap.js does not strictly follow specs

[devtools] _parseMap in SourceMap.js does not strictly follow specs

Also add tests for issue 611738

BUG=611738
Committed: https://crrev.com/51f1de9d9288bc3603f38d30341cc61e960bd44d
Cr-Commit-Position: refs/heads/master@{#415074}

[darwin, 2016-08-06 22:02:29]

antonin.hildebrand@gmail.com changed reviewers:
+ lushnikov+blink@chromium.org

[darwin, 2016-08-06 22:02:29]

antonin.hildebrand@gmail.com changed required reviewers:
+ lushnikov+blink@chromium.org

[darwin, 2016-08-06 22:02:29]

My first submission ever. Please forgive mistakes.

Random bits of experience of a first-time committer:

- Took me 2 days to properly compile blink_tests (due to problems with beta
version of my xcode installation.

- docs are good, but the workflow for submission is not super-easy for
first-timers.

- My google account is antonin.hildebrand@gmail.com, that is my user on
https://codereview.chromium.org and in other google services. But my git email
is antonin@hildebrand.cz. Presubmit checks got confused about it. So I tried to
rebase my commits and force antonin.hildebrand@gmail.com there. But it didn't
fix the warning. I'm not sure where is the presubmit check getting the info.
Maybe it is from global git config?

[darwin, 2016-08-15 22:47:24]

The CQ bit was checked by antonin.hildebrand@gmail.com

[commit-bot: I haz the power, 2016-08-15 22:48:01]

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-15 22:48:01]

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

[commit-bot: I haz the power, 2016-08-15 22:48:02]

All required reviewers (with asterisk prefixes) have not yet approved this CL.

No L-G-T-M from a valid reviewer yet. 
CQ run can only be started by full committers or once the patch has
received an L-G-T-M from a full committer.
Even if an L-G-T-M may have been provided, it was from a non-committer,
_not_ a full super star committer.
See http://www.chromium.org/getting-involved/become-a-committer
Note that this has nothing to do with OWNERS files.

[lushnikov, 2016-08-25 19:48:36]

lushnikov@chromium.org changed reviewers:
+ lushnikov@chromium.org

[lushnikov, 2016-08-25 19:48:37]

Thank you! You also want to add yourself in the src/AUTHORS file

https://codereview.chromium.org/2222713004/diff/1/third_party/WebKit/LayoutTe...
File
third_party/WebKit/LayoutTests/http/tests/inspector/compiler-script-mapping.html
(right):

https://codereview.chromium.org/2222713004/diff/1/third_party/WebKit/LayoutTe...
third_party/WebKit/LayoutTests/http/tests/inspector/compiler-script-mapping.html:480:
function testIssue611738(next)
let's name this testNameIndexes

https://codereview.chromium.org/2222713004/diff/1/third_party/WebKit/LayoutTe...
third_party/WebKit/LayoutTests/http/tests/inspector/compiler-script-mapping.html:503:
Related links:
let's remove the "related links"

[darwin, 2016-08-26 23:13:09]

An attempt to update the patch.

[lushnikov, 2016-08-26 23:16:15]

https://codereview.chromium.org/2222713004/diff/20001/AUTHORS
File AUTHORS (right):

https://codereview.chromium.org/2222713004/diff/20001/AUTHORS#newcode733
AUTHORS:733: Antonin Hildebrand <antonin.hildebrand@gmail.com>
let's put your name in alphabetic order (it's unfortunate that the sorting is
somewhat broken for a few last names; it shouldn't be)

[darwin, 2016-08-26 23:26:25]

Getting better at updating patches :)

[lushnikov, 2016-08-26 23:33:33]

great, thank you! lgtm

[lushnikov, 2016-08-26 23:33:38]

The CQ bit was checked by lushnikov@chromium.org

[commit-bot: I haz the power, 2016-08-26 23:34:08]

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-26 23:34:08]

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

[commit-bot: I haz the power, 2016-08-26 23:34:09]

All required reviewers (with asterisk prefixes) have not yet approved this CL.

No L-G-T-M from a valid reviewer yet. 
CQ run can only be started by full committers or once the patch has
received an L-G-T-M from a full committer.
Even if an L-G-T-M may have been provided, it was from a non-committer,
_not_ a full super star committer.
See http://www.chromium.org/getting-involved/become-a-committer
Note that this has nothing to do with OWNERS files.

[lushnikov, 2016-08-29 23:46:29]

Description was changed from

==========
[devtools] _parseMap in SourceMap.js does not strictly follow specs

Also add tests for issue 611738

BUG=611738
==========

to

==========
[devtools] _parseMap in SourceMap.js does not strictly follow specs

Also add tests for issue 611738

BUG=611738
==========

[lushnikov, 2016-08-29 23:46:29]

lushnikov@chromium.org changed reviewers:
- lushnikov+blink@chromium.org

[lushnikov, 2016-08-29 23:46:29]

lushnikov@chromium.org changed required reviewers:
- lushnikov+blink@chromium.org

[lushnikov, 2016-08-29 23:46:39]

The CQ bit was checked by lushnikov@chromium.org

[commit-bot: I haz the power, 2016-08-30 00:17:23]

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-30 04:07:55]

Description was changed from

==========
[devtools] _parseMap in SourceMap.js does not strictly follow specs

Also add tests for issue 611738

BUG=611738
==========

to

==========
[devtools] _parseMap in SourceMap.js does not strictly follow specs

Also add tests for issue 611738

BUG=611738
==========

[commit-bot: I haz the power, 2016-08-30 04:07:56]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2016-08-30 04:09:32]

Description was changed from

==========
[devtools] _parseMap in SourceMap.js does not strictly follow specs

Also add tests for issue 611738

BUG=611738
==========

to

==========
[devtools] _parseMap in SourceMap.js does not strictly follow specs

Also add tests for issue 611738

BUG=611738

Committed: https://crrev.com/51f1de9d9288bc3603f38d30341cc61e960bd44d
Cr-Commit-Position: refs/heads/master@{#415074}
==========

[commit-bot: I haz the power, 2016-08-30 04:09:33]

Patchset 3 (id:??) landed as
https://crrev.com/51f1de9d9288bc3603f38d30341cc61e960bd44d
Cr-Commit-Position: refs/heads/master@{#415074}