Skip to content
This repository was archived by the owner on Sep 19, 2018. It is now read-only.

Issue #177: Update assert_readonly implementation to use ES5 features#178

Open
DLehenbauer wants to merge 2 commits intow3c:masterfrom
MicrosoftEdge:fix-assert-readonly
Open

Issue #177: Update assert_readonly implementation to use ES5 features#178
DLehenbauer wants to merge 2 commits intow3c:masterfrom
MicrosoftEdge:fix-assert-readonly

Conversation

@DLehenbauer
Copy link
Contributor

@DLehenbauer DLehenbauer commented Jan 22, 2016

Determines if the property is readonly by inspecting it's descriptor and verifying that attempts to set throw under strict mode.


This change is Reviewable

@hoppipolla-critic-bot
Copy link

Critic review: https://critic.hoppipolla.co.uk/r/6136

This is an external review system which you may optionally use for the code review of your pull request.

In order to help critic track your changes, please do not make in-place history rewrites (e.g. via git rebase -i or git commit --amend) when updating this pull request.

@gsnedders
Copy link
Member

Because mentions in the title aren't cross-referenced (thanks GitHub!): #177.

zqzhang pushed a commit to zqzhang/realsense-extensions-crosswalk that referenced this pull request Apr 11, 2016
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants