DevTools: clicking console messages with traces should check for links

DevTools: clicking console messages with traces should check for links

Console messages with a trace can be clicked to toggle their expanded state.
The click handler was intercepting and consuming clicks on links that were part
of the message's text. As a result, some anchors stopped revealing in the
sources panel and navigated the target page instead.

This CL makes the handler check whether the click target was an anchor first to
allow the document's handler to follow relative links.

BUG=660521
Committed: https://crrev.com/67fdae4bf2f450c494f76623c6810e18443067e2
Cr-Commit-Position: refs/heads/master@{#430796}

[luoe, 2016-11-08 03:01:16]

Description was changed from

==========
DevTools: handle link clicks in console error messages

BUG=660521
==========

to

==========
DevTools: clicking relative links in console error messages should reveal in
sources

Console messages with a trace can be clicked to toggle their expanded state.
The click handler was intercepting and consuming clicks on links that were part
of the message's text. As a result, some anchors stopped revealing in the
sources panel and navigated the target page instead.

This CL makes the handler check whether the click target was an anchor first to
allow the document's handler to follow relative links.

BUG=660521
==========

[luoe, 2016-11-08 03:02:05]

Description was changed from

==========
DevTools: clicking relative links in console error messages should reveal in
sources

Console messages with a trace can be clicked to toggle their expanded state.
The click handler was intercepting and consuming clicks on links that were part
of the message's text. As a result, some anchors stopped revealing in the
sources panel and navigated the target page instead.

This CL makes the handler check whether the click target was an anchor first to
allow the document's handler to follow relative links.

BUG=660521
==========

to

==========
DevTools: clicking console messages with traces should check for links

Console messages with a trace can be clicked to toggle their expanded state.
The click handler was intercepting and consuming clicks on links that were part
of the message's text. As a result, some anchors stopped revealing in the
sources panel and navigated the target page instead.

This CL makes the handler check whether the click target was an anchor first to
allow the document's handler to follow relative links.

BUG=660521
==========

[luoe, 2016-11-08 03:03:20]

luoe@chromium.org changed reviewers:
+ dgozman@chromium.org

[luoe, 2016-11-08 03:03:20]

PTAL

[dgozman, 2016-11-08 19:52:52]

lgtm

[luoe, 2016-11-08 21:31:56]

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

[commit-bot: I haz the power, 2016-11-08 21:33:24]

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-11-08 23:33:16]

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

[commit-bot: I haz the power, 2016-11-08 23:33:18]

Dry run: This issue passed the CQ dry run.

[luoe, 2016-11-09 00:29:41]

The CQ bit was checked by luoe@chromium.org

[commit-bot: I haz the power, 2016-11-09 00:30:09]

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-11-09 00:39:00]

Description was changed from

==========
DevTools: clicking console messages with traces should check for links

Console messages with a trace can be clicked to toggle their expanded state.
The click handler was intercepting and consuming clicks on links that were part
of the message's text. As a result, some anchors stopped revealing in the
sources panel and navigated the target page instead.

This CL makes the handler check whether the click target was an anchor first to
allow the document's handler to follow relative links.

BUG=660521
==========

to

==========
DevTools: clicking console messages with traces should check for links

Console messages with a trace can be clicked to toggle their expanded state.
The click handler was intercepting and consuming clicks on links that were part
of the message's text. As a result, some anchors stopped revealing in the
sources panel and navigated the target page instead.

This CL makes the handler check whether the click target was an anchor first to
allow the document's handler to follow relative links.

BUG=660521
==========

[commit-bot: I haz the power, 2016-11-09 00:39:01]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-11-09 00:44:51]

Description was changed from

==========
DevTools: clicking console messages with traces should check for links

Console messages with a trace can be clicked to toggle their expanded state.
The click handler was intercepting and consuming clicks on links that were part
of the message's text. As a result, some anchors stopped revealing in the
sources panel and navigated the target page instead.

This CL makes the handler check whether the click target was an anchor first to
allow the document's handler to follow relative links.

BUG=660521
==========

to

==========
DevTools: clicking console messages with traces should check for links

Console messages with a trace can be clicked to toggle their expanded state.
The click handler was intercepting and consuming clicks on links that were part
of the message's text. As a result, some anchors stopped revealing in the
sources panel and navigated the target page instead.

This CL makes the handler check whether the click target was an anchor first to
allow the document's handler to follow relative links.

BUG=660521

Committed: https://crrev.com/67fdae4bf2f450c494f76623c6810e18443067e2
Cr-Commit-Position: refs/heads/master@{#430796}
==========

[commit-bot: I haz the power, 2016-11-09 00:44:52]

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