Add the ability to delete patchsets.

appengine/chromium_rietveld/new_static/components/cr-patchset-edit-dialog.html

 <!-- Copyright (c) 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="cr-butterbar.html">
 <link rel="import" href="cr-button.html">
 <link rel="import" href="cr-dialog.html">
 <link rel="import" href="cr-toolbar.html">
 
-<polymer-element name="cr-patchset-title-dialog" attributes="patchset">
+<polymer-element name="cr-patchset-edit-dialog" attributes="patchset">
     <template>
         <link rel="stylesheet" href="common.css">
         <link rel="stylesheet" href="forms.css">
 
+        <style>
+            img {
+                display: block;
+            }
+
+            .delete-action {
+                flex: 1;
+                display: flex;
+                justify-content: flex-end;
+            }
+
+            .delete-action > cr-button {
+                margin-right: 0;
+            }
+
+            .delete-confirmation {
+                display: flex;
+                align-items: center;
+                border: 1px solid #dcdcdc;
+                margin-left: -1px;

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

Negative margins!

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

Yeah I want the borders to overlap with the trash can icon. :/

+            }
+
+            .delete-confirmation div {
+                display: flex;
+                align-items: center;
+                margin: 0 16px;
+            }
+
+            .delete-confirmation cr-button {
+                height: 20px;
+            }
+        </style>
+
         <dialog is="cr-dialog" id="dialog" on-cancel="{{ cancel }}">
             <cr-butterbar message="{{ butterbarMessage }}"></cr-butterbar>
 
             <h1>Edit patchset {{ patchset.sequence }}</h1>
 
             <fieldset class="form-fields" disabled?="{{ disabled }}">
                 <div class="form-row">
                     <label class="form-label" for="title">Title</label>
                     <div class="form-field">
                         <input type="text" id="title" value="{{ title }}" autocomplete="off">
                     </div>
                 </div>
             </fieldset>
 
             <cr-toolbar>
                 <cr-button primary on-tap="{{ save }}">Save</cr-button>
                 <cr-button on-tap="{{ cancel }}">Cancel</cr-button>
+                <template if="{{ patchset.issue.patchsets.length > 1 }}">
+                    <div class="delete-action">
+                        <cr-button on-tap="{{ toggleDeleteAction }}" icon>
+                            <img src="../images/delete_icon_36.png" height="25">
+                        </cr-button>
+                        <template if="{{ showDeleteAction }}">
+                            <div class="delete-confirmation">
+                                <div>Permanently delete this patch?</div>
+                                <cr-button create on-tap="{{ deletePatchset }}">Delete</cr-button>
+                                <cr-button on-tap="{{ toggleDeleteAction }}">No</cr-button>
+                            </div>
+                        </template>
+                    </div>
+                </template>
             </cr-toolbar>
         </dialog>
     </template>
     <script>
-        Polymer("cr-patchset-title-dialog", {
+        Polymer("cr-patchset-edit-dialog", {
             created: function() {
                 this.patchset = null;
                 this.commit = true;
                 this.butterbarMessage = "";
                 this.sending = false;
                 this.title = "";
+                this.showDeleteAction = false;
+            },
+            toggleDeleteAction: function() {
+                this.showDeleteAction = !this.showDeleteAction;
+            },
+            deletePatchset: function() {
+                this.sending = true;

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

Is this.sending read anywhere? Maybe outside the component?

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

Is this.sending read anywhere? Maybe outside the component?

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

That's a bug, in the template it does disabled?="{{ disabled }}" it should be
sending.

+                this.patchset.delete()
+                    .then(this.deleteSuccess.bind(this))
+                    .catch(this.displayError.bind(this));
+            },
+            deleteSuccess: function() {
+                this.fire("issue-refresh");
+                this.close();
             },
             cancel: function(event) {
                 this.close();
             },
             reset: function() {
                 this.sending = false;
                 this.butterbarMessage = "";
                 this.commit = true;
                 this.title = this.patchset ? this.patchset.title : "";
+                this.showDeleteAction = false;
             },
             close: function() {
                 this.reset();
                 this.$.dialog.close();
             },
             save: function() {
                 this.sending = true;
                 this.butterbarMessage = "Saving...";
                 this.patchset.setTitle(this.title)
                     .then(this.saveSuccess.bind(this))
-                    .catch(this.saveFailure.bind(this));
+                    .catch(this.displayError.bind(this));
             },
             saveSuccess: function() {
                 this.close();
             },
-            saveFailure: function(error) {
+            displayError: function(error) {
                 // TODO(esprehn): We should show an better error message.
                 this.sending = false;
                 this.butterbarMessage = "Error: " + error.message;
             },
             showModal: function() {
                 if (!this.patchset)
                     return;
                 this.reset();
                 this.$.dialog.showModal();
             },
         });
     </script>
 </polymer-element>