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

Unified Diff: include.preload.js

Issue 5171007186796544: Issue 1517 - Always call parseInt() with a radix as second argument (Closed)
Patch Set: Created Oct. 31, 2014, 2:52 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 | « background.js ('k') | safari/ext/common.js » ('j') | 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
@@ -110,7 +110,7 @@
// and about:srcdoc. So we have to apply element hiding and collapsing
// from the parent frame, when inline frames are loaded.
var match = navigator.userAgent.match(/\bChrome\/(\d+)/);
- var fixInlineFrames = match && parseInt(match[1]) < 37;
+ var fixInlineFrames = match && parseInt(match[1], 10) < 37;
// use Shadow DOM if available to don't mess with web pages that
// rely on the order of their own <style> tags (#309). However we
« no previous file with comments | « background.js ('k') | safari/ext/common.js » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld