Add support for ProxyResolverErrorObserver to ProxyResolverMojo.

net/proxy/proxy_resolver_error_observer_mojo.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 "net/proxy/proxy_resolver_error_observer_mojo.h"
+
+#include "base/bind.h"
+#include "base/location.h"
+#include "base/message_loop/message_loop_proxy.h"
+#include "base/single_thread_task_runner.h"
+#include "base/threading/thread_checker.h"
+#include "mojo/common/common_type_converters.h"
+
+namespace net {
+
+class ProxyResolverErrorObserverMojo::Core

[Anand Mistry (off Chromium), 2015/03/20 03:12:32]

I think you can simplify this code and eliminate this class by using a WeakPtr<>
to post to the task runner in OnPACScriptError().

[Sam McNally, 2015/03/20 03:52:41]

Done.

+    : public base::RefCountedThreadSafe<ProxyResolverErrorObserverMojo::Core> {
+ public:
+  explicit Core(interfaces::ProxyResolverErrorObserverPtr error_observer);
+  void SendOnPacScriptError(int line_number, const base::string16& error);
+
+ private:
+  friend class base::RefCountedThreadSafe<Core>;
+
+  ~Core();
+
+  interfaces::ProxyResolverErrorObserverPtr error_observer_;
+  scoped_refptr<base::SingleThreadTaskRunner> task_runner_;
+  base::ThreadChecker thread_checker_;
+};
+
+// static
+scoped_ptr<ProxyResolverErrorObserver> ProxyResolverErrorObserverMojo::Create(
+    interfaces::ProxyResolverErrorObserverPtr error_observer) {
+  if (!error_observer)
+    return nullptr;
+
+  return scoped_ptr<ProxyResolverErrorObserver>(
+      new ProxyResolverErrorObserverMojo(error_observer.Pass()));
+}
+
+void ProxyResolverErrorObserverMojo::OnPACScriptError(
+    int line_number,
+    const base::string16& error) {
+  core_->SendOnPacScriptError(line_number, error);
+}
+
+ProxyResolverErrorObserverMojo::ProxyResolverErrorObserverMojo(
+    interfaces::ProxyResolverErrorObserverPtr error_observer)
+    : core_(new Core(error_observer.Pass())) {
+}
+
+ProxyResolverErrorObserverMojo::~ProxyResolverErrorObserverMojo() = default;
+
+ProxyResolverErrorObserverMojo::Core::Core(
+    interfaces::ProxyResolverErrorObserverPtr error_observer)
+    : error_observer_(error_observer.Pass()),
+      task_runner_(base::MessageLoopProxy::current()) {
+}
+
+void ProxyResolverErrorObserverMojo::Core::SendOnPacScriptError(
+    int line_number,
+    const base::string16& error) {
+  if (!task_runner_->RunsTasksOnCurrentThread()) {
+    task_runner_->PostTask(
+        FROM_HERE,
+        base::Bind(&ProxyResolverErrorObserverMojo::Core::SendOnPacScriptError,
+                   this, line_number, error));
+    return;
+  }
+  DCHECK(thread_checker_.CalledOnValidThread());
+  error_observer_->OnPacScriptError(line_number, mojo::String::From(error));
+}
+
+ProxyResolverErrorObserverMojo::Core::~Core() = default;
+
+}  // namespace net