googlearchive / chromedeveditor

Chrome Dev Editor is a developer tool for building apps on the Chrome platform - Chrome Apps and Web Apps, in JavaScript or Dart. (NO LONGER IN ACTIVE DEVELOPMENT)
BSD 3-Clause "New" or "Revised" License
2.92k stars 411 forks source link

System keyboard shortcut for changing keyboard layout crashes ace.js #1216

Open ussuri opened 10 years ago

ussuri commented 10 years ago

This happens at least on Linux, I'm going to check other OSes too.

The console says: "Breaking on exception: TypeError: Object 48 has no method 'toLowerCase'".

The stack is almost useless, because the source is minified, but it's this: t.codeToKeyString() onCommandKey() onCommandKey() b() o() ()

This might be impossible to fix, since it's not our code, but is worth investigating.

dinhvh commented 10 years ago

similarly, Ctrl/Cmd-L calls prompt(), which is not allowed on Chrome Apps. There's a few things with ACE going on in Spark.