Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(638)

Issue 3013039: Enhance plugin logging a bit. This adds logging in a few more places, especia... (Closed)

Created:
10 years, 4 months ago by brettw
Modified:
9 years, 7 months ago
Reviewers:
viettrungluu
CC:
chromium-reviews, John Grabowski, jam, pam+watch_chromium.org, darin-cc_chromium.org
Visibility:
Public.

Description

Enhance plugin logging a bit. This adds logging in a few more places, especially for Windows. This also changes the log level to ERROR so that when you opt in to plugin logging, you don't also have to set the log level to be INFO and put up with a low of log spew. TEST=none BUG=none Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=54008

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+42 lines, -29 lines) Patch
M webkit/glue/plugins/plugin_lib.cc View 5 chunks +11 lines, -9 lines 0 comments Download
M webkit/glue/plugins/plugin_lib_posix.cc View 4 chunks +6 lines, -6 lines 0 comments Download
M webkit/glue/plugins/plugin_lib_win.cc View 3 chunks +9 lines, -4 lines 0 comments Download
M webkit/glue/plugins/plugin_list.cc View 2 chunks +7 lines, -1 line 0 comments Download
M webkit/glue/plugins/plugin_list_posix.cc View 4 chunks +9 lines, -9 lines 0 comments Download

Messages

Total messages: 2 (0 generated)
brettw
10 years, 4 months ago (2010-07-28 17:23:15 UTC) #1
viettrungluu
10 years, 4 months ago (2010-07-28 18:13:01 UTC) #2
LGTM.

Powered by Google App Engine
This is Rietveld 408576698