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

Unified Diff: plugins/org.chromium.sdk.wipbackend.wk118685/src/org/chromium/sdk/internal/wip/WipScriptImpl.java

Issue 11829027: drop old backends (Closed) Base URL: https://chromedevtools.googlecode.com/svn/trunk
Patch Set: Created 7 years, 11 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: plugins/org.chromium.sdk.wipbackend.wk118685/src/org/chromium/sdk/internal/wip/WipScriptImpl.java
diff --git a/plugins/org.chromium.sdk.wipbackend.wk118685/src/org/chromium/sdk/internal/wip/WipScriptImpl.java b/plugins/org.chromium.sdk.wipbackend.wk118685/src/org/chromium/sdk/internal/wip/WipScriptImpl.java
deleted file mode 100644
index c46d11ebfe1f7e08f400460164f3a1567dbe1a68..0000000000000000000000000000000000000000
--- a/plugins/org.chromium.sdk.wipbackend.wk118685/src/org/chromium/sdk/internal/wip/WipScriptImpl.java
+++ /dev/null
@@ -1,115 +0,0 @@
-// Copyright (c) 2011 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.sdk.internal.wip;
-
-import java.util.List;
-
-import org.chromium.sdk.RelayOk;
-import org.chromium.sdk.Script;
-import org.chromium.sdk.SyncCallback;
-import org.chromium.sdk.internal.ScriptBase;
-import org.chromium.sdk.internal.liveeditprotocol.LiveEditProtocolParserAccess;
-import org.chromium.sdk.internal.liveeditprotocol.LiveEditResult;
-import org.chromium.sdk.internal.protocolparser.JsonProtocolParseException;
-import org.chromium.sdk.internal.wip.protocol.input.debugger.CallFrameValue;
-import org.chromium.sdk.internal.wip.protocol.input.debugger.SetScriptSourceData;
-import org.chromium.sdk.internal.wip.protocol.output.debugger.SetScriptSourceParams;
-import org.chromium.sdk.util.GenericCallback;
-import org.chromium.sdk.util.RelaySyncCallback;
-
-/**
- * Wip implementation of {@link Script}.
- */
-class WipScriptImpl extends ScriptBase<String> {
- private final WipScriptManager scriptManager;
-
- WipScriptImpl(WipScriptManager scriptManager, Descriptor<String> descriptor) {
- super(descriptor);
- this.scriptManager = scriptManager;
- }
-
- @Override
- public RelayOk setSourceOnRemote(String newSource, UpdateCallback callback,
- SyncCallback syncCallback) {
- return sendLiveEditRequest(newSource, false, callback, syncCallback);
- }
-
- @Override
- public RelayOk previewSetSource(String newSource, UpdateCallback callback,
- SyncCallback syncCallback) {
- return sendLiveEditRequest(newSource, true, callback, syncCallback);
- }
-
- private RelayOk sendLiveEditRequest(String newSource, final boolean preview,
- final UpdateCallback updateCallback,
- final SyncCallback syncCallback) {
-
- RelaySyncCallback relay = new RelaySyncCallback(syncCallback);
- final RelaySyncCallback.Guard guard = relay.newGuard();
-
- SetScriptSourceParams params = new SetScriptSourceParams(getId(), newSource, preview);
-
- GenericCallback<SetScriptSourceData> commandCallback =
- new GenericCallback<SetScriptSourceData>() {
- @Override
- public void success(SetScriptSourceData value) {
- RelayOk relayOk =
- possiblyUpdateCallFrames(preview, value, updateCallback, guard.getRelay());
- guard.discharge(relayOk);
- }
-
- @Override
- public void failure(Exception exception) {
- updateCallback.failure(exception.getMessage());
- }
- };
-
- WipCommandProcessor commandProcessor = scriptManager.getTabImpl().getCommandProcessor();
- return commandProcessor.send(params, commandCallback, guard.asSyncCallback());
- }
-
- private RelayOk possiblyUpdateCallFrames(boolean preview, final SetScriptSourceData data,
- final UpdateCallback updateCallback, RelaySyncCallback relay) {
-
- // TODO: support 'step-in recommended'.
-
- List<CallFrameValue> callFrames = null;
- if (!preview) {
- callFrames = data.callFrames();
- }
- if (callFrames == null) {
- dispatchResult(data.result(), updateCallback);
- return relay.finish();
- } else {
- GenericCallback<Void> setFramesCallback =
- new GenericCallback<Void>() {
- @Override public void success(Void value) {
- dispatchResult(data.result(), updateCallback);
- }
- @Override public void failure(Exception exception) {
- throw new RuntimeException(exception);
- }
- };
- WipContextBuilder contextBuilder = scriptManager.getTabImpl().getContextBuilder();
- return contextBuilder.updateStackTrace(callFrames, setFramesCallback,
- relay.getUserSyncCallback());
- }
- }
-
- private void dispatchResult(SetScriptSourceData.Result result, UpdateCallback updateCallback) {
- if (updateCallback != null) {
- LiveEditResult liveEditResult;
- try {
- liveEditResult =
- LiveEditProtocolParserAccess.get().parseLiveEditResult(result.getUnderlyingObject());
- } catch (JsonProtocolParseException e) {
- throw new RuntimeException("Failed to parse LiveEdit response", e);
- }
- ChangeDescription wrappedChangeDescription =
- UpdateResultParser.wrapChangeDescription(liveEditResult);
- updateCallback.success(null, wrappedChangeDescription);
- }
- }
-}

Powered by Google App Engine
This is Rietveld 408576698