media: Replace wtf/Assertions.h macros in favor of base/logging.h macros

media: Replace wtf/Assertions.h macros in favor of base/logging.h macros

Replace WTF_LOG macros with DVLOG in AutoplayExperimentHelper.cpp,
HTMLTrackElement.cpp, TextTrackLoader.cpp and VTTRegion.cpp.

BUG=596522
Committed: https://crrev.com/4c0107731d76ac0f1c0c8e4406eab568157eb146
Cr-Commit-Position: refs/heads/master@{#394762}

[Srirama, 2016-05-19 08:05:55]

Description was changed from

==========
fix


fix

BUG=
==========

to

==========
media: Replace wtf/Assertions.h macros in favor of base/logging.h macros 


Replace WTF_LOG macros with DVLOG in AutoplayExperimentHelper.cpp,
HTMLMediaElementEncryptedMedia.cpp, HTMLTrackElement.cpp,
TextTrackLoader.cpp and VTTRegion.cpp

BUG=596522
==========

[Srirama, 2016-05-19 08:09:47]

srirama.m@samsung.com changed reviewers:
+ fs@opera.com

[Srirama, 2016-05-19 08:09:48]

PTAL.
Cleaned up few more media files.

[fs, 2016-05-19 11:05:36]

https://codereview.chromium.org/1998553002/diff/1/third_party/WebKit/Source/c...
File third_party/WebKit/Source/core/html/AutoplayExperimentHelper.cpp (right):

https://codereview.chromium.org/1998553002/diff/1/third_party/WebKit/Source/c...
third_party/WebKit/Source/core/html/AutoplayExperimentHelper.cpp:43: DVLOG(3) <<
"autoplay experiment set to " << m_mode;
DVLOG_IF(3, m_mode != Mode::ExperimentOff) << ...

https://codereview.chromium.org/1998553002/diff/1/third_party/WebKit/Source/c...
File third_party/WebKit/Source/core/html/HTMLTrackElement.cpp (right):

https://codereview.chromium.org/1998553002/diff/1/third_party/WebKit/Source/c...
third_party/WebKit/Source/core/html/HTMLTrackElement.cpp:58:
DVLOG(TRACK_LOG_LEVEL) << "HTMLTrackElement - ", (void*)this;
<< (not ,)

https://codereview.chromium.org/1998553002/diff/1/third_party/WebKit/Source/c...
File third_party/WebKit/Source/core/loader/TextTrackLoader.cpp (right):

https://codereview.chromium.org/1998553002/diff/1/third_party/WebKit/Source/c...
third_party/WebKit/Source/core/loader/TextTrackLoader.cpp:141: DVLOG(3) <<
"fileFailedToParse";
Just drop this log-statement.

https://codereview.chromium.org/1998553002/diff/1/third_party/WebKit/Source/m...
File
third_party/WebKit/Source/modules/encryptedmedia/HTMLMediaElementEncryptedMedia.cpp
(right):

https://codereview.chromium.org/1998553002/diff/1/third_party/WebKit/Source/m...
third_party/WebKit/Source/modules/encryptedmedia/HTMLMediaElementEncryptedMedia.cpp:24:
#define EME_LOG_LEVEL 3
you'll need to add additional reviewers for this file. Maybe split it out into a
separate CL.

[Srirama, 2016-05-19 12:14:54]

https://codereview.chromium.org/1998553002/diff/1/third_party/WebKit/Source/c...
File third_party/WebKit/Source/core/html/AutoplayExperimentHelper.cpp (right):

https://codereview.chromium.org/1998553002/diff/1/third_party/WebKit/Source/c...
third_party/WebKit/Source/core/html/AutoplayExperimentHelper.cpp:43: DVLOG(3) <<
"autoplay experiment set to " << m_mode;
On 2016/05/19 11:05:36, fs wrote:
> DVLOG_IF(3, m_mode != Mode::ExperimentOff) << ...

Done.

https://codereview.chromium.org/1998553002/diff/1/third_party/WebKit/Source/c...
File third_party/WebKit/Source/core/html/HTMLTrackElement.cpp (right):

https://codereview.chromium.org/1998553002/diff/1/third_party/WebKit/Source/c...
third_party/WebKit/Source/core/html/HTMLTrackElement.cpp:58:
DVLOG(TRACK_LOG_LEVEL) << "HTMLTrackElement - ", (void*)this;
On 2016/05/19 11:05:36, fs wrote:
> << (not ,)

Done.

https://codereview.chromium.org/1998553002/diff/1/third_party/WebKit/Source/c...
File third_party/WebKit/Source/core/loader/TextTrackLoader.cpp (right):

https://codereview.chromium.org/1998553002/diff/1/third_party/WebKit/Source/c...
third_party/WebKit/Source/core/loader/TextTrackLoader.cpp:141: DVLOG(3) <<
"fileFailedToParse";
On 2016/05/19 11:05:36, fs wrote:
> Just drop this log-statement.

Done.

https://codereview.chromium.org/1998553002/diff/1/third_party/WebKit/Source/m...
File
third_party/WebKit/Source/modules/encryptedmedia/HTMLMediaElementEncryptedMedia.cpp
(right):

https://codereview.chromium.org/1998553002/diff/1/third_party/WebKit/Source/m...
third_party/WebKit/Source/modules/encryptedmedia/HTMLMediaElementEncryptedMedia.cpp:24:
#define EME_LOG_LEVEL 3
On 2016/05/19 11:05:36, fs wrote:
> you'll need to add additional reviewers for this file. Maybe split it out into
a
> separate CL.

Done.

[fs, 2016-05-19 12:18:08]

lgtm

[Srirama, 2016-05-19 12:26:47]

The CQ bit was checked by srirama.m@samsung.com

[Srirama, 2016-05-19 12:26:47]

The CQ bit was unchecked by srirama.m@samsung.com

[Srirama, 2016-05-19 12:27:22]

Description was changed from

==========
media: Replace wtf/Assertions.h macros in favor of base/logging.h macros 


Replace WTF_LOG macros with DVLOG in AutoplayExperimentHelper.cpp,
HTMLMediaElementEncryptedMedia.cpp, HTMLTrackElement.cpp,
TextTrackLoader.cpp and VTTRegion.cpp

BUG=596522
==========

to

==========
media: Replace wtf/Assertions.h macros in favor of base/logging.h macros 

Replace WTF_LOG macros with DVLOG in AutoplayExperimentHelper.cpp,
HTMLTrackElement.cpp, TextTrackLoader.cpp and VTTRegion.cpp.

BUG=596522
==========

[Srirama, 2016-05-19 12:27:30]

The CQ bit was checked by srirama.m@samsung.com

[commit-bot: I haz the power, 2016-05-19 12:27:46]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1998553002/20001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1998553002/20001

[commit-bot: I haz the power, 2016-05-19 12:29:20]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-05-19 12:29:21]

Try jobs failed on following builders:
  ios-device on tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios-device/builds...)
  ios-device-gn on tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios-device-gn/bui...)
  ios-simulator on tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios-simulator/bui...)
  ios-simulator-gn on tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios-simulator-gn/...)
  mac_chromium_gn_rel on tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_gn_r...)
  mac_chromium_rel_ng on tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_rel_...)

[Srirama, 2016-05-19 12:40:51]

The CQ bit was checked by srirama.m@samsung.com

[Srirama, 2016-05-19 12:40:51]

The patchset sent to the CQ was uploaded after l-g-t-m from fs@opera.com
Link to the patchset: https://codereview.chromium.org/1998553002/#ps40001
(title: "Rebase")

[commit-bot: I haz the power, 2016-05-19 12:41:02]

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

[commit-bot: I haz the power, 2016-05-19 14:09:42]

Description was changed from

==========
media: Replace wtf/Assertions.h macros in favor of base/logging.h macros 

Replace WTF_LOG macros with DVLOG in AutoplayExperimentHelper.cpp,
HTMLTrackElement.cpp, TextTrackLoader.cpp and VTTRegion.cpp.

BUG=596522
==========

to

==========
media: Replace wtf/Assertions.h macros in favor of base/logging.h macros 

Replace WTF_LOG macros with DVLOG in AutoplayExperimentHelper.cpp,
HTMLTrackElement.cpp, TextTrackLoader.cpp and VTTRegion.cpp.

BUG=596522
==========

[commit-bot: I haz the power, 2016-05-19 14:09:43]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2016-05-19 14:10:54]

Description was changed from

==========
media: Replace wtf/Assertions.h macros in favor of base/logging.h macros 

Replace WTF_LOG macros with DVLOG in AutoplayExperimentHelper.cpp,
HTMLTrackElement.cpp, TextTrackLoader.cpp and VTTRegion.cpp.

BUG=596522
==========

to

==========
media: Replace wtf/Assertions.h macros in favor of base/logging.h macros

Replace WTF_LOG macros with DVLOG in AutoplayExperimentHelper.cpp,
HTMLTrackElement.cpp, TextTrackLoader.cpp and VTTRegion.cpp.

BUG=596522

Committed: https://crrev.com/4c0107731d76ac0f1c0c8e4406eab568157eb146
Cr-Commit-Position: refs/heads/master@{#394762}
==========

[commit-bot: I haz the power, 2016-05-19 14:10:56]

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