Changeset 119470 in webkit
- Timestamp:
- Jun 5, 2012, 2:08:04 AM (14 years ago)
- Location:
- trunk/Source
- Files:
-
- 7 edited
-
WebCore/ChangeLog (modified) (1 diff)
-
WebCore/loader/EmptyClients.cpp (modified) (2 diffs)
-
WebCore/loader/EmptyClients.h (modified) (12 diffs)
-
WebCore/svg/graphics/SVGImage.cpp (modified) (1 diff)
-
WebKit/chromium/ChangeLog (modified) (1 diff)
-
WebKit/chromium/src/WebHelperPluginImpl.cpp (modified) (1 diff)
-
WebKit/chromium/src/WebPagePopupImpl.cpp (modified) (1 diff)
Legend:
- Unmodified
- Added
- Removed
-
trunk/Source/WebCore/ChangeLog
r119469 r119470 1 2012-06-05 Kent Tamura <tkent@chromium.org> 2 3 Move some function definitions in EmptyClients.h to EmptyClients.cpp 4 https://bugs.webkit.org/show_bug.cgi?id=88285 5 6 Reviewed by Ryosuke Niwa. 7 8 If we have definitions of functions with RefPtr<> in EmptyClients.h like 9 10 virtual void runOpenPanel(Frame*, PassRefPtr<FileChooser>) { } 11 12 and a source file includes EmptyClient.h, the source file needs to 13 include FileChooser.h though the source file doesn't use FileChooser 14 class explicitly because the definition of runOpenPanel() needs to call 15 FileChooser::deref(). 16 To avoid this, we had better move such functions to EmptyClients.cpp. 17 18 No new tests. No behavior changes. 19 20 * loader/EmptyClients.cpp: Move such functions from EmptyClients.h to 21 here, and move EmptyPopupMenu and EmptySearchPopupMenu too because 22 they're used only by EmptyChromeClient. 23 (WebCore): 24 (EmptyPopupMenu): 25 (WebCore::EmptyPopupMenu::show): 26 (WebCore::EmptyPopupMenu::hide): 27 (WebCore::EmptyPopupMenu::updateFromElement): 28 (WebCore::EmptyPopupMenu::disconnectClient): 29 (EmptySearchPopupMenu): 30 (WebCore::EmptySearchPopupMenu::popupMenu): 31 (WebCore::EmptySearchPopupMenu::saveRecentSearches): 32 (WebCore::EmptySearchPopupMenu::loadRecentSearches): 33 (WebCore::EmptySearchPopupMenu::enabled): 34 (WebCore::EmptyChromeClient::createPopupMenu): 35 (WebCore::EmptyChromeClient::createSearchPopupMenu): 36 (WebCore::EmptyChromeClient::createColorChooser): 37 (WebCore::EmptyChromeClient::runOpenPanel): 38 (WebCore::EmptyFrameLoaderClient::dispatchDecidePolicyForNewWindowAction): 39 (WebCore::EmptyFrameLoaderClient::dispatchDecidePolicyForNavigationAction): 40 (WebCore::EmptyFrameLoaderClient::dispatchWillSendSubmitEvent): 41 (WebCore::EmptyFrameLoaderClient::dispatchWillSubmitForm): 42 (WebCore::EmptyFrameLoaderClient::createDocumentLoader): 43 (WebCore::EmptyFrameLoaderClient::createFrame): 44 (WebCore::EmptyFrameLoaderClient::createPlugin): 45 (WebCore::EmptyFrameLoaderClient::createJavaAppletWidget): 46 (WebCore::EmptyFrameLoaderClient::createMediaPlayerProxyPlugin): 47 (WebCore::EmptyFrameLoaderClient::createNetworkingContext): 48 (WebCore::EmptyFrameLoaderClient::dispatchIntent): 49 (WebCore::EmptyTextCheckerClient::requestCheckingOfString): 50 (WebCore::EmptyEditorClient::registerUndoStep): 51 (WebCore::EmptyEditorClient::registerRedoStep): 52 (WebCore::EmptyContextMenuClient::customizeMenu): 53 * loader/EmptyClients.h: Move such function definitions to 54 EmptyClient.cpp, and append OVERRIDE. 55 (EmptyChromeClient): 56 (EmptyFrameLoaderClient): 57 (EmptyTextCheckerClient): 58 (EmptyEditorClient): 59 (EmptyContextMenuClient): 60 * svg/graphics/SVGImage.cpp: Remove unnecessary #includes. 61 1 62 2012-06-05 Alexander Pavlov <apavlov@chromium.org> 2 63 -
trunk/Source/WebCore/loader/EmptyClients.cpp
r113811 r119470 29 29 #include "EmptyClients.h" 30 30 31 #i f ENABLE(SVG) || ENABLE(PAGE_POPUP)31 #include "DocumentLoader.h" 32 32 #include "FileChooser.h" 33 33 #include "FormState.h" 34 #include "Frame.h" 35 #include "FrameNetworkingContext.h" 34 36 #include "HTMLFormElement.h" 37 38 #if ENABLE(INPUT_TYPE_COLOR) 39 #include "ColorChooser.h" 40 #endif 35 41 36 42 namespace WebCore { 37 43 44 #if ENABLE(SVG) || ENABLE(PAGE_POPUP) 38 45 void fillWithEmptyClients(Page::PageClients& pageClients) 39 46 { … … 54 61 pageClients.inspectorClient = dummyInspectorClient; 55 62 } 63 #endif 64 65 class EmptyPopupMenu : public PopupMenu { 66 public: 67 virtual void show(const IntRect&, FrameView*, int) { } 68 virtual void hide() { } 69 virtual void updateFromElement() { } 70 virtual void disconnectClient() { } 71 }; 72 73 class EmptySearchPopupMenu : public SearchPopupMenu { 74 public: 75 virtual PopupMenu* popupMenu() { return m_popup.get(); } 76 virtual void saveRecentSearches(const AtomicString&, const Vector<String>&) { } 77 virtual void loadRecentSearches(const AtomicString&, Vector<String>&) { } 78 virtual bool enabled() { return false; } 79 80 private: 81 RefPtr<EmptyPopupMenu> m_popup; 82 }; 83 84 PassRefPtr<PopupMenu> EmptyChromeClient::createPopupMenu(PopupMenuClient*) const 85 { 86 return adoptRef(new EmptyPopupMenu()); 87 } 88 89 PassRefPtr<SearchPopupMenu> EmptyChromeClient::createSearchPopupMenu(PopupMenuClient*) const 90 { 91 return adoptRef(new EmptySearchPopupMenu()); 92 } 93 94 #if ENABLE(INPUT_TYPE_COLOR) 95 PassOwnPtr<ColorChooser> EmptyChromeClient::createColorChooser(ColorChooserClient*, const Color&) 96 { 97 return nullptr; 98 } 99 #endif 100 101 void EmptyChromeClient::runOpenPanel(Frame*, PassRefPtr<FileChooser>) 102 { 103 } 104 105 void EmptyFrameLoaderClient::dispatchDecidePolicyForNewWindowAction(FramePolicyFunction, const NavigationAction&, const ResourceRequest&, PassRefPtr<FormState>, const String&) 106 { 107 } 108 109 void EmptyFrameLoaderClient::dispatchDecidePolicyForNavigationAction(FramePolicyFunction, const NavigationAction&, const ResourceRequest&, PassRefPtr<FormState>) 110 { 111 } 112 113 void EmptyFrameLoaderClient::dispatchWillSendSubmitEvent(PassRefPtr<FormState>) 114 { 115 } 116 117 void EmptyFrameLoaderClient::dispatchWillSubmitForm(FramePolicyFunction, PassRefPtr<FormState>) 118 { 119 } 120 121 PassRefPtr<DocumentLoader> EmptyFrameLoaderClient::createDocumentLoader(const ResourceRequest& request, const SubstituteData& substituteData) 122 { 123 return DocumentLoader::create(request, substituteData); 124 } 125 126 PassRefPtr<Frame> EmptyFrameLoaderClient::createFrame(const KURL&, const String&, HTMLFrameOwnerElement*, const String&, bool, int, int) 127 { 128 return 0; 129 } 130 131 PassRefPtr<Widget> EmptyFrameLoaderClient::createPlugin(const IntSize&, HTMLPlugInElement*, const KURL&, const Vector<String>&, const Vector<String>&, const String&, bool) 132 { 133 return 0; 134 } 135 136 PassRefPtr<Widget> EmptyFrameLoaderClient::createJavaAppletWidget(const IntSize&, HTMLAppletElement*, const KURL&, const Vector<String>&, const Vector<String>&) 137 { 138 return 0; 139 } 140 141 #if ENABLE(PLUGIN_PROXY_FOR_VIDEO) 142 PassRefPtr<Widget> EmptyFrameLoaderClient::createMediaPlayerProxyPlugin(const IntSize&, HTMLMediaElement*, const KURL&, const Vector<String>&, const Vector<String>&, const String&) 143 { 144 return 0; 145 } 146 #endif 147 148 PassRefPtr<FrameNetworkingContext> EmptyFrameLoaderClient::createNetworkingContext() 149 { 150 return PassRefPtr<FrameNetworkingContext>(); 151 } 152 153 #if ENABLE(WEB_INTENTS) 154 void EmptyFrameLoaderClient::dispatchIntent(PassRefPtr<IntentRequest>) 155 { 156 } 157 #endif 158 159 void EmptyTextCheckerClient::requestCheckingOfString(PassRefPtr<TextCheckingRequest>) 160 { 161 } 162 163 void EmptyEditorClient::registerUndoStep(PassRefPtr<UndoStep>) 164 { 165 } 166 167 void EmptyEditorClient::registerRedoStep(PassRefPtr<UndoStep>) 168 { 169 } 170 171 #if ENABLE(CONTEXT_MENUS) 172 #if USE(CROSS_PLATFORM_CONTEXT_MENUS) 173 PassOwnPtr<ContextMenu> EmptyContextMenuClient::customizeMenu(PassOwnPtr<ContextMenu>) 174 { 175 return nullptr; 176 } 177 #endif 178 #endif 56 179 57 180 } 58 #endif -
trunk/Source/WebCore/loader/EmptyClients.h
r119192 r119470 33 33 #include "DeviceMotionClient.h" 34 34 #include "DeviceOrientationClient.h" 35 #include "DocumentLoader.h"36 35 #include "DragClient.h" 37 #include "EditCommand.h"38 36 #include "EditorClient.h" 39 37 #include "TextCheckerClient.h" … … 41 39 #include "FocusDirection.h" 42 40 #include "FrameLoaderClient.h" 43 #include "FrameNetworkingContext.h"44 41 #include "InspectorClient.h" 45 42 #include "Page.h" 46 #include "PopupMenu.h"47 43 #include "ResourceError.h" 48 #include "SearchPopupMenu.h"49 44 50 45 #if USE(V8) 51 46 #include <v8.h> 52 #endif53 54 #if ENABLE(INPUT_TYPE_COLOR)55 #include "ColorChooser.h"56 #endif57 58 #if ENABLE(WEB_INTENTS)59 #include "IntentRequest.h"60 47 #endif 61 48 … … 77 64 class GraphicsContext3D; 78 65 79 class EmptyPopupMenu : public PopupMenu {80 public:81 virtual void show(const IntRect&, FrameView*, int) {}82 virtual void hide() {}83 virtual void updateFromElement() {}84 virtual void disconnectClient() {}85 };86 87 class EmptySearchPopupMenu : public SearchPopupMenu {88 public:89 virtual PopupMenu* popupMenu() { return m_popup.get(); }90 virtual void saveRecentSearches(const AtomicString&, const Vector<String>&) {}91 virtual void loadRecentSearches(const AtomicString&, Vector<String>&) {}92 virtual bool enabled() { return false; }93 94 private:95 RefPtr<EmptyPopupMenu> m_popup;96 };97 98 66 class EmptyChromeClient : public ChromeClient { 99 67 public: … … 151 119 virtual bool selectItemAlignmentFollowsMenuWritingDirection() { return false; } 152 120 virtual bool hasOpenedPopup() const OVERRIDE { return false; } 153 virtual PassRefPtr<PopupMenu> createPopupMenu(PopupMenuClient*) const { return adoptRef(new EmptyPopupMenu()); }154 virtual PassRefPtr<SearchPopupMenu> createSearchPopupMenu(PopupMenuClient*) const { return adoptRef(new EmptySearchPopupMenu()); }121 virtual PassRefPtr<PopupMenu> createPopupMenu(PopupMenuClient*) const OVERRIDE; 122 virtual PassRefPtr<SearchPopupMenu> createSearchPopupMenu(PopupMenuClient*) const OVERRIDE; 155 123 #if ENABLE(PAGE_POPUP) 156 124 virtual PagePopup* openPagePopup(PagePopupClient*, const IntRect&) OVERRIDE { return 0; } … … 203 171 204 172 #if ENABLE(INPUT_TYPE_COLOR) 205 virtual PassOwnPtr<ColorChooser> createColorChooser(ColorChooserClient*, const Color&) { return nullptr; }206 #endif 207 208 virtual void runOpenPanel(Frame*, PassRefPtr<FileChooser>) { }173 virtual PassOwnPtr<ColorChooser> createColorChooser(ColorChooserClient*, const Color&) OVERRIDE; 174 #endif 175 176 virtual void runOpenPanel(Frame*, PassRefPtr<FileChooser>) OVERRIDE; 209 177 virtual void loadIconForFiles(const Vector<String>&, FileIconLoader*) { } 210 178 … … 298 266 299 267 virtual void dispatchDecidePolicyForResponse(FramePolicyFunction, const ResourceResponse&, const ResourceRequest&) { } 300 virtual void dispatchDecidePolicyForNewWindowAction(FramePolicyFunction, const NavigationAction&, const ResourceRequest&, PassRefPtr<FormState>, const String&) { }301 virtual void dispatchDecidePolicyForNavigationAction(FramePolicyFunction, const NavigationAction&, const ResourceRequest&, PassRefPtr<FormState>) { }268 virtual void dispatchDecidePolicyForNewWindowAction(FramePolicyFunction, const NavigationAction&, const ResourceRequest&, PassRefPtr<FormState>, const String&) OVERRIDE; 269 virtual void dispatchDecidePolicyForNavigationAction(FramePolicyFunction, const NavigationAction&, const ResourceRequest&, PassRefPtr<FormState>) OVERRIDE; 302 270 virtual void cancelPolicyCheck() { } 303 271 304 272 virtual void dispatchUnableToImplementPolicy(const ResourceError&) { } 305 273 306 virtual void dispatchWillSendSubmitEvent(PassRefPtr<FormState>) { }307 virtual void dispatchWillSubmitForm(FramePolicyFunction, PassRefPtr<FormState>) { }274 virtual void dispatchWillSendSubmitEvent(PassRefPtr<FormState>) OVERRIDE; 275 virtual void dispatchWillSubmitForm(FramePolicyFunction, PassRefPtr<FormState>) OVERRIDE; 308 276 309 277 virtual void revertToProvisionalState(DocumentLoader*) { } … … 350 318 virtual void prepareForDataSourceReplacement() { } 351 319 352 virtual PassRefPtr<DocumentLoader> createDocumentLoader(const ResourceRequest& request, const SubstituteData& substituteData) { return DocumentLoader::create(request, substituteData); }320 virtual PassRefPtr<DocumentLoader> createDocumentLoader(const ResourceRequest&, const SubstituteData&) OVERRIDE; 353 321 virtual void setTitle(const StringWithDirection&, const KURL&) { } 354 322 … … 374 342 virtual void didRunInsecureContent(SecurityOrigin*, const KURL&) { } 375 343 virtual void didDetectXSS(const KURL&, bool) { } 376 virtual PassRefPtr<Frame> createFrame(const KURL&, const String&, HTMLFrameOwnerElement*, const String&, bool, int, int) { return 0; }377 virtual PassRefPtr<Widget> createPlugin(const IntSize&, HTMLPlugInElement*, const KURL&, const Vector<String>&, const Vector<String>&, const String&, bool) { return 0; }378 virtual PassRefPtr<Widget> createJavaAppletWidget(const IntSize&, HTMLAppletElement*, const KURL&, const Vector<String>&, const Vector<String>&) { return 0; }344 virtual PassRefPtr<Frame> createFrame(const KURL&, const String&, HTMLFrameOwnerElement*, const String&, bool, int, int) OVERRIDE; 345 virtual PassRefPtr<Widget> createPlugin(const IntSize&, HTMLPlugInElement*, const KURL&, const Vector<String>&, const Vector<String>&, const String&, bool) OVERRIDE; 346 virtual PassRefPtr<Widget> createJavaAppletWidget(const IntSize&, HTMLAppletElement*, const KURL&, const Vector<String>&, const Vector<String>&) OVERRIDE; 379 347 #if ENABLE(PLUGIN_PROXY_FOR_VIDEO) 380 virtual PassRefPtr<Widget> createMediaPlayerProxyPlugin(const IntSize&, HTMLMediaElement*, const KURL&, const Vector<String>&, const Vector<String>&, const String&) { return 0; }348 virtual PassRefPtr<Widget> createMediaPlayerProxyPlugin(const IntSize&, HTMLMediaElement*, const KURL&, const Vector<String>&, const Vector<String>&, const String&) OVERRIDE; 381 349 virtual void hideMediaPlayerProxyPlugin(Widget*) { } 382 350 virtual void showMediaPlayerProxyPlugin(Widget*) { } … … 408 376 #endif 409 377 410 virtual PassRefPtr<FrameNetworkingContext> createNetworkingContext() { return PassRefPtr<FrameNetworkingContext>(); }378 virtual PassRefPtr<FrameNetworkingContext> createNetworkingContext() OVERRIDE; 411 379 412 380 #if ENABLE(WEB_INTENTS) 413 virtual void dispatchIntent(PassRefPtr<IntentRequest>) { }381 virtual void dispatchIntent(PassRefPtr<IntentRequest>) OVERRIDE; 414 382 #endif 415 383 }; … … 429 397 430 398 virtual void getGuessesForWord(const String&, const String&, Vector<String>&) { } 431 virtual void requestCheckingOfString(PassRefPtr<TextCheckingRequest>) { }399 virtual void requestCheckingOfString(PassRefPtr<TextCheckingRequest>) OVERRIDE; 432 400 }; 433 401 … … 469 437 virtual void didSetSelectionTypesForPasteboard() { } 470 438 471 virtual void registerUndoStep(PassRefPtr<UndoStep>) { }472 virtual void registerRedoStep(PassRefPtr<UndoStep>) { }439 virtual void registerUndoStep(PassRefPtr<UndoStep>) OVERRIDE; 440 virtual void registerRedoStep(PassRefPtr<UndoStep>) OVERRIDE; 473 441 virtual void clearUndoRedoOperations() { } 474 442 … … 547 515 548 516 #if USE(CROSS_PLATFORM_CONTEXT_MENUS) 549 virtual PassOwnPtr<ContextMenu> customizeMenu(PassOwnPtr<ContextMenu>) { return nullptr; }517 virtual PassOwnPtr<ContextMenu> customizeMenu(PassOwnPtr<ContextMenu>) OVERRIDE; 550 518 #else 551 519 virtual PlatformMenuDescription getCustomMenuFromDefaultItems(ContextMenu*) { return 0; } -
trunk/Source/WebCore/svg/graphics/SVGImage.cpp
r116001 r119470 31 31 #include "SVGImage.h" 32 32 33 #include "CachedPage.h"34 33 #include "DocumentLoader.h" 35 34 #include "EmptyClients.h" 36 #include "FileChooser.h"37 #include "FileIconLoader.h"38 #include "FloatRect.h"39 #include "Frame.h"40 #include "FrameLoader.h"41 35 #include "FrameView.h" 42 #include "GraphicsContext.h"43 #include "HTMLFormElement.h"44 36 #include "ImageBuffer.h" 45 #include "ImageObserver.h"46 #include "Length.h"47 #include "Page.h"48 37 #include "RenderSVGRoot.h" 49 #include "RenderView.h"50 #include "ResourceError.h"51 38 #include "SVGDocument.h" 52 #include "SVGLength.h"53 #include "SVGRenderSupport.h"54 39 #include "SVGSVGElement.h" 55 40 #include "Settings.h" -
trunk/Source/WebKit/chromium/ChangeLog
r119468 r119470 1 2012-06-05 Kent Tamura <tkent@chromium.org> 2 3 Move some function definitions in EmptyClients.h to EmptyClients.cpp 4 https://bugs.webkit.org/show_bug.cgi?id=88285 5 6 Reviewed by Ryosuke Niwa. 7 8 * src/WebHelperPluginImpl.cpp: Remove unnecessary #includes, and add necessary #includes. 9 * src/WebPagePopupImpl.cpp: ditto. 10 1 11 2012-06-05 Noel Gordon <noel.gordon@gmail.com> 2 12 -
trunk/Source/WebKit/chromium/src/WebHelperPluginImpl.cpp
r119434 r119470 33 33 34 34 #include "Chrome.h" 35 #include "DocumentLoader.h" 35 36 #include "EmptyClients.h" 36 37 #include "FocusController.h" 37 #include "FileChooser.h"38 #include "FocusController.h"39 #include "FormState.h"40 38 #include "FrameView.h" 41 #include "HTMLFormElement.h"42 39 #include "Page.h" 43 #include "PagePopupClient.h"44 40 #include "PageWidgetDelegate.h" 45 41 #include "Settings.h" 46 #include "WebInputEvent.h"47 #include "WebInputEventConversion.h"48 42 #include "WebFrameImpl.h" 49 #include "WebPagePopup.h"50 43 #include "WebViewClient.h" 51 44 #include "WebViewImpl.h" -
trunk/Source/WebKit/chromium/src/WebPagePopupImpl.cpp
r118482 r119470 34 34 #include "Chrome.h" 35 35 #include "DOMWindowPagePopup.h" 36 #include "DocumentLoader.h" 36 37 #include "EmptyClients.h" 37 #include "FileChooser.h"38 38 #include "FocusController.h" 39 #include "F ormState.h"39 #include "Frame.h" 40 40 #include "FrameView.h" 41 #include "HTMLFormElement.h"42 41 #include "Page.h" 43 42 #include "PagePopupClient.h"
Note:
See TracChangeset
for help on using the changeset viewer.