Make favicon work properly when viewing job

Current implementation of controllers makes it hard to implement favicon
changes outside of build/job controllers. The problem is that if a job
is viewed, both controller are populated, which means they will both
send state change events to change favicon. This commit adds a check for
build controller, which allows to disable state change events when we're
viewing a job

app/controllers/build.coffee

@@ -8,6 +8,7 @@ Controller = Ember.Controller.extend GithubUrlPropertievs,
   commitBinding: 'build.commit'
   currentUserBinding: 'controllers.repo.currentUser'
   tabBinding: 'controllers.repo.tab'
+  sendFaviconStateChanges: true
 
   currentItemBinding: 'build'
 
@@ -24,7 +25,8 @@ Controller = Ember.Controller.extend GithubUrlPropertievs,
   ).property('commit.authorEmail')
 
   buildStateDidChange: (->
-    @send('faviconStateDidChange', @get('build.state'))
+    if @get('sendFaviconStateChanges')
+      @send('faviconStateDidChange', @get('build.state'))
   ).observes('build.state')
 
 `export default Controller`

app/routes/job.coffee

@@ -20,13 +20,24 @@ Route = TravisRoute.extend
 
     if build = model.get('build')
       build = @store.recordForId('build', build.get('id'))
-      @controllerFor('build').set('build', build)
+      buildController = @controllerFor('build')
+
+      # this is a hack to not set favicon changes from build
+      # controller while we're viewing job, this should go away
+      # after refactoring of controllers
+      buildController.set('sendFaviconStateChanges', false)
+
+      buildController.set('build', build)
 
   model: (params) ->
     @store.find('job', params.job_id)
 
   deactivate: ->
+    buildController = @controllerFor('build')
+    buildController.set('sendFaviconStateChanges', true)
     @controllerFor('build').set('build', null)
     @controllerFor('job').set('job', null)
 
+    @_super.apply(this, arguments)
+
 `export default Route`