Skip to content
This repository has been archived by the owner on Dec 11, 2019. It is now read-only.

Commit

Permalink
removes update to preview from preferences
Browse files Browse the repository at this point in the history
Fix #11638
  • Loading branch information
kjozwiak committed Oct 23, 2017
1 parent 5484ba6 commit e7b6947
Show file tree
Hide file tree
Showing 7 changed files with 7 additions and 20,561 deletions.
3 changes: 1 addition & 2 deletions app/index.js
Original file line number Diff line number Diff line change
Expand Up @@ -309,13 +309,12 @@ app.on('ready', () => {
})
// DO NOT TO THIS LIST - see above

// We need the initial state to read the UPDATE_TO_PREVIEW_RELEASES preference
loadAppStatePromise.then((initialImmutableState) => {
updater.init(
process.platform,
process.arch,
process.env.BRAVE_UPDATE_VERSION || app.getVersion(),
initialImmutableState.getIn(['settings', settings.UPDATE_TO_PREVIEW_RELEASES])
process.env.BRAVE_ENABLE_PREVIEW_UPDATES !== undefined
)

// This is fired by a menu entry
Expand Down
14 changes: 0 additions & 14 deletions app/renderer/components/preferences/advancedTab.js
Original file line number Diff line number Diff line change
Expand Up @@ -21,27 +21,13 @@ const {scaleSize} = require('../../../common/constants/toolbarUserInterfaceScale

// Utils
const {changeSetting} = require('../../lib/settingsUtil')
const platformUtil = require('../../../common/lib/platformUtil')
const isLinux = platformUtil.isLinux()

class AdvancedTab extends ImmutableComponent {
previewReleases () {
return isLinux
? null
: <SettingCheckbox
dataL10nId='updateToPreviewReleases'
data-test-id='update-to-preview-releases'
prefKey={settings.UPDATE_TO_PREVIEW_RELEASES}
settings={this.props.settings}
onChangeSetting={this.props.onChangeSetting} />
}

render () {
return <section>
<main className={css(styles.advancedTabMain)}>
<DefaultSectionTitle data-l10n-id='contentSettings' />
<SettingsList>
{this.previewReleases()}
<SettingCheckbox dataL10nId='useHardwareAcceleration' prefKey={settings.HARDWARE_ACCELERATION_ENABLED} settings={this.props.settings} onChangeSetting={this.props.onChangeSetting} />
<SettingCheckbox dataL10nId='useSmoothScroll' prefKey={settings.SMOOTH_SCROLL_ENABLED} settings={this.props.settings} onChangeSetting={this.props.onChangeSetting} />
<SettingCheckbox dataL10nId='sendCrashReports' prefKey={settings.SEND_CRASH_REPORTS} settings={this.props.settings} onChangeSetting={this.props.onChangeSetting} />
Expand Down
3 changes: 1 addition & 2 deletions js/about/preferences.js
Original file line number Diff line number Diff line change
Expand Up @@ -838,8 +838,7 @@ class AboutPreferences extends React.Component {
settings.PDFJS_ENABLED,
settings.TORRENT_VIEWER_ENABLED,
settings.SMOOTH_SCROLL_ENABLED,
settings.SEND_CRASH_REPORTS,
settings.UPDATE_TO_PREVIEW_RELEASES
settings.SEND_CRASH_REPORTS
]
if (settingsRequiringRestart.includes(key)) {
ipc.send(messages.PREFS_RESTART, key, value)
Expand Down
1 change: 0 additions & 1 deletion js/constants/appConfig.js
Original file line number Diff line number Diff line change
Expand Up @@ -193,7 +193,6 @@ module.exports = {
'advanced.smooth-scroll-enabled': false,
'advanced.send-crash-reports': true,
'advanced.send-usage-statistics': false,
'advanced.update-to-preview-releases': false,
'advanced.toolbar-ui-scale': 'normal',
'shutdown.clear-history': false,
'shutdown.clear-downloads': false,
Expand Down
1 change: 0 additions & 1 deletion js/constants/settings.js
Original file line number Diff line number Diff line change
Expand Up @@ -75,7 +75,6 @@ const settings = {
SMOOTH_SCROLL_ENABLED: 'advanced.smooth-scroll-enabled',
SEND_CRASH_REPORTS: 'advanced.send-crash-reports',
SEND_USAGE_STATISTICS: 'advanced.send-usage-statistics',
UPDATE_TO_PREVIEW_RELEASES: 'advanced.update-to-preview-releases',
ADBLOCK_CUSTOM_RULES: 'adblock.customRules',
TOOLBAR_UI_SCALE: 'advanced.toolbar-ui-scale',
// Sync settings
Expand Down
Loading

0 comments on commit e7b6947

Please sign in to comment.