Add the ability to delete patchsets.

appengine/chromium_rietveld/new_static/components/cr-patchset-header.html

Index: appengine/chromium_rietveld/new_static/components/cr-patchset-header.html
diff --git a/appengine/chromium_rietveld/new_static/components/cr-patchset-header.html b/appengine/chromium_rietveld/new_static/components/cr-patchset-header.html
index a1be5a5ce120b34dff4211805e872c8fc6970237..a94acaba62e0607b809955c4504ee483db16ba0b 100644
--- a/appengine/chromium_rietveld/new_static/components/cr-patchset-header.html
+++ b/appengine/chromium_rietveld/new_static/components/cr-patchset-header.html
@@ -2,7 +2,7 @@
 Use of this source code is governed by a BSD-style license that can be
 found in the LICENSE file. -->
 
-<link rel="import" href="cr-patchset-title-dialog.html">
+<link rel="import" href="cr-patchset-edit-dialog.html">
 
 <polymer-element name="cr-patchset-header" attributes="patchset">
     <template>
@@ -31,11 +31,6 @@ found in the LICENSE file. -->
         <div class="patchset-title">
             Patch {{ patchset.sequence }}<template if="{{ patchset.title }}">: {{ patchset.title }}</template>
         </div>
-        <div class="patchset-actions">
-            <template if="{{ userIsOwner }}">
-                <cr-action on-tap="{{ showEditTitleDialog }}">Edit</cr-action>
-            </template>
-        </div>
         <div class="patchset-comments">
             {{ patchset.messageCount - patchset.draftCount | pluralize("comment") }}
             <template if="{{ patchset.draftCount }}">
@@ -44,6 +39,11 @@ found in the LICENSE file. -->
             </template>
         </div>
         <div class="patchset-date">{{ patchset.created | formatDate }}</div>
+        <div class="patchset-actions">
+            <template if="{{ userIsOwner }}">
+                <cr-action on-tap="{{ showEditDialog }}">Edit</cr-action>
+            </template>
+        </div>
     </template>
     <script>
         Polymer("cr-patchset-header", {
@@ -64,15 +64,20 @@ found in the LICENSE file. -->
                     return "";
                 return date.relative();
             },
-            showEditTitleDialog: function(event) {
+            showEditDialog: function(event) {
                 // Don't collapse the patchset.
                 event.stopPropagation();
-                var dialog = document.createElement("cr-patchset-title-dialog");
+                var dialog = document.createElement("cr-patchset-edit-dialog");
                 dialog.patchset = this.patchset;
                 dialog.addEventListener("close", function() {
                     dialog.remove();
                 });
-                document.body.appendChild(dialog);
+                // TODO(esprehn): This is a hack, we need to be a child of the
+                // issue to dispatch a "refresh-issue" event. Instead we should
+                // stop using that event and use a real data model API that
+                // components interact with like <cr-datasource>.
+                var dialogs = document.querySelector("html /deep/ cr-issue::shadow #dialogs");

[ojan, 2015/02/02 06:23:39]

ewww

[esprehn, 2015/02/02 06:28:52]

Yeah this is super gross. I need to switch to a different way to dealing with
the data model. Having it all be top down and using this event to reload the
entire page isn't working. I probably also want some kind of dialog controller.

+                dialogs.appendChild(dialog);
                 dialog.showModal();
             },
         });