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

Issues Search
    My Issues | Starred     Open | Closed | All

Issue 10824101: Harfbuzz roll 3ab7b37bdebf0f8773493a1fee910b151c4de30f (Closed)

Created:
8 years, 4 months ago by bashi
Modified:
8 years, 4 months ago
CC:
chromium-reviews, tony
Base URL:
http://git.chromium.org/chromium/src.git@master
Visibility:
Public.

Description

Harfbuzz roll 3ab7b37bdebf0f8773493a1fee910b151c4de30f Eliminate some memory leaks. BUG=139709 TEST=ran layout tests. Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=150955

Patch Set 1 #

Patch Set 2 : rebased #

Unified diffs Side-by-side diffs Delta from patch set Stats (+23 lines, -56 lines) Patch
M third_party/harfbuzz/README.chromium View 1 1 chunk +3 lines, -3 lines 0 comments Download
M third_party/harfbuzz/chromium.patch View 3 chunks +2 lines, -38 lines 0 comments Download
M third_party/harfbuzz/src/harfbuzz-arabic.c View 1 chunk +9 lines, -3 lines 0 comments Download
M third_party/harfbuzz/src/harfbuzz-global.h View 1 chunk +0 lines, -4 lines 0 comments Download
M third_party/harfbuzz/src/harfbuzz-gpos.c View 1 chunk +3 lines, -3 lines 0 comments Download
M third_party/harfbuzz/src/harfbuzz-gsub.c View 1 chunk +3 lines, -3 lines 0 comments Download
M third_party/harfbuzz/src/harfbuzz-myanmar.c View 1 chunk +2 lines, -2 lines 0 comments Download
M third_party/harfbuzz/src/harfbuzz-tibetan.c View 1 chunk +1 line, -0 lines 0 comments Download

Messages

Total messages: 6 (0 generated)
bashi
Hi behdad, jshin, Could you take a look?
8 years, 4 months ago (2012-07-31 00:29:14 UTC) #1
jungshik at Google
LGTM Thanks and sorry for the delay.
8 years, 4 months ago (2012-08-03 22:56:28 UTC) #2
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/bashi@chromium.org/10824101/1
8 years, 4 months ago (2012-08-09 23:24:14 UTC) #3
commit-bot: I haz the power
Failed to apply patch for third_party/harfbuzz/README.chromium: While running patch -p1 --forward --force; patching file third_party/harfbuzz/README.chromium ...
8 years, 4 months ago (2012-08-09 23:24:16 UTC) #4
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/bashi@chromium.org/10824101/10001
8 years, 4 months ago (2012-08-09 23:28:55 UTC) #5
commit-bot: I haz the power
8 years, 4 months ago (2012-08-10 00:43:50 UTC) #6 
Change committed as 150955

Powered by Google App Engine
This is Rietveld 408576698