Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(358)

Side by Side Diff: appengine/chromium_rietveld/new_static/components/cr-issue-patchset.html

Issue 1152723005: Expand patchset if patchsetID provided in URL (Closed) Base URL: https://chromium.googlesource.com/infra/infra@master
Patch Set: Display patchset id Created 5 years, 6 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
OLDNEW
1 <!-- Copyright (c) 2014 The Chromium Authors. All rights reserved. 1 <!-- Copyright (c) 2014 The Chromium Authors. All rights reserved.
2 Use of this source code is governed by a BSD-style license that can be 2 Use of this source code is governed by a BSD-style license that can be
3 found in the LICENSE file. --> 3 found in the LICENSE file. -->
4 4
5 <link rel="import" href="../common/cr-action.html"> 5 <link rel="import" href="../common/cr-action.html">
6 <link rel="import" href="../common/cr-toolbar.html"> 6 <link rel="import" href="../common/cr-toolbar.html">
7 <link rel="import" href="cr-issue-cq-dry-run-dialog.html"> 7 <link rel="import" href="cr-issue-cq-dry-run-dialog.html">
8 <link rel="import" href="cr-issue-patchfile.html"> 8 <link rel="import" href="cr-issue-patchfile.html">
9 <link rel="import" href="cr-try-results.html"> 9 <link rel="import" href="cr-try-results.html">
10 <link rel="import" href="../diff/cr-diff-controls.html"> 10 <link rel="import" href="../diff/cr-diff-controls.html">
(...skipping 46 matching lines...) Expand 10 before | Expand all | Expand 10 after
57 </h2> 57 </h2>
58 <template if="{{ tryJobResults.length }}"> 58 <template if="{{ tryJobResults.length }}">
59 <cr-try-results results="{{ tryJobResults }}"></cr-try-resul ts> 59 <cr-try-results results="{{ tryJobResults }}"></cr-try-resul ts>
60 </template> 60 </template>
61 <template if="{{ patchset.mostRecent && !patchset.issue.closed & & !patchset.issue.commit }}"> 61 <template if="{{ patchset.mostRecent && !patchset.issue.closed & & !patchset.issue.commit }}">
62 <a class="choose-trybots" is="cr-action" on-tap="{{ handleAd dMore }}">Choose try jobs</a> / 62 <a class="choose-trybots" is="cr-action" on-tap="{{ handleAd dMore }}">Choose try jobs</a> /
63 <a class="trigger-dry-run" is="cr-action" on-tap="{{ showDry RunDialog }}">CQ dry run</a> 63 <a class="trigger-dry-run" is="cr-action" on-tap="{{ showDry RunDialog }}">CQ dry run</a>
64 </template> 64 </template>
65 </div> 65 </div>
66 </template> 66 </template>
67 <a class="patchset-id">(Patchset Id: {{ patchset.id }})</a>
jrobbins 2015/06/02 19:50:44 I realize you tried to limit visual clutter by mak
rmistry 2015/06/02 20:35:54 Done. We can revisit this if users complain about
67 <div id="dialogs"></div> 68 <div id="dialogs"></div>
68 </template> 69 </template>
69 <script> 70 <script>
70 Polymer({ 71 Polymer({
71 created: function() { 72 created: function() {
72 this.patchset = null; 73 this.patchset = null;
73 this.tryJobResults = null; 74 this.tryJobResults = null;
74 }, 75 },
75 showDryRunDialog: function() { 76 showDryRunDialog: function() {
76 if (this.patchset.issue.closed) 77 if (this.patchset.issue.closed)
(...skipping 31 matching lines...) Expand 10 before | Expand all | Expand 10 after
108 return Promise.all(this.getFiles().map(function(file) { 109 return Promise.all(this.getFiles().map(function(file) {
109 return file.hideDiff(); 110 return file.hideDiff();
110 })); 111 }));
111 }, 112 },
112 handleAddMore: function() { 113 handleAddMore: function() {
113 this.fire("show-try-dialog"); 114 this.fire("show-try-dialog");
114 }, 115 },
115 }); 116 });
116 </script> 117 </script>
117 </polymer-element> 118 </polymer-element>
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698