Bug 116883 - Web Inspector: Remove console.profiles from window.console API
Summary: Web Inspector: Remove console.profiles from window.console API
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Web Inspector (show other bugs)
Version: 528+ (Nightly build)
Hardware: All All
: P2 Normal
Assignee: Joseph Pecoraro
URL:
Keywords: InRadar
: 43471 (view as bug list)
Depends on:
Blocks:
 
Reported: 2013-05-28 13:21 PDT by Joseph Pecoraro
Modified: 2014-02-26 11:29 PST (History)
13 users (show)

See Also:


Attachments
[PATCH] Proposed Fix (37.51 KB, patch)
2013-08-14 15:01 PDT, Joseph Pecoraro
buildbot: commit-queue-
Details | Formatted Diff | Diff
Archive of layout-test-results from webkit-ews-07 for mac-mountainlion (453.62 KB, application/zip)
2013-08-14 16:22 PDT, Build Bot
no flags Details
Archive of layout-test-results from webkit-ews-16 for mac-mountainlion-wk2 (502.71 KB, application/zip)
2013-08-16 11:39 PDT, Build Bot
no flags Details
[PATCH] Proposed Fix (7.55 KB, patch)
2014-02-25 17:32 PST, Joseph Pecoraro
timothy: review+
Details | Formatted Diff | Diff
[PATCH] For Bots (10.11 KB, patch)
2014-02-25 18:02 PST, Joseph Pecoraro
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Joseph Pecoraro 2013-05-28 13:21:15 PDT
Blink did this. I agree, we should too.
<https://chromium.googlesource.com/chromium/blink/+/6896b8cc83bc337bca452e85b8db0939cc1873ca>

>     This change removes profiles property from window.console
>     objec and all related classes.
>     console.profile/profileEnd can still be used to record
>     profile but the profiles themselves should be retrieved
>     only using remote debugging protocol.
Comment 1 Radar WebKit Bug Importer 2013-05-28 13:21:26 PDT
<rdar://problem/14004223>
Comment 2 Joseph Pecoraro 2013-08-14 15:01:39 PDT
Created attachment 208762 [details]
[PATCH] Proposed Fix

Remove console.profiles and related code.
Comment 3 Build Bot 2013-08-14 16:22:04 PDT
Comment on attachment 208762 [details]
[PATCH] Proposed Fix

Attachment 208762 [details] did not pass mac-ews (mac):
Output: http://webkit-queues.appspot.com/results/1453986

New failing tests:
fast/profiler/document-dot-write.html
fast/profiler/nested-anonymous-functon.html
fast/profiler/anonymous-function-calls-built-in-functions.html
fast/profiler/built-in-function-calls-anonymous.html
fast/profiler/apply.html
fast/profiler/built-in-function-calls-user-defined-function.html
fast/profiler/multiple-and-different-scoped-anonymous-function-calls.html
http/tests/inspector/inspect-element.html
fast/profiler/nested-start-and-stop-profiler.html
fast/profiler/execution-context-and-eval-on-same-line.html
fast/profiler/multiple-and-different-scoped-function-calls.html
fast/profiler/one-execution-context.html
fast/profiler/compare-multiple-profiles.html
fast/profiler/multiple-anonymous-functions-called-from-the-same-function.html
fast/profiler/named-functions-with-display-names.html
fast/profiler/no-execution-context.html
fast/profiler/inline-event-handler.html
fast/profiler/multiple-frames.html
fast/profiler/profiling-from-a-nested-location-but-stop-profiling-outside-the-nesting.html
fast/profiler/anonymous-functions-with-display-names.html
fast/profiler/anonymous-function-calls-eval.html
fast/profiler/call.html
fast/profiler/profile-with-no-title.html
fast/profiler/many-calls-in-the-same-scope.html
fast/profiler/calling-the-function-that-started-the-profiler-from-another-scope.html
fast/profiler/constructor.html
fast/profiler/anonymous-event-handler.html
fast/profiler/anonymous-function-called-from-different-contexts.html
fast/profiler/event-handler.html
fast/profiler/profile-calls-in-included-file.html
Comment 4 Build Bot 2013-08-14 16:22:06 PDT
Created attachment 208769 [details]
Archive of layout-test-results from webkit-ews-07 for mac-mountainlion

The attached test failures were seen while running run-webkit-tests on the mac-ews.
Bot: webkit-ews-07  Port: mac-mountainlion  Platform: Mac OS X 10.8.4
Comment 5 Joseph Pecoraro 2013-08-14 17:48:03 PDT
Hmm, it looks like there are a bunch of fast/profiler tests that depend on console.profiles.
Comment 6 Build Bot 2013-08-16 11:39:24 PDT
Comment on attachment 208762 [details]
[PATCH] Proposed Fix

Attachment 208762 [details] did not pass mac-wk2-ews (mac-wk2):
Output: http://webkit-queues.appspot.com/results/1465745

New failing tests:
fast/profiler/document-dot-write.html
fast/profiler/nested-anonymous-functon.html
fast/profiler/anonymous-function-calls-built-in-functions.html
fast/profiler/multiple-frames.html
fast/profiler/apply.html
fast/profiler/built-in-function-calls-user-defined-function.html
http/tests/inspector/inspect-element.html
fast/profiler/execution-context-and-eval-on-same-line.html
fast/profiler/nested-start-and-stop-profiler.html
fast/profiler/multiple-and-different-scoped-anonymous-function-calls.html
fast/profiler/multiple-and-different-scoped-function-calls.html
fast/profiler/one-execution-context.html
fast/profiler/compare-multiple-profiles.html
editing/pasteboard/copy-image-with-alt-text.html
fast/profiler/multiple-anonymous-functions-called-from-the-same-function.html
fast/profiler/named-functions-with-display-names.html
fast/profiler/no-execution-context.html
fast/profiler/inline-event-handler.html
fast/profiler/built-in-function-calls-anonymous.html
fast/profiler/anonymous-functions-with-display-names.html
fast/profiler/anonymous-function-calls-eval.html
fast/profiler/call.html
fast/profiler/many-calls-in-the-same-scope.html
fast/profiler/calling-the-function-that-started-the-profiler-from-another-scope.html
fast/profiler/constructor.html
fast/profiler/anonymous-event-handler.html
fast/profiler/anonymous-function-called-from-different-contexts.html
fast/profiler/event-handler.html
fast/profiler/profile-calls-in-included-file.html
Comment 7 Build Bot 2013-08-16 11:39:26 PDT
Created attachment 208936 [details]
Archive of layout-test-results from webkit-ews-16 for mac-mountainlion-wk2

The attached test failures were seen while running run-webkit-tests on the mac-wk2-ews.
Bot: webkit-ews-16  Port: mac-mountainlion-wk2  Platform: Mac OS X 10.8.4
Comment 8 Timothy Hatcher 2014-01-10 15:37:16 PST
Moving to the right component.
Comment 9 Joseph Pecoraro 2014-02-25 16:31:32 PST
*** Bug 43471 has been marked as a duplicate of this bug. ***
Comment 10 Joseph Pecoraro 2014-02-25 17:30:10 PST
To keep tests working lets just move it to Internals. I have a patch, seems to work fine.
Comment 11 Joseph Pecoraro 2014-02-25 17:32:21 PST
Created attachment 225204 [details]
[PATCH] Proposed Fix

Lets see what the bots do. fast/profiler tests pass for me.
Comment 12 Joseph Pecoraro 2014-02-25 18:02:39 PST
Created attachment 225207 [details]
[PATCH] For Bots
Comment 13 Joseph Pecoraro 2014-02-26 11:29:52 PST
Landed <http://trac.webkit.org/changeset/164730>.