Remove references to Rietveld from bugs.webkit.org
authorddkilzer@apple.com <ddkilzer@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Thu, 16 Oct 2014 16:01:21 +0000 (16:01 +0000)
committerddkilzer@apple.com <ddkilzer@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Thu, 16 Oct 2014 16:01:21 +0000 (16:01 +0000)
Rietveld was an experiment that was added by Google.  It was
never enabled for everyone, so it's safe to remove the changes
that reference it.

* Bugzilla/Flag.pm: Remove code that prevented mail messages from
being sent for patches reviewed in Rietveld.
* attachment.cgi: Remove reference to the "rietveldreview"
template.
* template/en/custom/attachment/list.html.tmpl: Remove code
that handled the Rietveld review flag.
* template/en/custom/attachment/reviewform.html.tmpl: Remove code
that handled the "rietveldReview" CGI parameter.
* template/en/custom/attachment/rietveldreview.html.tmpl: Delete.
* template/en/custom/flag/list.html.tmpl: Delete.  After
removing the code to handle Rietveld reviews, there were no more
local WebKit changes to the custom template file, so it could be
removed.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@174773 268f45cc-cd09-0410-ab3c-d52691b4dbfc

Websites/bugs.webkit.org/Bugzilla/Flag.pm
Websites/bugs.webkit.org/attachment.cgi
Websites/bugs.webkit.org/template/en/custom/attachment/list.html.tmpl
Websites/bugs.webkit.org/template/en/custom/attachment/reviewform.html.tmpl
Websites/bugs.webkit.org/template/en/custom/attachment/rietveldreview.html.tmpl [deleted file]
Websites/bugs.webkit.org/template/en/custom/flag/list.html.tmpl [deleted file]

index 596423e..a727532 100644 (file)
@@ -937,12 +937,6 @@ sub notify {
         }
     }
 
-    #if WEBKIT_CHANGES
-    # Don't send a notification when the flag is in-rietveld,
-    # since it isn't a user visible flag, and that mail is spammy.
-    return if ($flag->type->name eq 'in-rietveld');
-    #endif // WEBKIT_CHANGES
-
     my $cc_list = $flag ? $flag->type->cc_list : $old_flag->type->cc_list;
     # Is there someone to notify?
     return unless ($addressee || $cc_list);
index c111cf8..50e9da9 100755 (executable)
@@ -133,10 +133,6 @@ elsif ($action eq "reviewform")
 {
     edit("reviewform");
 }
-elsif ($action eq "rietveldreview")
-{
-    edit("rietveldreview");
-}
 #endif // WEBKIT_CHANGES
 elsif ($action eq "update") 
 { 
index 343edfe..ea7ea24 100644 (file)
@@ -112,16 +112,10 @@ function toggle_display(link) {
 
         [% IF show_attachment_flags %]
           <td class="bz_attach_flags" valign="top">
-[%# if WEBKIT_CHANGES %]
-            [% IF attachment.flags.size == 0  || 
-               (attachment.flags.size == 1 && attachment.flags.0.type.name == 'in-rietveld') %]
-[%# endif // WEBKIT_CHANGES %]
+            [% IF attachment.flags.size == 0 %]
               <i>no flags</i>
             [% ELSE %]
               [% FOREACH flag = attachment.flags %]
-[%# if WEBKIT_CHANGES %]
-                [% IF flag.type.name != 'in-rietveld' %]
-[%# endif // WEBKIT_CHANGES %]
                 [% IF user.id %]
                   <span title="[% flag.setter.identity FILTER html %]">[% flag.setter.nick FILTER html %]</span>:
                 [% ELSIF flag.setter.name %]
@@ -139,9 +133,6 @@ function toggle_display(link) {
                     ([% flag.requestee.nick FILTER html %])
                   [% END %]
                 [% END %]<br>
-[%# if WEBKIT_CHANGES %]
-                [% END %]
-[%# endif // WEBKIT_CHANGES %]
               [% END %]
             [% END %]
           </td>
@@ -165,11 +156,6 @@ function toggle_display(link) {
           [% Hook.process("action") %]
 [%# if WEBKIT_CHANGES %]
           [% IF attachment.ispatch %]
-            [% FOREACH flag = attachment.flags %]
-              [% IF flag.type.name == 'in-rietveld' && flag.status == "+"  %]
-                | <a href="attachment.cgi?id=[% attachment.id %]&amp;action=rietveldreview&amp;GoAheadAndLogIn=1">Rietve
-              [% END %]
-            [% END %]
           <div class="statusBubble">
             <iframe src="https://webkit-queues.appspot.com/status-bubble/[% attachment.id %]"
                     style="width: 600px; height: 20px; border: none;" scrolling="no">
index 2ba094e..873d902 100644 (file)
 
 [% PROCESS global/variables.none.tmpl %]
 
-[% USE CGI %]
-[%  
-    # We use a query string rietveldreview=true to indicate when in rietveld mode. 
-    rietveldReview = CGI.param('rietveldreview')
-%]
-
 <style>
 * { font-size: small; }
 #comment { width: 100%; height: 100%; font-family: monospace; }
     }
     function onSubmitComments()
     {
-[% IF rietveldReview %]
-         var submitBtn = document.getElementById('submitBtn');
-         submitBtn.innerHTML = 'Fetching comments from Rietveld...';
-         submitBtn.disabled = true; // Disable since this is asynch.
-         top.getComments();
-         // Handle the error case of Rietveld not responding by giving
-         // the user another chance.
-         setTimeout(function() {
-             submitBtn.innerHTML = 'Fetching comments timed out ... retry?';
-             submitBtn.disabled = false;
-         }, 30000);
-         return false;
-[% ELSE %]
          // If someone just left the entire comment alone, then upload the form
          // with no comment at all.
          var textarea = document.getElementById("comment");
          if (textarea.defaultValue === textarea.value)
              textarea.value = "";
          return true;
-[% END %]
     }
 [% END %]
 
 </tr>
 <tr>
 <td colspan="4">
-[% IF rietveldReview %]
-Enter high-level comments below.  Make detailed comments above.  Submit will submit both.<br>
-[% ELSE %]
 Enter comments below:<br>
-[% END %]
 <textarea id=comment name="comment" wrap="soft">
-[% IF !rietveldReview %]
 [% quotedpatch FILTER html %]
-[% END %]
 </textarea><br>
 
 </tr>
diff --git a/Websites/bugs.webkit.org/template/en/custom/attachment/rietveldreview.html.tmpl b/Websites/bugs.webkit.org/template/en/custom/attachment/rietveldreview.html.tmpl
deleted file mode 100644 (file)
index cbd2cc1..0000000
+++ /dev/null
@@ -1,77 +0,0 @@
-[%# The contents of this file are subject to the Mozilla Public
-  # License Version 1.1 (the "License"); you may not use this file
-  # except in compliance with the License. You may obtain a copy of
-  # the License at http://www.mozilla.org/MPL/
-  #
-  # Software distributed under the License is distributed on an "AS
-  # IS" basis, WITHOUT WARRANTY OF ANY KIND, either express or
-  # implied. See the License for the specific language governing
-  # rights and limitations under the License.
-  #
-  # The Original Code is the Bugzilla Bug Tracking System.
-  #
-  # The Initial Developer of the Original Code is Netscape Communications
-  # Corporation. Portions created by Netscape are
-  # Copyright (C) 1998 Netscape Communications Corporation. All
-  # Rights Reserved.
-  #
-  # Contributor(s): Myk Melez <myk@mozilla.org>
-  #%]
-
-[% PROCESS global/variables.none.tmpl %]
-
-<!DOCTYPE HTML>
-<html>
-<head>
-<title>Rietveld review patch #[% attachment.id %] for [% terms.Bug %] #[% attachment.bug_id %]: [% bugsummary FILTER html %]</title>
-<script type="text/javascript">
-<!--
-/**
- * Handle the postMessage response from Rietveld.
- * The response will be the comments to combine with those left in the
- * comments box.
- */
-function updateComments(e)
-{
-    if (e.origin == 'http://wkrietveld.appspot.com') {
-        var reviewFormDoc = document.getElementById('reviewFormFrame').
-            contentDocument;
-        // Add the Rietveld comments to the comments field.
-        if (e.data) {
-            var commentBox = reviewFormDoc.getElementById('comment');
-
-            if (commentBox.value)
-                commentBox.value += '\n---------------------------------\n';
-            commentBox.value += e.data;
-        }
-        reviewFormDoc.forms[0].submit();
-    }
-}
-
-/**
- * Fetch comments from Rietveld via postMessage.
- */
-function getComments()
-{
-    document.getElementById('rietveldFrame').contentWindow.
-        postMessage('publish', 'http://wkrietveld.appspot.com');
-}
-
-// Listen for messages from Rietveld.
-if (window.postMessage)
-    window.addEventListener("message", updateComments, false);
-else
-    alert('Error: Please load this page in a browser that supports postMessage');
-//-->
-</script>
-
-[% IF !user.login %]
-[%# We shouldn't ever hit this, since all links to this page use GoAheadAndLogIn.
-    This is just here as a safeguard if anyone directly navigates to it. %]
-Please log in and refresh this page.
-[% ELSE %]
-<frameset rows="80%,20%">
-<frame id=rietveldFrame src="http://wkrietveld.appspot.com/[% attachment.bug_id  %]?email=[% user.login %]">
-<frame id=reviewFormFrame src="attachment.cgi?id=[% attachment.id %]&action=reviewform&rietveldreview=true">
-</frameset>
-[% END %]
diff --git a/Websites/bugs.webkit.org/template/en/custom/flag/list.html.tmpl b/Websites/bugs.webkit.org/template/en/custom/flag/list.html.tmpl
deleted file mode 100644 (file)
index 3181465..0000000
+++ /dev/null
@@ -1,228 +0,0 @@
-[%# The contents of this file are subject to the Mozilla Public
-  # License Version 1.1 (the "License"); you may not use this file
-  # except in compliance with the License. You may obtain a copy of
-  # the License at http://www.mozilla.org/MPL/
-  #
-  # Software distributed under the License is distributed on an "AS
-  # IS" basis, WITHOUT WARRANTY OF ANY KIND, either express or
-  # implied. See the License for the specific language governing
-  # rights and limitations under the License.
-  #
-  # The Original Code is the Bugzilla Bug Tracking System.
-  #
-  # The Initial Developer of the Original Code is Netscape Communications
-  # Corporation. Portions created by Netscape are
-  # Copyright (C) 1998 Netscape Communications Corporation. All
-  # Rights Reserved.
-  #
-  # Contributor(s): Myk Melez <myk@mozilla.org>
-  #%]
-
-[% IF user.id AND !read_only_flags %]
-
-[%# We list flags by looping twice over the flag types relevant for the bug.
-  # In the first loop, we display existing flags and then, for active types,
-  # we display UI for adding new flags.  In the second loop, we display UI 
-  # for adding additional new flags for those types for which a flag already
-  # exists but which are multiplicable (can have multiple flags of the type
-  # on a single bug/attachment).
-  #%]
-
-[% DEFAULT flag_table_id = "flags" %]
-
-<script src="[% 'js/flag.js' FILTER mtime %]" type="text/javascript"></script>
-
-<table id="[% flag_table_id FILTER html %]">
-  [% UNLESS flag_no_header %]
-    <tr>
-      <th colspan="3">
-        Flags:
-      </th>
-      [% IF any_flags_requesteeble %]
-        <th>
-          Requestee:
-        </th>
-      [% END %]
-    </tr>
-  [% END %]
-
-  [%# Step 1: Display every flag type (except inactive types with no flags). %]
-  [% FOREACH type = flag_types -%]
-
-    [%-# Step 1a: Display existing flag(s). %]
-    [% FOREACH flag = type.flags %]
-[%# if WEBKIT_CHANGES %]
-      [% IF type.name == 'in-rietveld' %]
-        <tr style='display:none'>
-      [% ELSE %]
-        <tr>
-      [% END %]
-[%# endif // WEBKIT_CHANGES %]
-        <td>
-          <span title="[% flag.setter.identity FILTER html %]">[% flag.setter.nick FILTER html %]</span>:
-        </td>
-        <td>
-          <label title="[% type.description FILTER html %]"
-                 for="flag-[% flag.id %]">
-            [%- type.name FILTER html FILTER no_break -%]</label>
-        </td>
-        <td>
-          <select id="flag-[% flag.id %]" name="flag-[% flag.id %]" 
-                  title="[% type.description FILTER html %]"
-                  onchange="toggleRequesteeField(this);"
-                  class="flag_select flag_type-[% type.id %]">
-            [%# Only display statuses the user is allowed to set. %]
-            [% IF user.can_request_flag(type) || flag.setter_id == user.id %]
-              <option value="X"></option>
-            [% END %]
-            [% IF type.is_active %]
-              [% IF (type.is_requestable && user.can_request_flag(type)) || flag.status == "?" %]
-                <option value="?" [% "selected" IF flag.status == "?" %]>?</option>
-              [% END %]
-              [% IF user.can_set_flag(type) || flag.status == "+" %]
-                <option value="+" [% "selected" IF flag.status == "+" %]>+</option>
-              [% END %]
-              [% IF user.can_set_flag(type) || flag.status == "-" %]
-                <option value="-" [% "selected" IF flag.status == "-" %]>-</option>
-              [% END %]
-            [% ELSE %]
-              <option value="[% flag.status %]" selected="selected">[% flag.status %]</option>
-            [% END %]
-          </select>
-        </td>
-        [% IF any_flags_requesteeble %]
-          <td>
-            [% IF (type.is_active && type.is_requestable && type.is_requesteeble) || flag.requestee %]
-              <span style="white-space: nowrap;">
-                [% SET flag_custom_list = [] %]
-                [% IF Param('usemenuforusers') %]
-                  [% flag_custom_list = flag.type.grant_list %]
-                  [% IF !(type.is_active && type.is_requestable && type.is_requesteeble) %]
-                    [%# We are here only because there was already a requestee. In this case,
-                        the only valid action is to remove the requestee or leave it alone;
-                        nothing else. %]
-                    [% flag_custom_list = [flag.requestee] %]
-                  [% END %]
-                [% END %]
-                [% INCLUDE global/userselect.html.tmpl
-                           name     => "requestee-$flag.id"
-                           id       => "requestee-$flag.id"
-                           value    => flag.requestee.login
-                           multiple => 0
-                           emptyok  => 1
-                           classes => ["requestee"]
-                           custom_userlist => flag_custom_list
-                %]
-              </span>
-            [% END %]
-          </td>
-        [% END %]
-      </tr>
-    [% END -%]
-
-    [%-# Step 1b: Display UI for setting flag. %]
-    [% IF (!type.flags || type.flags.size == 0) && type.is_active %]
-
-      [% PROCESS flag_row first_cell_empty = 1 addl_text = "" %]
-    [% END %]
-  [% END %]
-
-  [%# Step 2: Display flag type again (if type is multiplicable). %]
-  [% FOREACH type = flag_types %]
-    [% NEXT UNLESS type.flags && type.flags.size > 0 && type.is_multiplicable && type.is_active %]
-    [% IF !separator_displayed %]
-        <tr><td colspan="3"><hr></td></tr>
-        [% separator_displayed = 1 %]
-    [% END %]
-
-    [% PROCESS flag_row first_cell_empty = 0 addl_text = "addl." %]
-  [% END %]
-</table>
-
-[% ELSE %]
-  [%# The user is logged out. Display flags as read-only. %]
-  [% header_displayed = 0 %]
-  [% FOREACH type = flag_types %]
-    [% FOREACH flag = type.flags %]
-      [% IF !flag_no_header AND !header_displayed %]
-        <p><b>Flags:</b></p>
-        [% header_displayed = 1 %]
-      [% END %]
-      [% IF flag.setter.name %]
-        <span title="[% flag.setter.name FILTER html %]">[% flag.setter.nick FILTER html %]</span>:
-      [% ELSE %]
-        [% flag.setter.nick FILTER html %]:
-      [% END %]
-      [%+ type.name FILTER html FILTER no_break %][% flag.status %]
-      [% IF flag.requestee %]
-        [% IF flag.requestee.name %]
-          (<span title="[% flag.requestee.name FILTER html %]">[% flag.requestee.nick FILTER html %]</span>)
-        [% ELSE %]
-          ([% flag.requestee.nick FILTER html %])
-        [% END %]
-      [% END %]<br>
-    [% END %]
-  [% END %]
-[% END %]
-
-[%# Display a table row for unset flags %]
-
-[% BLOCK flag_row %]
-[%# if WEBKIT_CHANGES %]
-  [% IF type.name == 'in-rietveld' %]
-  <tr style='display:none'>
-  [% ELSE %]
-  <tr>
-  [% END %]
-[%# endif // WEBKIT_CHANGES %]
-  [% IF first_cell_empty %]
-    <td>&nbsp;</td>
-    <td>
-  [% ELSE %]
-    <td colspan="2">
-  [% END %]
-
-      [% addl_text FILTER html %]
-      <label title="[% type.description FILTER html %]" for="flag_type-[% type.id %]">
-        [%- type.name FILTER html FILTER no_break %]</label>
-    </td>
-    <td>
-      <select id="flag_type-[% type.id %]" name="flag_type-[% type.id %]"
-              title="[% type.description FILTER html %]"
-              [% " disabled=\"disabled\"" UNLESS (type.is_requestable && user.can_request_flag(type)) || user.can_set_flag(type) %]
-              onchange="toggleRequesteeField(this);"
-              class="flag_select flag_type-[% type.id %]">
-        <option value="X"></option>
-        [% IF type.is_requestable && user.can_request_flag(type) %]
-          <option value="?">?</option>
-        [% END %]
-        [% IF user.can_set_flag(type) %]
-          <option value="+">+</option>
-          <option value="-">-</option>
-        [% END %]
-      </select>
-    </td>
-    [% IF any_flags_requesteeble %]
-      <td>
-        [% IF type.is_requestable && type.is_requesteeble %]
-          <span style="white-space: nowrap;">
-            [% SET grant_list = [] %]
-            [% IF Param('usemenuforusers') %]
-              [% grant_list = type.grant_list %]
-            [% END %]
-            [% INCLUDE  global/userselect.html.tmpl
-                        name     => "requestee_type-$type.id"
-                        id       => "requestee_type-$type.id"
-                        multiple => type.is_multiplicable * 3
-                        emptyok  => !type.is_multiplicable
-                        value    => ""
-                        custom_userlist => grant_list
-                        classes => ["requestee"]
-            %]            
-            
-          </span>
-        [% END %]
-      </td>
-    [% END %]
-  </tr>
-[% END %]