RESOLVED FIXED 93915
[WK2] Track multiple Web Processes per context
https://bugs.webkit.org/show_bug.cgi?id=93915
Summary [WK2] Track multiple Web Processes per context
Alexey Proskuryakov
Reported 2012-08-13 17:33:59 PDT
Patch forthcoming.
Attachments
proposed patch (16.24 KB, patch)
2012-08-13 17:43 PDT, Alexey Proskuryakov
sam: review+
gyuyoung.kim: commit-queue-
For EWS (18.18 KB, patch)
2012-08-14 13:31 PDT, Alexey Proskuryakov
no flags
Alexey Proskuryakov
Comment 1 2012-08-13 17:43:33 PDT
Created attachment 158166 [details] proposed patch
WebKit Review Bot
Comment 2 2012-08-13 17:45:24 PDT
Attachment 158166 [details] did not pass style-queue: Failed to run "['Tools/Scripts/check-webkit-style', '--diff-files', u'Source/WebKit2/ChangeLog', u'Source/WebKit..." exit_code: 1 Source/WebKit2/UIProcess/WebContext.cpp:479: An else statement can be removed when the prior "if" concludes with a return, break, continue or goto statement. [readability/control_flow] [4] Source/WebKit2/UIProcess/WebContext.cpp:490: An else statement can be removed when the prior "if" concludes with a return, break, continue or goto statement. [readability/control_flow] [4] Source/WebKit2/UIProcess/WebContext.cpp:523: One line control clauses should not use braces. [whitespace/braces] [4] Total errors found: 3 in 4 files If any of these errors are false positives, please file a bug against check-webkit-style.
Sam Weinig
Comment 3 2012-08-13 18:14:51 PDT
Comment on attachment 158166 [details] proposed patch Beautiful. r=me.
Sam Weinig
Comment 4 2012-08-13 18:15:08 PDT
Oh, but please fix the builds first.
Gyuyoung Kim
Comment 5 2012-08-13 18:18:26 PDT
Comment on attachment 158166 [details] proposed patch Attachment 158166 [details] did not pass efl-ews (efl): Output: http://queues.webkit.org/results/13489431
Build Bot
Comment 6 2012-08-13 20:34:07 PDT
Comment on attachment 158166 [details] proposed patch Attachment 158166 [details] did not pass win-ews (win): Output: http://queues.webkit.org/results/13490375
Alexey Proskuryakov
Comment 7 2012-08-14 13:31:07 PDT
Created attachment 158405 [details] For EWS Checking build fixes.
WebKit Review Bot
Comment 8 2012-08-14 13:33:12 PDT
Attachment 158405 [details] did not pass style-queue: Failed to run "['Tools/Scripts/check-webkit-style', '--diff-files', u'Source/WebKit2/ChangeLog', u'Source/WebKit..." exit_code: 1 Source/WebKit2/UIProcess/WebContext.cpp:484: An else statement can be removed when the prior "if" concludes with a return, break, continue or goto statement. [readability/control_flow] [4] Source/WebKit2/UIProcess/WebContext.cpp:495: An else statement can be removed when the prior "if" concludes with a return, break, continue or goto statement. [readability/control_flow] [4] Source/WebKit2/UIProcess/WebContext.cpp:528: One line control clauses should not use braces. [whitespace/braces] [4] Total errors found: 3 in 5 files If any of these errors are false positives, please file a bug against check-webkit-style.
Alexey Proskuryakov
Comment 9 2012-08-14 14:07:01 PDT
Note You need to log in before you can comment on or make changes to this bug.