Diff
Modified: branches/safari-601.1.32-branch/Source/WebInspectorUI/ChangeLog (184254 => 184255)
--- branches/safari-601.1.32-branch/Source/WebInspectorUI/ChangeLog 2015-05-13 04:25:32 UTC (rev 184254)
+++ branches/safari-601.1.32-branch/Source/WebInspectorUI/ChangeLog 2015-05-13 04:25:36 UTC (rev 184255)
@@ -1,3 +1,28 @@
+2015-05-12 Dana Burkart
+Dana Burkart <[email protected]
[email protected]>
+
+ Merge r184108. rdar://problem/20903134
+
+ 2015-05-11 Timothy Hatcher <[email protected]>
+
+ Web Inspector: NavigationSidebarPanel leaks some event listeners
+ https://bugs.webkit.org/show_bug.cgi?id=144523
+
+ Reviewed by Joseph Pecoraro.
+
+ * UserInterface/Views/NavigationSidebarPanel.js:
+ (WebInspector.NavigationSidebarPanel):
+ (WebInspector.NavigationSidebarPanel.prototype.closed):
+ * UserInterface/Views/ResourceSidebarPanel.js:
+ (WebInspector.ResourceSidebarPanel.prototype.closed):
+ * UserInterface/Views/SearchSidebarPanel.js:
+ (WebInspector.SearchSidebarPanel.prototype.closed):
+ * UserInterface/Views/StorageSidebarPanel.js:
+ (WebInspector.StorageSidebarPanel.prototype.closed):
+ * UserInterface/Views/TimelineSidebarPanel.js:
+ (WebInspector.TimelineSidebarPanel.prototype.closed):
+
2015-05-06 Matt Baker <[email protected]>
Web Inspector: The text in the left pane overlaps the "Filter Time Events" field in the Timeline after the Web Inspector is resized
Modified: branches/safari-601.1.32-branch/Source/WebInspectorUI/UserInterface/Views/NavigationSidebarPanel.js (184254 => 184255)
--- branches/safari-601.1.32-branch/Source/WebInspectorUI/UserInterface/Views/NavigationSidebarPanel.js 2015-05-13 04:25:32 UTC (rev 184254)
+++ branches/safari-601.1.32-branch/Source/WebInspectorUI/UserInterface/Views/NavigationSidebarPanel.js 2015-05-13 04:25:36 UTC (rev 184255)
@@ -52,7 +52,8 @@
this.element.appendChild(this._topOverflowShadowElement);
}
- window.addEventListener("resize", this._updateContentOverflowShadowVisibility.bind(this));
+ this._boundUpdateContentOverflowShadowVisibility = this._updateContentOverflowShadowVisibility.bind(this);
+ window.addEventListener("resize", this._boundUpdateContentOverflowShadowVisibility);
this._filtersSetting = new WebInspector.Setting(identifier + "-navigation-sidebar-filters", {});
this._filterBar.filters = this._filtersSetting.value;
@@ -76,6 +77,12 @@
// Public
+ closed()
+ {
+ window.removeEventListener("resize", this._boundUpdateContentOverflowShadowVisibility);
+ WebInspector.Frame.removeEventListener(null, null, this);
+ }
+
get contentBrowser()
{
return this._contentBrowser;
Modified: branches/safari-601.1.32-branch/Source/WebInspectorUI/UserInterface/Views/ResourceSidebarPanel.js (184254 => 184255)
--- branches/safari-601.1.32-branch/Source/WebInspectorUI/UserInterface/Views/ResourceSidebarPanel.js 2015-05-13 04:25:32 UTC (rev 184254)
+++ branches/safari-601.1.32-branch/Source/WebInspectorUI/UserInterface/Views/ResourceSidebarPanel.js 2015-05-13 04:25:36 UTC (rev 184255)
@@ -56,6 +56,8 @@
closed()
{
+ super.closed();
+
WebInspector.Frame.removeEventListener(null, null, this);
WebInspector.frameResourceManager.removeEventListener(null, null, this);
WebInspector.debuggerManager.removeEventListener(null, null, this);
Modified: branches/safari-601.1.32-branch/Source/WebInspectorUI/UserInterface/Views/SearchSidebarPanel.js (184254 => 184255)
--- branches/safari-601.1.32-branch/Source/WebInspectorUI/UserInterface/Views/SearchSidebarPanel.js 2015-05-13 04:25:32 UTC (rev 184254)
+++ branches/safari-601.1.32-branch/Source/WebInspectorUI/UserInterface/Views/SearchSidebarPanel.js 2015-05-13 04:25:36 UTC (rev 184255)
@@ -64,6 +64,8 @@
closed()
{
+ super.closed();
+
WebInspector.Frame.removeEventListener(null, null, this);
}
Modified: branches/safari-601.1.32-branch/Source/WebInspectorUI/UserInterface/Views/StorageSidebarPanel.js (184254 => 184255)
--- branches/safari-601.1.32-branch/Source/WebInspectorUI/UserInterface/Views/StorageSidebarPanel.js 2015-05-13 04:25:32 UTC (rev 184254)
+++ branches/safari-601.1.32-branch/Source/WebInspectorUI/UserInterface/Views/StorageSidebarPanel.js 2015-05-13 04:25:36 UTC (rev 184255)
@@ -85,6 +85,8 @@
closed()
{
+ super.closed();
+
WebInspector.storageManager.removeEventListener(null, null, this);
WebInspector.applicationCacheManager.removeEventListener(null, null, this);
}
Modified: branches/safari-601.1.32-branch/Source/WebInspectorUI/UserInterface/Views/TimelineSidebarPanel.js (184254 => 184255)
--- branches/safari-601.1.32-branch/Source/WebInspectorUI/UserInterface/Views/TimelineSidebarPanel.js 2015-05-13 04:25:32 UTC (rev 184254)
+++ branches/safari-601.1.32-branch/Source/WebInspectorUI/UserInterface/Views/TimelineSidebarPanel.js 2015-05-13 04:25:36 UTC (rev 184255)
@@ -172,6 +172,8 @@
closed()
{
+ super.closed();
+
WebInspector.showReplayInterfaceSetting.removeEventListener(null, null, this);
WebInspector.replayManager.removeEventListener(null, null, this);
WebInspector.timelineManager.removeEventListener(null, null, this);
