Make ui::TextInputClient::IsEditCommandEnabled const.

blimp/engine/feature/engine_render_widget_feature_unittest.cc

 // 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.
 
 #include "blimp/engine/feature/engine_render_widget_feature.h"
 
 #include <memory>
 
 #include "base/logging.h"
 #include "base/memory/ptr_util.h"
@@ skipping 86 matching lines @@
   void AddMouseEventCallback(const MouseEventCallback& callback) override {}
   void RemoveMouseEventCallback(const MouseEventCallback& callback) override {}
   void GetWebScreenInfo(blink::WebScreenInfo* result) override {}
   bool GetScreenColorProfile(std::vector<char>* color_profile) override {
     return false; }
   void HandleCompositorProto(const std::vector<uint8_t>& proto) override {}
 
   bool Send(IPC::Message* msg) override { return false; }
 };
 
 class MockTextInputClient : public ui::TextInputClient {

[tapted, 2016/06/06 07:17:21]

Can this inherit from DummyTextInputClient instead?

[karandeepb, 2016/06/08 03:01:55]

Will do this in a separate CL - https://codereview.chromium.org/2044593004/

   ui::TextInputType text_input_type_;
 
  public:
   MockTextInputClient() : text_input_type_(ui::TEXT_INPUT_TYPE_TEXT) {}
   ~MockTextInputClient() override {}
 
   void SetCompositionText(const ui::CompositionText& composition) override {}
   void ConfirmCompositionText() override {}
   void ClearCompositionText() override {}
   void InsertText(const base::string16& text) override {}
@@ skipping 19 matching lines @@
   bool GetSelectionRange(gfx::Range* range) const override { return false; }
   bool SetSelectionRange(const gfx::Range& range) override { return false; }
   bool DeleteRange(const gfx::Range& range) override { return false; }
   void OnInputMethodChanged() override {}
   bool ChangeTextDirectionAndLayoutAlignment(
       base::i18n::TextDirection direction) override {
     return false;
   }
   void ExtendSelectionAndDelete(size_t before, size_t after) override {}
   void EnsureCaretInRect(const gfx::Rect& rect) override {}
-  bool IsEditCommandEnabled(int command_id) override { return false; }
+  bool IsEditCommandEnabled(int command_id) const override { return false; }
   void SetEditCommandForNextKeyEvent(int command_id) override {}
   bool GetTextFromRange(const gfx::Range& range,
                         base::string16* text) const override {
     *text = base::string16(base::ASCIIToUTF16("green apple"));
     return false;
   }
 };
 
 MATCHER_P(CompMsgEquals, contents, "") {
   if (contents.size() != arg.size())
@@ skipping 215 matching lines @@
 
   feature_.OnRenderWidgetCreated(1, &render_widget_host1_);
   feature_.OnRenderWidgetCreated(2, &render_widget_host2_);
   feature_.OnRenderWidgetInitialized(1, &render_widget_host1_);
   feature_.OnRenderWidgetDeleted(2, &render_widget_host2_);
   feature_.SendCompositorMessage(1, &render_widget_host1_, payload);
 }
 
 }  // namespace engine
 }  // namespace blimp