Cleanup ios/web_view.

ios/web_view/shell/shell_app_delegate.mm

Index: ios/web_view/shell/shell_app_delegate.mm
diff --git a/ios/web_view/shell/shell_app_delegate.mm b/ios/web_view/shell/shell_app_delegate.mm
index ab0692df702a439e5f369a32ff382e5e1b8ce1b3..eefdc17c53d9d9e466beba731d1df8b50f1e1ab8 100644
--- a/ios/web_view/shell/shell_app_delegate.mm
+++ b/ios/web_view/shell/shell_app_delegate.mm
@@ -49,4 +49,9 @@
   [CRIWV shutDown];
 }
 
+- (void)dealloc {
+  [self.window release];

[sdefresne, 2017/01/25 09:11:09]

I think setting this to nil will work (since the generated setter dealloc the
old instance) and is more correct (as self.window will not point to a possibly
deallocated object).

  self.window = nil;

Or if you want to have the -release call be explicit, modify the underlying
ivar:

  [_window release];
  _window = nil;

[Eugene But (OOO till 7-30), 2017/01/25 14:28:04]

FWIW: |[self.window release];| or |self.window = nil;| is discouraged by the
style guide:
https://google.github.io/styleguide/objcguide.xml#Avoid_Accessors_During_init...

Calling |[_window release];| is the right suggestion. I don't think |_window =
nil;| adds anything and such constructs is unusual in dealloc.
|

[michaeldo, 2017/01/25 17:23:47]

Done.

+  [super dealloc];
+}
+
 @end