Issue metadata
Sign in to add a comment
|
Chrome Extensions API method chrome.topSites.get returns strange results
Reported by
stepanya...@gmail.com,
Nov 28
|
||||||||||||||||||||||
Issue descriptionUserAgent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10_14_0) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/71.0.3578.55 YaBrowser/19.1.1.25 (beta) Yowser/2.5 Safari/537.36 Steps to reproduce the problem: 1. Manually add/remove some sites on the new tab page 2. Install extension that has topSites permission 3. Try to call chrome.topSites.get in extensions background page What is the expected behavior? I expect chrome.topSites.get to return the list of sites from new tab page What went wrong? It returns strange result, not the list of sites from new tab page. It seems some sites from history, but i'm not sure Did this work before? Yes 71.0.3578.62 Does this work in other browsers? Yes Chrome version: 72.0.3624.0 Channel: canary OS Version: OS X 10.14.0 Flash Version: Doesn't reproduced on 70.0.3538.110 and 71.0.3578.62
,
Nov 28
,
Nov 28
,
Nov 29
Thanks for filing the issue... Tried to reproduce the issue on reported chrome 72.0.3624.0 and 71.0.3578.62 using Mac 10.14.0. Attaching screencast for reference. Steps: ----- 1. Launched reported chrome 2. Installed extension as per attached screenshot 3. Enabled Developer mode and opened background page >> console >> Entered text chrome.topSites.get As we have observed that the functions on the console page (observed same behaviour on chrome 72.0.3624.0 and 71.0.3578.62) Reporter: Could you please check the attached screen cast and let us know if anything missed from our end and If possible provide a sample extension and screencast so that we can investigate the issue further. Thanks..!
,
Nov 29
I have investigated that problem starts to reproduce after the site deletion from NTP. Before the deletion everything is ok, take a look at screencast 1.mp4 But after deletion nothing is deleted or added to chrome.topSites (screencast 2.mp4)
,
Nov 29
Thank you for providing more feedback. Adding the requester to the cc list. For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
,
Nov 29
,
Nov 30
Able to reproduce the issue on the reported chrome 72.0.3624.0 and issue not seen on latest canary 72.0.3625.0 using Mac 10.14.0. As we are getting all good builds. Hence, providing the manual bisect information from https://omahaproxy.appspot.com/. Inconsistent behavior seen on Windows and Ubuntu. Bisect Info: ================ Good build: 70.0.3503.0 Bad build: 70.0.3504.0 CHANGELOG URL: https://chromium.googlesource.com/chromium/src/+log/70.0.3503.0..70.0.3504.0?pretty=fuller&n=10000 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1149458/ Kristi Park:Please confirm the issue and help in re-assigning if it is not related to your change. Adding RBS label for M-71 feel free to change it if not required. Thanks!
,
Nov 30
This is working as intended. chrome.topSites.get returns the list of most visited sites based on history, and this is what is shown on the NTP by default. Editing/adding/deleting those shortcuts on the NTP is a separate feature and does not affect this list. You can confirm this by clicking the gear icon and "Restore default shortcuts". The shortcuts should be what is returned by chrome.topSites.get. @ramyan, we may need to update the description in https://developer.chrome.com/extensions/topSites#method-get.
,
Dec 3
Kristi Park, thank you for your response. If it works as intended, I think it will be useful to have API to retrieve the list of shortcuts from NTP
,
Dec 3
Yes, that's definitely something we're considering. Updates will be posted in bug 911175. |
|||||||||||||||||||||||
►
Sign in to add a comment |
|||||||||||||||||||||||
Comment 1 by stepanya...@gmail.com
, Nov 28