Closed frames shouldn't include newtab

Fix #3258 Auditors: @bbondy Test Plan: New Tab, close it, look in History menu to ensure it's not among recently closed
brave · Sep 8, 2016 · 56e9ecc · 56e9ecc
1 parent 0a2141f
commit 56e9ecc
Showing 1 changed file with 1 addition and 1 deletion.
diff --git a/js/state/frameStateUtil.js b/js/state/frameStateUtil.js
@@ -413,7 +413,7 @@ function removeFrame (frames, tabs, closedFrames, frameProps, activeFrameKey) {
     0)
   }
 
-  if (!frameProps.get('isPrivate')) {
+  if (!frameProps.get('isPrivate') && frameProps.get('location') !== 'about:newtab') {
     frameProps = frameProps.set('isFullScreen', false)
     closedFrames = closedFrames.push(frameProps)
     if (frameProps.get('thumbnailBlob')) {