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

Unified Diff: include.preload.js

Issue 5096952043667456: Fixed: Social widgets removed from share popup in first-run page (Closed)
Patch Set: Created Nov. 14, 2013, 5:02 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
« no previous file with comments | « no previous file | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: include.preload.js
===================================================================
--- a/include.preload.js
+++ b/include.preload.js
@@ -81,7 +81,7 @@
var type = typeMap[tag];
chrome.extension.sendRequest({reqtype: "should-collapse", url: url, documentUrl: document.URL, type: type}, function(response)
{
- if (response && target.parentNode)
+ if (response === true && target.parentNode)
{
// <frame> cannot be removed, doing that will mess up the frameset
if (tag == "frame")
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld