Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(195)

Side by Side Diff: content/shell/common/layout_test.mojom

Issue 2955123002: Mojo-ify ShellViewHostMsg_LayoutDumpResponse. (Closed)
Patch Set: Created 3 years, 5 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
OLDNEW
1 // Copyright 2017 The Chromium Authors. All rights reserved. 1 // Copyright 2017 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 module content.mojom; 5 module content.mojom;
6 6
7 import "mojo/common/file_path.mojom"; 7 import "mojo/common/file_path.mojom";
8 import "ui/gfx/geometry/mojo/geometry.mojom"; 8 import "ui/gfx/geometry/mojo/geometry.mojom";
9 import "url/mojo/url.mojom"; 9 import "url/mojo/url.mojom";
10 10
(...skipping 18 matching lines...) Expand all
29 29
30 // The expected MD5 hash of the pixel results. 30 // The expected MD5 hash of the pixel results.
31 string expected_pixel_hash; 31 string expected_pixel_hash;
32 32
33 // The initial size of the test window. 33 // The initial size of the test window.
34 gfx.mojom.Size initial_size; 34 gfx.mojom.Size initial_size;
35 }; 35 };
36 36
37 interface LayoutTestControl { 37 interface LayoutTestControl {
38 // Asks a frame to dump its contents into a string and send them back over 38 // Asks a frame to dump its contents into a string and send them back over
39 // IPC. 39 // IPC.
alexmos 2017/06/27 21:35:48 nit: s/them/it/ and perhaps remove "over IPC"
Łukasz Anforowicz 2017/06/27 22:17:41 I've changed the entire comment.
40 LayoutDumpRequest(); 40 DumpFrameLayout() => (string frame_layout_dump);
41 41
42 // Replicates test config (for an already started test) to a new renderer 42 // Replicates test config (for an already started test) to a new renderer
43 // that hosts parts of the main test window. 43 // that hosts parts of the main test window.
44 ReplicateTestConfiguration(ShellTestConfiguration config); 44 ReplicateTestConfiguration(ShellTestConfiguration config);
45 45
46 // Sets the test config for a layout test that is being started. This message 46 // Sets the test config for a layout test that is being started. This message
47 // is sent only to a renderer that hosts parts of the main test window. 47 // is sent only to a renderer that hosts parts of the main test window.
48 SetTestConfiguration(ShellTestConfiguration config); 48 SetTestConfiguration(ShellTestConfiguration config);
49 49
50 // Sets up a secondary renderer (renderer that doesn't [yet] host parts of the 50 // Sets up a secondary renderer (renderer that doesn't [yet] host parts of the
51 // main test window) for a layout test. 51 // main test window) for a layout test.
52 SetupSecondaryRenderer(); 52 SetupSecondaryRenderer();
53 }; 53 };
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698