id	summary	reporter	owner	description	type	status	priority	milestone	component	version	resolution	keywords	cc
13300	The internalCommit argument in the image dialog seems to be totally unused	Piotrek Koszuliński		"The argument was added in #4246, because there's nothing about it in:

http://dev.ckeditor.com/browser/CKEditor/trunk/_source/plugins/image/dialogs/image.js?rev=4684

and plenty of it here:

http://dev.ckeditor.com/browser/CKEditor/trunk/_source/plugins/image/dialogs/image.js?rev=4686

For all these years I thought that it serves some purpose, but I cannot find any. In our tests and the one I performed manually `commit()` is never called with more arguments than two. Therefore I propose to remove these lines.

There's still a small chance that someone (perhaps some external plugin, because none of ours seem to rely on this argument either) uses this argument, so I propose to make this change in 4.5.0."	Bug	closed	Normal	CKEditor 4.5.0	General	3.0 RC	fixed