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

Issue 625502 link

Starred by 2 users

Issue metadata

Status: Fixed
Owner:
Last visit > 30 days ago
Closed: Aug 2016
Components:
EstimatedDays: ----
NextAction: ----
OS: Linux
Pri: 2
Type: Bug



Sign in to add a comment

Unnecessary html entity encoding of page titles in Inspect Devices

Reported by teo8...@gmail.com, Jul 3 2016

Issue description

UserAgent: Mozilla/5.0 (X11; Linux x86_64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/51.0.2704.106 Safari/537.36

Steps to reproduce the problem:
1. On an Android device, visit http://output.jsbin.com/zeyubi
2. Connect the device to the computer
3. In Desktop Chrome, go to Developer Tools -> More Tools -> Inspect Devices

What is the expected behavior?
The page you have opened with the connected mobile phone should be listed as "Let's get retarded" which is its title

What went wrong?
The page is listed as "Let's get retarded" (see screenshot)

There's absolutely no reason to encode the page title here.

Did this work before? N/A 

Chrome version: 51.0.2704.106  Channel: stable
OS Version: 
Flash Version: Shockwave Flash 22.0 r0
 
Screenshot from 2016-07-04 00-27-59.png
19.5 KB View Download
Owner: pfeldman@chromium.org
Status: Available (was: Unconfirmed)
Owner: eostroukhov@chromium.org
Status: Assigned (was: Available)
Status: Started (was: Assigned)
This also happens if you open that web page in the regular Chrome and navigate to chrome://inspect/#pages
Project Member

Comment 4 by bugdroid1@chromium.org, Aug 8 2016

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

commit ef93247015994a7da23e1021df735e048a20ad95
Author: eostroukhov <eostroukhov@google.com>
Date: Mon Aug 08 17:02:44 2016

[DevTools] Do not escape page titles

Page titles are used as strings, they should not be escaped.

R=dgozman, pfeldman
BUG= 625502 

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

[modify] https://crrev.com/ef93247015994a7da23e1021df735e048a20ad95/chrome/browser/devtools/devtools_targets_ui.cc

Status: Fixed (was: Started)

Sign in to add a comment