Fixed the editing of previously edited files

Fixed the editing of previously edited files


BUG=chromium-os:23768
TEST=


Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=113389

[Vladislav Kaznacheev, 2011-12-06 18:30:16]

Please review. This patch solves the problem of editing a file that is already a
"Copy of ...". Not completely though, as it does not check if the new name is
taken.

[dgozman, 2011-12-07 06:02:09]

LGTM

http://codereview.chromium.org/8818020/diff/1/chrome/browser/resources/file_m...
File chrome/browser/resources/file_manager/js/image_editor/gallery.js (right):

http://codereview.chromium.org/8818020/diff/1/chrome/browser/resources/file_m...
chrome/browser/resources/file_manager/js/image_editor/gallery.js:877: new
RegExp('^' + Ribbon.Item.COPY_SIGNATURE + '(.+) \\((\\d+)\\)$');
You can extract regExp constants and place them near COPY_SIGNATURE.

[Vladislav Kaznacheev, 2011-12-07 13:11:13]

http://codereview.chromium.org/8818020/diff/1/chrome/browser/resources/file_m...
File chrome/browser/resources/file_manager/js/image_editor/gallery.js (right):

http://codereview.chromium.org/8818020/diff/1/chrome/browser/resources/file_m...
chrome/browser/resources/file_manager/js/image_editor/gallery.js:877: new
RegExp('^' + Ribbon.Item.COPY_SIGNATURE + '(.+) \\((\\d+)\\)$');
On 2011/12/07 06:02:09, dgozman wrote:
> You can extract regExp constants and place them near COPY_SIGNATURE.

Done.

[commit-bot: I haz the power, 2011-12-07 13:11:39]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/kaznacheev@chromium.org/8818020/4002

[commit-bot: I haz the power, 2011-12-07 14:24:27]

Change committed as 113389