Add CRWNativeContent willBeDismissed.

ios/web/web_state/ui/crw_web_controller_container_view.mm

 // Copyright 2015 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #import "ios/web/web_state/ui/crw_web_controller_container_view.h"
 
 #include "base/ios/weak_nsobject.h"
 #include "base/logging.h"
 #include "base/mac/scoped_nsobject.h"
 #import "ios/web/public/web_state/ui/crw_content_view.h"
@@ skipping 152 matching lines @@
     [_webViewContentView setFrame:self.bounds];
     [self addSubview:_webViewContentView];
   }
 }
 
 - (id<CRWNativeContent>)nativeController {
   return _nativeController.get();
 }
 
 - (void)setNativeController:(id<CRWNativeContent>)nativeController {
+  if ([_nativeController respondsToSelector:@selector(willBeDismissed)]) {

[Eugene But (OOO till 7-30), 2016/12/21 15:26:35]

Should this be moved inside |(![_nativeController isEqual:nativeController])|
check? Otherwise |willBeDismissed| may be called even for cases when
nativeController is not changed, which does not sound right to me.

[Olivier, 2016/12/21 15:41:11]

The logic is to call in any navigation.
If the nativeContentProvider gives the same nativeController for two
navigations, it should be called twice. That is why I put it here.
I don't think it change a lot for OfflinePageNativeContent as BVC creates a new
one each time, so I can put it inside.

[Olivier, 2016/12/21 15:51:36]

Done.

+    [_nativeController willBeDismissed];
+  }
   if (![_nativeController isEqual:nativeController]) {
     base::WeakNSProtocol<id> oldController(_nativeController);
     [[oldController view] removeFromSuperview];
     _nativeController.reset([nativeController retain]);
     // TODO(crbug.com/503297): Re-enable this DCHECK once native controller
     // leaks are fixed.
     //    DCHECK(!oldController);
   }
 }
 
@@ skipping 152 matching lines @@
   else
     self.toolbarContainerView = nil;
 }
 
 - (void)removeAllToolbars {
   // Resetting the property will remove the toolbars from the hierarchy.
   self.toolbarContainerView = nil;
 }
 
 @end