mozilla/treeherder-ui — pull request #269 by tojonmz (2014-11-12 23:07:55+00:00)
Title: Bug 1062825 - Include job details in Log viewer page title

This work fixes Bugzilla bug [1062825](https://bugzilla.mozilla.org/show_bug.cgi?id=1062825).

In it, we add a descriptive page title for Log viewer, so primarily on hover, they can be differentiated.

Because angular template flicker in `

` cannot be presently suppressed by `ng-cloak` we instead add a static title and then switch it on page load to the buildApi data value. It's not a perfect transition, but it does result in a readable title on initial page rendering. Just like the main Treeherder index.html, it undergoes a brief flicker (on my local server to its http:// value), then to the final desired value.

Here is before and after the fix:

![logviewertitlecurrent](https://cloud.githubusercontent.com/assets/3660661/5020265/721eb722-6a95-11e4-9de0-d5b2c4aebc26.jpg)

![logviewertitleproposed](https://cloud.githubusercontent.com/assets/3660661/5020268/74aa924a-6a95-11e4-8382-5f52305e70cd.jpg)

I've tested it in a Chrome browser also, and the behavior seems to be generally similar, and fine. I ended up moving the entire html under the logviewer controller, as the new function lies within in the init(). I am not sure if this has an appreciable performance differences, but I couldn't see any.

When it did not exist inside that init() the new getter wasn't callable, as far as I could tell.

Tested on Windows:
FF Release **33.1**
Chrome Latest Release **38.0.2125.111 m**

Adding @camd for review and @edmorley for visibility.