Chromium Code Reviews

Unified Diff: tools/emacs/trybot.el

Issue 5826002: emacs: fix HTML urls to automatically load plain text (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 10 years ago
Use n/p to move between diff chunks; N/P to move between comments.
Jump to:
View side-by-side diff with in-line comments
« no previous file with comments | « no previous file | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: tools/emacs/trybot.el
diff --git a/tools/emacs/trybot.el b/tools/emacs/trybot.el
index b3820df6f9f3df993bcae9e375b777c7c10aa6e3..6bc966fff711cc10375f5423a2896a1aafb24ef6 100644
--- a/tools/emacs/trybot.el
+++ b/tools/emacs/trybot.el
@@ -149,7 +149,10 @@
(clipboard-yank)
(setq url (buffer-string))))
- ;; TODO: fixup URL to append /text if necessary.
+ ;; Append /text to the URL to get plain text output in the common
+ ;; case of getting a URL to the HTML build log.
+ (when (equal "stdio" (car (last (split-string url "/"))))
+ (setq url (concat url "/text")))
(let ((type-hint (cond ((string-match "/win/" url) 'win)
((string-match "/mac/" url) 'mac)
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine