New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 682622 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Last visit > 30 days ago
Closed: Feb 2017
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Linux , Windows , Mac
Pri: 3
Type: Bug



Sign in to add a comment

Both Yes and No options are seen in chrome://ntp-tiles-internals/ of incognito and guest mode

Project Member Reported by sc00335...@techmahindra.com, Jan 19 2017

Issue description

Chrome Version: 57.0.2986.0 dev
OS: Windows,Ubuntu 14.04

What steps will reproduce the problem?
(1)Open chrome://ntp-tiles-internals/ in Incognito/Guest mode and observe enabled options under topsites,suggestions,service

Expected: Either of the one [Yes or No] should be seen.
Actual: Instead both yes and no are seen.

This is a Non-regression issue seen from  57.0.2986.0 dev. Till 57.0.2985.0 chrome crashes on naviagting to above url in incognito/guest mode. 
 
Attaching screenshot for reference.. 
Actual_ntp-tiles-internals.png
135 KB View Download
Labels: OS-Mac
Able to reproduce the issue on Mac 10.12.1 using chrome version 57.0.2986.0
Project Member

Comment 3 by bugdroid1@chromium.org, Feb 27 2017

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/d6336797781587abf6c3fa648d034315714d1134

commit d6336797781587abf6c3fa648d034315714d1134
Author: sfiera <sfiera@chromium.org>
Date: Mon Feb 27 15:44:08 2017

chrome://ntp-tiles-internals: hide when incognito

Previously I just disabled the callbacks. In addition to that, send back
an empty configuration and site list, to make the page properly display
that everything is disabled and there are no sites in incognito.

Without this change, the "View JSON" button appears for popular sites,
but causes a crash because there's no PopularSites object.

BUG= 682622 , 696471 ,681774

Review-Url: https://codereview.chromium.org/2721603002
Cr-Commit-Position: refs/heads/master@{#453220}

[modify] https://crrev.com/d6336797781587abf6c3fa648d034315714d1134/components/ntp_tiles/webui/ntp_tiles_internals_message_handler.cc
[modify] https://crrev.com/d6336797781587abf6c3fa648d034315714d1134/components/ntp_tiles/webui/resources/ntp_tiles_internals.js

Comment 4 by sfiera@chromium.org, Feb 27 2017

Status: Fixed (was: Assigned)

Sign in to add a comment