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
NOTRY=true
NOPRESUBMIT=true

Review-Url: https://codereview.chromium.org/2490433002
Cr-Commit-Position: refs/heads/master@{#430796}
(cherry picked from commit 67fdae4bf2f450c494f76623c6810e18443067e2)

[luoe, 2016-11-10 17:41:02]

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

Review-Url: https://codereview.chromium.org/2490433002
Cr-Commit-Position: refs/heads/master@{#430796}
(cherry picked from commit 67fdae4bf2f450c494f76623c6810e18443067e2)
==========

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
NOTRY=true
NOPRESUBMIT=true

Review-Url: https://codereview.chromium.org/2490433002
Cr-Commit-Position: refs/heads/master@{#430796}
(cherry picked from commit 67fdae4bf2f450c494f76623c6810e18443067e2)
==========

[luoe, 2016-11-10 17:42:09]

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

[luoe, 2016-11-10 17:42:09]

This is another CL to merge a regression fix to M55.  A committer's l g t m is
needed.  Could a committer ptal?

[dgozman, 2016-11-10 18:43:42]

lgtm

[luoe, 2016-11-10 18:44:44]

The CQ bit was checked by luoe@chromium.org

[commit-bot: I haz the power, 2016-11-10 18:45:18]

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-10 18:54:12]

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
NOTRY=true
NOPRESUBMIT=true

Review-Url: https://codereview.chromium.org/2490433002
Cr-Commit-Position: refs/heads/master@{#430796}
(cherry picked from commit 67fdae4bf2f450c494f76623c6810e18443067e2)
==========

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
NOTRY=true
NOPRESUBMIT=true

Review-Url: https://codereview.chromium.org/2490433002
Cr-Commit-Position: refs/heads/master@{#430796}
(cherry picked from commit 67fdae4bf2f450c494f76623c6810e18443067e2)
==========

[commit-bot: I haz the power, 2016-11-10 18:54:13]

Committed patchset #1 (id:1)