Bug 11250 - File upload form control isn't finished on Windows
Summary: File upload form control isn't finished on Windows
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Forms (show other bugs)
Version: 420+
Hardware: PC Windows XP
: P2 Normal
Assignee: Brett Wilson (Google)
URL:
Keywords:
Depends on:
Blocks: 11267
  Show dependency treegraph
 
Reported: 2006-10-10 17:35 PDT by Brett Wilson (Google)
Modified: 2007-07-04 22:52 PDT (History)
1 user (show)

See Also:


Attachments
Patch to make it work (5.08 KB, patch)
2006-10-12 13:48 PDT, Brett Wilson (Google)
sam: review-
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-10-10 17:35:45 PDT
It appears there is a Browse button that doesn't do anything and no text field currently.
Comment 1 Brett Wilson (Google) 2006-10-12 13:48:19 PDT
Created attachment 11053 [details]
Patch to make it work

This patch separates out the file upload controls from TemporaryLinkStubs into a new file. It implements the basic functionality necessary to open the standard Windows open file dialog and to set the filename back into the control.

Unimplemented is fancy splitting of the filename, so it will just get cut off if the control isn't wide enough.

This also has a spacing change to the vcproj for WebCore. Visual Studio will "fix" this every time somebody changes the file, so we should fix it to prevent everybody from having to manually factor it out of their diff.
Comment 2 Sam Weinig 2006-11-04 13:14:13 PST
Comment on attachment 11053 [details]
Patch to make it work

I'm r-  this due to a couple of style issues I see.

Please put the curly brace on the same line as the namespace declaration.

+namespace WebCore
+{

The curly brace for functions should go on it's own line and the content should be indented 4 spaces.  I'm also not sure the commented is need.

+// static
+PassRefPtr<FileChooser> FileChooser::create(Document* doc, RenderFileUploadControl* control) {
+  return new FileChooser(doc, control);
+}

Otherwise, it's looking good.