Bug 156759 - Web Inspector: Use debounce proxies in a couple more places
Summary: Web Inspector: Use debounce proxies in a couple more places
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Web Inspector (show other bugs)
Version: WebKit Local Build
Hardware: All All
: P2 Normal
Assignee: Timothy Hatcher
URL:
Keywords: InRadar
Depends on:
Blocks:
 
Reported: 2016-04-19 14:20 PDT by Timothy Hatcher
Modified: 2016-04-20 14:59 PDT (History)
7 users (show)

See Also:

Attachments
Patch (9.22 KB, patch)
2016-04-19 14:26 PDT, Timothy Hatcher 		joepeck: review+
timothy: commit-queue-
Details | Formatted Diff | Diff
Patch (37.42 KB, patch)
2016-04-19 15:36 PDT, Timothy Hatcher 		no flags Details | Formatted Diff | Diff
Patch (37.42 KB, patch)
2016-04-19 15:38 PDT, Timothy Hatcher 		no flags Details | Formatted Diff | Diff
Show Obsolete (2) View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Timothy Hatcher 2016-04-19 14:20:59 PDT 
Replace some old *Soon() functions with the soon proxy.
Comment 1 Timothy Hatcher 2016-04-19 14:26:08 PDT 
Created attachment 276757 [details]
Patch
Comment 2 Radar WebKit Bug Importer 2016-04-19 14:27:52 PDT 
<rdar://problem/25813633>
Comment 3 Radar WebKit Bug Importer 2016-04-19 14:27:53 PDT 
<rdar://problem/25813634>
Comment 4 Timothy Hatcher 2016-04-19 15:36:47 PDT 
Created attachment 276764 [details]
Patch
Comment 5 Timothy Hatcher 2016-04-19 15:38:39 PDT 
Created attachment 276765 [details]
Patch
Comment 6 Timothy Hatcher 2016-04-19 15:39:27 PDT 
Stupid webkit-patch…
Comment 7 Joseph Pecoraro 2016-04-20 14:37:49 PDT 
Comment on attachment 276757 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=276757&action=review

r=me, you can cq+ once the other lands

> Source/WebInspectorUI/UserInterface/Views/NavigationSidebarPanel.js:36
> -        this.contentView.element.addEventListener("scroll", this._updateContentOverflowShadowVisibilitySoon.bind(this));
> +        this.contentView.element.addEventListener("scroll", this.soon._updateContentOverflowShadowVisibility);

This may have just blown my mind.
Comment 8 Timothy Hatcher 2016-04-20 14:58:53 PDT 
Comment on attachment 276757 [details]
Patch

https://trac.webkit.org/changeset/199790