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

Issues Search
    My Issues | Starred     Open | Closed | All

Issue 12379062: Remove Debug restriction on Bluetooth log messages (Closed)

Created:
7 years, 9 months ago by keybuk
Modified:
7 years, 9 months ago
CC:
chromium-reviews, dbeam+watch-options_chromium.org, stevenjb+watch_chromium.org, oshima+watch_chromium.org
Visibility:
Public.

Description

Remove Debug restriction on Bluetooth log messages We frequently need to access these in real builds after all. BUG=none TEST=chrome --vmodule=*bluetooth*=3 Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=187064

Patch Set 1 #

Patch Set 2 : Rebase #

Patch Set 3 : Rebase again (base files missing?) #

Unified diffs Side-by-side diffs Delta from patch set Stats (+65 lines, -74 lines) Patch
M chrome/browser/ui/webui/options/chromeos/bluetooth_options_handler.cc View 10 chunks +14 lines, -14 lines 0 comments Download
M chromeos/dbus/bluetooth_adapter_client.cc View 5 chunks +6 lines, -8 lines 0 comments Download
M chromeos/dbus/bluetooth_agent_service_provider.cc View 1 chunk +2 lines, -2 lines 0 comments Download
M chromeos/dbus/bluetooth_device_client.cc View 4 chunks +6 lines, -6 lines 0 comments Download
M chromeos/dbus/bluetooth_input_client.cc View 1 chunk +0 lines, -2 lines 0 comments Download
M chromeos/dbus/bluetooth_manager_client.cc View 4 chunks +3 lines, -5 lines 0 comments Download
M chromeos/dbus/bluetooth_node_client.cc View 1 chunk +0 lines, -2 lines 0 comments Download
M chromeos/dbus/introspectable_client.cc View 1 chunk +0 lines, -1 line 0 comments Download
M device/bluetooth/bluetooth_adapter_chromeos.cc View 11 chunks +18 lines, -18 lines 0 comments Download
M device/bluetooth/bluetooth_device_chromeos.cc View 12 chunks +16 lines, -16 lines 0 comments Download

Messages

Total messages: 10 (0 generated)
keybuk
7 years, 9 months ago (2013-03-02 00:21:43 UTC) #1
stevenjb
lgtm
7 years, 9 months ago (2013-03-02 02:44:32 UTC) #2
satorux1
LGTM
7 years, 9 months ago (2013-03-04 01:07:10 UTC) #3
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/keybuk@chromium.org/12379062/1
7 years, 9 months ago (2013-03-07 17:20:59 UTC) #4
commit-bot: I haz the power
Presubmit check for 12379062-1 failed and returned exit status 1. INFO:root:Found 10 file(s). Running presubmit ...
7 years, 9 months ago (2013-03-07 17:21:08 UTC) #5
keybuk
nkostylev for chrome/browser/ui/webui/options/chromeos/OWNERS
7 years, 9 months ago (2013-03-07 17:30:00 UTC) #6
keybuk
dpolukhin for chrome/browser/ui/webui/options/chromeos/OWNERS
7 years, 9 months ago (2013-03-08 18:46:58 UTC) #7
Dmitry Polukhin
LGTM for chrome/browser/ui/webui/options/chromeos/OWNERS But please rebase and run trybots before commit.
7 years, 9 months ago (2013-03-08 19:54:53 UTC) #8
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/keybuk@chromium.org/12379062/13001
7 years, 9 months ago (2013-03-08 20:50:25 UTC) #9
commit-bot: I haz the power
7 years, 9 months ago (2013-03-08 23:00:12 UTC) #10
Message was sent while issue was closed. 
Change committed as 187064

Powered by Google App Engine
This is Rietveld 408576698