[ios clean] ToolsMenu Mediator observes Webstate.

ios/clean/chrome/browser/ui/tools/tools_mediator.h

 // Copyright 2017 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.
 
 #ifndef IOS_CLEAN_CHROME_BROWSER_UI_TOOLS_TOOLS_MEDIATOR_H_
 #define IOS_CLEAN_CHROME_BROWSER_UI_TOOLS_TOOLS_MEDIATOR_H_
 
 #import <Foundation/Foundation.h>
 
 @protocol ToolsConsumer;
 @class ToolsMenuConfiguration;
+namespace web {
+class WebState;
+}
 
 // A mediator object that sets a ToolsMenuVC appeareance based on various data
 // sources.
 @interface ToolsMediator : NSObject
-- (instancetype)initWithConsumer:(id<ToolsConsumer>)consumer
-                andConfiguration:(ToolsMenuConfiguration*)menuConfiguration
+- (nullable instancetype)initWithConsumer:(nonnull id<ToolsConsumer>)consumer
+                         andConfiguration:

[marq (ping after 24h), 2017/05/30 09:24:54]

Naming nit: Don't use 'and' to connect keywords in method names. 'and' is only
used to indicate a second action the method performs
("-doSomethingWithThis:andDoSomethingElseWithThis:"). 

Generally keywords after the first in initializers shouldn't need prepositions
or conjunctions, so 'initWithConsumer:configuration:' should be sufficient.

[sczs, 2017/05/31 03:05:48]

Thanks for the tip! Done.

+                             (nonnull ToolsMenuConfiguration*)menuConfiguration
     NS_DESIGNATED_INITIALIZER;
-- (instancetype)init NS_UNAVAILABLE;
+- (nullable instancetype)init NS_UNAVAILABLE;
+
+// The WebState whose properties this object mediates. This can change during
+// the lifetime of this object and may be null.
+@property(nonatomic, assign, nullable) web::WebState* webState;

[sczs, 2017/05/31 03:05:47]

Thought it would be a good idea to start using Nullability annotations after the
WSL discussion. I know we still haven't decided if we are going to use them, or
which style to follow. So I can remove these if you have any objection or
concern. 

[marq (ping after 24h), 2017/05/31 10:40:31]

We have them in other places. As a matter of general ObjC style, it's under
heavy discussion; I myself don't have a clear idea about what I'd like to see.

[sczs, 2017/05/31 23:19:26]

Acknowledged.

 @end
 
 #endif  // IOS_CLEAN_CHROME_BROWSER_UI_TOOLS_TOOLS_MEDIATOR_H_