Summary: | Remove Page::javaScriptURLsAreAllowed setting. | ||||||||
---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Andreas Kling <kling> | ||||||
Component: | WebCore Misc. | Assignee: | Andreas Kling <kling> | ||||||
Status: | RESOLVED FIXED | ||||||||
Severity: | Normal | CC: | koivisto, sam, webkit.review.bot | ||||||
Priority: | P2 | ||||||||
Version: | 528+ (Nightly build) | ||||||||
Hardware: | Unspecified | ||||||||
OS: | Unspecified | ||||||||
Attachments: |
|
Description
Andreas Kling
2012-10-21 16:33:51 PDT
Created attachment 169817 [details]
Patch
Comment on attachment 169817 [details]
Patch
r=me
Comment on attachment 169817 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=169817&action=review > Source/WebKit/win/WebView.cpp:-6183 > -HRESULT WebView::setJavaScriptURLsAreAllowed(BOOL areAllowed) > -{ > - m_page->setJavaScriptURLsAreAllowed(areAllowed); > - return S_OK; > -} > - Removing this will break the COM ABI. Just remove the implementation and have it return E_NOTIMPL instead. Created attachment 169819 [details]
Patch v2
Ok, let's not break the COM ABI.
Comment on attachment 169819 [details] Patch v2 Clearing flags on attachment: 169819 Committed r132023: <http://trac.webkit.org/changeset/132023> All reviewed patches have been landed. Closing bug. |