Minor code cleanup. I didn't merge this into the didChooseFilesForOpenPanel message, since the necessity of sandbox extensions depends on ENABLE(SANDBOX_EXTENSIONS). I don't want to conditionally define the same message more than one way since we don't do that anywhere else that I'm aware of.
Created attachment 316383 [details] Proposed Fix
Comment on attachment 316383 [details] Proposed Fix Clearing flags on attachment: 316383 Committed r219887: <http://trac.webkit.org/changeset/219887>
All reviewed patches have been landed. Closing bug.
Comment on attachment 316383 [details] Proposed Fix View in context: https://bugs.webkit.org/attachment.cgi?id=316383&action=review > Source/WebKit/WebProcess/WebPage/WebPage.cpp:3841 > + for (size_t i = 0; i < handles.size(); ++i) { I would prefer a modern for loop here. Presuming that there is a begin/end function on HandleArray.
Re-opened since this is blocked by bug 174867
Created attachment 316455 [details] Followup fix
Comment on attachment 316455 [details] Followup fix Heh. Better.
Comment on attachment 316455 [details] Followup fix Clearing flags on attachment: 316455 Committed r219952: <http://trac.webkit.org/changeset/219952>