diff options
| author | agl@chromium.org <agl@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2009-07-09 17:27:51 +0000 |
|---|---|---|
| committer | agl@chromium.org <agl@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2009-07-09 17:27:51 +0000 |
| commit | 47c2188f2fc336b6040d6047afca32eb8e9779d8 (patch) | |
| tree | 9e0df23594de8c9c89cf8d5a8aaa4157d824d514 /chrome | |
| parent | 5156c3cf4c3162755d16b69c7842763a8f277c4c (diff) | |
| download | chromium_src-47c2188f2fc336b6040d6047afca32eb8e9779d8.zip chromium_src-47c2188f2fc336b6040d6047afca32eb8e9779d8.tar.gz chromium_src-47c2188f2fc336b6040d6047afca32eb8e9779d8.tar.bz2 | |
Linux: add LOG(FATAL) to try and catch an error.
Crash reports like:
http://crash/reportview?product=Chrome_Linux&version=3.0.192.0&date=&signature=WebCore::TextRunWalker::nextScriptRun()-3F2080B
are popping up and I don't know why. I suspect that we might we
returning an empty family string from fontconfig, so I'm adding a
LOG(FATAL) to catch this. We'll see in the crash reports from next
week if it's getting hit.
R=evan
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@20271 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'chrome')
| -rw-r--r-- | chrome/renderer/renderer_sandbox_support_linux.cc | 8 |
1 files changed, 8 insertions, 0 deletions
diff --git a/chrome/renderer/renderer_sandbox_support_linux.cc b/chrome/renderer/renderer_sandbox_support_linux.cc index a08fff1..758bf57 100644 --- a/chrome/renderer/renderer_sandbox_support_linux.cc +++ b/chrome/renderer/renderer_sandbox_support_linux.cc @@ -32,6 +32,14 @@ std::string getFontFamilyForCharacters(const uint16_t* utf16, size_t num_utf16) reply.ReadString(&pickle_iter, &family_name); } + if (!family_name.size()) { + LOG(ERROR) << "Starting code point dump:"; + for (size_t i = 0; i < num_utf16; ++i) + LOG(ERROR) << " " << utf16[i]; + LOG(FATAL) << "Bug caught: requested font family for " << num_utf16 + << " code points and got an empty string on return"; + } + return family_name; } |