If you create a local html file, load it in Safari, enable the Web Inspector and move to the Resrouce view, update the local html file, refersh the page, and view the code in the Resources view, it doesn't change. Essentially, the update code does not appear. * STEPS TO REPRODUCE 1. Launch WebKit nightly 2. Open the attached sample HTML file in Safari 3. Enable the "Develop" menu via the Safari Advanced preferences 4. Via the Develop menu, enable the Web Inspector 5. Switch to the Resrouces view inside the Web Inspector and view the code 6. Open the HTML file from its location on disk an modify it; for example, just below line 5 (which includes "console.log('page loaded');"), add "document.getElementById('target_div');" 7. Swich back to Safari, refresh the page, and view the code in the Resources view * CURRENT RESULT The code in the Resources section of the Web Inspector aren't updated; the original code is still visible (see the attached screen recording). * INTENDED RESULT If chages are made to the local file, its associated code should always update the Resources section of the Web Inspector upon refresh. rdar://problem/20755411
Created attachment 252207 [details] Patch
Comment on attachment 252207 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=252207&action=review r=me > Source/WebInspectorUI/UserInterface/Views/ResourceSidebarPanel.js:-167 > - // The navigation path needs update when the main frame changes, since all resources are under the main frame. > - this.contentBrowser.updateHierarchicalPathForCurrentContentView(); Is this updateHierarchicalPathForCurrentContentView call still needed? If not, we should get rid of it, because this is was the only caller! > Source/WebInspectorUI/UserInterface/Views/ResourceSidebarPanel.js:183 > + // Cookie restoration will attempt to re-select the resource we were showing. Nit: "State restoration" instead of "Cookie restoration"?
Comment on attachment 252207 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=252207&action=review >> Source/WebInspectorUI/UserInterface/Views/ResourceSidebarPanel.js:-167 >> - this.contentBrowser.updateHierarchicalPathForCurrentContentView(); > > Is this updateHierarchicalPathForCurrentContentView call still needed? > > If not, we should get rid of it, because this is was the only caller! Stand by, I have a new use for it in a coming patch.
Comment on attachment 252207 [details] Patch Clearing flags on attachment: 252207 Committed r183713: <http://trac.webkit.org/changeset/183713>
All reviewed patches have been landed. Closing bug.