Bug 196195
| Summary: | [GTK] Regression in inspector/console/console-screenshot.html | ||
|---|---|---|---|
| Product: | WebKit | Reporter: | Diego Pino <dpino> |
| Component: | New Bugs | Assignee: | Nobody <webkit-unassigned> |
| Status: | RESOLVED FIXED | ||
| Severity: | Normal | CC: | magomez |
| Priority: | P2 | ||
| Version: | WebKit Nightly Build | ||
| Hardware: | Unspecified | ||
| OS: | Unspecified | ||
Diego Pino
Test is failing since introduction of this feature in r242992 (Web Inspector: provide a way to capture a screenshot of a node from within the page https://bugs.webkit.org/show_bug.cgi?id=194279)
Diff:
--- /WebKit/WebKitBuild/Release/layout-test-results/inspector/console/console-screenshot-expected.txt
+++ /WebKit/WebKitBuild/Release/layout-test-results/inspector/console/console-screenshot-actual.txt
@@ -7,11 +7,15 @@
== Running test suite: console.screenshot
-- Running test case: console.screenshot.SingleNode
PASS: The added message should be an image.
-PASS: The image should be a 2x2 red square.
+FAIL: The image should be a 2x2 red square.
+ Expected: "data:image/png;base64,iVBORw0KGgoAAAANSUhEUgAAAAIAAAACCAYAAABytg0kAAAAAXNSR0IArs4c6QAAABNJREFUCB1j/M/AAEQMDEwgAgQAHxcCAmtAm/sAAAAASUVORK5CYII="
+ Actual: "data:image/png;base64,iVBORw0KGgoAAAANSUhEUgAAAAIAAAACCAYAAABytg0kAAAABHNCSVQICAgIfAhkiAAAABRJREFUCJlj/M/A8J+BgYGBiQEKAB8XAgK+3Bg8AAAAAElFTkSuQmCC"
-- Running test case: console.screenshot.MultipleNodes
PASS: The added message should be an image.
-PASS: The image should be a 2x2 blue square.
+FAIL: The image should be a 2x2 blue square.
+ Expected: "data:image/png;base64,iVBORw0KGgoAAAANSUhEUgAAAAIAAAACCAYAAABytg0kAAAAAXNSR0IArs4c6QAAABRJREFUCB1jZGD4/58BCJhABAgAAB0ZAgJSPDJ6AAAAAElFTkSuQmCC"
+ Actual: "data:image/png;base64,iVBORw0KGgoAAAANSUhEUgAAAAIAAAACCAYAAABytg0kAAAABHNCSVQICAgIfAhkiAAAABRJREFUCJljZGD4/5+BgYGBiQEKAB0ZAgLFFMilAAAAAElFTkSuQmCC"
It seems there are differences in the expected and actual Base64 encodings.
| Attachments | ||
|---|---|---|
| Add attachment proposed patch, testcase, etc. |
Miguel Gomez
This is passing since r244868. Closing bug and updating expectations.