To this pull request be merge, there is 3 possibilities: 
1 remove this peace of code, because as you said it does nothing and let one test fail. 
2  override this code to not update content that is not editable, as in the comment isContentEditable() do not work as expected in my tests
3  Remove this code and remove the "should not update test" ignoring this error. The browser don't enable fields for edit, on this case, only direct code can do this.

I leave the code to use the pull request for discussions.