ios/web/webui/web_ui_mojo_inttest.mm - Issue 2916473002: [ObjC ARC] Converts ios/web:web to ARC.

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

Unified Diff: ios/web/webui/web_ui_mojo_inttest.mm

Issue 2916473002: [ObjC ARC] Converts ios/web:web to ARC. (Closed)

Patch Set: Tweaks to autorelease pool Created 3 years, 6 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:

Index: ios/web/webui/web_ui_mojo_inttest.mm

diff --git a/ios/web/webui/web_ui_mojo_inttest.mm b/ios/web/webui/web_ui_mojo_inttest.mm

index 97916af543d08a09094c16c7d477a5742fa9ed32..9725fb511a122dc13f4d51dd9e5f22cb3d44b195 100644

--- a/ios/web/webui/web_ui_mojo_inttest.mm

+++ b/ios/web/webui/web_ui_mojo_inttest.mm

@@ -146,6 +146,10 @@ class WebUIMojoTest : public WebIntTest {

new TestWebUIControllerFactory(ui_handler_.get()));

}

+ void TearDown() override {

+ web_state_.reset();

+ WebIntTest::TearDown();

+ }

Eugene But (OOO till 7-30) 2017/06/12 06:12:52 nit: Keep a linebreak after |TearDown|?

PL 2017/06/14 00:31:11 Oh yes, thanks!

// Returns WebState which loads test WebUI page.

WebStateImpl* web_state() { return web_state_.get(); }

// Returns UI handler which communicates with WebUI page.