[chrome/browser/s*] Convert VLOGs to DVLOGs

[chrome/browser/s*] Convert VLOGs to DVLOGs

Note: s* includes -->(safe_browsing, service_process, sessions, signin, ssl, supervised_user, sync modules)

BUG=101424
Committed: https://crrev.com/14caff1993c280bd5979706f50d0e454ba3b9772
Cr-Commit-Position: refs/heads/master@{#302962}

[anujsharma, 2014-11-03 12:59:18]

anujk.sharma@samsung.com changed reviewers:
+ jochen@chromium.org, sky@chromium.org

[anujsharma, 2014-11-03 12:59:18]

PTAL

[anujsharma, 2014-11-04 10:28:55]

anujk.sharma@samsung.com changed reviewers:
+ bauerb@chromium.org, blundell@chromium.org, maniscalco@chromium.org,
mattm@chromium.org, palmer@chromium.org, vitalybuka@chromium.org
- jochen@chromium.org

[anujsharma, 2014-11-04 10:32:11]

On 2014/11/03 12:59:18, anujsharma wrote:
> PTAL

@blundell- PTAL in signin module
@sky- PTAL in sessions module
@maniscalco- PTAL in test module
@vitalybuka- PTAL in service_process module
@palmer- PTAL in ssl module
@mattm -PTAL in safe-browsing module
@bauerb- PTAL in supervised_user module

[Bernhard Bauer, 2014-11-04 12:02:56]

bauerb@chromium.org changed reviewers:
+ antrim@chromium.org

[Bernhard Bauer, 2014-11-04 12:02:57]

+antrim for Chrome OS supervised users.

I was under the impression that Chrome OS generally only builds in Release mode,
so DVLOGs wouldn't get logged.

[blundell, 2014-11-04 15:06:00]

blundell@chromium.org changed reviewers:
+ rogerta@chromium.org

[blundell, 2014-11-04 15:06:01]

blundell -> rogerta

[Roger Tawa OOO till Jul 10th, 2014-11-04 16:10:05]

lgtm chrome/browser/signin

[Vitaly Buka (NO REVIEWS), 2014-11-04 16:46:25]

lgtm

[palmer, 2014-11-04 19:44:23]

Rubberstamp LGTM

[mattm, 2014-11-04 22:59:31]

lgtm

[anujsharma, 2014-11-05 14:37:46]

On 2014/11/04 10:32:11, anujsharma wrote:
> On 2014/11/03 12:59:18, anujsharma wrote:
> > PTAL
> 
> @blundell- PTAL in signin module
> @sky- PTAL in sessions module
> @maniscalco- PTAL in test module
> @vitalybuka- PTAL in service_process module
> @palmer- PTAL in ssl module
> @mattm -PTAL in safe-browsing module
> @bauerb- PTAL in supervised_user module

Thanks Roger, Vitaly, Chromium Palmer and matt for lgtm.

@sky,antrim,maniscalco - could you also give thumbs up for this CL.
Thanks!!

[sky, 2014-11-05 15:42:27]

What specific files do you need me to look at? In general prefer local owners.
Is there not a more local owner for the files you're going to ask me to review?

[anujsharma, 2014-11-05 17:39:42]

On 2014/11/05 15:42:27, sky wrote:
> What specific files do you need me to look at? In general prefer local owners.
> Is there not a more local owner for the files you're going to ask me to
review?

Sorry sky for the inconvenience caused. I want to you
src/chrome/browser/sessions module. Actually for this module, only you and Marja
are the OWNERS, thats why i send review request to you. Sorry if i am wrong
somewhere. 
Thanks!!

[sky, 2014-11-05 20:03:41]

sessions LGTM

[anujsharma, 2014-11-06 02:53:18]

On 2014/11/05 20:03:41, sky wrote:
> sessions LGTM

Thanks sky for lgtm.

[anujsharma, 2014-11-06 02:54:59]

On 2014/11/04 12:02:57, Bernhard Bauer wrote:
> +antrim for Chrome OS supervised users.
> 
> I was under the impression that Chrome OS generally only builds in Release
mode,
> so DVLOGs wouldn't get logged.

@Bauer, antrim,
I have removed Chrome OS supervised users from this CL.
Thanks for your time.

[anujsharma, 2014-11-06 02:55:10]

The CQ bit was checked by anujk.sharma@samsung.com

[commit-bot: I haz the power, 2014-11-06 02:56:09]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/688993004/40001

[commit-bot: I haz the power, 2014-11-06 04:28:45]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2014-11-06 04:29:21]

Patchset 3 (id:??) landed as
https://crrev.com/14caff1993c280bd5979706f50d0e454ba3b9772
Cr-Commit-Position: refs/heads/master@{#302962}