[SCM] WebKit Debian packaging branch, debian/experimental, updated. upstream/1.3.3-10851-g50815da
bweinstein at apple.com
bweinstein at apple.com
Wed Dec 22 18:26:41 UTC 2010
The following commit has been merged in the debian/experimental branch:
commit 00e2dc900d775656266c31af06022521d6105298
Author: bweinstein at apple.com <bweinstein at apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Date: Fri Dec 10 23:41:04 2010 +0000
When we are creating a ContextMenuItem, only call setSubMenu if we have
a subMenu to set.
Reviewed by Ada Chan.
* platform/ContextMenuItem.cpp:
(WebCore::ContextMenuItem::ContextMenuItem):
git-svn-id: http://svn.webkit.org/repository/webkit/trunk@73813 268f45cc-cd09-0410-ab3c-d52691b4dbfc
diff --git a/WebCore/ChangeLog b/WebCore/ChangeLog
index ec0aa9f..315f2bf 100644
--- a/WebCore/ChangeLog
+++ b/WebCore/ChangeLog
@@ -1,3 +1,13 @@
+2010-12-10 Brian Weinstein <bweinstein at apple.com>
+
+ Reviewed by Ada Chan.
+
+ When we are creating a ContextMenuItem, only call setSubMenu if we have
+ a subMenu to set.
+
+ * platform/ContextMenuItem.cpp:
+ (WebCore::ContextMenuItem::ContextMenuItem):
+
2010-12-10 Kenneth Russell <kbr at google.com>
Unreviewed, another build fix. #include <wtf/UnusedParam.h> needed
diff --git a/WebCore/platform/ContextMenuItem.cpp b/WebCore/platform/ContextMenuItem.cpp
index bd321d9..0c2a6ea 100644
--- a/WebCore/platform/ContextMenuItem.cpp
+++ b/WebCore/platform/ContextMenuItem.cpp
@@ -39,7 +39,8 @@ ContextMenuItem::ContextMenuItem(ContextMenuItemType type, ContextMenuAction act
, m_enabled(true)
, m_checked(false)
{
- setSubMenu(subMenu);
+ if (subMenu)
+ setSubMenu(subMenu);
}
ContextMenuItem::ContextMenuItem(ContextMenuItemType type, ContextMenuAction action, const String& title, bool enabled, bool checked)
--
WebKit Debian packaging
More information about the Pkg-webkit-commits
mailing list