In general, one should avoid changing third party library code directly (to make upgrades easier). In this case, I don't think there is a good alternative, so I am OK with this change. It would be nice to make the commit message for tiny_mce_popup.js specific to just changing the dialog size from 10 to 100 (so this PR would have multiple commits), but that is a nit.