Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code

Unified Diff: devtools-panel.js

Issue 29740563: Noissue - In the devtools panel, preserve space for missing document domain (Closed)
Patch Set: Created April 2, 2018, 6:47 p.m.
Use n/p to move between diff chunks; N/P to move between comments.
Jump to:
View side-by-side diff with in-line comments
Download patch
« devtools-panel.html ('K') | « devtools-panel.html ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: devtools-panel.js
===================================================================
--- a/devtools-panel.js
+++ b/devtools-panel.js
@@ -83,8 +83,6 @@
{
let row = document.importNode(template, true);
row.dataset.type = request.type;
-
- row.querySelector(".domain").textContent = request.docDomain;
row.querySelector(".type").textContent = request.type;
let urlElement = row.querySelector(".resource-link");
@@ -109,6 +107,9 @@
}
}
+ if (request.docDomain)
+ row.querySelector(".domain").textContent = request.docDomain;
+
if (filter)
{
let filterElement = row.querySelector(".filter");
« devtools-panel.html ('K') | « devtools-panel.html ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld