My favorites | Sign in
Logo
             
New issue | Search
for
| Advanced search | Search tips
Issue 2929: "Encoding" menu disabled when visit a page at the first time
3 people starred this issue and may be notified of changes. Back to list
 
Reported by xlyuan@chromium.org, Sep 29, 2008
Build: 0.2.153.1 (Official Build 2519)
 
[Steps]
1 Launch Chrome
2 Navigate to any web page, say google.com
3 Select "Control the current page" drop down menu 
4 Observe

[Results]
The "Encoding" menu is disabled

[Expected]
The "Encoding" menu should be always enabled

[Notes]
Not reproduce after reload the current page


Encoding menu.jpg
114 KB   View   Download
Comment 1 by mal.chromium, Sep 29, 2008
(No comment was entered for this change.)
Labels: Area-BrowserUI I18N are
Comment 2 by mal.chromium, Sep 29, 2008
(No comment was entered for this change.)
Labels: -are -area-i18n
Comment 3 by thatan...@google.com, Oct 03, 2008
(No comment was entered for this change.)
Cc: thatan...@google.com
Labels: I18N-jp
Comment 4 by jshin@chromium.org, Oct 16, 2008
This is for sure Mstone-1.0. I've begun to take a look along with  issue 3315 


Status: Assigned
Owner: js...@chromium.org
Cc: -js...@chromium.org
Labels: Mstone-1.0
Comment 5 by xlyuan@chromium.org, Oct 17, 2008
I just want to mention that: 

When open js or css file, the "Encoding" menu is always disabled, so if "Auto detect"
is off or "Default encoding" is not the right one, there is no way to get non-ASCII
characters show correctly in js or css file.

Test:
1. Launch English Chrome
2. Go to http://login.sina.com.cn/
3. View source of this page
4. Open the corresponding css or js link in a new tab
5. Observe the non-ASCII characters
Comment 6 by jshin@chromium.org, Oct 17, 2008
JS/CSS issue was filed as  issue 3565 
Comment 7 by tak...@google.com, Oct 22, 2008
(No comment was entered for this change.)
Cc: hb...@chromium.org tak...@google.com xly...@chromium.org
Comment 8 by jshin@chromium.org, Oct 22, 2008
(No comment was entered for this change.)
Status: Started
Comment 9 by niranjan@chromium.org, Oct 22, 2008
Moving this out of the 1.0 list as per https://spreadsheets.google.com/a/google.com/ccc?key=pv1RidUkOQETG1XcAcFyRYA
Labels: -Mstone-1.0 Mstone-X
Comment 10 by mal.chromium, Oct 22, 2008
(No comment was entered for this change.)
Labels: -Pri-2 -Mstone-X Pri-1 Mstone-1.0 Regression dev-release-block
Comment 11 by jshin@chromium.org, Oct 23, 2008
fixed in r3844 on trunk (also merged to 1.0 branch by Mark). 


Status: Fixed
Comment 12 by jshin@chromium.org, Oct 23, 2008
Xiaolu, can you verify the fix? 

Glen said he still saw the encoding menu disabled occasionally? 
To Glen, can you reproduce failure reliably? Specific web pages, file types, a
sequence of pages?  Or is it just sporadic?  


Cc: g...@chromium.org
Comment 13 by xlyuan@chromium.org, Oct 24, 2008
Hmm, three cases:

1. Fixed for part of web pages, like:

http://www.chinamobile.com/
http://www.icbc.com.cn/icbc/

2. Still reproduce on some web pages, what those pages have in common:

When "Save page as", "Save As" dialog shows:
File name: download
Save as type: Chrome HTML

Instead of:
File name: The title of the page
Save as type: Web Page, Complete

URLs:
http://www.autohome.com.cn/
http://fund.eastmoney.com/

3. This web page is more weird, when open it at the first time, the "Encoding" menu
is enabled, then switch to a new tab, then switch back, the "Encoding" menu becomes
disabled.

URL:
http://www.zaobao.com/

Verified on build:
0.3.154.6 (Official Build 3810)
0.3.155.0 (Developer Build 3908)



Status: Assigned
Comment 14 by jshin@chromium.org, Oct 24, 2008
The condition for enabling the encoding menu is |IsSavableType && IsSavableURL|.  So, 
the 2nd case makes sense. Now, I wonder why those web pages are regarded as 'not 
savable'.  I'll debug it. 

Hmm, the 3rd case is indeed strange. I'll also look into it. 



Comment 15 by jshin@chromium.org, Oct 27, 2008
This time, it should be really fixed (r4001 , 
http://codereview.chromium.org/8206/show )




Comment 16 by xlyuan@chromium.org, Oct 29, 2008
Fixed on trunk build 0.3.155.0 (Developer Build 4200).

I will close this bug when the fix landed to the branch.
Comment 17 by niranjan@chromium.org, Nov 05, 2008
Xiaolu, please mark this as verified when you verify this on the release branch.
Status: Fixed
Comment 18 by progame, Nov 06, 2008
i notice it with 0.3.154.9
in this site
http://statcomp.ats.ucla.edu/WebTools/JavaTools/popup.htm
after i click the "here" link to open a popup (in the encoding menu of the chrome 
icon in the popup's top left corner)
Comment 19 by mberkow...@chromium.org, Nov 07, 2008
Verified in 0.3.154.12 (Official Build 4978).
Status: Verified
Comment 20 by jshin@chromium.org, Nov 07, 2008
RE: comment 18 
0.3.154.9 does not have the patch, yet. It'll be in next dev release. However, even 
in trunk build, the pop-up menu has the encoding menu disabled. Thanks for spotting 
it. Let's deal with it in  issue 3769 .


Comment 21 by progame, Nov 07, 2008
you mean issue 3765 (i think)

i just love beta testing... thanks for giving me the platform to do it :) and for 
making a great piece of software of course


Sign in to add a comment

Powered by Google Project Hosting