Bug 50377 - [chromium] Remove dead code from PNG encoder
Summary: [chromium] Remove dead code from PNG encoder
Alias: None
Product: WebKit
Classification: Unclassified
Component: Canvas (show other bugs)
Version: 528+ (Nightly build)
Hardware: PC Windows 7
: P2 Normal
Assignee: Nobody
Depends on:
Reported: 2010-12-02 00:29 PST by noel gordon
Modified: 2010-12-02 20:36 PST (History)
6 users (show)

See Also:

patch (7.40 KB, patch)
2010-12-02 00:34 PST, noel gordon
levin: review+
Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description noel gordon 2010-12-02 00:29:54 PST
Remove dead code, and webkit style these files.
Comment 1 noel gordon 2010-12-02 00:34:27 PST
Created attachment 75358 [details]
Comment 2 David Levin 2010-12-02 09:35:00 PST
Comment on attachment 75358 [details]

View in context: https://bugs.webkit.org/attachment.cgi?id=75358&action=review

> WebCore/platform/image-encoders/skia/PNGImageEncoder.cpp:2
> + * Copyright (c) 2010, Google Inc. All rights reserved.

fwiw for some reason, WebKit tends to keep all the years (as opposed to Chromium which only keeps the most recent). Also, the years are comma separated and no comma after the last. While this style is followed regularly, I'll admit that this is a super minor issue, so this is fine for this change.
Comment 3 WebKit Commit Bot 2010-12-02 11:49:45 PST
Comment on attachment 75358 [details]

Rejecting patch 75358 from commit-queue.

Failed to run "['./WebKitTools/Scripts/webkit-patch', '--status-host=queues.webkit.org', '--bot-id=eseidel-cq-sl', 'build', '--no-clean', '--no-update', '--build-style=both']" exit_code: 1
ERROR: Working directory has local commits, pass --force-clean to continue.

Full output: http://queues.webkit.org/results/6759020
Comment 4 David Levin 2010-12-02 16:10:09 PST
Comment on attachment 75358 [details]

If at first you don't succeed,...
Comment 5 Eric Seidel (no email) 2010-12-02 16:28:21 PST
Sigh.  You hit bug 49798.

I'm not sure how we're hitting this so frequently.
Comment 6 Adam Barth 2010-12-02 17:09:16 PST
That bug shouldn't be that hard to fix.  We just need to force-clean before building.
Comment 7 Eric Seidel (no email) 2010-12-02 17:16:21 PST
We need to force clean at some point.  Not sure we when exactly, since we don't want to ditch the patch.   I'm not sure how it's ending up with a commit left over.  i assume from a previously failed commit?
Comment 8 Jian Li 2010-12-02 20:36:28 PST
Committed as http://trac.webkit.org/changeset/73233.