isa/gnome-shell
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

modalDialog: drop global.get_current_time() arguments to close

Browse Source 
Modal dialogs take an optional timestamp in their close method.
If the timestamp is not passed in, then global.get_current_time()
is used.

Some callers of the close method pass in global.get_current_time()
unnecessarly (since it's the default).

This commit drops the argument for those cases.

https://bugzilla.gnome.org/show_bug.cgi?id=694296
This commit is contained in:
Ray Strode 2013-02-20 13:12:42 -05:00
parent 8640e18d1f
commit e9f18c6d8d
2 changed files with 4 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
js/ui/endSessionDialog.js
View File

@ -395,7 +395,7 @@ const EndSessionDialog = new Lang.Class({
cancel: function() {
this._stopTimer();
this._dbusImpl.emit_signal('Canceled', null);
this.close(global.get_current_time());
this.close();
},
_confirm: function(signal) {
@ -454,7 +454,7 @@ const EndSessionDialog = new Lang.Class({
let item = new ListItem(app, reason);
item.connect('activate',
Lang.bind(this, function() {
this.close(global.get_current_time());
this.close();
}));
this._applicationList.add(item.actor, { x_fill: true });
this._stopTimer();

4
js/ui/extensionDownloader.js
View File

@ -215,7 +215,7 @@ const InstallExtensionDialog = new Lang.Class({
},
_onCancelButtonPressed: function(button, event) {
this.close(global.get_current_time());
this.close();
this._invocation.return_value(GLib.Variant.new('(s)', ['cancelled']));
},
@ -257,7 +257,7 @@ const InstallExtensionDialog = new Lang.Class({
gotExtensionZipFile(session, message, uuid, dir, callback, errback);
}));
this.close(global.get_current_time());
this.close();
}
});