Bug 74163 - [chromium] Remove un-used functions from TiledLayerChromium
Summary: [chromium] Remove un-used functions from TiledLayerChromium
Status: RESOLVED WONTFIX
Alias: None
Product: WebKit
Classification: Unclassified
Component: Layout and Rendering (show other bugs)
Version: 528+ (Nightly build)
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Shawn Singh
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2011-12-08 21:01 PST by Shawn Singh
Modified: 2011-12-08 23:32 PST (History)
4 users (show)

See Also:

Attachments
Patch (3.09 KB, patch)
2011-12-08 21:04 PST, Shawn Singh 		jamesr: review-
Details | Formatted Diff | Diff
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Shawn Singh 2011-12-08 21:01:43 PST 
This code seems to be un-used, and compiles successfully when its removed.  Enne, could you please confirm this code can actually be removed?  Thanks in advance.
Comment 1 Shawn Singh 2011-12-08 21:04:39 PST 
Created attachment 118524 [details]
Patch
Comment 2 Adrienne Walker 2011-12-08 21:08:45 PST 
It compiles successfully because there's a base version of protectVisibleTiles in LayerChromium and it's virtual.  NonCompositedContentHost calls this function.
Comment 3 James Robinson 2011-12-08 21:36:11 PST 
Comment on attachment 118524 [details]
Patch

I agree, pretty sure this is called and necessary.
Comment 4 Shawn Singh 2011-12-08 23:32:39 PST 
ok cool, thanks anyway.