Bug 83733

Summary: [chromium] Remove unused compositeToTexture / compositeOffscreen setting
Product: WebKit Reporter: James Robinson <jamesr>
Component: New BugsAssignee: James Robinson <jamesr>
Status: RESOLVED FIXED    
Severity: Normal CC: abarth, backer, cc-bugs, dglazkov, enne, fishd, piman, tkent, tkent+wkapi, webkit.review.bot, wjmaclean
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
Patch enne: review+, webkit.review.bot: commit-queue-

Description James Robinson 2012-04-11 17:17:36 PDT
[chromium] Remove unused compositeToTexture / compositeOffscreen setting
Comment 1 James Robinson 2012-04-11 17:19:17 PDT
Created attachment 136794 [details]
Patch
Comment 2 James Robinson 2012-04-11 17:20:32 PDT
This depends on https://chromiumcodereview.appspot.com/9969181/ to remove the one caller to WebSettings::setCompositeToTextureEnabled().
Comment 3 Adrienne Walker 2012-04-11 21:56:57 PDT
Comment on attachment 136794 [details]
Patch

Code changes look good to me. I am happy to get rid of this dead code.
Comment 4 WebKit Review Bot 2012-04-11 21:57:17 PDT
Please wait for approval from abarth@webkit.org, dglazkov@chromium.org, fishd@chromium.org, jamesr@chromium.org or tkent@chromium.org before submitting, as this patch contains changes to the Chromium public API. See also https://trac.webkit.org/wiki/ChromiumWebKitAPI.
Comment 5 WebKit Review Bot 2012-04-11 22:18:58 PDT
Comment on attachment 136794 [details]
Patch

Attachment 136794 [details] did not pass chromium-ews (chromium-xvfb):
Output: http://queues.webkit.org/results/12395047
Comment 6 James Robinson 2012-04-13 14:01:23 PDT
Committed r114166: <http://trac.webkit.org/changeset/114166>