From 59246babea450635dde14b39be1492ff61e029c6 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Florian=20M=C3=BCllner?= Date: Mon, 11 Jun 2012 15:19:56 +0200 Subject: [PATCH] search: Remove createResultContainer() hook It is now unused since the contacts search provider was the only consumer, so remove it. https://bugzilla.gnome.org/show_bug.cgi?id=677442 --- js/ui/search.js | 13 ------------- js/ui/searchDisplay.js | 5 +---- 2 files changed, 1 insertion(+), 17 deletions(-) diff --git a/js/ui/search.js b/js/ui/search.js index df5e192ab..447a4395d 100644 --- a/js/ui/search.js +++ b/js/ui/search.js @@ -141,19 +141,6 @@ const SearchProvider = new Lang.Class({ throw new Error('Not implemented'); }, - /** - * createResultContainer: - * - * Search providers may optionally override this to render their - * results in a custom fashion. The default implementation - * will create a vertical list. - * - * Returns: An instance of SearchResultDisplay. - */ - createResultContainerActor: function() { - return null; - }, - /** * createResultActor: * @resultMeta: Object with result metadata diff --git a/js/ui/searchDisplay.js b/js/ui/searchDisplay.js index 9ca5198f9..508751eba 100644 --- a/js/ui/searchDisplay.js +++ b/js/ui/searchDisplay.js @@ -275,10 +275,7 @@ const SearchResults = new Lang.Class({ x_fill: true, y_fill: true }); providerBox.add(resultDisplayBin, { expand: true }); - let resultDisplay = provider.createResultContainerActor(); - if (resultDisplay == null) { - resultDisplay = new GridSearchResults(provider); - } + let resultDisplay = new GridSearchResults(provider); resultDisplayBin.set_child(resultDisplay.actor); this._providerMeta.push({ provider: provider,