1182438 - Editing the context URL should change/remove the favicon, as it's no longer relevant

Status: RESOLVED WONTFIX

(Hello (Loop) :: Client, defect, P3)

Description

[[SV Manager] Florin Mezei, QA (:FlorinMezei)]

Reproducible with: 
- Firefox 40 Beta 3 - BuildID: 20150709163524
- Firefox 40 Beta 2 - BuildID: 20150706172413

Reproducible on: Windows 7 x64, Windows 10 x86, Mac OS X 10.8.5

Steps to reproduce:
1. On one machine (machine A) open Firefox with a fresh profile and navigate to a web page (e.g. http://www.nytimes.com/).
2. Click on the Hello icon and then "Get Started".
3. Check "Let's talk about" -> "Start a conversation".
4. Copy the conversation link and open it on another machine (machine B) and join the conversation.
5. On machine A choose to edit the conversation context (pen icon overlayed on the video area).
6. Change the URL to something else (e.g. http://espn.go.com/) and save.

Expected results:
Since the original favicon is no longer relevant to the new URL it should be: removed, replaced with a default favicon, or updated to the correct favicon (not sure if even possible).

Actual results:
The favicon from the original URL keeps displaying in:
- the Hello panel
- the conversation window (machine A)
- the link clicker side (machine B)

Comment 1

[Robert Kaiser]

[Tracking Requested - why for this release]:
Nominating Hello Conversation Context issues for 40, as the amount of issues makes the feature questionable for release with this train.

Comment 2

[Ritu Kothari (:ritu) (Inactive, please n-i to RyanVM, jcristau, or pascal)]

Adding a tracking flag for FF40. It would be nice to get a fix for this in time for 40 release as this is a new feature shipping in FF40.

Comment 3

[Mike de Boer [:mikedeboer]]

Requesting tracking-, because this is not something we intend to fix for the Fx40 train.

Comment 4

[Ritu Kothari (:ritu) (Inactive, please n-i to RyanVM, jcristau, or pascal)]

Untracking for FF40 as per Mike's suggestion. Showing the default icon would be nice but since we are in the end game in FF40 beta, let's leave it as is. We should try to fix it for FF41+

Comment 5

[Ritu Kothari (:ritu) (Inactive, please n-i to RyanVM, jcristau, or pascal)]

Mike, is this something that is being investigated/fixed at the moment? I do not think this is a release blocking scenario so I will set the FF41 status as won't fix. But is this worth while to track for 42?

Comment 6

[Mike de Boer [:mikedeboer]]

No, and also not a high enough priority for Fx43.

Comment 7

[Ian Bicking (:ianbicking)]

Editable context is no longer part of the product, so this state can no longer happen.