[Extensions] Isolate ExtensionFunction results_ and error_

chrome/browser/extensions/api/input_ime/input_ime_api.cc

 // Copyright (c) 2012 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 "chrome/browser/extensions/api/input_ime/input_ime_api.h"
 
 #include "base/lazy_instance.h"
 #include "chrome/browser/chrome_notification_types.h"
 #include "chrome/common/extensions/api/input_ime.h"
 #include "content/public/browser/notification_registrar.h"
@@ skipping 300 matching lines @@
           segment_info.style =
               InputMethodEngineBase::SEGMENT_STYLE_NO_UNDERLINE;
         }
         segments.push_back(segment_info);
       }
     }
     int selection_start =
         params.selection_start ? *params.selection_start : params.cursor;
     int selection_end =
         params.selection_end ? *params.selection_end : params.cursor;
+    std::string error;
     success = engine->SetComposition(params.context_id, params.text.c_str(),
                                      selection_start, selection_end,
-                                     params.cursor, segments, &error_);
+                                     params.cursor, segments, &error);
+    SetError(error);
   }
   std::unique_ptr<base::ListValue> output =
       SetComposition::Results::Create(success);
   return RespondNow(ArgumentList(std::move(output)));
 }
 
 ExtensionFunction::ResponseAction InputImeCommitTextFunction::Run() {
   bool success = false;
   InputImeEventRouter* event_router =
       GetInputImeEventRouter(Profile::FromBrowserContext(browser_context()));
   InputMethodEngineBase* engine =
       event_router ? event_router->GetActiveEngine(extension_id()) : nullptr;
   if (engine) {
     std::unique_ptr<CommitText::Params> parent_params(
         CommitText::Params::Create(*args_));
     const CommitText::Params::Parameters& params = parent_params->parameters;
+    std::string error;
     success =
-        engine->CommitText(params.context_id, params.text.c_str(), &error_);
+        engine->CommitText(params.context_id, params.text.c_str(), &error);
+    SetError(error);
   }
   std::unique_ptr<base::ListValue> output =
       CommitText::Results::Create(success);
   return RespondNow(ArgumentList(std::move(output)));
 }
 
 ExtensionFunction::ResponseAction InputImeSendKeyEventsFunction::Run() {
   InputImeEventRouter* event_router =
       GetInputImeEventRouter(Profile::FromBrowserContext(browser_context()));
   InputMethodEngineBase* engine =
@@ skipping 58 matching lines @@
 InputImeEventRouter* GetInputImeEventRouter(Profile* profile) {
   if (!profile)
     return nullptr;
   if (profile->HasOffTheRecordProfile())
     profile = profile->GetOffTheRecordProfile();
   return extensions::InputImeEventRouterFactory::GetInstance()->GetRouter(
       profile);
 }
 
 }  // namespace extensions