Bug 1293822 - Have worker own /home/worker files; r=dustin

This effectively reverts the change in 2f0d6ea822b5 (bug 1163057)
to own the files by root:root. Since that change landed, we su
to the "worker" user early during task startup. So there is no more
need to have /home/worker owned by root:root.

MozReview-Commit-ID: 77q9APiDKpb

--HG--
extra : rebase_source : 6f45af9c51fa6c0f9f14ea183b008e38f9388664

testing/docker/desktop-test/Dockerfile

@@ -60,11 +60,6 @@ RUN npm install -g taskcluster-vcs@2.3.12 \
  && rm -rf ~/.npm
 ENV PATH $PATH:/home/worker/bin
 
-# Remove once running under 'worker' user.  This is necessary for pulseaudio to start
-# XXX: change this back to worker:worker once permissions issues are resolved
-RUN            chown -R root:root /home/worker
-
-
 # TODO Re-enable worker when bug 1093833 lands
 #USER          worker
 
@@ -97,5 +92,7 @@ ADD motd /etc/taskcluster-motd
 ADD taskcluster-interactive-shell /bin/taskcluster-interactive-shell
 RUN chmod +x /bin/taskcluster-interactive-shell
 
+RUN chown -R worker:worker /home/worker
+
 # Set a default command useful for debugging
 CMD ["/bin/bash", "--login"]

testing/docker/desktop1604-test/Dockerfile

@@ -61,11 +61,6 @@ RUN npm install -g npm@^2.0.0 \
  && rm -rf ~/.npm
 ENV PATH $PATH:/home/worker/bin
 
-# Remove once running under 'worker' user.  This is necessary for pulseaudio to start
-# XXX: change this back to worker:worker once permissions issues are resolved
-RUN            chown -R root:root /home/worker
-
-
 # TODO Re-enable worker when bug 1093833 lands
 #USER          worker
 
@@ -98,5 +93,7 @@ ADD motd /etc/taskcluster-motd
 ADD taskcluster-interactive-shell /bin/taskcluster-interactive-shell
 RUN chmod +x /bin/taskcluster-interactive-shell
 
+RUN chown -R worker:worker /home/worker
+
 # Set a default command useful for debugging
 CMD ["/bin/bash", "--login"]