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

Issue 4566001: Fix for bug 8764 that breaks prebuilts for files with more than just keyword pairs. (Closed)

Created:
10 years, 1 month ago by sosa
Modified:
9 years, 6 months ago
Reviewers:
oshima
CC:
chromium-os-reviews_chromium.org, Mandeep Singh Baines, anush, sosa
Visibility:
Public.

Description

Fix for bug 8764 that breaks prebuilts for files with more than just keyword pairs. I've also modified the unit test so it actually tests for this case. Change-Id: I6688d3be7bf1d8962031fcf1d684ddcaab43ba9f BUG=8764 TEST=Unittests Committed: http://chrome-svn/viewvc/chromeos?view=rev&revision=170bde0

Patch Set 1 #

Patch Set 2 : Add unittest #

Patch Set 3 : Remove unnecessary check of = in string #

Unified diffs Side-by-side diffs Delta from patch set Stats (+5 lines, -3 lines) Patch
M prebuilt.py View 1 2 1 chunk +2 lines, -2 lines 0 comments Download
M prebuilt_unittest.py View 1 chunk +3 lines, -1 line 0 comments Download

Messages

Total messages: 1 (0 generated)
oshima
10 years, 1 month ago (2010-11-05 03:49:08 UTC) #1
LGTM

Thank you for prompt fix!

Powered by Google App Engine
This is Rietveld 408576698