Summary: | Web Inspector: message dispatch metrics should use high-resolution timing data | ||||||||||||
---|---|---|---|---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Brian Burg <burg> | ||||||||||
Component: | Web Inspector | Assignee: | BJ Burg <bburg> | ||||||||||
Status: | RESOLVED FIXED | ||||||||||||
Severity: | Normal | CC: | bburg, buildbot, graouts, joepeck, rniwa, timothy, webkit-bug-importer | ||||||||||
Priority: | P2 | Keywords: | InRadar | ||||||||||
Version: | 528+ (Nightly build) | ||||||||||||
Hardware: | All | ||||||||||||
OS: | All | ||||||||||||
Bug Depends on: | 42434 | ||||||||||||
Bug Blocks: | 147066 | ||||||||||||
Attachments: |
|
Description
Brian Burg
2014-07-31 12:45:15 PDT
Created attachment 259888 [details]
Proposed Fix
Comment on attachment 259888 [details] Proposed Fix View in context: https://bugs.webkit.org/attachment.cgi?id=259888&action=review Looks good to me. cq- so you don't enable it by default. > Source/WebInspectorUI/UserInterface/Protocol/InspectorBackend.js:44 > - this.dumpInspectorTimeStats = false; > + this.dumpInspectorTimeStats = true; > this.dumpInspectorProtocolMessages = false; > - this.warnForLongMessageHandling = false; > + this.warnForLongMessageHandling = true; I don't think you want to commit these. Note, you could write a test for these if you wanted. You would just enable these dump/warn flags at the start then trigger some messages. But I'm not sure that would be too valuable. I wonder how our jsmin.py is doing minifying with new syntax (template strings, arrow functions, etc). I wonder if the first time we have a newline in a template string if we will run into issues. Comment on attachment 259888 [details] Proposed Fix Attachment 259888 [details] did not pass mac-ews (mac): Output: http://webkit-queues.webkit.org/results/106488 Number of test failures exceeded the failure limit. Created attachment 259894 [details]
Archive of layout-test-results from ews101 for mac-mavericks
The attached test failures were seen while running run-webkit-tests on the mac-ews.
Bot: ews101 Port: mac-mavericks Platform: Mac OS X 10.9.5
(In reply to comment #4) > I wonder how our jsmin.py is doing minifying with new syntax (template > strings, arrow functions, etc). I wonder if the first time we have a newline > in a template string if we will run into issues. We already have some, I think. Comment on attachment 259888 [details] Proposed Fix Attachment 259888 [details] did not pass mac-wk2-ews (mac-wk2): Output: http://webkit-queues.webkit.org/results/106480 Number of test failures exceeded the failure limit. Created attachment 259896 [details]
Archive of layout-test-results from ews107 for mac-mavericks-wk2
The attached test failures were seen while running run-webkit-tests on the mac-wk2-ews.
Bot: ews107 Port: mac-mavericks-wk2 Platform: Mac OS X 10.9.5
Created attachment 259897 [details]
Proposed Fix
Comment on attachment 259888 [details] Proposed Fix View in context: https://bugs.webkit.org/attachment.cgi?id=259888&action=review > Source/WebInspectorUI/UserInterface/Base/Utilities.js:1129 > +function timestampMS() Just timestamp()? > Source/WebInspectorUI/UserInterface/Protocol/MessageDispatcher.js:68 > - if (this._dispatchTimeout) > + if (WebInspector._dispatchTimeout) I think we should use this. The other places should change. Committed r188946: <http://trac.webkit.org/changeset/188946> |