printing: Use base::MessageLoop.

printing: Use base::MessageLoop.

BUG=236029
R=abodenha@chromium.org

Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=198551

[xhwang, 2013-04-30 17:32:45]

ping

[xhwang, 2013-05-04 05:39:19]

thestig@: please OWNERS review. Thanks!

[Lei Zhang, 2013-05-06 04:05:55]

lgtm

https://codereview.chromium.org/14113054/diff/1/printing/printed_document.cc
File printing/printed_document.cc (right):

https://codereview.chromium.org/14113054/diff/1/printing/printed_document.cc#...
printing/printed_document.cc:222: cookie_(cookie) {}
The rest of the file doesn't use '{}' -> leave it alone?

https://codereview.chromium.org/14113054/diff/1/printing/printing_context_win.cc
File printing/printing_context_win.cc (right):

https://codereview.chromium.org/14113054/diff/1/printing/printing_context_win...
printing/printing_context_win.cc:481: DCHECK(!base::MessageLoop::current()
How about this?

DCHECK(!base::MessageLoop::current() ||
       !base::MessageLoop::current()->NestableTasksAllowed())'

[xhwang, 2013-05-06 16:07:19]

https://codereview.chromium.org/14113054/diff/1/printing/printed_document.cc
File printing/printed_document.cc (right):

https://codereview.chromium.org/14113054/diff/1/printing/printed_document.cc#...
printing/printed_document.cc:222: cookie_(cookie) {}
On 2013/05/06 04:05:55, Lei Zhang wrote:
> The rest of the file doesn't use '{}' -> leave it alone?

Done.

https://codereview.chromium.org/14113054/diff/1/printing/printing_context_win.cc
File printing/printing_context_win.cc (right):

https://codereview.chromium.org/14113054/diff/1/printing/printing_context_win...
printing/printing_context_win.cc:481: DCHECK(!base::MessageLoop::current()
On 2013/05/06 04:05:55, Lei Zhang wrote:
> How about this?
> 
> DCHECK(!base::MessageLoop::current() ||
>        !base::MessageLoop::current()->NestableTasksAllowed())'

Done.

[commit-bot: I haz the power, 2013-05-06 16:07:53]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/xhwang@chromium.org/14113054/17001

[commit-bot: I haz the power, 2013-05-06 22:43:58]

Change committed as 198551