Project: chromium Issues People Development process History Sign in
New issue
Advanced search Search tips
Issue 111900 [uber page] Sub-page URLS are not invoking the correct page
Starred by 23 users Reported by pcha...@gmail.com, Jan 30 2012 Back to list
Status: Verified
Owner: csilv@chromium.org
Closed: Feb 2012
Cc: csilv@chromium.org, anan...@chromium.org, annapop@chromium.org, dim...@chromium.org, kochi@chromium.org, tturchetto@chromium.org, ligim...@chromium.org, jhawkins@chromium.org, estade@chromium.org, nkostylev@chromium.org, krisr@chromium.org
Components:
OS: ----
Pri: 1
Type: Bug-Regression

Blocking:
issue chromium-os:25716
issue 208643

Restricted
  • Only users with EditIssue permission may comment.


Sign in to add a comment
Chromium Version     : 18.0.1023.0 (Build de développement 119645 Windows)
URLs (if applicable) :
Other browsers tested: Google Chrome stable 16.0.912.77 m
Add OK or FAIL after other browsers where you have tested this issue:
Safari 5:
Firefox 4.x:
IE 7/8/9:

What steps will reproduce the problem?
1. Right-click on an extension icon
2. clik on "manage extensions" (cf screenshot 1)
3.

What is the expected result?
Extensions panel is displayed directly

What happens instead?
Options panel is displayed instead (cf screenshot 2) 

Please provide any additional information below. Attach a screenshot if
possible.
 
Chromium_extensions-1.png
31.4 KB View Download
Chromium_extensions-2.png
76.7 KB View Download
Comment 1 by dharani@google.com, Jan 30 2012
Labels: -Area-Undefined Area-UI Feature-Extensions
Comment 2 by pavanv@chromium.org, Jan 31 2012
Labels: -Type-Bug Type-Regression Mstone-18 ReleaseBlock-Beta
Status: Untriaged
Chrome: 18.0.1023.0 and 18.0.1025.0
Doesn't happen on 18.0.1017.0

Must've been introduced with the new layout for chrome://settings 
Comment 3 by dharani@google.com, Jan 31 2012
Labels: Feature-Preferences
Comment 4 by pavanv@chromium.org, Jan 31 2012
Labels: -Pri-2 Pri-1
Increasing the priority since other options with shortcuts are leading to settings master page as well

eg: ctrl+shift+del redirects to chrome://settings instead of chrome://settings/clearBrowserData
Owner: csilv@chromium.org
Status: Assigned
Comment 6 by csilv@chromium.org, Jan 31 2012
Summary: [uber page] Sub-page URLS are not invoking the correct page (was: NULL)
Comment 7 by csilv@chromium.org, Jan 31 2012
Status: Started
Comment 8 by csilv@chromium.org, Jan 31 2012
Issue 112192 has been merged into this issue.
Comment 9 by csilv@chromium.org, Jan 31 2012
Cc: estade@chromium.org csilv@chromium.org jhawkins@chromium.org
Issue 112188 has been merged into this issue.
Issue 112133 has been merged into this issue.
Cc: ligim...@chromium.org dim...@chromium.org anan...@chromium.org
Issue 112215 has been merged into this issue.
Project Member Comment 12 by bugdroid1@chromium.org, Feb 1 2012
The following revision refers to this bug:
    http://src.chromium.org/viewvc/chrome?view=rev&revision=120006

------------------------------------------------------------------------
r120006 | csilv@chromium.org | Tue Jan 31 17:53:44 PST 2012

Changed paths:
 M http://src.chromium.org/viewvc/chrome/trunk/src/chrome/browser/resources/uber/uber.js?r1=120006&r2=120005&pathrev=120006

[uber page] Fix url handling for uber-page sub-paths.

BUG= 111900 
TEST=Invoke 'Clear Browsing Data' from the wrench menu, should bring up the correct dialog.
Review URL: https://chromiumcodereview.appspot.com/9309018
------------------------------------------------------------------------
Project Member Comment 13 by bugdroid1@chromium.org, Feb 1 2012
The following revision refers to this bug:
    http://src.chromium.org/viewvc/chrome?view=rev&revision=120045

------------------------------------------------------------------------
r120045 | csilv@chromium.org | Wed Feb 01 00:18:16 PST 2012

Changed paths:
 M http://src.chromium.org/viewvc/chrome/trunk/src/chrome/browser/browser_about_handler.cc?r1=120045&r2=120044&pathrev=120045
 M http://src.chromium.org/viewvc/chrome/trunk/src/chrome/browser/ui/browser.cc?r1=120045&r2=120044&pathrev=120045

[uber page] Improve URL mapping for extension URLS.

BUG= 111900 
TEST=Verify that the 'Manage extensions' contextual menu invokes the extensions page.
Review URL: https://chromiumcodereview.appspot.com/9318019
------------------------------------------------------------------------
Blocking: chromium-os:25716
Comment 15 by dhw@chromium.org, Feb 1 2012
Issue 112256 has been merged into this issue.
Issue 112207 has been merged into this issue.
Labels: Merge-Requested
Verified in a fresh developer build.  There are two changelists to merge, detailed above:

    r120006
    r120045


Comment 18 by kareng@google.com, Feb 1 2012
Labels: -Merge-Requested Merge-Approved
Cc: tturchetto@chromium.org krisr@chromium.org
Issue chromium-os:25643 has been merged into this issue.
Cc: annapop@chromium.org
Issue chromium-os:25739 has been merged into this issue.
Issue chromium-os:25738 has been merged into this issue.
Comment 22 by pcha...@gmail.com, Feb 2 2012
Fixed in 18.0.1027.0 (Build de développement 120133)

Thanks
Project Member Comment 23 by bugdroid1@chromium.org, Feb 2 2012
Labels: -merge-approved merge-merged-1025
The following revision refers to this bug:
    http://src.chromium.org/viewvc/chrome?view=rev&revision=120232

------------------------------------------------------------------------
r120232 | csilv@chromium.org | Thu Feb 02 12:03:37 PST 2012

Changed paths:
 M http://src.chromium.org/viewvc/chrome/branches/1025/src/chrome/browser/resources/uber/uber.js?r1=120232&r2=120231&pathrev=120232

Merge 120006 - [uber page] Fix url handling for uber-page sub-paths.

BUG= 111900 
TEST=Invoke 'Clear Browsing Data' from the wrench menu, should bring up the correct dialog.
Review URL: https://chromiumcodereview.appspot.com/9309018

TBR=csilv@chromium.org
Review URL: https://chromiumcodereview.appspot.com/9325005
------------------------------------------------------------------------
Project Member Comment 24 by bugdroid1@chromium.org, Feb 2 2012
The following revision refers to this bug:
    http://src.chromium.org/viewvc/chrome?view=rev&revision=120233

------------------------------------------------------------------------
r120233 | csilv@chromium.org | Thu Feb 02 12:05:04 PST 2012

Changed paths:
 M http://src.chromium.org/viewvc/chrome/branches/1025/src/chrome/browser/browser_about_handler.cc?r1=120233&r2=120232&pathrev=120233
 M http://src.chromium.org/viewvc/chrome/branches/1025/src/chrome/browser/ui/browser.cc?r1=120233&r2=120232&pathrev=120233

Merge 120045 - [uber page] Improve URL mapping for extension URLS.

BUG= 111900 
TEST=Verify that the 'Manage extensions' contextual menu invokes the extensions page.
Review URL: https://chromiumcodereview.appspot.com/9318019

TBR=csilv@chromium.org
Review URL: https://chromiumcodereview.appspot.com/9325006
------------------------------------------------------------------------
Issue 112502 has been merged into this issue.
Issue 112505 has been merged into this issue.
Verified fix, build 18.0.1025.3 (thanks so much for merging and rolling it out!). In particular verified all individual scenarios as above.

Comment 28 by kareng@google.com, Feb 3 2012
Status: Verified
Comment 29 by dhw@chromium.org, Feb 6 2012
Issue 112874 has been merged into this issue.
Cc: kochi@chromium.org nkostylev@chromium.org
Issue chromium-os:25716 has been merged into this issue.
Issue 112801 has been merged into this issue.
Project Member Comment 32 by bugdroid1@chromium.org, Oct 13 2012
Blocking: -chromium-os:25716 chromium-os:25716
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 33 by laforge@google.com, Oct 17 2012
Labels: -Feature-Preferences Feature-Settings MovedToSettings
Project Member Comment 34 by bugdroid1@chromium.org, Mar 9 2013
Labels: -Type-Regression -Area-UI -Feature-Extensions -Mstone-18 -Feature-Settings Type-Bug-Regression Cr-UI-Settings Cr-Platform-Extensions Cr-UI M-18
Project Member Comment 35 by bugdroid1@chromium.org, Mar 13 2013
Labels: -Restrict-AddIssueComment-Commit Restrict-AddIssueComment-EditIssue
Sign in to add a comment