Skip to content
This repository has been archived by the owner on Nov 3, 2021. It is now read-only.

Commit

Permalink
Revert "Bug 1152969 - merge pull request #29689 from Cwiiis:bug115296…
Browse files Browse the repository at this point in the history
…9-can-save-empty-bookmark-name to mozilla-b2g:master"

This reverts commit 84de27d, reversing
changes made to 42fd693.
  • Loading branch information
rvandermeulen committed Apr 23, 2015
1 parent 6a7d642 commit 54739ed
Show file tree
Hide file tree
Showing 2 changed files with 1 addition and 10 deletions.
4 changes: 1 addition & 3 deletions apps/bookmark/js/bookmark_editor.js
Expand Up @@ -131,9 +131,7 @@ var BookmarkEditor = {
*/
_submit: function(event) {
event.preventDefault();
if (!this.saveButton.disabled) {
this.save();
}
this.save();
},

_clearTitle: function bookmarkEditor_clearTitle(event) {
Expand Down
7 changes: 0 additions & 7 deletions apps/bookmark/test/unit/bookmark_editor_test.js
Expand Up @@ -150,13 +150,6 @@ suite('bookmark_editor.js >', function() {
test('Bookmarks with blank title should not be saved >', function() {
assert.isTrue(BookmarkEditor.saveButton.disabled,
'Blank title, add button should be disabled');

var saveSpy = sinon.spy(BookmarkEditor, 'save');

BookmarkEditor.form.dispatchEvent(new CustomEvent('submit'));
assert.equal(saveSpy.callCount, 0);

BookmarkEditor.save.restore();
});

test('Check save button typing website name ', function() {
Expand Down

0 comments on commit 54739ed

Please sign in to comment.