 Chromium Code Reviews
 Chromium Code Reviews Issue 1998553002:
  media: Replace wtf/Assertions.h macros in favor of base/logging.h macros   (Closed) 
  Base URL: https://chromium.googlesource.com/chromium/src.git@master
    
  
    Issue 1998553002:
  media: Replace wtf/Assertions.h macros in favor of base/logging.h macros   (Closed) 
  Base URL: https://chromium.googlesource.com/chromium/src.git@master| Index: third_party/WebKit/Source/core/html/HTMLTrackElement.cpp | 
| diff --git a/third_party/WebKit/Source/core/html/HTMLTrackElement.cpp b/third_party/WebKit/Source/core/html/HTMLTrackElement.cpp | 
| index e51919a8b1b8a36f673e34231a816f0fea93954d..e7879309870d9bb324c4c444fab698f153ceebc5 100644 | 
| --- a/third_party/WebKit/Source/core/html/HTMLTrackElement.cpp | 
| +++ b/third_party/WebKit/Source/core/html/HTMLTrackElement.cpp | 
| @@ -34,6 +34,8 @@ | 
| #include "core/html/track/LoadableTextTrack.h" | 
| #include "platform/Logging.h" | 
| +#define TRACK_LOG_LEVEL 3 | 
| + | 
| namespace blink { | 
| using namespace HTMLNames; | 
| @@ -53,7 +55,7 @@ inline HTMLTrackElement::HTMLTrackElement(Document& document) | 
| : HTMLElement(trackTag, document) | 
| , m_loadTimer(this, &HTMLTrackElement::loadTimerFired) | 
| { | 
| - WTF_LOG(Media, "HTMLTrackElement::HTMLTrackElement - %p", this); | 
| + DVLOG(TRACK_LOG_LEVEL) << "HTMLTrackElement - ", (void*)this; | 
| 
fs
2016/05/19 11:05:36
<< (not ,)
 
Srirama
2016/05/19 12:14:53
Done.
 | 
| } | 
| DEFINE_NODE_FACTORY(HTMLTrackElement) | 
| @@ -64,7 +66,7 @@ HTMLTrackElement::~HTMLTrackElement() | 
| Node::InsertionNotificationRequest HTMLTrackElement::insertedInto(ContainerNode* insertionPoint) | 
| { | 
| - WTF_LOG(Media, "HTMLTrackElement::insertedInto"); | 
| + DVLOG(TRACK_LOG_LEVEL) << "insertedInto"; | 
| // Since we've moved to a new parent, we may now be able to load. | 
| scheduleLoad(); | 
| @@ -137,7 +139,7 @@ bool HTMLTrackElement::isURLAttribute(const Attribute& attribute) const | 
| void HTMLTrackElement::scheduleLoad() | 
| { | 
| - WTF_LOG(Media, "HTMLTrackElement::scheduleLoad"); | 
| + DVLOG(TRACK_LOG_LEVEL) << "scheduleLoad"; | 
| // 1. If another occurrence of this algorithm is already running for this text track and its track element, | 
| // abort these steps, letting that other algorithm take care of this element. | 
| @@ -162,7 +164,7 @@ void HTMLTrackElement::scheduleLoad() | 
| void HTMLTrackElement::loadTimerFired(Timer<HTMLTrackElement>*) | 
| { | 
| - WTF_LOG(Media, "HTMLTrackElement::loadTimerFired"); | 
| + DVLOG(TRACK_LOG_LEVEL) << "loadTimerFired"; | 
| // 6. [X] Set the text track readiness state to loading. | 
| setReadyState(LOADING); | 
| @@ -223,7 +225,7 @@ bool HTMLTrackElement::canLoadUrl(const KURL& url) | 
| return false; | 
| if (!document().contentSecurityPolicy()->allowMediaFromSource(url)) { | 
| - WTF_LOG(Media, "HTMLTrackElement::canLoadUrl(%s) -> rejected by Content Security Policy", urlForLoggingTrack(url).utf8().data()); | 
| + DVLOG(TRACK_LOG_LEVEL) << "canLoadUrl(" << urlForLoggingTrack(url) << ") -> rejected by Content Security Policy"; | 
| return false; | 
| } |