Bug 92082

Summary: [Chromium] Build fix for Android after r123424
Product: WebKit Reporter: Peter Beverloo <peter>
Component: New BugsAssignee: Peter Beverloo <peter>
Status: RESOLVED FIXED    
Severity: Normal CC: abarth, pilgrim
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: Unspecified   
OS: Unspecified   
Bug Depends on:    
Bug Blocks: 92010    
Attachments:
Description Flags
Patch abarth: review+

Peter Beverloo
Reported 2012-07-24 01:50:45 PDT
[Chromium] Buildfix for Android after r123424
Attachments
Patch (1.45 KB, patch)
2012-07-24 01:52 PDT, Peter Beverloo 		abarth: review+
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Peter Beverloo
Comment 1 2012-07-24 01:52:27 PDT
Created attachment 153991 [details] Patch
Peter Beverloo
Comment 2 2012-07-24 01:55:36 PDT
Committed r123440: <http://trac.webkit.org/changeset/123440>
Adam Barth
Comment 3 2012-07-24 01:55:54 PDT
You don't need to include LayoutTestSupport.h?
Peter Beverloo
Comment 4 2012-07-24 01:57:53 PDT
(In reply to comment #3) > You don't need to include LayoutTestSupport.h? The include for PlatformSupport.h was removed in favor of LayoutTestSupport.h, which is still around on line 29 and required for the isRunningLayoutTest() function. Thank you! This landed with the "unreviewed" line, but it's good to capture it in the bug.
Note You need to log in before you can comment on or make changes to this bug.