Chromium Code Reviews| Index: ios/web/web_state/ui/wk_web_view_configuration_provider_unittest.mm |
| diff --git a/ios/web/web_state/ui/wk_web_view_configuration_provider_unittest.mm b/ios/web/web_state/ui/wk_web_view_configuration_provider_unittest.mm |
| index db1cd90a464c81ae9bc7b483e462c876e11361d9..aec2fa0eaca342481379df5074fa1c1764ea9049 100644 |
| --- a/ios/web/web_state/ui/wk_web_view_configuration_provider_unittest.mm |
| +++ b/ios/web/web_state/ui/wk_web_view_configuration_provider_unittest.mm |
| @@ -68,6 +68,9 @@ TEST_F(WKWebViewConfigurationProviderTest, ConfigurationOwnerhip) { |
| other_provider.GetWebViewConfiguration().processPool); |
| } |
| +#if 0 |
|
rohitrao (ping after 24h)
2015/06/10 16:18:22
We can check the current SDK version instead of #i
justincohen
2015/06/10 16:33:36
shreyasv is OK with fixing it shortly. It's a sma
|
| +// TODO(shreyasv) Disabled for usaging of nil on latest Xcode beta. |
| +// crbug.com/498841 |
| // Tests that internal configuration object can not be changed by clients. |
| TEST_F(WKWebViewConfigurationProviderTest, ConfigurationProtection) { |
| CR_TEST_REQUIRES_WK_WEB_VIEW(); |
| @@ -93,6 +96,7 @@ TEST_F(WKWebViewConfigurationProviderTest, ConfigurationProtection) { |
| EXPECT_EQ(userContentController.get(), |
| provider.GetWebViewConfiguration().userContentController); |
| } |
| +#endif |
| // Tests that |HasWebViewConfiguration| returns false by default. |
| TEST_F(WKWebViewConfigurationProviderTest, NoConfigurationByDefault) { |