[Redesign] Remove AphrontTable restriction on Search Engine

Summary: Ref T8099, functionally I prefer to be able to set anything 'table-like' with `setTable` for design consistency. This looses the restriction and did some light grepping for other missed cases.

Test Plan: Test new UI, grep for other missing cases.

Reviewers: btrahan, epriestley

Reviewed By: epriestley

Subscribers: epriestley, Korvin

Maniphest Tasks: T8099

Differential Revision: https://secure.phabricator.com/D13471

src/applications/harbormaster/controller/HarbormasterBuildableViewController.php

@@ -303,7 +303,7 @@ final class HarbormasterBuildableViewController
 
       $lint = id(new PHUIObjectBoxView())
         ->setHeader($lint_header)
-        ->appendChild($lint_table);
+        ->setTable($lint_table);
     } else {
       $lint = null;
     }
@@ -327,7 +327,7 @@ final class HarbormasterBuildableViewController
 
       $unit = id(new PHUIObjectBoxView())
         ->setHeader($unit_header)
-        ->appendChild($unit_table);
+        ->setTable($unit_table);
     } else {
       $unit = null;
     }

src/applications/harbormaster/controller/HarbormasterUnitMessagesController.php

@@ -40,7 +40,7 @@ final class HarbormasterUnitMessagesController
 
     $unit = id(new PHUIObjectBoxView())
       ->setHeaderText(pht('Unit Tests'))
-      ->appendChild($unit_table);
+      ->setTable($unit_table);
 
     $crumbs = $this->buildApplicationCrumbs();
     $this->addBuildableCrumb($crumbs, $buildable);

src/applications/owners/controller/PhabricatorOwnersDetailController.php

@@ -276,7 +276,7 @@ final class PhabricatorOwnersDetailController
 
     $box = id(new PHUIObjectBoxView())
       ->setHeaderText(pht('Paths'))
-      ->appendChild($table);
+      ->setTable($table);
 
     if ($info) {
       $box->setInfoView($info);

src/applications/people/query/PhabricatorPeopleLogSearchEngine.php

@@ -187,8 +187,7 @@ final class PhabricatorPeopleLogSearchEngine
     }
 
     $result = new PhabricatorApplicationSearchResultView();
-    $result->setContent($table);
-    $result->setCollapsed(true);
+    $result->setTable($table);
 
     return $result;
   }

src/applications/repository/query/PhabricatorRepositoryPushLogSearchEngine.php

@@ -103,13 +103,13 @@ final class PhabricatorRepositoryPushLogSearchEngine
     PhabricatorSavedQuery $query,
     array $handles) {
 
-    $content = id(new DiffusionPushLogListView())
+    $table = id(new DiffusionPushLogListView())
       ->setUser($this->requireViewer())
       ->setHandles($handles)
       ->setLogs($logs);
 
     return id(new PhabricatorApplicationSearchResultView())
-      ->setContent($content);
+      ->setTable($table);
   }
 
 }

src/applications/search/view/PhabricatorApplicationSearchResultView.php

@@ -34,7 +34,7 @@ final class PhabricatorApplicationSearchResultView extends Phobject {
     return $list;
   }
 
-  public function setTable(AphrontTableView $table) {
+  public function setTable($table) {
     $this->table = $table;
     return $this;
   }