Commit 42229ab0 authored by Jack Franklin's avatar Jack Franklin Committed by Commit Bot

Update layout tests to use Linkifier.linkInfo

https://crrev.com/c/2544131 renamed the method to remove the underscore
but shipped both names in order to avoid layout test breakages. Once
that's landed, we can land this change, and then remove the old
`_linkInfo` method as it will then be entirely unused.

Bug: 1149403
Change-Id: I21bcf71fb8e87fe4202d07ee83b3bbf481905f91
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2546760
Commit-Queue: Jack Franklin <jacktfranklin@chromium.org>
Reviewed-by: default avatarTim van der Lippe <tvanderlippe@chromium.org>
Cr-Commit-Position: refs/heads/master@{#829280}
parent e64c6cc4
......@@ -46,7 +46,7 @@
TestRunner.addResult('\nText: ' + element.deepTextContent());
var links = element.querySelectorAll('.devtools-link');
for (var link of links) {
var info = Components.Linkifier._linkInfo(link);
var info = Components.Linkifier.linkInfo(link);
if (info && info.url) {
TestRunner.addResult('Linked url: ' + info.url);
if (info.lineNumber !== null || info.columnNumber !== null)
......
......@@ -207,7 +207,7 @@
var linkifier = new Components.Linkifier();
var anchorURI = uiSourceCode.url();
var anchor = linkifier.linkifyScriptLocation(SDK.targetManager.mainTarget(), null, anchorURI, 10, {columnNumber: 1});
var info = Components.Linkifier._linkInfo(anchor);
var info = Components.Linkifier.linkInfo(anchor);
Common.Revealer.reveal(info.uiLocation).then(function() {
TestRunner.addResult('Selection: ' + panel.visibleView.textEditor.selection().toString());
dumpSelection('Showed anchor in ' + anchorURI.split('/').pop() + ' with line 333 column 3');
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment