Bug 59888

Summary: [android] OS(ANDROID) does not imply PLATFORM(ANDROID)
Product: WebKit Reporter: Holger Freyther <zecke>
Component: WebKit Misc.Assignee: Holger Freyther <zecke>
Status: RESOLVED FIXED    
Severity: Normal CC: abarth, eric, steveblock
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: PC   
OS: OS X 10.5   
Bug Depends on:    
Bug Blocks: 60907    
Attachments:
Description Flags
Patch steveblock: review+

Description Holger Freyther 2011-05-01 13:55:47 PDT
It is possible to build other ports for the Android Operating System as well. PLATFORM(ANDROID) should not be automatically enabled.
Comment 1 Holger Freyther 2011-05-01 13:59:41 PDT
Created attachment 91840 [details]
Patch
Comment 2 Adam Barth 2011-05-11 20:47:01 PDT
Comment on attachment 91840 [details]
Patch

That makes a lot of sense.  It's probably a good idea to double-check with Steve Block before landing.
Comment 3 Steve Block 2011-05-12 01:59:33 PDT
Comment on attachment 91840 [details]
Patch

Thanks Adam and Holger. This looks correct to me.

Assuming that Adam's r- was just to give me chance to have a look, r+
Comment 4 Adam Barth 2011-05-12 10:04:34 PDT
> Assuming that Adam's r- was just to give me chance to have a look, r+

It was actually just a mis-click.  :)
Comment 5 Holger Freyther 2011-05-13 08:09:47 PDT
Landed in r86432.