Bug 11820 - Make the Windows port compile again
Summary: Make the Windows port compile again
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Platform (show other bugs)
Version: 420+
Hardware: PC Windows XP
: P2 Normal
Assignee: Brett Wilson (Google)
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2006-12-13 11:05 PST by Brett Wilson (Google)
Modified: 2006-12-15 12:57 PST (History)
0 users

See Also:


Attachments
Fix windows build bustage as of r18196 (86.85 KB, patch)
2006-12-13 13:19 PST, Brett Wilson (Google)
darin: review-
Details | Formatted Diff | Diff
Address comments, fix Windows build bustage as of r18218 (88.06 KB, patch)
2006-12-14 16:11 PST, Brett Wilson (Google)
no flags Details | Formatted Diff | Diff
New patch to fix Windows build bustage up to r18233 (88.47 KB, patch)
2006-12-15 12:16 PST, Brett Wilson (Google)
ap: review+
Details | Formatted Diff | Diff
diff of Spinneret.sln from previous patch (2.87 KB, patch)
2006-12-15 12:39 PST, Brett Wilson (Google)
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Brett Wilson (Google) 2006-12-13 11:05:04 PST
The windows build is busted again.
Comment 1 Brett Wilson (Google) 2006-12-13 13:19:22 PST
Created attachment 11833 [details]
Fix windows build bustage as of r18196

This patch makes the Windows build compile and run again. As part of this, I moved some of the Windows-specific callback clients from WebCore to WebKit where they belong. These are the *Client files, which I created files mirroring the Mac structure for them that are currently empty, and moved the implementations out of TemporaryLinkStubs, because that's WebCore only.
Comment 2 Darin Adler 2006-12-14 09:55:04 PST
Comment on attachment 11833 [details]
Fix windows build bustage as of r18196

The change to FormState.h is incorrect. People who manipulate FormState objects do need to include HTMLFormElement.h, but FormState.h itself does not. You can successfully call values() on a FormState object without ever including HTMLFormElement.h for example.

The rest of this looks OK.
Comment 3 Brett Wilson (Google) 2006-12-14 16:11:25 PST
Created attachment 11847 [details]
Address comments, fix Windows build bustage as of r18218

This addresses the review comments about FormState.h, and I also added a few more things to TemporaryLinkStubs to fix things that were broken since I uploaded that last patch.
Comment 4 Brett Wilson (Google) 2006-12-15 12:16:33 PST
Created attachment 11866 [details]
New patch to fix Windows build bustage up to r18233

Updated patch. Exactly one line changed from previous patch, PopupMenu::PopupMenu in TemporaryLinkStubs changed.
Comment 5 Alexey Proskuryakov 2006-12-15 12:26:13 PST
Comment on attachment 11866 [details]
New patch to fix Windows build bustage up to r18233

r=me

+#pragma warning(push, 0)

It's sad that these are needed, but removing them can wait.
Comment 6 Brett Wilson (Google) 2006-12-15 12:39:54 PST
Created attachment 11867 [details]
diff of Spinneret.sln from previous patch
Comment 7 Alexey Proskuryakov 2006-12-15 12:57:02 PST
Committed revision 18236.