Bug 91637 – Roll out r121610 and r122487 which may have been causing flaky crashes

• Status: RESOLVED
• Resolution: FIXED
• Component: New Bugs

[Joshua Bell]

Roll out r121610 and r122487 which may have been causing flaky crashes

[Joshua Bell]

I don't have strong evidence that r121610 is behind the flakes described in webkit.org/b/90469 but we should rule it out. Will upload a patch here once I've got a try run.

[Joshua Bell]

Created attachment 153047 [details]
Patch

[Kentaro Hara]

Comment on attachment 153047 [details]
Patch

OK. Try it.

[WebKit Review Bot]

Attachment 153047 [details] did not pass style-queue:

Failed to run "['Tools/Scripts/check-webkit-style', '--diff-files', u'LayoutTests/ChangeLog', u'LayoutTests/plug..." exit_code: 1
Source/WebCore/bindings/v8/NPObjectWrapper.cpp:151:  Place brace on its own line for function definitions.  [whitespace/braces] [4]
Source/WebCore/bindings/v8/NPObjectWrapper.cpp:176:  One line control clauses should not use braces.  [whitespace/braces] [4]
Source/WebCore/bindings/v8/NPObjectWrapper.cpp:174:  An else statement can be removed when the prior "if" concludes with a return, break, continue or goto statement.  [readability/control_flow] [4]
Source/WebCore/bindings/v8/NPObjectWrapper.cpp:178:  One line control clauses should not use braces.  [whitespace/braces] [4]
Source/WebCore/bindings/v8/ScriptController.cpp:48:  Alphabetical sorting problem.  [build/include_order] [4]
Source/WebCore/bindings/v8/NPObjectWrapper.h:46:  The parameter name "object" adds no information, so it should be removed.  [readability/parameter_name] [5]
Source/WebCore/bindings/v8/NPObjectWrapper.h:52:  The parameter name "obj" adds no information, so it should be removed.  [readability/parameter_name] [5]
Source/WebCore/bindings/v8/NPObjectWrapper.h:56:  The parameter name "obj" adds no information, so it should be removed.  [readability/parameter_name] [5]
Source/WebCore/bindings/v8/NPObjectWrapper.h:60:  The parameter name "obj" adds no information, so it should be removed.  [readability/parameter_name] [5]
Source/WebCore/bindings/v8/NPObjectWrapper.h:61:  The parameter name "obj" adds no information, so it should be removed.  [readability/parameter_name] [5]
Source/WebCore/bindings/v8/NPObjectWrapper.h:62:  The parameter name "obj" adds no information, so it should be removed.  [readability/parameter_name] [5]
Source/WebCore/bindings/v8/NPObjectWrapper.h:63:  The parameter name "obj" adds no information, so it should be removed.  [readability/parameter_name] [5]
Source/WebCore/bindings/v8/NPObjectWrapper.h:64:  The parameter name "obj" adds no information, so it should be removed.  [readability/parameter_name] [5]
Source/WebCore/bindings/v8/NPObjectWrapper.h:65:  The parameter name "obj" adds no information, so it should be removed.  [readability/parameter_name] [5]
Source/WebCore/bindings/v8/NPObjectWrapper.h:66:  The parameter name "obj" adds no information, so it should be removed.  [readability/parameter_name] [5]
Source/WebCore/bindings/v8/NPObjectWrapper.h:67:  The parameter name "obj" adds no information, so it should be removed.  [readability/parameter_name] [5]
Source/WebCore/bindings/v8/NPObjectWrapper.h:68:  The parameter name "obj" adds no information, so it should be removed.  [readability/parameter_name] [5]
Source/WebCore/bindings/v8/NPObjectWrapper.h:69:  The parameter name "obj" adds no information, so it should be removed.  [readability/parameter_name] [5]
Source/WebCore/bindings/v8/NPObjectWrapper.h:70:  The parameter name "obj" adds no information, so it should be removed.  [readability/parameter_name] [5]
Source/WebCore/bindings/v8/NPObjectWrapper.h:71:  The parameter name "npp" adds no information, so it should be removed.  [readability/parameter_name] [5]
Source/WebCore/bindings/v8/NPObjectWrapper.h:71:  The parameter name "obj" adds no information, so it should be removed.  [readability/parameter_name] [5]
Source/WebCore/bindings/v8/NPObjectWrapper.h:71:  Missing space after ,  [whitespace/comma] [3]
Source/WebCore/bindings/v8/NPObjectWrapper.h:74:  The parameter name "obj" adds no information, so it should be removed.  [readability/parameter_name] [5]
Source/WebCore/bindings/v8/NPObjectWrapper.h:78:  The parameter name "obj" adds no information, so it should be removed.  [readability/parameter_name] [5]
Total errors found: 24 in 12 files


If any of these errors are false positives, please file a bug against check-webkit-style.

[Kentaro Hara]

Maybe you need to land it manually to avoid style check errors.

[Joshua Bell]

Landed manually as r123110:

http://trac.webkit.org/changeset/123110

... and looking promising. 0 of the 17 runs have shown flaky crashes in the "s*" layout tests after the patch landing, compared to a roughly 95% showing a flaky crash prior to that revision. Will give it a day before updating TestExpectations, though.