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

Issue 862753 link

Starred by 1 user

Issue metadata

Status: Assigned
Owner:
Last visit > 30 days ago
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 3
Type: Bug



Sign in to add a comment

Splitting QueryHomepageTitle from HomepageClient interface

Project Member Reported by romax@chromium.org, Jul 11

Issue description

Quoting from a CL comment by dimich@: 
"""
Instead of adding this method to the HomepageClient, I'd make it a member method of MostVisitedSites (a sibling of InitiateNotificationForNewTiles(), where it is used).

This way, HomepageClient always would have only one 'direction' of info flow (from Java HomepageManager into cc side) and it'll make it simpler for subsequent reading/understanding.

""

With offline discussion, we decided to do it later to make HomepageClient only have one-way information flow (from Java to native) and probably splitting the QueryHomepageTitle out to another interface used between components/ and chrome/browser/.
 
Owner: romax@chromium.org
Status: Assigned (was: Untriaged)

Sign in to add a comment