My favorites | Sign in
Project Home Downloads Wiki Issues Code Search
New issue   Search
for
  Advanced search   Search tips   Subscriptions
Issue 56325: NTP replacement extensions need a way to launch an app
2 people starred this issue and may be notified of changes. Back to list
Status:  Verified
Owner:  asargent@chromium.org
Closed:  Sep 2010
Cc:  a...@chromium.org, erik...@chromium.org
M-8

Restricted
  • Only users with EditIssue permission may comment.


Sign in to add a comment
 
Project Member Reported by asargent@chromium.org, Sep 20, 2010
We added the extension.management API (currently in experimental, but hopefully moving to supported in M7) so that New Tab Page replacement extensions don't prevent people from being able to use apps. With the API as-is, they are limited to launching apps in a regular tab instead of being able to duplicate the experience you get from the built-in new tab page (pinned tabs, panels, full screen, etc. depending on settings). 


Sep 23, 2010
#1 bugdroid1@gmail.com
The following revision refers to this bug:
    http://src.chromium.org/viewvc/chrome?view=rev&revision=60334

------------------------------------------------------------------------
r60334 | asargent@chromium.org | Thu Sep 23 13:19:56 PDT 2010

Changed paths:
 M http://src.chromium.org/viewvc/chrome/trunk/src/chrome/browser/extensions/extension_management_api.h?r1=60334&r2=60333&pathrev=60334
 A http://src.chromium.org/viewvc/chrome/trunk/src/chrome/test/data/extensions/management/packaged_app/main.html?r1=60334&r2=60333&pathrev=60334
 M http://src.chromium.org/viewvc/chrome/trunk/src/chrome/browser/extensions/extension_management_api_browsertest.cc?r1=60334&r2=60333&pathrev=60334
 M http://src.chromium.org/viewvc/chrome/trunk/src/chrome/common/extensions/docs/experimental.management.html?r1=60334&r2=60333&pathrev=60334
 A http://src.chromium.org/viewvc/chrome/trunk/src/chrome/test/data/extensions/management/launch_app/manifest.json?r1=60334&r2=60333&pathrev=60334
 M http://src.chromium.org/viewvc/chrome/trunk/src/chrome/browser/extensions/extension_function_dispatcher.cc?r1=60334&r2=60333&pathrev=60334
 M http://src.chromium.org/viewvc/chrome/trunk/src/chrome/common/extensions/api/extension_api.json?r1=60334&r2=60333&pathrev=60334
 A http://src.chromium.org/viewvc/chrome/trunk/src/chrome/test/data/extensions/management/launch_app?r1=60334&r2=60333&pathrev=60334
 A http://src.chromium.org/viewvc/chrome/trunk/src/chrome/test/data/extensions/management/launch_app/background.html?r1=60334&r2=60333&pathrev=60334
 A http://src.chromium.org/viewvc/chrome/trunk/src/chrome/test/data/extensions/management/packaged_app/manifest.json?r1=60334&r2=60333&pathrev=60334
 A http://src.chromium.org/viewvc/chrome/trunk/src/chrome/test/data/extensions/management/packaged_app?r1=60334&r2=60333&pathrev=60334
 A http://src.chromium.org/viewvc/chrome/trunk/src/chrome/test/data/extensions/management/simple_extension/manifest.json?r1=60334&r2=60333&pathrev=60334
 M http://src.chromium.org/viewvc/chrome/trunk/src/chrome/browser/extensions/extension_management_api.cc?r1=60334&r2=60333&pathrev=60334
 A http://src.chromium.org/viewvc/chrome/trunk/src/chrome/test/data/extensions/management/launch_app/test.js?r1=60334&r2=60333&pathrev=60334
 A http://src.chromium.org/viewvc/chrome/trunk/src/chrome/test/data/extensions/management/simple_extension?r1=60334&r2=60333&pathrev=60334

Add a launchApp method to extension management API.

This makes it possible for new tab replacement extensions to recreate the same
app launch experience as the built-in NTP (pinned tabs, launch containers,
etc.)

Also add the extension version to data we report about extensions via
the management API.

BUG=56325,55474
TEST=An extension using chrome.experimental.management.launchApp(<id>) should
work to launch an app the same way that app gets launched from the built-in
new tab page.



Review URL: http://codereview.chromium.org/3381019
------------------------------------------------------------------------
Sep 23, 2010
#2 bugdroid1@gmail.com
The following revision refers to this bug:
    http://src.chromium.org/viewvc/chrome?view=rev&revision=60357

------------------------------------------------------------------------
r60357 | asargent@chromium.org | Thu Sep 23 14:44:20 PDT 2010

Changed paths:
 A http://src.chromium.org/viewvc/chrome/branches/517/src/chrome/test/data/extensions/management/launch_app/manifest.json?r1=60357&r2=60356&pathrev=60357
 M http://src.chromium.org/viewvc/chrome/branches/517/src/chrome/browser/extensions/extension_function_dispatcher.cc?r1=60357&r2=60356&pathrev=60357
 M http://src.chromium.org/viewvc/chrome/branches/517/src/chrome/common/extensions/api/extension_api.json?r1=60357&r2=60356&pathrev=60357
 A http://src.chromium.org/viewvc/chrome/branches/517/src/chrome/test/data/extensions/management/launch_app?r1=60357&r2=60356&pathrev=60357
 A http://src.chromium.org/viewvc/chrome/branches/517/src/chrome/test/data/extensions/management/launch_app/background.html?r1=60357&r2=60356&pathrev=60357
 A http://src.chromium.org/viewvc/chrome/branches/517/src/chrome/test/data/extensions/management/packaged_app/manifest.json?r1=60357&r2=60356&pathrev=60357
 A http://src.chromium.org/viewvc/chrome/branches/517/src/chrome/test/data/extensions/management/packaged_app?r1=60357&r2=60356&pathrev=60357
 A http://src.chromium.org/viewvc/chrome/branches/517/src/chrome/test/data/extensions/management/simple_extension/manifest.json?r1=60357&r2=60356&pathrev=60357
 M http://src.chromium.org/viewvc/chrome/branches/517/src/chrome/browser/extensions/extension_management_api.cc?r1=60357&r2=60356&pathrev=60357
 A http://src.chromium.org/viewvc/chrome/branches/517/src/chrome/test/data/extensions/management/launch_app/test.js?r1=60357&r2=60356&pathrev=60357
 A http://src.chromium.org/viewvc/chrome/branches/517/src/chrome/test/data/extensions/management/simple_extension?r1=60357&r2=60356&pathrev=60357
 M http://src.chromium.org/viewvc/chrome/branches/517/src/chrome/browser/extensions/extension_management_api.h?r1=60357&r2=60356&pathrev=60357
 A http://src.chromium.org/viewvc/chrome/branches/517/src/chrome/test/data/extensions/management/packaged_app/main.html?r1=60357&r2=60356&pathrev=60357
 M http://src.chromium.org/viewvc/chrome/branches/517/src/chrome/browser/extensions/extension_management_api_browsertest.cc?r1=60357&r2=60356&pathrev=60357
 M http://src.chromium.org/viewvc/chrome/branches/517/src/chrome/common/extensions/docs/experimental.management.html?r1=60357&r2=60356&pathrev=60357

Merge 60334 - Add a launchApp method to extension management API.

This makes it possible for new tab replacement extensions to recreate the same
app launch experience as the built-in NTP (pinned tabs, launch containers,
etc.)

Also add the extension version to data we report about extensions via
the management API.

BUG=56325,55474
TEST=An extension using chrome.experimental.management.launchApp(<id>) should
work to launch an app the same way that app gets launched from the built-in
new tab page.



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

TBR=asargent@chromium.org
Review URL: http://codereview.chromium.org/3402027
------------------------------------------------------------------------
Sep 23, 2010
#3 asargent@chromium.org
(No comment was entered for this change.)
Status: Fixed
Sep 23, 2010
#4 bugdroid1@gmail.com
The following revision refers to this bug:
    http://src.chromium.org/viewvc/chrome?view=rev&revision=60375

------------------------------------------------------------------------
r60375 | asargent@chromium.org | Thu Sep 23 16:13:41 PDT 2010

Changed paths:
 M http://src.chromium.org/viewvc/chrome/branches/517/src/chrome/browser/extensions/extension_function_dispatcher.cc?r1=60375&r2=60374&pathrev=60375
 M http://src.chromium.org/viewvc/chrome/branches/517/src/chrome/common/extensions/api/extension_api.json?r1=60375&r2=60374&pathrev=60375
 D http://src.chromium.org/viewvc/chrome/branches/517/src/chrome/test/data/extensions/management/launch_app?r1=60375&r2=60374&pathrev=60375
 D http://src.chromium.org/viewvc/chrome/branches/517/src/chrome/test/data/extensions/management/packaged_app?r1=60375&r2=60374&pathrev=60375
 M http://src.chromium.org/viewvc/chrome/branches/517/src/chrome/browser/extensions/extension_management_api.cc?r1=60375&r2=60374&pathrev=60375
 D http://src.chromium.org/viewvc/chrome/branches/517/src/chrome/test/data/extensions/management/simple_extension?r1=60375&r2=60374&pathrev=60375
 M http://src.chromium.org/viewvc/chrome/branches/517/src/chrome/browser/extensions/extension_management_api.h?r1=60375&r2=60374&pathrev=60375
 M http://src.chromium.org/viewvc/chrome/branches/517/src/chrome/browser/extensions/extension_management_api_browsertest.cc?r1=60375&r2=60374&pathrev=60375
 M http://src.chromium.org/viewvc/chrome/branches/517/src/chrome/common/extensions/docs/experimental.management.html?r1=60375&r2=60374&pathrev=60375

Revert 60357 - Merge 60334 - Add a launchApp method to extension management API.

This makes it possible for new tab replacement extensions to recreate the same
app launch experience as the built-in NTP (pinned tabs, launch containers,
etc.)

Also add the extension version to data we report about extensions via
the management API.

BUG=56325,55474
TEST=An extension using chrome.experimental.management.launchApp(<id>) should
work to launch an app the same way that app gets launched from the built-in
new tab page.



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

TBR=asargent@chromium.org
Review URL: http://codereview.chromium.org/3402027

TBR=asargent@chromium.org
Review URL: http://codereview.chromium.org/3416022
------------------------------------------------------------------------
Sep 24, 2010
#5 bugdroid1@gmail.com
The following revision refers to this bug:
    http://src.chromium.org/viewvc/chrome?view=rev&revision=60501

------------------------------------------------------------------------
r60501 | asargent@chromium.org | Fri Sep 24 11:48:05 PDT 2010

Changed paths:
 A http://src.chromium.org/viewvc/chrome/branches/517/src/chrome/test/data/extensions/management/launch_app/manifest.json?r1=60501&r2=60500&pathrev=60501
 M http://src.chromium.org/viewvc/chrome/branches/517/src/chrome/browser/extensions/extension_function_dispatcher.cc?r1=60501&r2=60500&pathrev=60501
 M http://src.chromium.org/viewvc/chrome/branches/517/src/chrome/common/extensions/api/extension_api.json?r1=60501&r2=60500&pathrev=60501
 A http://src.chromium.org/viewvc/chrome/branches/517/src/chrome/test/data/extensions/management/launch_app?r1=60501&r2=60500&pathrev=60501
 A http://src.chromium.org/viewvc/chrome/branches/517/src/chrome/test/data/extensions/management/launch_app/background.html?r1=60501&r2=60500&pathrev=60501
 A http://src.chromium.org/viewvc/chrome/branches/517/src/chrome/test/data/extensions/management/packaged_app/manifest.json?r1=60501&r2=60500&pathrev=60501
 A http://src.chromium.org/viewvc/chrome/branches/517/src/chrome/test/data/extensions/management/packaged_app?r1=60501&r2=60500&pathrev=60501
 A http://src.chromium.org/viewvc/chrome/branches/517/src/chrome/test/data/extensions/management/simple_extension/manifest.json?r1=60501&r2=60500&pathrev=60501
 M http://src.chromium.org/viewvc/chrome/branches/517/src/chrome/browser/extensions/extension_management_api.cc?r1=60501&r2=60500&pathrev=60501
 A http://src.chromium.org/viewvc/chrome/branches/517/src/chrome/test/data/extensions/management/launch_app/test.js?r1=60501&r2=60500&pathrev=60501
 A http://src.chromium.org/viewvc/chrome/branches/517/src/chrome/test/data/extensions/management/simple_extension?r1=60501&r2=60500&pathrev=60501
 M http://src.chromium.org/viewvc/chrome/branches/517/src/chrome/browser/extensions/extension_management_api.h?r1=60501&r2=60500&pathrev=60501
 A http://src.chromium.org/viewvc/chrome/branches/517/src/chrome/test/data/extensions/management/packaged_app/main.html?r1=60501&r2=60500&pathrev=60501
 M http://src.chromium.org/viewvc/chrome/branches/517/src/chrome/browser/extensions/extension_management_api_browsertest.cc?r1=60501&r2=60500&pathrev=60501
 M http://src.chromium.org/viewvc/chrome/branches/517/src/chrome/common/extensions/docs/experimental.management.html?r1=60501&r2=60500&pathrev=60501

Manual merge of 60334 - Add a launchApp method to extension management API.

This makes it possible for new tab replacement extensions to recreate the same
app launch experience as the built-in NTP (pinned tabs, launch containers,
etc.)

Also add the extension version to data we report about extensions via
the management API.

BUG=56325, 55474
TEST=An extension using chrome.experimental.management.launchApp(<id>) should
work to launch an app the same way that app gets launched from the built-in
new tab page.



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

TBR=asargent@chromium.org

Review URL: http://codereview.chromium.org/3462011
------------------------------------------------------------------------
Jul 19, 2011
#6 mberkow...@chromium.org
Verified in Chrome 14.0.822.0 (r92488).
Status: Verified
Oct 12, 2012
#7 bugdro...@chromium.org
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.
Labels: Restrict-AddIssueComment-Commit
Mar 10, 2013
#8 bugdro...@chromium.org
(No comment was entered for this change.)
Labels: -Area-Internals -Feature-Extensions -Feature-Apps -Mstone-8 Cr-Platform-Extensions M-8 Cr-Platform-Apps Cr-Internals
Mar 13, 2013
#9 bugdro...@chromium.org
(No comment was entered for this change.)
Labels: -Restrict-AddIssueComment-Commit Restrict-AddIssueComment-EditIssue
Sign in to add a comment

Powered by Google Project Hosting