Fix visibility of the PhabricatorWorker::doWork() methods

Summary: Ref T6822. This method is only called from within the `PhabricatorWorker::executeTask()` and `PhabricatorWorker::scheduleTask()` methods.

Test Plan: `grep`ped for `->doWork`.

Reviewers: #blessed_reviewers, epriestley

Reviewed By: #blessed_reviewers, epriestley

Subscribers: Korvin, epriestley

Maniphest Tasks: T6822

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

src/applications/harbormaster/worker/HarbormasterBuildWorker.php

@@ -5,7 +5,7 @@
  */
 final class HarbormasterBuildWorker extends HarbormasterWorker {
 
-  public function doWork() {
+  protected function doWork() {
     $data = $this->getTaskData();
     $id = idx($data, 'buildID');
     $viewer = $this->getViewer();

src/applications/harbormaster/worker/HarbormasterTargetWorker.php

@@ -30,7 +30,7 @@ final class HarbormasterTargetWorker extends HarbormasterWorker {
     return $target;
   }
 
-  public function doWork() {
+  protected function doWork() {
     $target = $this->loadBuildTarget();
     $build = $target->getBuild();
     $viewer = $this->getViewer();

src/applications/metamta/PhabricatorMetaMTAWorker.php

@@ -11,7 +11,7 @@ final class PhabricatorMetaMTAWorker
     return ($task->getFailureCount() * 15);
   }
 
-  public function doWork() {
+  protected function doWork() {
     $message = $this->loadMessage();
     if (!$message) {
       throw new PhabricatorWorkerPermanentFailureException(

src/applications/repository/worker/PhabricatorRepositoryCommitParserWorker.php

@@ -27,7 +27,7 @@ abstract class PhabricatorRepositoryCommitParserWorker
     return $this->commit = $commit;
   }
 
-  final public function doWork() {
+  final protected function doWork() {
     if (!$this->loadCommit()) {
       return;
     }

src/applications/repository/worker/PhabricatorRepositoryPushMailWorker.php

@@ -3,7 +3,7 @@
 final class PhabricatorRepositoryPushMailWorker
   extends PhabricatorWorker {
 
-  public function doWork() {
+  protected function doWork() {
     $viewer = PhabricatorUser::getOmnipotentUser();
 
     $task_data = $this->getTaskData();

src/applications/search/worker/PhabricatorSearchWorker.php

@@ -2,7 +2,7 @@
 
 final class PhabricatorSearchWorker extends PhabricatorWorker {
 
-  public function doWork() {
+  protected function doWork() {
     $data = $this->getTaskData();
 
     $phid = idx($data, 'documentPHID');

src/infrastructure/sms/worker/PhabricatorSMSDemultiplexWorker.php

@@ -3,7 +3,7 @@
 final class PhabricatorSMSDemultiplexWorker
   extends PhabricatorSMSWorker {
 
-  public function doWork() {
+  protected function doWork() {
     $viewer = PhabricatorUser::getOmnipotentUser();
 
     $task_data = $this->getTaskData();

src/infrastructure/sms/worker/PhabricatorSMSSendWorker.php

@@ -11,7 +11,7 @@ final class PhabricatorSMSSendWorker
     return phutil_units('1 minute in seconds');
   }
 
-  public function doWork() {
+  protected function doWork() {
     $viewer = PhabricatorUser::getOmnipotentUser();
 
     $task_data = $this->getTaskData();