From d0f1466cddd77685e68f54046eaeaacd36e226e1 Mon Sep 17 00:00:00 2001 From: Dave Miller Date: Wed, 17 Feb 2016 04:04:59 -0800 Subject: [PATCH] Default javaScriptEnabled to true Summary:public Default javaScriptEnabled to true on Android WebView Also remove an old (about 6 weeks old) warning about a back compat supported property Reviewed By: nicklockwood Differential Revision: D2939482 fb-gh-sync-id: 2d476c3365f657da27ea370a033b23154750c2ea shipit-source-id: 2d476c3365f657da27ea370a033b23154750c2ea --- .../Components/WebView/WebView.android.js | 20 ++++++++----------- Libraries/Components/WebView/WebView.ios.js | 10 ---------- 2 files changed, 8 insertions(+), 22 deletions(-) diff --git a/Libraries/Components/WebView/WebView.android.js b/Libraries/Components/WebView/WebView.android.js index 2c8e2312d..e9577463f 100644 --- a/Libraries/Components/WebView/WebView.android.js +++ b/Libraries/Components/WebView/WebView.android.js @@ -142,6 +142,12 @@ var WebView = React.createClass({ }; }, + getDefaultProps: function() { + return { + javaScriptEnabled : true, + }; + }, + componentWillMount: function() { if (this.props.startInLoadingState) { this.setState({viewState: WebViewState.LOADING}); @@ -170,16 +176,6 @@ var WebView = React.createClass({ webViewStyles.push(styles.hidden); } - var {javaScriptEnabled, domStorageEnabled} = this.props; - if (this.props.javaScriptEnabledAndroid) { - console.warn('javaScriptEnabledAndroid is deprecated. Use javaScriptEnabled instead'); - javaScriptEnabled = this.props.javaScriptEnabledAndroid; - } - if (this.props.domStorageEnabledAndroid) { - console.warn('domStorageEnabledAndroid is deprecated. Use domStorageEnabled instead'); - domStorageEnabled = this.props.domStorageEnabledAndroid; - } - var source = this.props.source || {}; if (this.props.html) { source.html = this.props.html; @@ -201,8 +197,8 @@ var WebView = React.createClass({ source={resolveAssetSource(source)} injectedJavaScript={this.props.injectedJavaScript} userAgent={this.props.userAgent} - javaScriptEnabled={javaScriptEnabled} - domStorageEnabled={domStorageEnabled} + javaScriptEnabled={this.props.javaScriptEnabled} + domStorageEnabled={this.props.domStorageEnabled} contentInset={this.props.contentInset} automaticallyAdjustContentInsets={this.props.automaticallyAdjustContentInsets} onLoadingStart={this.onLoadingStart} diff --git a/Libraries/Components/WebView/WebView.ios.js b/Libraries/Components/WebView/WebView.ios.js index 3b37331ae..ec6340213 100644 --- a/Libraries/Components/WebView/WebView.ios.js +++ b/Libraries/Components/WebView/WebView.ios.js @@ -285,16 +285,6 @@ var WebView = React.createClass({ RCTWebViewManager.startLoadWithResult(!!shouldStart, event.nativeEvent.lockIdentifier); }); - var {javaScriptEnabled, domStorageEnabled} = this.props; - if (this.props.javaScriptEnabledAndroid) { - console.warn('javaScriptEnabledAndroid is deprecated. Use javaScriptEnabled instead'); - javaScriptEnabled = this.props.javaScriptEnabledAndroid; - } - if (this.props.domStorageEnabledAndroid) { - console.warn('domStorageEnabledAndroid is deprecated. Use domStorageEnabled instead'); - domStorageEnabled = this.props.domStorageEnabledAndroid; - } - var decelerationRate = processDecelerationRate(this.props.decelerationRate); var source = this.props.source || {};