Bug 206158 - [LFC][Multithreading] Introduce ConcurrentLayoutContext
Summary: [LFC][Multithreading] Introduce ConcurrentLayoutContext
Status: NEW
Alias: None
Product: WebKit
Classification: Unclassified
Component: Layout and Rendering (show other bugs)
Version: WebKit Nightly Build
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: zalan
URL:
Keywords:
Depends on:
Blocks: LFC
  Show dependency tree / graph
 
Reported: 2020-01-12 17:00 PST by zalan
Modified: 2020-08-03 09:40 PDT (History)
5 users (show)

See Also:

Attachments
Patch (23.03 KB, patch)
2020-01-12 17:09 PST, zalan 		no flags Details | Formatted Diff | Diff
Patch (23.04 KB, patch)
2020-01-12 18:00 PST, zalan 		no flags Details | Formatted Diff | Diff
Patch (22.29 KB, patch)
2020-01-25 05:46 PST, zalan 		no flags Details | Formatted Diff | Diff
Resize screen recording (33.37 MB, video/quicktime)
2020-01-25 05:53 PST, zalan 		no flags Details
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 zalan 2020-01-12 17:00:07 PST 
see bug 205601
Comment 1 zalan 2020-01-12 17:09:32 PST 
Created attachment 387497 [details]
Patch
Comment 2 zalan 2020-01-12 18:00:27 PST 
Created attachment 387499 [details]
Patch
Comment 3 Simon Fraser (smfr) 2020-01-13 11:08:11 PST 
Comment on attachment 387499 [details]
Patch

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

> Source/WebCore/ChangeLog:8
> +        BackgroundLayoutContext is here to take care of multithreaded layout on the formatting context level.

Not sure I like the term "background". Maybe "Threaded" or "Parallel" or "DependencyTree"?
Comment 4 zalan 2020-01-13 11:09:47 PST 
(In reply to Simon Fraser (smfr) from comment #3)
> Comment on attachment 387499 [details]
> Patch
> 
> View in context:
> https://bugs.webkit.org/attachment.cgi?id=387499&action=review
> 
> > Source/WebCore/ChangeLog:8
> > +        BackgroundLayoutContext is here to take care of multithreaded layout on the formatting context level.
> 
> Not sure I like the term "background". Maybe "Threaded" or "Parallel" or
> "DependencyTree"?
Threaded is my fav.
Comment 5 Sam Weinig 2020-01-13 13:33:07 PST 
Concurrent?
Comment 6 zalan 2020-01-13 14:06:08 PST 
(In reply to Sam Weinig from comment #5)
> Concurrent?
That's a pretty good one!
Comment 7 Sam Weinig 2020-01-13 17:52:12 PST 
(In reply to zalan from comment #6)
> (In reply to Sam Weinig from comment #5)
> > Concurrent?
> That's a pretty good one!

Hopefully the most accurate too (insert obvious Rob Pike clip here: https://www.youtube.com/watch?v=cN_DpYBzKso)?
Comment 8 zalan 2020-01-13 18:51:27 PST 
(In reply to Sam Weinig from comment #7)
> (In reply to zalan from comment #6)
> > (In reply to Sam Weinig from comment #5)
> > > Concurrent?
> > That's a pretty good one!
> 
> Hopefully the most accurate too (insert obvious Rob Pike clip here:
> https://www.youtube.com/watch?v=cN_DpYBzKso)?
wow this talk!
Comment 9 zalan 2020-01-20 07:06:42 PST 
slightly related bug 206489
Comment 10 zalan 2020-01-25 05:46:56 PST 
Created attachment 388769 [details]
Patch
Comment 11 zalan 2020-01-25 05:53:35 PST 
Created attachment 388770 [details]
Resize screen recording