Added an observer for 'pref::kDefaultCharset' change I forgot to add in rXXXX...

Added an observer for 'pref::kDefaultCharset' change I forgot to add in r15113 (http://src.chromium.org/viewvc/chrome?view=rev&revision=15113) . 
Besides, set referrer_charset_ (used in guessing the C-D charset in GetFileNameFromCD) to pref::kDefaultCharset without any q values appended.

TEST=1. Set the default charset in Options | Advanced | Fonts & Languages to ISO-8859-1. 
     2. Type http://i18nl10n.com/moztest/random21.yyy in the omnibox and the filename in the download bar should start with "ÇѱÛ21"
     3. Change the default charset to Korean
     4. Try #2 again and the filename in the download bar should start with '한글21'. 

BUG=1148
(http://crbug.com/1148 )




Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=20965

[wtc, 2009-07-16 01:14:47]

LGTM.

http://codereview.chromium.org/149705/diff/1006/1007
File chrome/browser/net/chrome_url_request_context.cc (right):

http://codereview.chromium.org/149705/diff/1006/1007#newcode291
Line 291: referrer_charset_ = default_charset;
Question: so referrer_charset_ should contain a single
charset and should not have the 'q' value?

[jungshik at Google, 2009-07-16 06:48:23]

2009/7/15  <wtc@chromium.org>:
> LGTM.

thank you for the review.

>
> http://codereview.chromium.org/149705/diff/1006/1007
> File chrome/browser/net/chrome_url_request_context.cc (right):
>
> http://codereview.chromium.org/149705/diff/1006/1007#newcode291
> Line 291: referrer_charset_ = default_charset;
> Question: so referrer_charset_ should contain a single
> charset and should not have the 'q' value?

yes. it's not sent over to a server but just used internally by us (net_utils).


>
> http://codereview.chromium.org/149705
>