Fix content browser test failure by enabling WebRuntimeFeatures::enableInputModeAttribute

content/renderer/render_view_browsertest.cc

Index: content/renderer/render_view_browsertest.cc
diff --git a/content/renderer/render_view_browsertest.cc b/content/renderer/render_view_browsertest.cc
index 7c1715e9e80eb603b309bdd4a38bf577793c37f9..c525517dcd8cfeccdae4e824024e878b0aa02129 100644
--- a/content/renderer/render_view_browsertest.cc
+++ b/content/renderer/render_view_browsertest.cc
@@ -34,6 +34,7 @@
 #include "third_party/WebKit/public/web/WebDataSource.h"
 #include "third_party/WebKit/public/web/WebFrame.h"
 #include "third_party/WebKit/public/web/WebHistoryItem.h"
+#include "third_party/WebKit/public/web/WebRuntimeFeatures.h"
 #include "third_party/WebKit/public/web/WebView.h"
 #include "third_party/WebKit/public/web/WebWindowFeatures.h"
 #include "ui/base/keycodes/keyboard_codes.h"
@@ -119,6 +120,17 @@ class RenderViewImplTest : public RenderViewTest {
     mock_keyboard_.reset(new MockKeyboard());
   }
 
+  virtual ~RenderViewImplTest() {}
+
+  virtual void SetUp() OVERRIDE {
+    RenderViewTest::SetUp();
+    // This test depends on Blink flag InputModeAttribute, which is enabled
+    // under only test. Content browser test doesn't enable the feature so we
+    // need enable it manually.
+    // TODO(yoichio): Remove this if InputMode feature is enabled by default.
+    WebKit::WebRuntimeFeatures::enableInputModeAttribute(true);

[jochen (gone - plz use gerrit), 2013/08/22 06:28:45]

nit. add using WebKit::WebRuntimeFeatures to the list starting at line 59

[yoichio, 2013/08/22 06:48:37]

Done.

+  }
+
   RenderViewImpl* view() {
     return static_cast<RenderViewImpl*>(view_);
   }
@@ -771,10 +783,7 @@ TEST_F(RenderViewImplTest, DontIgnoreBackAfterNavEntryLimit) {
 
 // Test that our IME backend sends a notification message when the input focus
 // changes.
-// crbug.com/276821:
-// Because Blink change cause this test failed, we first disabled this test and
-// fix later.
-TEST_F(RenderViewImplTest, DISABLED_OnImeTypeChanged) {
+TEST_F(RenderViewImplTest, OnImeTypeChanged) {
   // Enable our IME backend code.
   view()->OnSetInputMethodActive(true);