Unhide 'window.console' by renaming console variable in layout tests

Unhide 'window.console' by renaming console variable in layout tests

There is a 'console' variable in layout tests which is hiding 'window.console',
so 'window.console' is not working in layout tests.
Renamed it from 'console' to 'consoleDiv' so that 'window.console'
can be used for debugging purpose.

Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=198132

[Srirama, 2015-06-23 14:01:12]

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

[Srirama, 2015-06-23 14:01:12]

Came across this while working on some of the failed layout test cases. Though
it doesn't add any value, it will be easy for debugging layout tests with
window.console.
Please check if it is ok to land this change.

[philipj_slow, 2015-07-01 08:32:25]

LGTM with nit. Also remove BUG=none from description, I've been told.

https://codereview.chromium.org/1203693002/diff/1/LayoutTests/media/video-tes...
File LayoutTests/media/video-test.js (right):

https://codereview.chromium.org/1203693002/diff/1/LayoutTests/media/video-tes...
LayoutTests/media/video-test.js:4: var consoleLog = null;
I'd call it consoleDiv or consoleElement to avoid any confusion with
console.log() or the impression that this is a method.

[Srirama, 2015-07-01 09:48:38]

Thanks for the review

https://codereview.chromium.org/1203693002/diff/1/LayoutTests/media/video-tes...
File LayoutTests/media/video-test.js (right):

https://codereview.chromium.org/1203693002/diff/1/LayoutTests/media/video-tes...
LayoutTests/media/video-test.js:4: var consoleLog = null;
On 2015/07/01 08:32:25, philipj wrote:
> I'd call it consoleDiv or consoleElement to avoid any confusion with
> console.log() or the impression that this is a method.

Done.

[Srirama, 2015-07-01 09:48:45]

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

[Srirama, 2015-07-01 09:48:46]

The patchset sent to the CQ was uploaded after l-g-t-m from philipj@opera.com
Link to the patchset: https://codereview.chromium.org/1203693002/#ps20001
(title: "addressed nits")

[commit-bot: I haz the power, 2015-07-01 09:49:04]

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

[commit-bot: I haz the power, 2015-07-01 10:52:12]

Committed patchset #2 (id:20001) as
https://src.chromium.org/viewvc/blink?view=rev&revision=198132