Bug 38254 - Regression on fast/text/international/khmer-selection.html
Summary: Regression on fast/text/international/khmer-selection.html
Status: UNCONFIRMED
Alias: None
Product: WebKit
Classification: Unclassified
Component: WebCore Misc. (show other bugs)
Version: 528+ (Nightly build)
Hardware: PC Linux
: P2 Normal
Assignee: Nobody
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2010-04-28 04:40 PDT by Marcus Bulach
Modified: 2010-06-11 11:53 PDT (History)
5 users (show)

See Also:


Attachments
Patch (9.46 KB, patch)
2010-04-28 04:42 PDT, Marcus Bulach
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Marcus Bulach 2010-04-28 04:40:46 PDT
Reverts 58340 (https://bugs.webkit.org/show_bug.cgi?id=38158) due to regression on fast/text/international/khmer-selection.html
Comment 1 Marcus Bulach 2010-04-28 04:42:29 PDT
Created attachment 54548 [details]
Patch
Comment 2 WebKit Review Bot 2010-04-28 04:45:33 PDT
Attachment 54548 [details] did not pass style-queue:

Failed to run "['WebKitTools/Scripts/check-webkit-style', '--no-squash']" exit_code: 1
WebCore/platform/graphics/chromium/HarfbuzzSkia.cpp:142:  Use 0 instead of NULL.  [readability/null] [5]
Total errors found: 1 in 6 files


If any of these errors are false positives, please file a bug against check-webkit-style.
Comment 3 Jeremy Orlow 2010-04-28 04:46:18 PDT
Comment on attachment 54548 [details]
Patch

r=me

Leave this bug open after you commit to track the fact that this has been reverted.
Comment 4 WebKit Commit Bot 2010-04-28 04:58:53 PDT
Comment on attachment 54548 [details]
Patch

Clearing flags on attachment: 54548

Committed r58404: <http://trac.webkit.org/changeset/58404>
Comment 5 WebKit Commit Bot 2010-04-28 04:58:58 PDT
All reviewed patches have been landed.  Closing bug.
Comment 6 James Robinson 2010-05-03 17:55:51 PDT
Reverting a patch is fine, but when you do please reopen the original bug or at least CC the original patch author.  I didn't know this was reverted until I saw that the same errors were showing up.
Comment 7 James Robinson 2010-05-03 18:01:05 PDT
Ah, you did CC me.  However the bugzilla emails never mentioned reverting anything.  Please do reopen the original bug when reverting a patch in the future.  Do you happen to remember what the failures were or what platforms they occurred on?
Comment 8 Jeremy Orlow 2010-05-04 02:15:17 PDT
Sorry...this got mixed up with a lot of other failures those days.  I think I was going to land by hand and then changed my mind to commit bot forgetting that it needed to stay open.  I thought I documented somewhere why this was reverted, but now I can't find it, and I can't remember.  I guess maybe commit again while giving the gardener a heads up?  Sorry.
Comment 9 Jeremy Orlow 2010-05-04 02:15:41 PDT
Re-opening to track.
Comment 10 Marcus Bulach 2010-05-04 03:19:48 PDT
Sorry James, I should've reopened the original bug and added more info here, my bad..

IIRC, the problem was linux only:

http://test-results.appspot.com/dashboards/flakiness_dashboard.html#showExpectations=true&tests=khmer-selection.html