Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(28)

Unified Diff: mojo/public/java/bindings/src/org/chromium/mojo/bindings/AutoCloseableRouter.java

Issue 2250183003: Make the fuchsia mojo/public repo the source of truth. (Closed) Base URL: https://github.com/domokit/mojo.git@master
Patch Set: Created 4 years, 4 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: mojo/public/java/bindings/src/org/chromium/mojo/bindings/AutoCloseableRouter.java
diff --git a/mojo/public/java/bindings/src/org/chromium/mojo/bindings/AutoCloseableRouter.java b/mojo/public/java/bindings/src/org/chromium/mojo/bindings/AutoCloseableRouter.java
deleted file mode 100644
index 8ccd4e6347240a4c8edee52ee3729661bee9823b..0000000000000000000000000000000000000000
--- a/mojo/public/java/bindings/src/org/chromium/mojo/bindings/AutoCloseableRouter.java
+++ /dev/null
@@ -1,118 +0,0 @@
-// Copyright 2014 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.
-
-package org.chromium.mojo.bindings;
-
-import org.chromium.mojo.system.Core;
-import org.chromium.mojo.system.MessagePipeHandle;
-
-import java.util.concurrent.Executor;
-
-/**
- * Wrapper around {@link Router} that will close the connection when not referenced anymore.
- */
-class AutoCloseableRouter implements Router {
-
- /**
- * The underlying router.
- */
- private final Router mRouter;
-
- /**
- * The executor to close the underlying router.
- */
- private final Executor mExecutor;
-
- /**
- * Flags to keep track if this router owns an unclosed handle.
- */
- private boolean mOwnsHandle;
-
- /**
- * Constructor.
- */
- public AutoCloseableRouter(Core core, Router router) {
- mRouter = router;
- mExecutor = ExecutorFactory.getExecutorForCurrentThread(core);
- mOwnsHandle = true;
- }
-
- /**
- * @see Router#setIncomingMessageReceiver(MessageReceiverWithResponder)
- */
- @Override
- public void setIncomingMessageReceiver(MessageReceiverWithResponder incomingMessageReceiver) {
- mRouter.setIncomingMessageReceiver(incomingMessageReceiver);
- }
-
- /**
- * @see HandleOwner#passHandle()
- */
- @Override
- public MessagePipeHandle passHandle() {
- mOwnsHandle = false;
- return mRouter.passHandle();
- }
-
- /**
- * @see MessageReceiver#accept(Message)
- */
- @Override
- public boolean accept(Message message) {
- return mRouter.accept(message);
- }
-
- /**
- * @see MessageReceiverWithResponder#acceptWithResponder(Message, MessageReceiver)
- */
- @Override
- public boolean acceptWithResponder(Message message, MessageReceiver responder) {
- return mRouter.acceptWithResponder(message, responder);
-
- }
-
- /**
- * @see Router#start()
- */
- @Override
- public void start() {
- mRouter.start();
- }
-
- /**
- * @see Router#setErrorHandler(ConnectionErrorHandler)
- */
- @Override
- public void setErrorHandler(ConnectionErrorHandler errorHandler) {
- mRouter.setErrorHandler(errorHandler);
- }
-
- /**
- * @see java.io.Closeable#close()
- */
- @Override
- public void close() {
- mRouter.close();
- mOwnsHandle = false;
- }
-
- /**
- * @see Object#finalize()
- */
- @Override
- protected void finalize() throws Throwable {
- if (mOwnsHandle) {
- mExecutor.execute(new Runnable() {
-
- @Override
- public void run() {
- close();
- }
- });
- throw new IllegalStateException("Warning: Router objects should be explicitly closed " +
- "when no longer required otherwise you may leak handles.");
- }
- super.finalize();
- }
-}

Powered by Google App Engine
This is Rietveld 408576698