[Webkit-unassigned] [Bug 97032] Web Inspector: load sourcemaps asynchronously, part 1/2

[bugzilla-daemon at webkit.org]

https://bugs.webkit.org/show_bug.cgi?id=97032


Vsevolod Vlasov <vsevik at chromium.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |vsevik at chromium.org




--- Comment #8 from Vsevolod Vlasov <vsevik at chromium.org>  2012-09-26 07:35:26 PST ---
Few thoughts:
1) rawLocationToUILocation should never return null. This method is manly used to stop on the breakpoint: we don't want to end up in the situation when we are stopped but the user can't see the execution line.

2) I'm fine (to some extent) if uiLocationToRawLocation returns null at some point. This is actually happening for snippets and that will happen even more often when we support editing of files other than those loaded from the network (a script with syntax error is a similar situation).

3) rawLocationToUILocation should always fallback to "temporary" uiSourceCodes.
This could be implemented based on the https://bugs.webkit.org/show_bug.cgi?id=97680. The patch in this bug is not yet landed so I suggest that we wait another week before moving on with your patch.

-- 
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.