Bug 179419

Summary: REGRESSION(r224566): [GTK][WPE] Many inspector tests are failing after r224566
Product: WebKit Reporter: Carlos Garcia Campos <cgarcia>
Component: WebKitGTKAssignee: Nobody <webkit-unassigned>
Status: RESOLVED FIXED    
Severity: Normal CC: aboya, bburg, bugs-noreply, clopez, commit-queue, joepeck
Priority: P2 Keywords: Gtk, LayoutTestFailure, Regression
Version: WebKit Nightly Build   
Hardware: Unspecified   
OS: Unspecified   
See Also: https://bugs.webkit.org/show_bug.cgi?id=161212
https://bugs.webkit.org/show_bug.cgi?id=179471
Attachments:
Description Flags
Patch none

Carlos Garcia Campos
Reported 2017-11-08 01:39:35 PST
All of them fail with the following error: Uncaught exception in Inspector page: ReferenceError: Can't find variable: CodeMirror [TestCombined.js:36658:15] This is because Test.html, even when combined, is trying to load CodeMirror from External/CodeMirror, but CodeMirror was already combined for Main.html. We need to also combine CodeMirror for Test.html. I don't know why this works in Mac.
Attachments
Patch (3.42 KB, patch)
2017-11-08 01:42 PST, Carlos Garcia Campos 		no flags
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Carlos Garcia Campos
Comment 1 2017-11-08 01:42:00 PST
Created attachment 326316 [details] Patch
Carlos Garcia Campos
Comment 2 2017-11-08 03:43:59 PST
Joe, Brian I guess this is ok for Mac too, at least EWS didn't complain.
Carlos Garcia Campos
Comment 3 2017-11-08 23:48:17 PST
*** Bug 179430 has been marked as a duplicate of this bug. ***
Carlos Garcia Campos
Comment 4 2017-11-08 23:51:27 PST
Comment on attachment 326316 [details] Patch Ok, I'm landing this because it broke many tests. Joe, Brian if you have any objection or comment I'll fix it in a follow up.
WebKit Commit Bot
Comment 5 2017-11-09 00:12:07 PST
Comment on attachment 326316 [details] Patch Clearing flags on attachment: 326316 Committed r224621: <https://trac.webkit.org/changeset/224621>
WebKit Commit Bot
Comment 6 2017-11-09 00:12:09 PST
All reviewed patches have been landed. Closing bug.
Note You need to log in before you can comment on or make changes to this bug.