Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(94)

Side by Side Diff: Source/web/ChromeClientImpl.cpp

Issue 23819007: Have Node::document() return a reference instead of a pointer (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: Rebase on master Created 7 years, 3 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch | Annotate | Revision Log
« no previous file with comments | « Source/web/AutofillPopupMenuClient.cpp ('k') | Source/web/ContextMenuClientImpl.cpp » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 /* 1 /*
2 * Copyright (C) 2009 Google Inc. All rights reserved. 2 * Copyright (C) 2009 Google Inc. All rights reserved.
3 * Copyright (C) 2010 Nokia Corporation and/or its subsidiary(-ies). 3 * Copyright (C) 2010 Nokia Corporation and/or its subsidiary(-ies).
4 * 4 *
5 * Redistribution and use in source and binary forms, with or without 5 * Redistribution and use in source and binary forms, with or without
6 * modification, are permitted provided that the following conditions are 6 * modification, are permitted provided that the following conditions are
7 * met: 7 * met:
8 * 8 *
9 * * Redistributions of source code must retain the above copyright 9 * * Redistributions of source code must retain the above copyright
10 * notice, this list of conditions and the following disclaimer. 10 * notice, this list of conditions and the following disclaimer.
(...skipping 741 matching lines...) Expand 10 before | Expand all | Expand 10 after
752 setCursor(cursor); 752 setCursor(cursor);
753 } 753 }
754 754
755 void ChromeClientImpl::formStateDidChange(const Node* node) 755 void ChromeClientImpl::formStateDidChange(const Node* node)
756 { 756 {
757 if (m_webView->client()) 757 if (m_webView->client())
758 m_webView->client()->didChangeFormState(WebNode(const_cast<Node*>(node)) ); 758 m_webView->client()->didChangeFormState(WebNode(const_cast<Node*>(node)) );
759 759
760 // The current history item is not updated yet. That happens lazily when 760 // The current history item is not updated yet. That happens lazily when
761 // WebFrame::currentHistoryItem is requested. 761 // WebFrame::currentHistoryItem is requested.
762 WebFrameImpl* webframe = WebFrameImpl::fromFrame(node->document()->frame()); 762 WebFrameImpl* webframe = WebFrameImpl::fromFrame(node->document().frame());
763 if (webframe->client()) 763 if (webframe->client())
764 webframe->client()->didUpdateCurrentHistoryItem(webframe); 764 webframe->client()->didUpdateCurrentHistoryItem(webframe);
765 } 765 }
766 766
767 void ChromeClientImpl::getPopupMenuInfo(PopupContainer* popupContainer, 767 void ChromeClientImpl::getPopupMenuInfo(PopupContainer* popupContainer,
768 WebPopupMenuInfo* info) 768 WebPopupMenuInfo* info)
769 { 769 {
770 const Vector<PopupItem*>& inputItems = popupContainer->popupData(); 770 const Vector<PopupItem*>& inputItems = popupContainer->popupData();
771 771
772 WebVector<WebMenuItemInfo> outputItems(inputItems.size()); 772 WebVector<WebMenuItemInfo> outputItems(inputItems.size());
(...skipping 243 matching lines...) Expand 10 before | Expand all | Expand 10 after
1016 { 1016 {
1017 } 1017 }
1018 1018
1019 void NavigatorContentUtilsClientImpl::registerProtocolHandler(const String& sche me, const String& baseURL, const String& url, const String& title) 1019 void NavigatorContentUtilsClientImpl::registerProtocolHandler(const String& sche me, const String& baseURL, const String& url, const String& title)
1020 { 1020 {
1021 m_webView->client()->registerProtocolHandler(scheme, baseURL, url, title); 1021 m_webView->client()->registerProtocolHandler(scheme, baseURL, url, title);
1022 } 1022 }
1023 #endif 1023 #endif
1024 1024
1025 } // namespace WebKit 1025 } // namespace WebKit
OLDNEW
« no previous file with comments | « Source/web/AutofillPopupMenuClient.cpp ('k') | Source/web/ContextMenuClientImpl.cpp » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698