Title: [241360] branches/safari-608.1.5.1-branch/Source/WebInspectorUI
- Revision
- 241360
- Author
- [email protected]
- Date
- 2019-02-13 01:02:38 -0800 (Wed, 13 Feb 2019)
Log Message
Cherry-pick r240997. rdar://problem/47805023
Web Inspector: REGRESSION (r240947): Resources tab: can't select main frame after refreshing page
https://bugs.webkit.org/show_bug.cgi?id=194254
<rdar://problem/47805023>
Reviewed by Devin Rousso.
* UserInterface/Views/TreeOutline.js:
(WI.TreeOutline.prototype.selectionControllerSelectionDidChange):
TreeOutline should always call the TreeElement `select` and `deselect`
methods while processing selection changes. Having notifications
suppressed by `this._suppressNextSelectionDidChangeEvent` should only
affect the dispatching of TreeOutline events.
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@240997 268f45cc-cd09-0410-ab3c-d52691b4dbfc
Modified Paths
Diff
Modified: branches/safari-608.1.5.1-branch/Source/WebInspectorUI/ChangeLog (241359 => 241360)
--- branches/safari-608.1.5.1-branch/Source/WebInspectorUI/ChangeLog 2019-02-13 09:02:36 UTC (rev 241359)
+++ branches/safari-608.1.5.1-branch/Source/WebInspectorUI/ChangeLog 2019-02-13 09:02:38 UTC (rev 241360)
@@ -1,5 +1,40 @@
2019-02-13 Babak Shafiei <[email protected]>
+ Cherry-pick r240997. rdar://problem/47805023
+
+ Web Inspector: REGRESSION (r240947): Resources tab: can't select main frame after refreshing page
+ https://bugs.webkit.org/show_bug.cgi?id=194254
+ <rdar://problem/47805023>
+
+ Reviewed by Devin Rousso.
+
+ * UserInterface/Views/TreeOutline.js:
+ (WI.TreeOutline.prototype.selectionControllerSelectionDidChange):
+ TreeOutline should always call the TreeElement `select` and `deselect`
+ methods while processing selection changes. Having notifications
+ suppressed by `this._suppressNextSelectionDidChangeEvent` should only
+ affect the dispatching of TreeOutline events.
+
+
+ git-svn-id: https://svn.webkit.org/repository/webkit/trunk@240997 268f45cc-cd09-0410-ab3c-d52691b4dbfc
+
+ 2019-02-05 Matt Baker <[email protected]>
+
+ Web Inspector: REGRESSION (r240947): Resources tab: can't select main frame after refreshing page
+ https://bugs.webkit.org/show_bug.cgi?id=194254
+ <rdar://problem/47805023>
+
+ Reviewed by Devin Rousso.
+
+ * UserInterface/Views/TreeOutline.js:
+ (WI.TreeOutline.prototype.selectionControllerSelectionDidChange):
+ TreeOutline should always call the TreeElement `select` and `deselect`
+ methods while processing selection changes. Having notifications
+ suppressed by `this._suppressNextSelectionDidChangeEvent` should only
+ affect the dispatching of TreeOutline events.
+
+2019-02-13 Babak Shafiei <[email protected]>
+
Cherry-pick r240952. rdar://problem/47559021
Web Inspector: Resources: missing resource data for document on reload
Modified: branches/safari-608.1.5.1-branch/Source/WebInspectorUI/UserInterface/Views/TreeOutline.js (241359 => 241360)
--- branches/safari-608.1.5.1-branch/Source/WebInspectorUI/UserInterface/Views/TreeOutline.js 2019-02-13 09:02:36 UTC (rev 241359)
+++ branches/safari-608.1.5.1-branch/Source/WebInspectorUI/UserInterface/Views/TreeOutline.js 2019-02-13 09:02:38 UTC (rev 241360)
@@ -812,8 +812,7 @@
if (treeElement) {
if (treeElement.listItemElement)
treeElement.listItemElement.classList.remove("selected");
- if (!this._suppressNextSelectionDidChangeEvent)
- treeElement.deselect();
+ treeElement.deselect();
}
}
@@ -823,8 +822,8 @@
if (treeElement) {
if (treeElement.listItemElement)
treeElement.listItemElement.classList.add("selected");
- if (!this._suppressNextSelectionDidChangeEvent)
- treeElement.select();
+ const omitFocus = true;
+ treeElement.select(omitFocus);
}
}
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes
