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

Unified Diff: Tools/GardeningServer/scripts/svn-log.js

Issue 402603007: Get sheriff-o-matic data from auto-sheriff.appspot.com. (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: update and delete old code Created 6 years, 5 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 side-by-side diff with in-line comments
Download patch
Index: Tools/GardeningServer/scripts/svn-log.js
diff --git a/Tools/GardeningServer/scripts/svn-log.js b/Tools/GardeningServer/scripts/svn-log.js
deleted file mode 100644
index cd53f288af89c50208e661fe0c32e4f45db73842..0000000000000000000000000000000000000000
--- a/Tools/GardeningServer/scripts/svn-log.js
+++ /dev/null
@@ -1,145 +0,0 @@
-/*
- * Copyright (C) 2011 Apple Inc. All rights reserved.
- * Copyright (C) 2012 Google Inc. All rights reserved.
- *
- * Redistribution and use in source and binary forms, with or without
- * modification, are permitted provided that the following conditions
- * are met:
- * 1. Redistributions of source code must retain the above copyright
- * notice, this list of conditions and the following disclaimer.
- * 2. Redistributions in binary form must reproduce the above copyright
- * notice, this list of conditions and the following disclaimer in the
- * documentation and/or other materials provided with the distribution.
- *
- * THIS SOFTWARE IS PROVIDED BY APPLE INC. AND ITS CONTRIBUTORS ``AS IS''
- * AND ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT LIMITED TO,
- * THE IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR A PARTICULAR
- * PURPOSE ARE DISCLAIMED. IN NO EVENT SHALL APPLE INC. OR ITS CONTRIBUTORS
- * BE LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL, SPECIAL, EXEMPLARY, OR
- * CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT LIMITED TO, PROCUREMENT OF
- * SUBSTITUTE GOODS OR SERVICES; LOSS OF USE, DATA, OR PROFITS; OR BUSINESS
- * INTERRUPTION) HOWEVER CAUSED AND ON ANY THEORY OF LIABILITY, WHETHER IN
- * CONTRACT, STRICT LIABILITY, OR TORT (INCLUDING NEGLIGENCE OR OTHERWISE)
- * ARISING IN ANY WAY OUT OF THE USE OF THIS SOFTWARE, EVEN IF ADVISED OF
- * THE POSSIBILITY OF SUCH DAMAGE.
- */
-
-var trac = trac || {};
-
-(function() {
-
-function findUsingRegExp(string, regexp)
-{
- var match = regexp.exec(string);
- if (match)
- return match[1];
- return null;
-}
-
-function findReviewer(message)
-{
- var regexp = /(?:^|\n)\s*(?:TB)?R=(.+)/;
- var reviewers = findUsingRegExp(message, regexp);
- if (!reviewers)
- return null;
- return reviewers.replace(/\s*,\s*/g, ', ');
-}
-
-function findBugID(message)
-{
- var regexp = /(?:^|\n)\s*BUG=(.+)/;
- var value = findUsingRegExp(message, regexp);
- if (!value)
- return null;
- var result = value.split(/\s*,\s*/).map(function(id) {
- var parsedID = parseInt(id.replace(/[^\d]/g, ''), 10);
- return isNaN(parsedID) ? 0 : parsedID;
- }).filter(function(id) {
- return !!id;
- });
- return result.length ? result : null;
-}
-
-function findRevision(message)
-{
- var regexp = /git-svn-id: svn:\/\/svn.chromium.org\/blink\/trunk@(\d+)/;
- var svnRevision = parseInt(findUsingRegExp(message, regexp), 10);
- return isNaN(svnRevision) ? null : svnRevision;
-}
-
-function parseCommitMessage(message) {
- var lines = message.split('\n');
- var title = '';
- lines.some(function(line) {
- if (line) {
- title = line;
- return true;
- }
- });
- var summary = lines.join('\n').trim();
- return {
- title: title,
- summary: summary,
- bugID: findBugID(summary),
- reviewer: findReviewer(summary),
- revision: findRevision(summary),
- };
-}
-
-// FIXME: Consider exposing this method for unit testing.
-function parseCommitData(responseXML)
-{
- var commits = Array.prototype.map.call(responseXML.getElementsByTagName('entry'), function(logentry) {
- var author = logentry.getElementsByTagName('author')[0].textContent.trim();
- var time = logentry.getElementsByTagName('published')[0].textContent;
- var titleElement = logentry.getElementsByTagName('title')[0];
- var title = titleElement ? titleElement.textContent : null;
-
- // FIXME: This isn't a very high-fidelity reproduction of the commit message,
- // but it's good enough for our purposes.
- var message = parseCommitMessage(logentry.getElementsByTagName('content')[0].textContent);
-
- return {
- 'revision': message.revision,
- 'title': title || message.title,
- 'time': time,
- 'summary': message.title,
- 'author': author,
- 'reviewer': message.reviewer,
- 'bugID': message.bugID,
- 'message': message.summary,
- 'revertedRevision': undefined,
- };
- });
- return commits;
-}
-
-trac._queryParam = function(params)
-{
- var result = []
- Object.keys(params, function(key, value) {
- result.push(encodeURIComponent(key) + '=' + encodeURIComponent(value));
- });
- // FIXME: Remove the conversion of space to plus. This is just here
- // to remain compatible with jQuery.param, but there's no reason to
- // deviate from the built-in encodeURIComponent behavior.
- return result.join('&').replace(/%20/g, '+');
-}
-
-trac.changesetURL = function(revision)
-{
- var queryParameters = {
- view: 'rev',
- revision: revision,
- };
- return config.kBlinkRevisionURL + '?' + trac._queryParam(queryParameters);
-};
-
-trac.recentCommitData = function(path, limit)
-{
- return net.xml('http://blink.lc/blink/atom').then(function(commitData) {
- return parseCommitData(commitData);
- });
-};
-
-})();

Powered by Google App Engine
This is Rietveld 408576698