PassRefPtr<FileChoose> should be FileChoose*.
Though some implementations of ChromeClient::chooseIconForFiles() might have ownership of the FileChooser instance, we don't need to use PassRefPtr<>. Such implementations can store the FileChooser pointer to RefPtr<FileChooser>.
Created attachment 53273 [details]
Probably we want to change it for Chrome::runOpenPanel() too?
Comment on attachment 53273 [details]
Clearing flags on attachment: 53273
Committed r57903: <http://trac.webkit.org/changeset/57903>
All reviewed patches have been landed. Closing bug.