Bug 92021

Summary: [chromium] Fix build on Ubuntu Precise.
Product: WebKit Reporter: Daniel Cheng <dcheng>
Component: PlatformAssignee: Daniel Cheng <dcheng>
Status: RESOLVED FIXED    
Severity: Normal CC: enne, nayankk, piman, webkit.review.bot
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
Patch none

Daniel Cheng
Reported 2012-07-23 12:54:25 PDT
Otherwise, building ToT Chrome in debug mode with the components build results in the following warning: ../../third_party/WebKit/Source/WebKit/chromium/tests/CCResourceProviderTest.cpp: In member function 'virtual void {anonymous}::CCResourceProviderTest_DeleteOwnedResources_Test::TestBody()': ../../third_party/WebKit/Source/WebKit/chromium/tests/CCResourceProviderTest.cpp:242:36: error: vari able 'ids' set but not used [-Werror=unused-but-set-variable] cc1plus: all warnings being treated as errors
Attachments
Patch (1.59 KB, patch)
2012-07-23 12:57 PDT, Daniel Cheng 		no flags
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Daniel Cheng
Comment 1 2012-07-23 12:57:49 PDT
Created attachment 153846 [details] Patch
Antoine Labour
Comment 2 2012-07-23 13:05:30 PDT
LGTM
Adrienne Walker
Comment 3 2012-07-23 13:09:21 PDT
Comment on attachment 153846 [details] Patch R=me.
WebKit Review Bot
Comment 4 2012-07-23 13:22:48 PDT
Comment on attachment 153846 [details] Patch Clearing flags on attachment: 153846 Committed r123370: <http://trac.webkit.org/changeset/123370>
WebKit Review Bot
Comment 5 2012-07-23 13:22:51 PDT
All reviewed patches have been landed. Closing bug.
Nayan Kumar K
Comment 6 2012-07-24 03:03:19 PDT
*** Bug 91974 has been marked as a duplicate of this bug. ***
Note You need to log in before you can comment on or make changes to this bug.