Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(3538)

Unified Diff: chrome/browser/safe_browsing/threat_details.cc

Issue 2713233002: Update ThreatDOMDetails to be able to collect non-resource HTML Elements based on their attributes. (Closed)
Patch Set: Sync Created 3 years, 10 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chrome/browser/safe_browsing/threat_details.cc
diff --git a/chrome/browser/safe_browsing/threat_details.cc b/chrome/browser/safe_browsing/threat_details.cc
index 830b1f5608635bd8c7589b871638058237053d3c..9e8559fcf3d07ddbe119e407a648c46cd57c62ec 100644
--- a/chrome/browser/safe_browsing/threat_details.cc
+++ b/chrome/browser/safe_browsing/threat_details.cc
@@ -496,8 +496,11 @@ void ThreatDetails::AddDOMDetails(
for (size_t i = 0; i < params.size() && i < kMaxDomNodes; ++i) {
SafeBrowsingHostMsg_ThreatDOMDetails_Node node = params[i];
DVLOG(1) << node.url << ", " << node.tag_name << ", " << node.parent;
- ClientSafeBrowsingReportRequest::Resource* resource =
- AddUrl(node.url, node.parent, node.tag_name, &(node.children));
+ ClientSafeBrowsingReportRequest::Resource* resource = nullptr;
+ if (!node.url.is_empty()) {
+ resource = AddUrl(node.url, node.parent, node.tag_name, &(node.children));
+ }
+ // Check for a tag_name to avoid adding the summary node to the DOM.
if (!node.tag_name.empty()) {
AddDomElement(frame_tree_node_id, frame_url.spec(), node.node_id,
node.tag_name, node.parent_node_id, resource);

Powered by Google App Engine
This is Rietveld 408576698