New issue
Advanced search Search tips
Starred by 10 users

Issue metadata

Status: Fixed
Owner:
Closed: Apr 2012
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 3
Type: Bug

Restricted
  • Only users with EditIssue permission may comment.



Sign in to add a comment
link

Issue 96006: Re-enable acceleration for apps background pages

Reported by sadrul@chromium.org, Sep 9 2011 Project Member

Issue description

GPU acceleration is currently disabled for apps background pages (see  crbug.com/96005 ). It should eventually be turned back on.

Related:  http://crbug.com/64512  and  http://crbug.com/64841 
 

Comment 1 by bugdroid1@chromium.org, Sep 9 2011

Project Member
The following revision refers to this bug:
    http://src.chromium.org/viewvc/chrome?view=rev&revision=100404

------------------------------------------------------------------------
r100404 | sadrul@chromium.org | Fri Sep 09 07:27:58 PDT 2011

Changed paths:
 M http://src.chromium.org/viewvc/chrome/trunk/src/chrome/browser/tab_contents/background_contents.cc?r1=100404&r2=100403&pathrev=100404

apps/extensions: Disable all kinds of acceleration for background pages.

This was already done for extensions background pages. This change disables acceleration for apps background pages too.

BUG= 96005 , 96006 
TEST=none

Review URL: http://codereview.chromium.org/7787012
------------------------------------------------------------------------

Comment 2 by yoz@chromium.org, Sep 19 2011

Labels: -Pri-2 Pri-3 Feature-GPU
Owner: sadrul@chromium.org
What are the conditions for "eventually"?

Comment 3 by asargent@chromium.org, Oct 1 2011

Status: Assigned
(marking as assigned since it has an owner)

Comment 4 by vangelis@google.com, Oct 3 2011

yoz@ , is there a good use-case for adding gpu acceleration to background pages?

Comment 5 by sadrul@chromium.org, Oct 7 2011

Labels: Mstone-15 Merge-Requested

Comment 6 by sadrul@chromium.org, Oct 7 2011

yoz@ For this particular case, at least  crbug.com/96005  needs fixing first.

Comment 7 by kareng@google.com, Oct 7 2011

Labels: -Merge-Requested Merge-Approved

Comment 8 by bugdroid1@chromium.org, Oct 7 2011

Project Member
Labels: merge-merged-874
The following revision refers to this bug:
    http://src.chromium.org/viewvc/chrome?view=rev&revision=104570

------------------------------------------------------------------------
r104570 | sadrul@chromium.org | Fri Oct 07 14:29:53 PDT 2011

Changed paths:
 M http://src.chromium.org/viewvc/chrome/branches/874/src/chrome/browser/tab_contents/background_contents.cc?r1=104570&r2=104569&pathrev=104570

Merge 100404 - apps/extensions: Disable all kinds of acceleration for background pages.

This was already done for extensions background pages. This change disables acceleration for apps background pages too.

BUG= 96005 , 96006 
TEST=none

Review URL: http://codereview.chromium.org/7787012

TBR=sadrul@chromium.org
Review URL: http://codereview.chromium.org/8198012
------------------------------------------------------------------------

Comment 9 by kareng@google.com, Oct 17 2011

Labels: -Merge-Approved Merge-Merged

Comment 10 by saintlou@chromium.org, Apr 13 2012

Status: Fixed

Comment 11 by bugdroid1@chromium.org, Oct 13 2012

Project Member
Labels: Restrict-AddIssueComment-Commit
This issue has been closed for some time. No one will pay attention to new comments.
If you are seeing this bug or have new data, please click New Issue to start a new bug.

Comment 12 by bugdroid1@chromium.org, Mar 10 2013

Project Member
Labels: -Feature-Apps -Feature-GPU -Mstone-15 Cr-Platform-Apps Cr-Internals-GPU M-15

Comment 13 by bugdroid1@chromium.org, Mar 13 2013

Project Member
Labels: -Restrict-AddIssueComment-Commit Restrict-AddIssueComment-EditIssue

Sign in to add a comment