Skip to content
This repository has been archived by the owner on Jul 22, 2024. It is now read-only.

Commit

Permalink
Improved set session code
Browse files Browse the repository at this point in the history
  • Loading branch information
@keianhzo
keianhzo committed May 7, 2020
1 parent adaf6dc commit 3f180f8
Show file tree
Hide file tree
Showing 2 changed files with 30 additions and 34 deletions.
34 changes: 24 additions & 10 deletions app/src/common/shared/org/mozilla/vrbrowser/ui/widgets/WindowWidget.java
View file
Original file line number Diff line number Diff line change
Expand Up @@ -94,6 +94,11 @@ public class WindowWidget extends UIWidget implements SessionChangeListener,
public static final int SESSION_RELEASE_DISPLAY = 0;
public static final int SESSION_DO_NOT_RELEASE_DISPLAY = 1;

@IntDef(value = { ACTIVATE, DEACTIVATE, DO_NOT_CHANGE})
public @interface SetSessionActiveState {}
public static final int ACTIVATE = 0;
public static final int DEACTIVATE = 1;
public static final int DO_NOT_CHANGE = 2;

private Surface mSurface;
private int mWidth;
Expand Down Expand Up @@ -1107,21 +1112,35 @@ public void draw(Canvas aCanvas) {
}
}

public void setSession(@NonNull Session aSession) {
setSession(aSession, SESSION_RELEASE_DISPLAY);
public void setSession(@NonNull Session aSession, @SetSessionActiveState int aSessionState) {
setSession(aSession, SESSION_RELEASE_DISPLAY, aSessionState);
}

public void setSession(@NonNull Session aSession, @OldSessionDisplayAction int aDisplayAction) {
public void setSession(@NonNull Session aSession, @OldSessionDisplayAction int aDisplayAction, @SetSessionActiveState int aSessionState) {
if (mSession != aSession) {
Session oldSession = mSession;
if (oldSession != null) {
cleanListeners(oldSession);
switch (aSessionState) {
case ACTIVATE:
mSession.setActive(true);
break;
case DEACTIVATE:
mSession.setActive(false);
break;
case DO_NOT_CHANGE:
break;
}
if (aDisplayAction == SESSION_RELEASE_DISPLAY) {
oldSession.releaseDisplay();
}
}

mSession = aSession;

setupListeners(mSession);
SessionStore.get().setActiveSession(mSession);

mViewModel.setIsPrivateSession(mSession.isPrivateMode());

if (oldSession != null) {
Expand Down Expand Up @@ -1161,9 +1180,7 @@ public void onStackSession(Session aSession) {
// e.g. tab opened via window.open()
aSession.updateLastUse();
Session current = mSession;
setupListeners(aSession);
setSession(aSession);
SessionStore.get().setActiveSession(aSession);
setSession(aSession, DEACTIVATE);
current.captureBackgroundBitmap(getWindowWidth(), getWindowHeight()).thenAccept(aVoid -> current.setActive(false));
mWidgetManager.getWindows().showTabAddedNotification();

Expand All @@ -1173,10 +1190,7 @@ public void onStackSession(Session aSession) {
@Override
public void onUnstackSession(Session aSession, Session aParent) {
if (mSession == aSession) {
aParent.setActive(true);
setupListeners(aParent);
setSession(aParent);
SessionStore.get().setActiveSession(aParent);
setSession(aParent, DEACTIVATE);
SessionStore.get().destroySession(aSession);
}
}
Expand Down
30 changes: 6 additions & 24 deletions app/src/common/shared/org/mozilla/vrbrowser/ui/widgets/Windows.java
View file
Original file line number Diff line number Diff line change
Expand Up @@ -1213,21 +1213,14 @@ public void onTabSelect(Session aTab) {
Session moveTo = targetWindow.getSession();
moveFrom.surfaceDestroyed();
moveTo.surfaceDestroyed();
windowToMove.setupListeners(moveTo);
windowToMove.setSession(moveTo, WindowWidget.SESSION_DO_NOT_RELEASE_DISPLAY);
targetWindow.setupListeners(moveFrom);
targetWindow.setSession(moveFrom, WindowWidget.SESSION_DO_NOT_RELEASE_DISPLAY);
SessionStore.get().setActiveSession(targetWindow.getSession());
windowToMove.setSession(moveTo, WindowWidget.SESSION_DO_NOT_RELEASE_DISPLAY, WindowWidget.DO_NOT_CHANGE);
targetWindow.setSession(moveFrom, WindowWidget.SESSION_DO_NOT_RELEASE_DISPLAY, WindowWidget.DO_NOT_CHANGE);
windowToMove.setActiveWindow(false);
targetWindow.setActiveWindow(true);

} else {
setFirstPaint(targetWindow, aTab);
targetWindow.getSession().setActive(false);
targetWindow.setupListeners(aTab);
aTab.setActive(true);
targetWindow.setSession(aTab);
SessionStore.get().setActiveSession(aTab);
targetWindow.setSession(aTab, WindowWidget.DEACTIVATE);
}
}

Expand All @@ -1238,16 +1231,12 @@ public void addTab(WindowWidget targetWindow) {
public void addTab(@NonNull WindowWidget targetWindow, @Nullable String aUri) {
Session session = SessionStore.get().createSuspendedSession(aUri, targetWindow.getSession().isPrivateMode());
setFirstPaint(targetWindow, session);
targetWindow.getSession().setActive(false);
targetWindow.setupListeners(session);
session.setActive(true);
targetWindow.setSession(session);
targetWindow.setSession(session, WindowWidget.DEACTIVATE);
if (aUri == null || aUri.isEmpty()) {
session.loadHomePage();
} else {
session.loadUri(aUri);
}
SessionStore.get().setActiveSession(session);
}

public void addBackgroundTab(WindowWidget targetWindow, String aUri) {
Expand Down Expand Up @@ -1303,9 +1292,7 @@ public void onTabsClose(ArrayList<Session> aTabs) {
Session tab = available.get(0);
if (tab != null) {
setFirstPaint(window, tab);
window.setupListeners(tab);
tab.setActive(true);
window.setSession(tab);
window.setSession(tab, WindowWidget.DO_NOT_CHANGE);
}

available.remove(0);
Expand All @@ -1320,8 +1307,6 @@ public void onTabsClose(ArrayList<Session> aTabs) {
cache.removeBitmap(session.getId());
SessionStore.get().destroySession(session);
}

SessionStore.get().setActiveSession(targetWindow.getSession());
}

@Override
Expand All @@ -1342,10 +1327,7 @@ public void onTabsReceived(@NonNull List<TabData> aTabs) {

if (i == 0 && !fullscreen) {
// Set the first received tab of the list the current one.
SessionStore.get().setActiveSession(session);
targetWindow.setupListeners(session);
targetWindow.getSession().setActive(false);
targetWindow.setSession(session);
targetWindow.setSession(session, WindowWidget.DEACTIVATE);
}
}

Expand Down

0 comments on commit 3f180f8

Please sign in to comment.