Title: [199689] trunk
- Revision
- 199689
- Author
- joep...@webkit.org
- Date
- 2016-04-18 14:53:02 -0700 (Mon, 18 Apr 2016)
Log Message
Unreviewed revert accidental ChangeLog modifications.
Modified Paths
Diff
Modified: trunk/LayoutTests/ChangeLog (199688 => 199689)
--- trunk/LayoutTests/ChangeLog 2016-04-18 21:50:24 UTC (rev 199688)
+++ trunk/LayoutTests/ChangeLog 2016-04-18 21:53:02 UTC (rev 199689)
@@ -1,23 +1,3 @@
-2016-04-18 Joseph Pecoraro <pecor...@apple.com>
-
- Web Inspector: sourceMappingURL not used when sourceURL is set
- https://bugs.webkit.org/show_bug.cgi?id=156021
- <rdar://problem/25438417>
-
- Reviewed by Timothy Hatcher.
-
- * inspector/debugger/scriptParsed.html:
- * inspector/debugger/search-scripts.html:
- * inspector/debugger/setBreakpointByUrl-sourceURL.html:
- * inspector/debugger/sourceURLs.html:
- Update tests that need to handle sourceURL separately.
-
- * inspector/model/resources/relationship-named.js: Added.
- * inspector/model/resources/relationship-normal.js: Added.
- * inspector/model/script-resource-relationship-expected.txt: Added.
- * inspector/model/script-resource-relationship.html: Added.
- Tests for Script and Resource relationships.
-
2016-04-18 Brent Fulgham <bfulg...@apple.com>
CSP: Remove stubs for dynamically-added favicons (via link rel="icon")
Modified: trunk/Source/_javascript_Core/ChangeLog (199688 => 199689)
--- trunk/Source/_javascript_Core/ChangeLog 2016-04-18 21:50:24 UTC (rev 199688)
+++ trunk/Source/_javascript_Core/ChangeLog 2016-04-18 21:53:02 UTC (rev 199689)
@@ -1,27 +1,3 @@
-2016-04-18 Joseph Pecoraro <pecor...@apple.com>
-
- Web Inspector: sourceMappingURL not used when sourceURL is set
- https://bugs.webkit.org/show_bug.cgi?id=156021
- <rdar://problem/25438417>
-
- Reviewed by Timothy Hatcher.
-
- Clean up Debugger.sourceParsed to separately include:
-
- - url ("resource URL", "source url" in JSC APIs)
- - sourceURL - //# sourceURL directive
-
- By always having the resource URL the Web Inspector frontend
- can better match this Script to a Resource of the same URL,
- and decide to use the sourceURL if it is available when
- appropriate.
-
- * inspector/protocol/Debugger.json:
- * inspector/agents/InspectorDebuggerAgent.cpp:
- (Inspector::InspectorDebuggerAgent::setBreakpointByUrl):
- (Inspector::InspectorDebuggerAgent::didParseSource):
- Send the new sourceParsed parameters.
-
2016-04-18 Filip Pizlo <fpi...@apple.com>
ToThis should have a fast path based on type info flags
Modified: trunk/Source/WebInspectorUI/ChangeLog (199688 => 199689)
--- trunk/Source/WebInspectorUI/ChangeLog 2016-04-18 21:50:24 UTC (rev 199688)
+++ trunk/Source/WebInspectorUI/ChangeLog 2016-04-18 21:53:02 UTC (rev 199689)
@@ -1,68 +1,3 @@
-2016-04-18 Joseph Pecoraro <pecor...@apple.com>
-
- Web Inspector: sourceMappingURL not used when sourceURL is set
- https://bugs.webkit.org/show_bug.cgi?id=156021
- <rdar://problem/25438417>
-
- Reviewed by Timothy Hatcher.
-
- Previously Debugger.sourceParsed only providing the sourceURL, and
- wiping out the resourceURL, meant that a Script from a Resource that
- set a sourceURL directive would fail to be associated with its Resource.
-
- This would result in duplicated tree elements in the Resources Sidebar,
- one for the Resource, and one for the Script. With the Script getting
- ultimately getting the SourceMap resources. However, since the frontend
- prefers Resources over Scripts when possible, an error that generated
- from the script would point to a location in the Resource, not following
- source maps.
-
- By always providing the resource URL in Debugger.sourceParsed, a Script
- can better be associated with its Resource. The result is now a single
- shared tree element in the Resources Sidebar, and the Resource getting
- the SourceMap resources. Now the script error goes through the Resource
- to its SourceMap resources as we would expect.
-
- * UserInterface/Protocol/DebuggerObserver.js:
- (WebInspector.DebuggerObserver):
- (WebInspector.DebuggerObserver.prototype.scriptParsed):
- We now have to handle two different signatures of scriptParsed. One
- for legacy, and one for non-legacy. Cache that value early on, since
- scriptParsed happens a lot.
-
- * UserInterface/Protocol/InspectorBackend.js:
- (InspectorBackend.Agent.prototype.hasEventParameter):
- Runtime check a protocol event to see if it has a parameter. This
- is used to check if Debugger.sourceParsed is legacy or not based
- on if it has the legacy "hasSourceURL" parameter.
-
- * UserInterface/Models/Script.js:
- (WebInspector.Script):
- (WebInspector.Script.prototype.get sourceURL):
- Treat sourceURL and url separately.
-
- (WebInspector.Script.prototype.get displayName):
- Handle both the url and sourceURL in displayName.
-
- * UserInterface/Controllers/DebuggerManager.js:
- (WebInspector.DebuggerManager.prototype.get knownNonResourceScripts):
- (WebInspector.DebuggerManager.prototype.debuggerDidPause):
- (WebInspector.DebuggerManager.prototype.scriptDidParse):
- * UserInterface/Protocol/RemoteObject.js:
- (WebInspector.RemoteObject.prototype.findFunctionSourceCodeLocation):
- Update code that checks the sourceURL to explicitly use sourceURL.
-
- * UserInterface/Controllers/SourceMapManager.js:
- (WebInspector.SourceMapManager.prototype.downloadSourceMap):
- For legacy backends, or in case we get a resource that has an incomplete
- baseURL, attempt to get an absolute URL based on the main resource.
-
- * UserInterface/Views/DebuggerSidebarPanel.js:
- (WebInspector.DebuggerSidebarPanel.prototype._addScript):
- * UserInterface/Views/ResourceSidebarPanel.js:
- (WebInspector.ResourceSidebarPanel.prototype._scriptWasAdded):
- Ignore scripts without a url or sourceURL.
-
2016-04-18 Timothy Hatcher <timo...@apple.com>
Web Inspector: NavigationSidebarPanel does a lot of style recalc and layout on busy sites
_______________________________________________
webkit-changes mailing list
webkit-changes@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-changes
