| Index: Tools/GardeningServer/ui/ct-results-detail.html
|
| diff --git a/Tools/GardeningServer/ui/ct-results-detail.html b/Tools/GardeningServer/ui/ct-results-detail.html
|
| deleted file mode 100644
|
| index b5230a46dc40c580dd87a7a810e5304c20eefa55..0000000000000000000000000000000000000000
|
| --- a/Tools/GardeningServer/ui/ct-results-detail.html
|
| +++ /dev/null
|
| @@ -1,124 +0,0 @@
|
| -<!--
|
| -Copyright 2014 The Chromium Authors. All rights reserved.
|
| -Use of this source code is governed by a BSD-style license that can be
|
| -found in the LICENSE file.
|
| --->
|
| -
|
| -<link rel="import" href="ct-results-comparison.html">
|
| -
|
| -<polymer-element name="ct-results-detail" attributes="failure builder">
|
| - <template>
|
| - <style>
|
| - :host {
|
| - display: block;
|
| - }
|
| - </style>
|
| - <template if="{{!_urlGroups.length}}">
|
| - No results to display.
|
| - </template>
|
| - <template repeat="{{urlGroup in _urlGroups}}">
|
| - <template if="{{urlGroup.urls[_kUnknownKind]}}">
|
| - <ct-test-output type="{{urlGroup.type}}" url="{{urlGroup.urls[_kUnknownKind]}}"></ct-test-output>
|
| - </template>
|
| - <template if="{{!urlGroup.urls[_kUnknownKind]}}">
|
| - <ct-results-comparison type="{{urlGroup.type}}" expectedUrl="{{urlGroup.urls[_kExpectedKind]}}"
|
| - actualUrl="{{urlGroup.urls[_kActualKind]}}" diffUrl="{{urlGroup.urls[_kDiffKind]}}"></ct-results-comparison>
|
| - </template>
|
| - </template>
|
| - </template>
|
| - <script>
|
| - Polymer({
|
| - failure: null,
|
| - // FIXME: Initializing builder gives a JS error. Presumably because
|
| - // ct-results-by-builder sets builder="{{builders[selected]}}". But,
|
| - // it seems wrong that the way the parent uses this element constrains
|
| - // what the element can do. Polymer bug?
|
| - // builder: '',
|
| -
|
| - _urlGroups: [],
|
| - _kExpectedKind: results.kExpectedKind,
|
| - _kActualKind: results.kActualKind,
|
| - _kDiffKind: results.kDiffKind,
|
| - _kUnknownKind: results.kUnknownKind,
|
| -
|
| - observe: {
|
| - failure: '_update',
|
| - builder: '_update',
|
| - },
|
| -
|
| - _isStdioStep: function(result, step) {
|
| - return result.actual == 'UNKNOWN' || step == 'compile';
|
| - },
|
| -
|
| - _update: function() {
|
| - if (!this.failure || !this.builder)
|
| - return;
|
| -
|
| - // FIXME: If the types of groups doesn't change, then it'd be better to do this
|
| - // update in place so that the user doesn't see a flicker.
|
| - this._urlGroups = [];
|
| -
|
| - var result = this.failure.resultNodesByBuilder[this.builder];
|
| - // FIXME: There's probably a less hacky way to check this.
|
| - if (result.actual == 'FAIL' || this._isStdioStep(result, this.failure.step))
|
| - this._updateUrls();
|
| - else
|
| - this._updateWebkitTestUrls();
|
| - },
|
| -
|
| - _updateWebkitTestUrls: function() {
|
| - var result = this.failure.resultNodesByBuilder[this.builder];
|
| - var failureInfo = results.failureInfo(this.failure.testName, this.builder, result.actual);
|
| -
|
| - // FIXME: Move this logic to a proper model class so that the network requests this makes
|
| - // can be easily mocked out in tests.
|
| - results.fetchResultsURLs(failureInfo).then(function(resultsUrls) {
|
| - var resultsUrlsByTypeAndKind = {};
|
| - resultsUrls.forEach(function(url) {
|
| - var resultType = results.resultType(url);
|
| - if (!resultsUrlsByTypeAndKind[resultType])
|
| - resultsUrlsByTypeAndKind[resultType] = {};
|
| - resultsUrlsByTypeAndKind[resultType][results.resultKind(url)] = url;
|
| - });
|
| -
|
| - Object.keys(resultsUrlsByTypeAndKind, function(resultType, resultsUrlsByKind) {
|
| - this._urlGroups.push({
|
| - type: resultType,
|
| - urls: resultsUrlsByKind,
|
| - });
|
| - }.bind(this));
|
| - }.bind(this));
|
| - },
|
| -
|
| - _updateUrls: function() {
|
| - // FIXME: Along with _updateWebkitTestUrls, move this logic to a proper model class
|
| - // so that the network requests this makes can be easily mocked out in tests.
|
| -
|
| - var result = this.failure.resultNodesByBuilder[this.builder];
|
| -
|
| - // FIXME: We only store build logs by the test name, not the testsuite.testname,
|
| - // which means that two failing tests from different test suites conflict!
|
| - var testPart = this._isStdioStep(result, this.failure.step) ? 'stdio' : this.failure.testName.split('.')[1];
|
| - // Fix url for parameterized gtests, e.g.: /2 -> _2.
|
| - testPart = testPart.replace(/\/(\d+)$/, "_$1");
|
| - var url = result.masterUrl +
|
| - '/builders/' + encodeURIComponent(this.builder);
|
| -
|
| - // FIXME: Make failure groups aware of their own url
|
| - if (this.failure.testName)
|
| - url +=
|
| - '/builds/' + result.lastFailingBuild +
|
| - '/steps/' + this.failure.step +
|
| - '/logs/' + testPart;
|
| -
|
| - var resultsUrlsByKind = {};
|
| - resultsUrlsByKind[this._kUnknownKind] = url;
|
| -
|
| - this._urlGroups.push({
|
| - type: results.kTextType,
|
| - urls: resultsUrlsByKind,
|
| - });
|
| - },
|
| - });
|
| - </script>
|
| -</polymer-element>
|
|
|