Bug 108002

Summary: Remove ENABLE_3D_PLUGIN
Product: WebKit Reporter: Laszlo Gombos <laszlo.gombos>
Component: PlatformAssignee: Nobody <webkit-unassigned>
Status: RESOLVED FIXED    
Severity: Normal CC: apatrick, commit-queue, eric, jam, japhet, jbauman, kbr, webkit.review.bot
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
3D_PLUGIN usage
none
proposed change none

Laszlo Gombos
Reported 2013-01-25 22:18:08 PST
ENABLE_3D_PLUGIN is turned on for the Chromium port (only) as a result of the following 2 commits: - http://trac.webkit.org/changeset/71210 - http://trac.webkit.org/changeset/88548 It is hard for me to tell if this flag is still needed but as the flag is turned on currently for chromium removing it will result in a functional change in WebCore/loader/SubframeLoader.cpp .
Attachments
3D_PLUGIN usage (1016 bytes, patch)
2013-01-25 22:24 PST, Laszlo Gombos 		no flags
DetailsFormatted DiffDiff
proposed change (2.28 KB, patch)
2013-04-10 06:46 PDT, Laszlo Gombos 		no flags
DetailsFormatted DiffDiff
Show Obsolete (1) View All Add attachment proposed patch, testcase, etc.
Laszlo Gombos
Comment 1 2013-01-25 22:24:35 PST
Created attachment 184856 [details] 3D_PLUGIN usage It might very well be that this flag is needed, but I could not convince myself just yet that it is needed. It would help if someone from the chromium team could comment on this.
Kenneth Russell
Comment 2 2013-02-01 11:34:37 PST
CC'ing jbauman. I don't know whether this is still needed for Pepper 3D, but if not then I think it's safe to remove.
Laszlo Gombos
Comment 3 2013-04-10 06:46:28 PDT
Created attachment 197259 [details] proposed change
WebKit Commit Bot
Comment 4 2013-04-10 07:34:31 PDT
Comment on attachment 197259 [details] proposed change Clearing flags on attachment: 197259 Committed r148093: <http://trac.webkit.org/changeset/148093>
WebKit Commit Bot
Comment 5 2013-04-10 07:34:34 PDT
All reviewed patches have been landed. Closing bug.
Note You need to log in before you can comment on or make changes to this bug.