From 10ec924f8b4fa7354eb5144e19ac0b7e616e8793 Mon Sep 17 00:00:00 2001
From: NejcZdovc <nejc.zdovc@3zsistemi.si>
Date: Tue, 13 Jun 2017 09:07:49 +0200
Subject: [PATCH] Converts SiteInfo into redux component

Resolves #9412

Auditors: @bsclifton @bridiver

Test Plan:
---
 app/renderer/components/main/siteInfo.js | 37 +++++++++++++-----------
 1 file changed, 20 insertions(+), 17 deletions(-)

diff --git a/app/renderer/components/main/siteInfo.js b/app/renderer/components/main/siteInfo.js
index f23bedaf4a5..274ba2ddc34 100644
--- a/app/renderer/components/main/siteInfo.js
+++ b/app/renderer/components/main/siteInfo.js
@@ -3,17 +3,17 @@
  * You can obtain one at http://mozilla.org/MPL/2.0/. */
 
 const React = require('react')
-const PropTypes = require('prop-types')
 const {StyleSheet, css} = require('aphrodite/no-important')
 
 // Components
-const ImmutableComponent = require('../immutableComponent')
+const ReduxComponent = require('../reduxComponent')
 const Dialog = require('../common/dialog')
 const Button = require('../common/button')
 
 // Actions
 const appActions = require('../../../../js/actions/appActions')
 const webviewActions = require('../../../../js/actions/webviewActions')
+const windowActions = require('../../../../js/actions/windowActions')
 
 // Utils
 const cx = require('../../../../js/lib/classSet')
@@ -26,29 +26,37 @@ const platformUtil = require('../../../common/lib/platformUtil')
 const globalStyles = require('../styles/global')
 const commonStyles = require('../styles/commonStyles')
 
-class SiteInfo extends ImmutableComponent {
-  constructor () {
-    super()
+class SiteInfo extends React.Component {
+  constructor (props) {
+    super(props)
     this.onAllowRunInsecureContent = this.onAllowRunInsecureContent.bind(this)
     this.onDenyRunInsecureContent = this.onDenyRunInsecureContent.bind(this)
     this.onViewCertificate = this.onViewCertificate.bind(this)
   }
+
   onAllowRunInsecureContent () {
     appActions.changeSiteSetting(siteUtil.getOrigin(this.location),
       'runInsecureContent', true, this.isPrivate)
     appActions.loadURLRequested(this.props.frameProps.get('tabId'), this.location)
-    this.props.onHide()
+    this.onHide()
   }
+
   onDenyRunInsecureContent () {
     appActions.removeSiteSetting(siteUtil.getOrigin(this.location),
       'runInsecureContent', this.isPrivate)
     appActions.loadURLRequested(this.props.frameProps.get('tabId'), this.location)
-    this.props.onHide()
+    this.onHide()
   }
+
   onViewCertificate () {
-    this.props.onHide()
+    this.onHide()
     webviewActions.showCertificate()
   }
+
+  onHide () {
+    windowActions.setSiteInfoVisible(false)
+  }
+
   get isExtendedValidation () {
     return this.props.frameProps.getIn(['security', 'isExtendedValidation'])
   }
@@ -183,7 +191,7 @@ class SiteInfo extends ImmutableComponent {
             <Button l10nId='dismissAllowRunInsecureContent'
               className='primaryButton'
               testId='dismissAllowRunInsecureContentButton'
-              onClick={this.props.onHide}
+              onClick={this.onHide}
             />
           </div>
           {viewCertificateButton}
@@ -196,7 +204,7 @@ class SiteInfo extends ImmutableComponent {
             <Button l10nId='dismissDenyRunInsecureContent'
               className='whiteButton'
               testId='dismissDenyRunInsecureContentButton'
-              onClick={this.props.onHide}
+              onClick={this.onHide}
             />
             <Button l10nId='denyRunInsecureContent'
               className='primaryButton'
@@ -231,7 +239,7 @@ class SiteInfo extends ImmutableComponent {
         </div>
     }
 
-    return <Dialog onHide={this.props.onHide} className='siteInfo' isClickDismiss>
+    return <Dialog onHide={this.onHide} className='siteInfo' isClickDismiss>
       <div onClick={(e) => e.stopPropagation()}
         className={cx({
           [css(commonStyles.flyoutDialog)]: true,
@@ -252,10 +260,7 @@ class SiteInfo extends ImmutableComponent {
   }
 }
 
-SiteInfo.propTypes = {
-  frameProps: PropTypes.object,
-  onHide: PropTypes.func
-}
+module.exports = ReduxComponent.connect(SiteInfo)
 
 const styles = StyleSheet.create({
   flexJustifyEnd: {
@@ -302,5 +307,3 @@ const styles = StyleSheet.create({
     maxWidth: '500px'
   }
 })
-
-module.exports = SiteInfo