Skip to content

[sailfish-browser] Virtualize background tabs when another application is opened#231

Open
rainemak wants to merge 1 commit intosailfishos:masterfrom
rainemak:virtualizeOnBackground
Open

[sailfish-browser] Virtualize background tabs when another application is opened#231
rainemak wants to merge 1 commit intosailfishos:masterfrom
rainemak:virtualizeOnBackground

Conversation

@rainemak
Copy link
Copy Markdown
Member

Current implementation virtualizes backgrounded tabs when low memory
notification (warning / critical) is received even if the browser is
at foreground. Virtualizing backgrounded tabs when another application
is brought foreground guarantees that there are no background tabs
when another application is running regardless of low memory
notification.

@rainemak rainemak force-pushed the virtualizeOnBackground branch from 654ecb5 to 43b72e8 Compare November 11, 2014 07:28
Comment thread src/webpages.cpp Outdated
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

When the browser is semi-permanently on background, the timestamp is updated on every backgroundChanged(). That prevents the memory-pressure notification and is a small behavior change not described in the commit.

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

True that. Will later PR.

…n is opened

Current implementation virtualizes backgrounded tabs when low memory
notification (warning / critical) is received even if the browser is
at foreground. Virtualizing backgrounded tabs when another application
is brought foreground guarantees that there are no background tabs
when another application is running regardless of low memory
notification.
@rainemak rainemak force-pushed the virtualizeOnBackground branch from 43b72e8 to 32b0f43 Compare November 11, 2014 09:01
@rainemak
Copy link
Copy Markdown
Member Author

Updated.

Comment thread src/webpages.cpp
Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Oops. This sentence got copied here as well. Should not be here.

Another way to implement this kind of early release would be to do this when screen is dimmed meaning that browser goes to background and it is not requesting screen to be on.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants