Make ModuleSystem::DumpException() handle empty handles.

Make ModuleSystem::DumpException() handle empty handles.


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

[Aaron Boodman, 2012-08-24 13:37:45]

lgtm, thanks for quick fix

[commit-bot: I haz the power, 2012-08-24 18:03:46]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/koz@chromium.org/10874052/1

[commit-bot: I haz the power, 2012-08-24 18:03:55]

Presubmit check for 10874052-1 failed and returned exit status 1.


Running presubmit commit checks ...

** Presubmit ERRORS **
Missing LGTM from an OWNER for files in these directories:
    chrome

Presubmit checks took 3.8s to calculate.

[Aaron Boodman, 2012-08-24 18:24:54]

OWNER review please?

[Lei Zhang, 2012-08-24 18:31:47]

lgtm with nit below

https://chromiumcodereview.appspot.com/10874052/diff/1/chrome/renderer/module...
File chrome/renderer/module_system.cc (right):

https://chromiumcodereview.appspot.com/10874052/diff/1/chrome/renderer/module...
chrome/renderer/module_system.cc:73:
message->GetScriptResourceName()->ToString());
nit: weird spacing.
How about:
    resource_name =
        *v8::String::Utf8Value(message->GetScriptResourceName()->ToString());

[Aaron Boodman, 2012-08-24 18:47:33]

Agree, but the author of this patch is away, and we need to get it landed, so
I'm going to go ahead without. Will fix the nit myself subsequently.

[commit-bot: I haz the power, 2012-08-24 18:47:47]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/koz@chromium.org/10874052/1

[commit-bot: I haz the power, 2012-08-24 18:48:27]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/koz@chromium.org/10874052/1

[commit-bot: I haz the power, 2012-08-24 21:05:48]

Change committed as 153283