Move incognito newtab check into props

Auditors: @bridiver Related to #7912
brave · Mar 29, 2017 · ecca4ff · ecca4ff
1 parent e6fba42
commit ecca4ff
Show file tree

Hide file tree

Showing 2 changed files with 12 additions and 3 deletions.
diff --git a/js/about/newtab.js b/js/about/newtab.js
@@ -235,7 +235,7 @@ class NewTabPage extends React.Component {
       return <div className='empty' />
     }
 
-    if (window.chrome.extension.inIncognitoContext) {
+    if (this.props.isIncognito) {
       return <div className='empty' />
     }
 
@@ -312,5 +312,7 @@ class NewTabPage extends React.Component {
 
 module.exports = {
   component: NewTabPage,
-  AboutNewTab: React.createElement(DragDropContext(HTML5Backend)(NewTabPage))
+  AboutNewTab: React.createElement(DragDropContext(HTML5Backend)(NewTabPage), {
+    isIncognito: window.chrome && window.chrome.extension && window.chrome.extension.inIncognitoContext
+  })
 }
diff --git a/test/unit/about/newTabPageTest.js b/test/unit/about/newTabPageTest.js
@@ -42,7 +42,7 @@ describe('NewTab component unit tests', function () {
     randomSpy.restore()
   })
 
-  let wrapper
+  let wrapper, incognitoWrapper
   const backgroundImage = {
     style: {
       backgroundImage: 'url(testing123.jpg)'
@@ -80,6 +80,9 @@ describe('NewTab component unit tests', function () {
     wrapper = shallow(
       <NewTabPage />
     )
+    incognitoWrapper = shallow(
+      <NewTabPage isIncognito />
+    )
   })
 
   describe('Object properties', function () {
@@ -148,6 +151,10 @@ describe('NewTab component unit tests', function () {
         wrapper.setState({showEmptyPage: true})
         assert.equal(wrapper.find('div.empty').length, 1)
       })
+
+      it('renders an empty div if isIncognito props is true', function () {
+        assert.equal(incognitoWrapper.find('div.empty').length, 1)
+      })
     })
 
     describe('dashboard', function () {