chrome/test/data/webui/print_preview/print_preview.h - Issue 2893003003: Print Preview: Merge NativeLayerStubs for tests

Side by Side Diff

Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.

Keyboard Shortcuts

	File
u :	up to issue
j / k :	jump to file after / before current file
J / K :	jump to next file with a comment after / before current file
	Side-by-side diff
i :	toggle intra-line diffs
e :	expand all comments
c :	collapse all comments
s :	toggle showing all comments
n / p :	next / previous diff chunk or comment
N / P :	next / previous comment
<Up> / <Down> :	next / previous line

	Issue
u :	up to list of issues
j / k :	jump to patch after / before current patch
o / <Enter> :	open current patch in side-by-side view
i :	open current patch in unified diff view

	Issue List
j / k :	jump to issue after / before current issue
o / <Enter> :	open current issue

Side by Side Diff: chrome/test/data/webui/print_preview/print_preview.h

Issue 2893003003: Print Preview: Merge NativeLayerStubs for tests (Closed)

Patch Set: Fix closure error Created 3 years, 7 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

OLD	NEW
1 // Copyright (c) 2011 The Chromium Authors. All rights reserved.	1 // Copyright (c) 2011 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 #ifndef CHROME_TEST_DATA_WEBUI_PRINT_PREVIEW_H_	5 #ifndef CHROME_TEST_DATA_WEBUI_PRINT_PREVIEW_PRINT_PREVIEW_H_
	dpapad 2017/05/19 20:59:29 Is this file (and the corresponding cc file) neces Is this file (and the corresponding cc file) necessary at all? Is it used somewhere? AFAIU chrome/test/data/webui/print_preview/print_preview.js automatically generates a corresponding C++ file at out/<out_foder>/gen/chrome/test/data/webui/print_preview-gen.cc rbpotter 2017/05/20 00:43:59 Done in https://codereview.chromium.org/2894243004 Show quoted text On 2017/05/19 20:59:29, dpapad wrote: > Is this file (and the corresponding cc file) necessary at all? Is it used > somewhere? > > AFAIU chrome/test/data/webui/print_preview/print_preview.js automatically > generates a corresponding C++ file at > out/<out_foder>/gen/chrome/test/data/webui/print_preview-gen.cc Done in https://codereview.chromium.org/2894243004/
6 #define CHROME_TEST_DATA_WEBUI_PRINT_PREVIEW_H_	6 #define CHROME_TEST_DATA_WEBUI_PRINT_PREVIEW_PRINT_PREVIEW_H_

7	7

8 #include "base/macros.h"	8 #include "base/macros.h"

9 #include "chrome/test/base/web_ui_browser_test.h"	9 #include "chrome/test/base/web_ui_browser_test.h"

10	10

11 class PrintPreviewWebUITest : public WebUIBrowserTest {	11 class PrintPreviewWebUITest : public WebUIBrowserTest {

12 public:	12 public:

13 PrintPreviewWebUITest();	13 PrintPreviewWebUITest();

14 ~PrintPreviewWebUITest() override;	14 ~PrintPreviewWebUITest() override;

15	15

16 private:	16 private:

17 DISALLOW_COPY_AND_ASSIGN(PrintPreviewWebUITest);	17 DISALLOW_COPY_AND_ASSIGN(PrintPreviewWebUITest);

18 };	18 };

19	19

20 #endif // CHROME_TEST_DATA_WEBUI_PRINT_PREVIEW_H_	20 #endif // CHROME_TEST_DATA_WEBUI_PRINT_PREVIEW_PRINT_PREVIEW_H_

OLD	NEW