Skip to content

Conversation

@pfrazee
Copy link
Contributor

@pfrazee pfrazee commented Sep 30, 2016

Per #2749

Adds the following webview tags:

  • disableimages
  • disablejavascript
  • disablewebgl
  • disablewebaudio
  • displayinsecurecontent
  • runinsecurecontent

@pfrazee
Copy link
Contributor Author

pfrazee commented Sep 30, 2016

I started to write some tests for this, but ran into problems. I started with images, and found that no error is emitted when images are disabled (loading fails silently).

Let me know what we want to do about that.

@zcbenz
Copy link
Contributor

zcbenz commented Oct 3, 2016

Adding attribute for every property in webPreferences makes the code hard to maintain, I think we should probably add a generic attribute to set webPreferences like the features string of window.open instead, or add the events as described in #2749 (comment).

@pfrazee
Copy link
Contributor Author

pfrazee commented Oct 3, 2016

Ok. I'll submit a new PR with a webpreferences string

On Oct 3, 2016, at 7:28 AM, Cheng Zhao [email protected] wrote:

Adding attribute for every property in webPreferences makes the code hard to maintain, I think we should probably add a generic attribute to set webPreferences like the features string of window.open instead, or add the events as described in #2749 (comment).


You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.

@pfrazee
Copy link
Contributor Author

pfrazee commented Oct 14, 2016

Closing. This PR is replace by #7631.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants