Properly throw java exceptions from shouldOverrideUrlLoading

base/android/java_message_handler_factory.h

+// Copyright 2016 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.
+
+#ifndef BASE_ANDROID_JAVA_MESSAGE_HANDLER_FACTORY_H_
+#define BASE_ANDROID_JAVA_MESSAGE_HANDLER_FACTORY_H_
+
+#include "base/android/scoped_java_ref.h"
+#include "base/message_loop/message_pump.h"
+
+namespace base {
+
+class MessagePumpForUI;
+
+namespace android {
+
+/**
+ * Factory for creating the Java-side system message handler - only used for

[nyquist, 2016/08/18 15:50:02]

Nit: Should be C++-style comment

[gsennton, 2016/08/18 18:56:54]

Done.

+ * testing.
+ */
+class JavaMessageHandlerFactory {
+ public:
+  virtual ~JavaMessageHandlerFactory() {}
+  virtual base::android::ScopedJavaLocalRef<jobject> CreateMessageHandler(
+      JNIEnv* env,
+      base::MessagePump::Delegate* delegate,
+      MessagePumpForUI* message_pump,
+      WaitableEvent* test_done_event) = 0;
+};
+
+}  // namespace android
+}  // namespace base
+
+#endif  // BASE_ANDROID_JAVA_MESSAGE_HANDLER_FACTORY_H_