Bug 81313

Summary: Move parse_bug_id into config/urls in preparation for re-use elsewhere.
Product: WebKit Reporter: Eric Seidel (no email) <eric>
Component: New BugsAssignee: Eric Seidel (no email) <eric>
Status: RESOLVED FIXED    
Severity: Normal CC: abarth, ojan, webkit.review.bot
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
Patch abarth: review+, abarth: commit-queue+

Eric Seidel (no email)
Reported 2012-03-15 23:07:50 PDT
Move parse_bug_id into config/urls in preparation for re-use elsewhere.
Attachments
Patch (10.32 KB, patch)
2012-03-15 23:08 PDT, Eric Seidel (no email) 		abarth: review+
abarth: commit-queue+
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Eric Seidel (no email)
Comment 1 2012-03-15 23:08:20 PDT
Created attachment 132206 [details] Patch
Adam Barth
Comment 2 2012-03-15 23:10:58 PDT
Comment on attachment 132206 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=132206&action=review > Tools/Scripts/webkitpy/common/config/urls.py:56 > +def parse_bug_id(message): message ? Not sure what the best parameter name is here. > Tools/Scripts/webkitpy/common/config/urls_unittest.py:35 > + def test_parse_bug_id(self): Should we add any multiline test cases since that's needed by at least one caller.
Eric Seidel (no email)
Comment 3 2012-03-15 23:18:46 PDT
Committed r110948: <http://trac.webkit.org/changeset/110948>
Note You need to log in before you can comment on or make changes to this bug.