Title: [181949] releases/WebKitGTK/webkit-2.8/Source/WebKit2
- Revision
- 181949
- Author
- carlo...@webkit.org
- Date
- 2015-03-25 04:25:31 -0700 (Wed, 25 Mar 2015)
Log Message
Merge r181850 - [GTK][WK2] webkitBackForwardListChanged() should operate on a Vector reference
https://bugs.webkit.org/show_bug.cgi?id=142963
Reviewed by Carlos Garcia Campos.
* UIProcess/API/gtk/WebKitBackForwardList.cpp:
(webkitBackForwardListChanged): This function only reads from the passed-in
Vector of removed items, so only a const lvalue reference to the Vector
is needed.
* UIProcess/API/gtk/WebKitBackForwardListPrivate.h:
Modified Paths
Diff
Modified: releases/WebKitGTK/webkit-2.8/Source/WebKit2/ChangeLog (181948 => 181949)
--- releases/WebKitGTK/webkit-2.8/Source/WebKit2/ChangeLog 2015-03-25 11:23:54 UTC (rev 181948)
+++ releases/WebKitGTK/webkit-2.8/Source/WebKit2/ChangeLog 2015-03-25 11:25:31 UTC (rev 181949)
@@ -1,3 +1,16 @@
+2015-03-23 Zan Dobersek <zdober...@igalia.com>
+
+ [GTK][WK2] webkitBackForwardListChanged() should operate on a Vector reference
+ https://bugs.webkit.org/show_bug.cgi?id=142963
+
+ Reviewed by Carlos Garcia Campos.
+
+ * UIProcess/API/gtk/WebKitBackForwardList.cpp:
+ (webkitBackForwardListChanged): This function only reads from the passed-in
+ Vector of removed items, so only a const lvalue reference to the Vector
+ is needed.
+ * UIProcess/API/gtk/WebKitBackForwardListPrivate.h:
+
2015-03-23 Carlos Garcia Campos <cgar...@igalia.com>
Unreviewed. Update OptionsGTK.cmake and NEWS for 2.8.0 release.
Modified: releases/WebKitGTK/webkit-2.8/Source/WebKit2/UIProcess/API/gtk/WebKitBackForwardList.cpp (181948 => 181949)
--- releases/WebKitGTK/webkit-2.8/Source/WebKit2/UIProcess/API/gtk/WebKitBackForwardList.cpp 2015-03-25 11:23:54 UTC (rev 181948)
+++ releases/WebKitGTK/webkit-2.8/Source/WebKit2/UIProcess/API/gtk/WebKitBackForwardList.cpp 2015-03-25 11:25:31 UTC (rev 181949)
@@ -128,13 +128,13 @@
return list;
}
-void webkitBackForwardListChanged(WebKitBackForwardList* backForwardList, WebBackForwardListItem* webAddedItem, Vector<RefPtr<WebBackForwardListItem>> webRemovedItems)
+void webkitBackForwardListChanged(WebKitBackForwardList* backForwardList, WebBackForwardListItem* webAddedItem, const Vector<RefPtr<WebBackForwardListItem>>& webRemovedItems)
{
WebKitBackForwardListItem* addedItem = webkitBackForwardListGetOrCreateItem(backForwardList, webAddedItem);
GList* removedItems = nullptr;
WebKitBackForwardListPrivate* priv = backForwardList->priv;
- for (const auto& webItem : webRemovedItems) {
+ for (auto& webItem : webRemovedItems) {
removedItems = g_list_prepend(removedItems, g_object_ref(priv->itemsMap.get(webItem.get()).get()));
priv->itemsMap.remove(webItem.get());
}
Modified: releases/WebKitGTK/webkit-2.8/Source/WebKit2/UIProcess/API/gtk/WebKitBackForwardListPrivate.h (181948 => 181949)
--- releases/WebKitGTK/webkit-2.8/Source/WebKit2/UIProcess/API/gtk/WebKitBackForwardListPrivate.h 2015-03-25 11:23:54 UTC (rev 181948)
+++ releases/WebKitGTK/webkit-2.8/Source/WebKit2/UIProcess/API/gtk/WebKitBackForwardListPrivate.h 2015-03-25 11:25:31 UTC (rev 181949)
@@ -32,6 +32,6 @@
WebKitBackForwardList* webkitBackForwardListCreate(WebKit::WebBackForwardList*);
WebKitBackForwardListItem* webkitBackForwardListItemGetOrCreate(WebKit::WebBackForwardListItem*);
WebKit::WebBackForwardListItem* webkitBackForwardListItemGetItem(WebKitBackForwardListItem*);
-void webkitBackForwardListChanged(WebKitBackForwardList*, WebKit::WebBackForwardListItem* webAddedItem, Vector<RefPtr<WebKit::WebBackForwardListItem>>);
+void webkitBackForwardListChanged(WebKitBackForwardList*, WebKit::WebBackForwardListItem* webAddedItem, const Vector<RefPtr<WebKit::WebBackForwardListItem>>&);
#endif // WebKitBackForwardListPrivate_h
_______________________________________________
webkit-changes mailing list
webkit-changes@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-changes
