Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(363)

Issues Search
    My Issues | Starred     Open | Closed | All

Unified Diff: grit/gather/chrome_html_unittest.py

Issue 10386189: Add chrome_html gatherer, which inlines html and automatically generates image set… (Closed) Base URL: http://git.chromium.org/external/grit-i18n.git@master
Patch Set: chrome_html comment. Created 8 years, 7 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« grit/gather/chrome_html.py ('K') | « grit/gather/chrome_html.py ('k') | grit/gather/interface.py » ('j') | grit/node/structure.py » ('J')
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: grit/gather/chrome_html_unittest.py
diff --git a/grit/gather/chrome_html_unittest.py b/grit/gather/chrome_html_unittest.py
new file mode 100644
index 0000000000000000000000000000000000000000..399180f86bdf0c784219caeda471f8c0db56f199
--- /dev/null
+++ b/grit/gather/chrome_html_unittest.py
@@ -0,0 +1,196 @@
+#!/usr/bin/env python
+# Copyright (c) 2012 The Chromium Authors. All rights reserved.
+# Use of this source code is governed by a BSD-style license that can be
+# found in the LICENSE file.
+
+'''Unit tests for grit.gather.chrome_html'''
+
+
+import os
+import sys
+if __name__ == '__main__':
+ sys.path.append(os.path.join(os.path.dirname(sys.argv[0]), '../..'))
+
+import tempfile
+import unittest
+
+from grit.gather import chrome_html
+
+
+class TempDir(object):
+ def __init__(self, file_data):
+ self.files = []
+ self.dirs = []
+ self.tmp_dir_name = tempfile.gettempdir()
+ for name in file_data:
+ file_path = self.tmp_dir_name + '/' + name
+ dir_path = file_path.rsplit('/', 1)[0]
Jói 2012/05/23 10:22:46 I think you want os.path.split
flackr 2012/05/23 19:34:14 Done.
+ if not os.path.exists(dir_path):
+ self.dirs.append(dir_path)
+ os.makedirs(dir_path)
+ self.files.append(file_path)
+ f = open(file_path, 'w')
+ f.write(file_data[name])
+ f.close()
+
+ def CleanUp(self):
+ self.dirs.reverse()
+ for name in self.files:
+ os.unlink(name)
+ for name in self.dirs:
+ os.removedirs(name)
Jói 2012/05/23 10:22:46 This could fail depending on the order of director
flackr 2012/05/23 19:34:14 That should be fine. I just didn't want it to leav
+
+ def GetPath(self, name):
+ return self.tmp_dir_name + '/' + name
+
+
+class ChromeHtmlUnittest(unittest.TestCase):
+ '''Unit tests for ChromeHtml.'''
+
+ def testFileResources(self):
+ '''Tests inlined image file resources with available high DPI assets.'''
+
+ tmp_dir = TempDir({
+ 'index.html': '''
+ <!DOCTYPE HTML>
+ <html>
+ <head>
+ <link rel="stylesheet" href="test.css">
+ </head>
+ <body>
+ <!-- Don't need a body. -->
+ </body>
+ </html>
+ ''',
+
+ 'test.css': '''
+ .image {
+ background: url('test.png');
+ }
+ ''',
+
+ 'test.png': '''
+ PNG DATA
+ ''',
+
+ '1.4x/test.png': '''
+ 1.4x PNG DATA
+ ''',
+
+ '1.8x/test.png': '''
+ 1.8x PNG DATA
+ ''',
+ })
+
+ html = chrome_html.ChromeHtml(tmp_dir.GetPath('index.html'))
+ html.SetDefines({'scale_factors': '1.4x,1.8x'})
+ html.Parse()
+ self.failUnlessEqual(html.GetData('en', 'utf-8'), '''
+ <!DOCTYPE HTML>
+ <html>
+ <head>
+ <style>
+ .image {
+ background: -webkit-image-set(url("data:image/png;base64,CiAgICAgICAgUE5HIERBVEEKICAgICAg") 1x, url("data:image/png;base64,CiAgICAgICAgMS40eCBQTkcgREFUQQogICAgICA=") 1.4x, url("data:image/png;base64,CiAgICAgICAgMS44eCBQTkcgREFUQQogICAgICA=") 1.8x);
+ }
+ </style>
+ </head>
+ <body>
+ <!-- Don't need a body. -->
+ </body>
+ </html>
+ ''')
+ tmp_dir.CleanUp()
+
+ def testFileResourcesNoFile(self):
+ '''Tests inlined image file resources without available high DPI assets.'''
+
+ tmp_dir = TempDir({
+ 'index.html': '''
+ <!DOCTYPE HTML>
+ <html>
+ <head>
+ <link rel="stylesheet" href="test.css">
+ </head>
+ <body>
+ <!-- Don't need a body. -->
+ </body>
+ </html>
+ ''',
+
+ 'test.css': '''
+ .image {
+ background: url('test.png');
+ }
+ ''',
+
+ 'test.png': '''
+ PNG DATA
+ ''',
+ })
+
+ html = chrome_html.ChromeHtml(tmp_dir.GetPath('index.html'))
+ html.SetDefines({'scale_factors': '2x'})
+ html.Parse()
+ self.failUnlessEqual(html.GetData('en', 'utf-8'), '''
+ <!DOCTYPE HTML>
+ <html>
+ <head>
+ <style>
+ .image {
+ background: -webkit-image-set(url("data:image/png;base64,CiAgICAgICAgUE5HIERBVEEKICAgICAg") 1x);
+ }
+ </style>
+ </head>
+ <body>
+ <!-- Don't need a body. -->
+ </body>
+ </html>
+ ''')
+ tmp_dir.CleanUp()
+
+ def testThemeResources(self):
+ '''Tests inserting high DPI chrome://theme references.'''
+
+ tmp_dir = TempDir({
+ 'index.html': '''
+ <!DOCTYPE HTML>
+ <html>
+ <head>
+ <link rel="stylesheet" href="test.css">
+ </head>
+ <body>
+ <!-- Don't need a body. -->
+ </body>
+ </html>
+ ''',
+
+ 'test.css': '''
+ .image {
+ background: url('chrome://theme/IDR_RESOURCE_NAME');
+ }
+ ''',
+ })
+
+ html = chrome_html.ChromeHtml(tmp_dir.GetPath('index.html'))
+ html.SetDefines({'scale_factors': '2x'})
+ html.Parse()
+ self.failUnlessEqual(html.GetData('en', 'utf-8'), '''
+ <!DOCTYPE HTML>
+ <html>
+ <head>
+ <style>
+ .image {
+ background: -webkit-image-set(url("chrome://theme/IDR_RESOURCE_NAME") 1x, url("chrome://theme/IDR_RESOURCE_NAME@2x") 2x);
+ }
+ </style>
+ </head>
+ <body>
+ <!-- Don't need a body. -->
+ </body>
+ </html>
+ ''')
+ tmp_dir.CleanUp()
+
Jói 2012/05/23 10:22:46 two blank lines between top-level elements
flackr 2012/05/23 19:34:14 Done.
+if __name__ == '__main__':
+ unittest.main()
« grit/gather/chrome_html.py ('K') | « grit/gather/chrome_html.py ('k') | grit/gather/interface.py » ('j') | grit/node/structure.py » ('J')

Powered by Google App Engine
This is Rietveld 408576698