[DRT] LTC:: Move printing related APIs from LayoutTestController to Internals
authorkaustubh@motorola.com <kaustubh@motorola.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Wed, 5 Sep 2012 11:36:03 +0000 (11:36 +0000)
committerkaustubh@motorola.com <kaustubh@motorola.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Wed, 5 Sep 2012 11:36:03 +0000 (11:36 +0000)
https://bugs.webkit.org/show_bug.cgi?id=92735

Reviewed by Hajime Morita.

.:

Move numberOfPages, pageProperty & pageSizeAndMarginsInPixels in Internals and remove duplicated code from DumprenderTree & WebkitTestRunner.
Added symbols for GTK builds for corresponding APIs.

* Source/autotools/symbols.filter:

Source/WebCore:

Move numberOfPages, pageProperty & pageSizeAndMarginsInPixels in Internals and remove duplicated code from DumprenderTree & WebkitTestRunner.

Covered by existing test cases.

* testing/Internals.cpp:
(WebCore::Internals::numberOfPages):
(WebCore):
(WebCore::Internals::pageProperty):
(WebCore::Internals::pageSizeAndMarginsInPixels):
* testing/Internals.h:
(Internals):
* testing/Internals.idl:

Source/WebKit/efl:

Move numberOfPages, pageProperty & pageSizeAndMarginsInPixels in Internals and remove duplicated code from DumprenderTree & WebkitTestRunner.

* WebCoreSupport/DumpRenderTreeSupportEfl.cpp:
* WebCoreSupport/DumpRenderTreeSupportEfl.h:

Source/WebKit/gtk:

Move numberOfPages, pageProperty & pageSizeAndMarginsInPixels in Internals and remove duplicated code from DumprenderTree & WebkitTestRunner.

* WebCoreSupport/DumpRenderTreeSupportGtk.cpp:
* WebCoreSupport/DumpRenderTreeSupportGtk.h:
(DumpRenderTreeSupportGtk):

Source/WebKit/qt:

Move numberOfPages, pageProperty & pageSizeAndMarginsInPixels in Internals and remove duplicated code from DumprenderTree & WebkitTestRunner.

* WebCoreSupport/DumpRenderTreeSupportQt.cpp:
(DumpRenderTreeSupportQt::evaluateScriptInIsolatedWorld):
* WebCoreSupport/DumpRenderTreeSupportQt.h:

Source/WebKit/win:

Move numberOfPages, pageProperty & pageSizeAndMarginsInPixels in Internals and remove duplicated code from DumprenderTree & WebkitTestRunner.
Kept the function defination and declaration to keep binary compatibility for IWebFramePrivate.idl

* WebFrame.cpp:
(WebFrame::numberOfPages):

Source/WebKit2:

Move numberOfPages, pageProperty & pageSizeAndMarginsInPixels in Internals and remove duplicated code from DumprenderTree & WebkitTestRunner.
Add symbols for Mac Win builds for corresponding newly added APIs.

* win/WebKit2.def:
* win/WebKit2CFLite.def:

Tools:

Move numberOfPages, pageProperty & pageSizeAndMarginsInPixels in Internals and remove duplicated code from DumprenderTree & WebkitTestRunner.

* DumpRenderTree/TestRunner.cpp:
(TestRunner::staticFunctions):
* DumpRenderTree/TestRunner.h:
(TestRunner):
* DumpRenderTree/blackberry/TestRunnerBlackBerry.cpp:
(TestRunner::addUserStyleSheet):
* DumpRenderTree/chromium/DRTTestRunner.cpp:
(DRTTestRunner::DRTTestRunner):
(DRTTestRunner::hasCustomPageSizeStyle):
* DumpRenderTree/chromium/DRTTestRunner.h:
(DRTTestRunner):
* DumpRenderTree/efl/TestRunnerEfl.cpp:
* DumpRenderTree/gtk/TestRunnerGtk.cpp:
* DumpRenderTree/mac/TestRunnerMac.mm:
* DumpRenderTree/qt/TestRunnerQt.cpp:
(TestRunner::evaluateScriptInIsolatedWorld):
* DumpRenderTree/qt/TestRunnerQt.h:
(TestRunner):
* DumpRenderTree/win/TestRunnerWin.cpp:
* DumpRenderTree/wx/TestRunnerWx.cpp:
(TestRunner::abortModal):
* WebKitTestRunner/InjectedBundle/Bindings/TestRunner.idl:
* WebKitTestRunner/InjectedBundle/TestRunner.cpp:
(WTR):
* WebKitTestRunner/InjectedBundle/TestRunner.h:
(TestRunner):

LayoutTests:

Move numberOfPages, pageProperty & pageSizeAndMarginsInPixels in Internals and remove duplicated code from DumprenderTree & WebkitTestRunner.
Fixed test cases to use internals API instead of testRunner.

* printing/numberOfPages-expected.txt:
* printing/page-count-layout-overflow.html:
* printing/page-count-relayout-shrink.html:
* printing/page-count-with-one-word.html:
* printing/page-format-data-display-none.html:
* printing/page-format-data-expected.txt:
* printing/page-format-data.html:
* printing/page-rule-selection-expected.txt:
* printing/page-rule-selection.html:
* printing/resources/paged-media-test-utils.js:
(numberOfPagesShouldBe):
* printing/script-tests/numberOfPages.js:
* printing/zoomed-document.html:

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@127577 268f45cc-cd09-0410-ab3c-d52691b4dbfc

49 files changed:
ChangeLog
LayoutTests/ChangeLog
LayoutTests/printing/numberOfPages-expected.txt
LayoutTests/printing/page-count-layout-overflow.html
LayoutTests/printing/page-count-relayout-shrink.html
LayoutTests/printing/page-count-with-one-word.html
LayoutTests/printing/page-format-data-display-none.html
LayoutTests/printing/page-format-data-expected.txt
LayoutTests/printing/page-format-data.html
LayoutTests/printing/page-rule-selection-expected.txt
LayoutTests/printing/page-rule-selection.html
LayoutTests/printing/resources/paged-media-test-utils.js
LayoutTests/printing/script-tests/numberOfPages.js
LayoutTests/printing/zoomed-document.html
Source/WebCore/ChangeLog
Source/WebCore/testing/Internals.cpp
Source/WebCore/testing/Internals.h
Source/WebCore/testing/Internals.idl
Source/WebKit/efl/ChangeLog
Source/WebKit/efl/WebCoreSupport/DumpRenderTreeSupportEfl.cpp
Source/WebKit/efl/WebCoreSupport/DumpRenderTreeSupportEfl.h
Source/WebKit/gtk/ChangeLog
Source/WebKit/gtk/WebCoreSupport/DumpRenderTreeSupportGtk.cpp
Source/WebKit/gtk/WebCoreSupport/DumpRenderTreeSupportGtk.h
Source/WebKit/qt/ChangeLog
Source/WebKit/qt/WebCoreSupport/DumpRenderTreeSupportQt.cpp
Source/WebKit/qt/WebCoreSupport/DumpRenderTreeSupportQt.h
Source/WebKit/win/ChangeLog
Source/WebKit/win/WebFrame.cpp
Source/WebKit2/ChangeLog
Source/WebKit2/win/WebKit2.def
Source/WebKit2/win/WebKit2CFLite.def
Source/autotools/symbols.filter
Tools/ChangeLog
Tools/DumpRenderTree/TestRunner.cpp
Tools/DumpRenderTree/TestRunner.h
Tools/DumpRenderTree/blackberry/TestRunnerBlackBerry.cpp
Tools/DumpRenderTree/chromium/DRTTestRunner.cpp
Tools/DumpRenderTree/chromium/DRTTestRunner.h
Tools/DumpRenderTree/efl/TestRunnerEfl.cpp
Tools/DumpRenderTree/gtk/TestRunnerGtk.cpp
Tools/DumpRenderTree/mac/TestRunnerMac.mm
Tools/DumpRenderTree/qt/TestRunnerQt.cpp
Tools/DumpRenderTree/qt/TestRunnerQt.h
Tools/DumpRenderTree/win/TestRunnerWin.cpp
Tools/DumpRenderTree/wx/TestRunnerWx.cpp
Tools/WebKitTestRunner/InjectedBundle/Bindings/TestRunner.idl
Tools/WebKitTestRunner/InjectedBundle/TestRunner.cpp
Tools/WebKitTestRunner/InjectedBundle/TestRunner.h

index 5982b82..acf3820 100644 (file)
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,15 @@
+2012-09-05  Kaustubh Atrawalkar  <kaustubh@motorola.com>
+
+        [DRT] LTC:: Move printing related APIs from LayoutTestController to Internals
+        https://bugs.webkit.org/show_bug.cgi?id=92735
+
+        Reviewed by Hajime Morita.
+
+        Move numberOfPages, pageProperty & pageSizeAndMarginsInPixels in Internals and remove duplicated code from DumprenderTree & WebkitTestRunner.
+        Added symbols for GTK builds for corresponding APIs.
+
+        * Source/autotools/symbols.filter:
+
 2012-09-04  Julien Chaffraix  <jchaffraix@webkit.org>
 
         REGRESSION(r120832): RenderLayer::clampScrollOffset doesn't properly clamp
index e1fab43..74d03f5 100644 (file)
@@ -1,3 +1,27 @@
+2012-09-05  Kaustubh Atrawalkar  <kaustubh@motorola.com>
+
+        [DRT] LTC:: Move printing related APIs from LayoutTestController to Internals
+        https://bugs.webkit.org/show_bug.cgi?id=92735
+
+        Reviewed by Hajime Morita.
+
+        Move numberOfPages, pageProperty & pageSizeAndMarginsInPixels in Internals and remove duplicated code from DumprenderTree & WebkitTestRunner.
+        Fixed test cases to use internals API instead of testRunner.
+
+        * printing/numberOfPages-expected.txt:
+        * printing/page-count-layout-overflow.html:
+        * printing/page-count-relayout-shrink.html:
+        * printing/page-count-with-one-word.html:
+        * printing/page-format-data-display-none.html:
+        * printing/page-format-data-expected.txt:
+        * printing/page-format-data.html:
+        * printing/page-rule-selection-expected.txt:
+        * printing/page-rule-selection.html:
+        * printing/resources/paged-media-test-utils.js:
+        (numberOfPagesShouldBe):
+        * printing/script-tests/numberOfPages.js:
+        * printing/zoomed-document.html:
+
 2012-09-05  Dominik Röttsches  <dominik.rottsches@intel.com>
 
         [EFL] Unreviewed gardening after r127513
index 159bdd7..3a0a0f4 100644 (file)
@@ -1,4 +1,4 @@
-Test for testRunner.numberOfPages()
+Test for internals.numberOfPages()
 
 On success, you will see a series of "PASS" messages, followed by "TEST COMPLETE".
 
index 3a95623..571f7d7 100644 (file)
@@ -21,7 +21,7 @@ function runTest()
     for (var i = 0; i < 7; i++)
         lines = lines.concat(lines);
     document.body.innerHTML = '<p>' + lines.join('') + '</p>';
-    document.body.textContent = 'When printing, layout overflow must be cleared after current document size is queried: ' +  (testRunner.numberOfPages() > 1 ? 'PASS' : 'FAIL');
+    document.body.textContent = 'When printing, layout overflow must be cleared after current document size is queried: ' +  (internals.numberOfPages() > 1 ? 'PASS' : 'FAIL');
 }
 
 </script>
index 89c79f9..76d2cc1 100644 (file)
@@ -17,7 +17,7 @@ function runTest()
         document.body.appendChild(block.cloneNode(true));
 
     testRunner.dumpAsText();
-    var pages = testRunner.numberOfPages();
+    var pages = internals.numberOfPages();
     document.body.textContent = 'Document has ' + pages + ' pages.';
 }
 
index 1db9ad9..11a8af0 100644 (file)
@@ -13,7 +13,7 @@ if (window.testRunner) {
     // Test the range around A4 portrait paper size
     for (var i = 530; i < 560; i++) {
         for (var j = 730; j < 760; j++) {
-            var numberOfPages = testRunner.numberOfPages(i, j);
+            var numberOfPages = internals.numberOfPages(i, j);
 
             if (numberOfPages != 1) {
                 pass = false;
index 0a04eb9..a1e0d5a 100644 (file)
@@ -22,7 +22,7 @@ if (window.testRunner) {
     testRunner.dumpAsText();
 
     // The test passes if the following line doesn't cause a crash.
-    testRunner.pageSizeAndMarginsInPixels(0, 100, 200, 1, 2, 3, 4);
+    internals.pageSizeAndMarginsInPixels(0, 100, 200, 1, 2, 3, 4);
 
     document.getElementById("output").innerHTML = "PASS";
 }
index 8362895..f62d014 100644 (file)
@@ -15,68 +15,68 @@ Test visibility: visible
 PASS internals.isPageBoxVisible(document, 0) is true
 
 Test default page size
-PASS testRunner.pageSizeAndMarginsInPixels(0, 100, 200, 1, 2, 3, 4) is '(100, 200)' + pxMargins(1, 2, 3, 4)
+PASS internals.pageSizeAndMarginsInPixels(0, 100, 200, 1, 2, 3, 4) is '(100, 200)' + pxMargins(1, 2, 3, 4)
 Test auto page size
-PASS testRunner.pageSizeAndMarginsInPixels(0, 100, 200, 1, 2, 3, 4) is '(100, 200)' + pxMargins(1, 2, 3, 4)
+PASS internals.pageSizeAndMarginsInPixels(0, 100, 200, 1, 2, 3, 4) is '(100, 200)' + pxMargins(1, 2, 3, 4)
 Test landscape
-PASS testRunner.pageSizeAndMarginsInPixels(0, 100, 200, 1, 2, 3, 4) is '(200, 100)' + pxMargins(1, 2, 3, 4)
-PASS testRunner.pageSizeAndMarginsInPixels(0, 200, 100, 1, 2, 3, 4) is '(200, 100)' + pxMargins(1, 2, 3, 4)
+PASS internals.pageSizeAndMarginsInPixels(0, 100, 200, 1, 2, 3, 4) is '(200, 100)' + pxMargins(1, 2, 3, 4)
+PASS internals.pageSizeAndMarginsInPixels(0, 200, 100, 1, 2, 3, 4) is '(200, 100)' + pxMargins(1, 2, 3, 4)
 Test portrait
-PASS testRunner.pageSizeAndMarginsInPixels(0, 100, 200, 1, 2, 3, 4) is '(100, 200)' + pxMargins(1, 2, 3, 4)
-PASS testRunner.pageSizeAndMarginsInPixels(0, 200, 100, 1, 2, 3, 4) is '(100, 200)' + pxMargins(1, 2, 3, 4)
+PASS internals.pageSizeAndMarginsInPixels(0, 100, 200, 1, 2, 3, 4) is '(100, 200)' + pxMargins(1, 2, 3, 4)
+PASS internals.pageSizeAndMarginsInPixels(0, 200, 100, 1, 2, 3, 4) is '(100, 200)' + pxMargins(1, 2, 3, 4)
 Test a5
-PASS testRunner.pageSizeAndMarginsInPixels(0, 100, 200, 1, 2, 3, 4) is mmSize(148, 210) + pxMargins(1, 2, 3, 4)
+PASS internals.pageSizeAndMarginsInPixels(0, 100, 200, 1, 2, 3, 4) is mmSize(148, 210) + pxMargins(1, 2, 3, 4)
 Test a5 landscape
-PASS testRunner.pageSizeAndMarginsInPixels(0, 100, 200, 1, 2, 3, 4) is mmSize(210, 148) + pxMargins(1, 2, 3, 4)
+PASS internals.pageSizeAndMarginsInPixels(0, 100, 200, 1, 2, 3, 4) is mmSize(210, 148) + pxMargins(1, 2, 3, 4)
 Test portrait a4
-PASS testRunner.pageSizeAndMarginsInPixels(0, 100, 200, 1, 2, 3, 4) is mmSize(210, 297) + pxMargins(1, 2, 3, 4)
+PASS internals.pageSizeAndMarginsInPixels(0, 100, 200, 1, 2, 3, 4) is mmSize(210, 297) + pxMargins(1, 2, 3, 4)
 Test a3
-PASS testRunner.pageSizeAndMarginsInPixels(0, 100, 200, 1, 2, 3, 4) is mmSize(297, 420) + pxMargins(1, 2, 3, 4)
+PASS internals.pageSizeAndMarginsInPixels(0, 100, 200, 1, 2, 3, 4) is mmSize(297, 420) + pxMargins(1, 2, 3, 4)
 Test b5
-PASS testRunner.pageSizeAndMarginsInPixels(0, 100, 200, 1, 2, 3, 4) is mmSize(176, 250) + pxMargins(1, 2, 3, 4)
+PASS internals.pageSizeAndMarginsInPixels(0, 100, 200, 1, 2, 3, 4) is mmSize(176, 250) + pxMargins(1, 2, 3, 4)
 Test b4
-PASS testRunner.pageSizeAndMarginsInPixels(0, 100, 200, 1, 2, 3, 4) is mmSize(250, 353) + pxMargins(1, 2, 3, 4)
+PASS internals.pageSizeAndMarginsInPixels(0, 100, 200, 1, 2, 3, 4) is mmSize(250, 353) + pxMargins(1, 2, 3, 4)
 Test letter
-PASS testRunner.pageSizeAndMarginsInPixels(0, 100, 200, 1, 2, 3, 4) is inchSize(8.5, 11) + pxMargins(1, 2, 3, 4)
+PASS internals.pageSizeAndMarginsInPixels(0, 100, 200, 1, 2, 3, 4) is inchSize(8.5, 11) + pxMargins(1, 2, 3, 4)
 Test legal
-PASS testRunner.pageSizeAndMarginsInPixels(0, 100, 200, 1, 2, 3, 4) is inchSize(8.5, 14) + pxMargins(1, 2, 3, 4)
+PASS internals.pageSizeAndMarginsInPixels(0, 100, 200, 1, 2, 3, 4) is inchSize(8.5, 14) + pxMargins(1, 2, 3, 4)
 Test ledger
-PASS testRunner.pageSizeAndMarginsInPixels(0, 100, 200, 1, 2, 3, 4) is inchSize(11, 17) + pxMargins(1, 2, 3, 4)
+PASS internals.pageSizeAndMarginsInPixels(0, 100, 200, 1, 2, 3, 4) is inchSize(11, 17) + pxMargins(1, 2, 3, 4)
 Test 10cm
-PASS testRunner.pageSizeAndMarginsInPixels(0, 100, 200, 1, 2, 3, 4) is mmSize(100, 100) + pxMargins(1, 2, 3, 4)
+PASS internals.pageSizeAndMarginsInPixels(0, 100, 200, 1, 2, 3, 4) is mmSize(100, 100) + pxMargins(1, 2, 3, 4)
 Test 20cm x 30cm
-PASS testRunner.pageSizeAndMarginsInPixels(0, 100, 200, 1, 2, 3, 4) is mmSize(200, 300) + pxMargins(1, 2, 3, 4)
+PASS internals.pageSizeAndMarginsInPixels(0, 100, 200, 1, 2, 3, 4) is mmSize(200, 300) + pxMargins(1, 2, 3, 4)
 Test 10000px 20000px
-PASS testRunner.pageSizeAndMarginsInPixels(0, 100, 200, 1, 2, 3, 4) is '(10000, 20000)' + pxMargins(1, 2, 3, 4)
+PASS internals.pageSizeAndMarginsInPixels(0, 100, 200, 1, 2, 3, 4) is '(10000, 20000)' + pxMargins(1, 2, 3, 4)
 Test invalid page size: -10cm
-PASS testRunner.pageSizeAndMarginsInPixels(0, 100, 200, 1, 2, 3, 4) is '(10000, 20000)' + pxMargins(1, 2, 3, 4)
+PASS internals.pageSizeAndMarginsInPixels(0, 100, 200, 1, 2, 3, 4) is '(10000, 20000)' + pxMargins(1, 2, 3, 4)
 Test invalid page size: a4 a4
-PASS testRunner.pageSizeAndMarginsInPixels(0, 100, 200, 1, 2, 3, 4) is '(10000, 20000)' + pxMargins(1, 2, 3, 4)
+PASS internals.pageSizeAndMarginsInPixels(0, 100, 200, 1, 2, 3, 4) is '(10000, 20000)' + pxMargins(1, 2, 3, 4)
 Test invalid page size: landscape portrait
-PASS testRunner.pageSizeAndMarginsInPixels(0, 100, 200, 1, 2, 3, 4) is '(10000, 20000)' + pxMargins(1, 2, 3, 4)
+PASS internals.pageSizeAndMarginsInPixels(0, 100, 200, 1, 2, 3, 4) is '(10000, 20000)' + pxMargins(1, 2, 3, 4)
 Test invalid page size: 10cm letter
-PASS testRunner.pageSizeAndMarginsInPixels(0, 100, 200, 1, 2, 3, 4) is '(10000, 20000)' + pxMargins(1, 2, 3, 4)
+PASS internals.pageSizeAndMarginsInPixels(0, 100, 200, 1, 2, 3, 4) is '(10000, 20000)' + pxMargins(1, 2, 3, 4)
 Test invalid page size: 10cm 10cm 10cm
-PASS testRunner.pageSizeAndMarginsInPixels(0, 100, 200, 1, 2, 3, 4) is '(10000, 20000)' + pxMargins(1, 2, 3, 4)
+PASS internals.pageSizeAndMarginsInPixels(0, 100, 200, 1, 2, 3, 4) is '(10000, 20000)' + pxMargins(1, 2, 3, 4)
 Test invalid page size: empty
-PASS testRunner.pageSizeAndMarginsInPixels(0, 100, 200, 1, 2, 3, 4) is '(10000, 20000)' + pxMargins(1, 2, 3, 4)
+PASS internals.pageSizeAndMarginsInPixels(0, 100, 200, 1, 2, 3, 4) is '(10000, 20000)' + pxMargins(1, 2, 3, 4)
 
 Test 0px margins
-PASS testRunner.pageSizeAndMarginsInPixels(0, 100, 200, 1, 2, 3, 4) is '(10000, 20000)' + pxMargins(0, 0, 0, 0)
+PASS internals.pageSizeAndMarginsInPixels(0, 100, 200, 1, 2, 3, 4) is '(10000, 20000)' + pxMargins(0, 0, 0, 0)
 Test non-zero margins
-PASS testRunner.pageSizeAndMarginsInPixels(0, 100, 200, 1, 2, 3, 4) is '(10000, 20000)' + pxMargins(10, 20, 40, 80)
+PASS internals.pageSizeAndMarginsInPixels(0, 100, 200, 1, 2, 3, 4) is '(10000, 20000)' + pxMargins(10, 20, 40, 80)
 Test percentage margin
-PASS testRunner.pageSizeAndMarginsInPixels(0, 100, 200, 1, 2, 3, 4) is '(10000, 20000)' + pxMargins(800, 400, 200, 100)
+PASS internals.pageSizeAndMarginsInPixels(0, 100, 200, 1, 2, 3, 4) is '(10000, 20000)' + pxMargins(800, 400, 200, 100)
 Test margin: auto 4% 2% 1%
-PASS testRunner.pageSizeAndMarginsInPixels(0, 100, 200, 1, 2, 3, 4) is '(10000, 20000)' + pxMargins(1, 400, 200, 100)
+PASS internals.pageSizeAndMarginsInPixels(0, 100, 200, 1, 2, 3, 4) is '(10000, 20000)' + pxMargins(1, 400, 200, 100)
 Test margin: 8% auto 2% 1%
-PASS testRunner.pageSizeAndMarginsInPixels(0, 100, 200, 1, 2, 3, 4) is '(10000, 20000)' + pxMargins(800, 2, 200, 100)
+PASS internals.pageSizeAndMarginsInPixels(0, 100, 200, 1, 2, 3, 4) is '(10000, 20000)' + pxMargins(800, 2, 200, 100)
 Test margin: 8% 4% auto 1%
-PASS testRunner.pageSizeAndMarginsInPixels(0, 100, 200, 1, 2, 3, 4) is '(10000, 20000)' + pxMargins(800, 400, 3, 100)
+PASS internals.pageSizeAndMarginsInPixels(0, 100, 200, 1, 2, 3, 4) is '(10000, 20000)' + pxMargins(800, 400, 3, 100)
 Test margin: 8% 4% 2% auto
-PASS testRunner.pageSizeAndMarginsInPixels(0, 100, 200, 1, 2, 3, 4) is '(10000, 20000)' + pxMargins(800, 400, 200, 4)
+PASS internals.pageSizeAndMarginsInPixels(0, 100, 200, 1, 2, 3, 4) is '(10000, 20000)' + pxMargins(800, 400, 200, 4)
 Test visibility: hidden
-PASS testRunner.pageSizeAndMarginsInPixels(0, 100, 200, 1, 2, 3, 4) is '(10000, 20000)' + pxMargins(800, 400, 200, 4)
+PASS internals.pageSizeAndMarginsInPixels(0, 100, 200, 1, 2, 3, 4) is '(10000, 20000)' + pxMargins(800, 400, 200, 4)
 
 PASS successfullyParsed is true
 
index fdc135b..890fb64 100644 (file)
         // Page size tests
 
         debug("Test default page size");
-        shouldBe("testRunner.pageSizeAndMarginsInPixels(0, 100, 200, 1, 2, 3, 4)", "'(100, 200)' + pxMargins(1, 2, 3, 4)");
+        shouldBe("internals.pageSizeAndMarginsInPixels(0, 100, 200, 1, 2, 3, 4)", "'(100, 200)' + pxMargins(1, 2, 3, 4)");
 
         debug("Test auto page size");
         appendStyle("@page {size:auto;}");
-        shouldBe("testRunner.pageSizeAndMarginsInPixels(0, 100, 200, 1, 2, 3, 4)", "'(100, 200)' + pxMargins(1, 2, 3, 4)");
+        shouldBe("internals.pageSizeAndMarginsInPixels(0, 100, 200, 1, 2, 3, 4)", "'(100, 200)' + pxMargins(1, 2, 3, 4)");
 
         debug("Test landscape");
         appendStyle("@page {size:landscape;}");
-        shouldBe("testRunner.pageSizeAndMarginsInPixels(0, 100, 200, 1, 2, 3, 4)", "'(200, 100)' + pxMargins(1, 2, 3, 4)");
-        shouldBe("testRunner.pageSizeAndMarginsInPixels(0, 200, 100, 1, 2, 3, 4)", "'(200, 100)' + pxMargins(1, 2, 3, 4)");
+        shouldBe("internals.pageSizeAndMarginsInPixels(0, 100, 200, 1, 2, 3, 4)", "'(200, 100)' + pxMargins(1, 2, 3, 4)");
+        shouldBe("internals.pageSizeAndMarginsInPixels(0, 200, 100, 1, 2, 3, 4)", "'(200, 100)' + pxMargins(1, 2, 3, 4)");
 
         debug("Test portrait");
         appendStyle("@page {size:portrait;}");
-        shouldBe("testRunner.pageSizeAndMarginsInPixels(0, 100, 200, 1, 2, 3, 4)", "'(100, 200)' + pxMargins(1, 2, 3, 4)");
-        shouldBe("testRunner.pageSizeAndMarginsInPixels(0, 200, 100, 1, 2, 3, 4)", "'(100, 200)' + pxMargins(1, 2, 3, 4)");
+        shouldBe("internals.pageSizeAndMarginsInPixels(0, 100, 200, 1, 2, 3, 4)", "'(100, 200)' + pxMargins(1, 2, 3, 4)");
+        shouldBe("internals.pageSizeAndMarginsInPixels(0, 200, 100, 1, 2, 3, 4)", "'(100, 200)' + pxMargins(1, 2, 3, 4)");
 
         debug("Test a5");
         appendStyle("@page {size:a5;}");
-        shouldBe("testRunner.pageSizeAndMarginsInPixels(0, 100, 200, 1, 2, 3, 4)", "mmSize(148, 210) + pxMargins(1, 2, 3, 4)");
+        shouldBe("internals.pageSizeAndMarginsInPixels(0, 100, 200, 1, 2, 3, 4)", "mmSize(148, 210) + pxMargins(1, 2, 3, 4)");
 
         debug("Test a5 landscape");
         appendStyle("@page {size:a5 landscape;}");
-        shouldBe("testRunner.pageSizeAndMarginsInPixels(0, 100, 200, 1, 2, 3, 4)", "mmSize(210, 148) + pxMargins(1, 2, 3, 4)");
+        shouldBe("internals.pageSizeAndMarginsInPixels(0, 100, 200, 1, 2, 3, 4)", "mmSize(210, 148) + pxMargins(1, 2, 3, 4)");
 
         debug("Test portrait a4");
         appendStyle("@page {size:portrait a4;}");
-        shouldBe("testRunner.pageSizeAndMarginsInPixels(0, 100, 200, 1, 2, 3, 4)", "mmSize(210, 297) + pxMargins(1, 2, 3, 4)");
+        shouldBe("internals.pageSizeAndMarginsInPixels(0, 100, 200, 1, 2, 3, 4)", "mmSize(210, 297) + pxMargins(1, 2, 3, 4)");
 
         debug("Test a3");
         appendStyle("@page {size:a3;}");
-        shouldBe("testRunner.pageSizeAndMarginsInPixels(0, 100, 200, 1, 2, 3, 4)", "mmSize(297, 420) + pxMargins(1, 2, 3, 4)");
+        shouldBe("internals.pageSizeAndMarginsInPixels(0, 100, 200, 1, 2, 3, 4)", "mmSize(297, 420) + pxMargins(1, 2, 3, 4)");
 
         debug("Test b5");
         appendStyle("@page {size:b5;}");
-        shouldBe("testRunner.pageSizeAndMarginsInPixels(0, 100, 200, 1, 2, 3, 4)", "mmSize(176, 250) + pxMargins(1, 2, 3, 4)");
+        shouldBe("internals.pageSizeAndMarginsInPixels(0, 100, 200, 1, 2, 3, 4)", "mmSize(176, 250) + pxMargins(1, 2, 3, 4)");
 
         debug("Test b4");
         appendStyle("@page {size:b4;}");
-        shouldBe("testRunner.pageSizeAndMarginsInPixels(0, 100, 200, 1, 2, 3, 4)", "mmSize(250, 353) + pxMargins(1, 2, 3, 4)");
+        shouldBe("internals.pageSizeAndMarginsInPixels(0, 100, 200, 1, 2, 3, 4)", "mmSize(250, 353) + pxMargins(1, 2, 3, 4)");
 
         debug("Test letter");
         appendStyle("@page {size:letter;}");
-        shouldBe("testRunner.pageSizeAndMarginsInPixels(0, 100, 200, 1, 2, 3, 4)", "inchSize(8.5, 11) + pxMargins(1, 2, 3, 4)");
+        shouldBe("internals.pageSizeAndMarginsInPixels(0, 100, 200, 1, 2, 3, 4)", "inchSize(8.5, 11) + pxMargins(1, 2, 3, 4)");
 
         debug("Test legal");
         appendStyle("@page {size:legal;}");
-        shouldBe("testRunner.pageSizeAndMarginsInPixels(0, 100, 200, 1, 2, 3, 4)", "inchSize(8.5, 14) + pxMargins(1, 2, 3, 4)");
+        shouldBe("internals.pageSizeAndMarginsInPixels(0, 100, 200, 1, 2, 3, 4)", "inchSize(8.5, 14) + pxMargins(1, 2, 3, 4)");
 
         debug("Test ledger");
         appendStyle("@page {size:ledger;}");
-        shouldBe("testRunner.pageSizeAndMarginsInPixels(0, 100, 200, 1, 2, 3, 4)", "inchSize(11, 17) + pxMargins(1, 2, 3, 4)");
+        shouldBe("internals.pageSizeAndMarginsInPixels(0, 100, 200, 1, 2, 3, 4)", "inchSize(11, 17) + pxMargins(1, 2, 3, 4)");
 
         debug("Test 10cm");
         appendStyle("@page {size:10cm;}");
-        shouldBe("testRunner.pageSizeAndMarginsInPixels(0, 100, 200, 1, 2, 3, 4)", "mmSize(100, 100) + pxMargins(1, 2, 3, 4)");
+        shouldBe("internals.pageSizeAndMarginsInPixels(0, 100, 200, 1, 2, 3, 4)", "mmSize(100, 100) + pxMargins(1, 2, 3, 4)");
 
         debug("Test 20cm x 30cm");
         appendStyle("@page {size:20cm 30cm;}");
-        shouldBe("testRunner.pageSizeAndMarginsInPixels(0, 100, 200, 1, 2, 3, 4)", "mmSize(200, 300) + pxMargins(1, 2, 3, 4)");
+        shouldBe("internals.pageSizeAndMarginsInPixels(0, 100, 200, 1, 2, 3, 4)", "mmSize(200, 300) + pxMargins(1, 2, 3, 4)");
 
         debug("Test 10000px 20000px");
         appendStyle("@page {size:10000px 20000px;}");
-        shouldBe("testRunner.pageSizeAndMarginsInPixels(0, 100, 200, 1, 2, 3, 4)", "'(10000, 20000)' + pxMargins(1, 2, 3, 4)");
+        shouldBe("internals.pageSizeAndMarginsInPixels(0, 100, 200, 1, 2, 3, 4)", "'(10000, 20000)' + pxMargins(1, 2, 3, 4)");
 
         debug("Test invalid page size: -10cm");
         appendStyle("@page {size:-10cm;}");
-        shouldBe("testRunner.pageSizeAndMarginsInPixels(0, 100, 200, 1, 2, 3, 4)", "'(10000, 20000)' + pxMargins(1, 2, 3, 4)");
+        shouldBe("internals.pageSizeAndMarginsInPixels(0, 100, 200, 1, 2, 3, 4)", "'(10000, 20000)' + pxMargins(1, 2, 3, 4)");
 
         debug("Test invalid page size: a4 a4");
         appendStyle("@page {size:a4 a4;}");
-        shouldBe("testRunner.pageSizeAndMarginsInPixels(0, 100, 200, 1, 2, 3, 4)", "'(10000, 20000)' + pxMargins(1, 2, 3, 4)");
+        shouldBe("internals.pageSizeAndMarginsInPixels(0, 100, 200, 1, 2, 3, 4)", "'(10000, 20000)' + pxMargins(1, 2, 3, 4)");
 
         debug("Test invalid page size: landscape portrait");
         appendStyle("@page {size:landscape portrait;}");
-        shouldBe("testRunner.pageSizeAndMarginsInPixels(0, 100, 200, 1, 2, 3, 4)", "'(10000, 20000)' + pxMargins(1, 2, 3, 4)");
+        shouldBe("internals.pageSizeAndMarginsInPixels(0, 100, 200, 1, 2, 3, 4)", "'(10000, 20000)' + pxMargins(1, 2, 3, 4)");
 
         debug("Test invalid page size: 10cm letter");
         appendStyle("@page {size:10cm letter;}");
-        shouldBe("testRunner.pageSizeAndMarginsInPixels(0, 100, 200, 1, 2, 3, 4)", "'(10000, 20000)' + pxMargins(1, 2, 3, 4)");
+        shouldBe("internals.pageSizeAndMarginsInPixels(0, 100, 200, 1, 2, 3, 4)", "'(10000, 20000)' + pxMargins(1, 2, 3, 4)");
 
         debug("Test invalid page size: 10cm 10cm 10cm");
         appendStyle("@page {size:10cm 10cm 10cm;}");
-        shouldBe("testRunner.pageSizeAndMarginsInPixels(0, 100, 200, 1, 2, 3, 4)", "'(10000, 20000)' + pxMargins(1, 2, 3, 4)");
+        shouldBe("internals.pageSizeAndMarginsInPixels(0, 100, 200, 1, 2, 3, 4)", "'(10000, 20000)' + pxMargins(1, 2, 3, 4)");
 
         debug("Test invalid page size: empty");
         appendStyle("@page {size:;}");
-        shouldBe("testRunner.pageSizeAndMarginsInPixels(0, 100, 200, 1, 2, 3, 4)", "'(10000, 20000)' + pxMargins(1, 2, 3, 4)");
+        shouldBe("internals.pageSizeAndMarginsInPixels(0, 100, 200, 1, 2, 3, 4)", "'(10000, 20000)' + pxMargins(1, 2, 3, 4)");
 
         debug("");
 
 
         debug("Test 0px margins");
         appendStyle("@page {margin:0px;}");
-        shouldBe("testRunner.pageSizeAndMarginsInPixels(0, 100, 200, 1, 2, 3, 4)", "'(10000, 20000)' + pxMargins(0, 0, 0, 0)");
+        shouldBe("internals.pageSizeAndMarginsInPixels(0, 100, 200, 1, 2, 3, 4)", "'(10000, 20000)' + pxMargins(0, 0, 0, 0)");
 
         debug("Test non-zero margins");
         appendStyle("@page {margin:10px 20px 40px 80px;}");
-        shouldBe("testRunner.pageSizeAndMarginsInPixels(0, 100, 200, 1, 2, 3, 4)", "'(10000, 20000)' + pxMargins(10, 20, 40, 80)");
+        shouldBe("internals.pageSizeAndMarginsInPixels(0, 100, 200, 1, 2, 3, 4)", "'(10000, 20000)' + pxMargins(10, 20, 40, 80)");
 
         debug("Test percentage margin");
         appendStyle("@page {margin: 8% 4% 2% 1%;}");
-        shouldBe("testRunner.pageSizeAndMarginsInPixels(0, 100, 200, 1, 2, 3, 4)", "'(10000, 20000)' + pxMargins(800, 400, 200, 100)");
+        shouldBe("internals.pageSizeAndMarginsInPixels(0, 100, 200, 1, 2, 3, 4)", "'(10000, 20000)' + pxMargins(800, 400, 200, 100)");
 
         debug("Test margin: auto 4% 2% 1%");
         appendStyle("@page {margin: auto 4% 2% 1%;}");
-        shouldBe("testRunner.pageSizeAndMarginsInPixels(0, 100, 200, 1, 2, 3, 4)", "'(10000, 20000)' + pxMargins(1, 400, 200, 100)");
+        shouldBe("internals.pageSizeAndMarginsInPixels(0, 100, 200, 1, 2, 3, 4)", "'(10000, 20000)' + pxMargins(1, 400, 200, 100)");
 
         debug("Test margin: 8% auto 2% 1%");
         appendStyle("@page {margin: 8% auto 2% 1%;}");
-        shouldBe("testRunner.pageSizeAndMarginsInPixels(0, 100, 200, 1, 2, 3, 4)", "'(10000, 20000)' + pxMargins(800, 2, 200, 100)");
+        shouldBe("internals.pageSizeAndMarginsInPixels(0, 100, 200, 1, 2, 3, 4)", "'(10000, 20000)' + pxMargins(800, 2, 200, 100)");
 
         debug("Test margin: 8% 4% auto 1%");
         appendStyle("@page {margin: 8% 4% auto 1%;}");
-        shouldBe("testRunner.pageSizeAndMarginsInPixels(0, 100, 200, 1, 2, 3, 4)", "'(10000, 20000)' + pxMargins(800, 400, 3, 100)");
+        shouldBe("internals.pageSizeAndMarginsInPixels(0, 100, 200, 1, 2, 3, 4)", "'(10000, 20000)' + pxMargins(800, 400, 3, 100)");
 
         debug("Test margin: 8% 4% 2% auto");
         appendStyle("@page {margin: 8% 4% 2% auto;}");
-        shouldBe("testRunner.pageSizeAndMarginsInPixels(0, 100, 200, 1, 2, 3, 4)", "'(10000, 20000)' + pxMargins(800, 400, 200, 4)");
+        shouldBe("internals.pageSizeAndMarginsInPixels(0, 100, 200, 1, 2, 3, 4)", "'(10000, 20000)' + pxMargins(800, 400, 200, 4)");
 
         debug("Test visibility: hidden");
         appendStyle("@page {visibility:hidden}");
-        shouldBe("testRunner.pageSizeAndMarginsInPixels(0, 100, 200, 1, 2, 3, 4)", "'(10000, 20000)' + pxMargins(800, 400, 200, 4)");
+        shouldBe("internals.pageSizeAndMarginsInPixels(0, 100, 200, 1, 2, 3, 4)", "'(10000, 20000)' + pxMargins(800, 400, 200, 4)");
 
         debug("");
 
index 00c0fa9..8550833 100644 (file)
@@ -4,55 +4,55 @@ On success, you will see a series of "PASS" messages, followed by "TEST COMPLETE
 
 
 Default margin is auto.
-PASS testRunner.pageProperty('margin-left', 0) is "auto"
-PASS testRunner.pageProperty('margin-left', 1) is "auto"
-PASS testRunner.pageProperty('margin-left', 2) is "auto"
-PASS testRunner.pageProperty('margin-left', 3) is "auto"
-PASS testRunner.pageProperty('margin-left', 4) is "auto"
+PASS internals.pageProperty('margin-left', 0) is "auto"
+PASS internals.pageProperty('margin-left', 1) is "auto"
+PASS internals.pageProperty('margin-left', 2) is "auto"
+PASS internals.pageProperty('margin-left', 3) is "auto"
+PASS internals.pageProperty('margin-left', 4) is "auto"
 Set margin to 200px via user style sheet.
-PASS testRunner.pageProperty('margin-left', 0) is "200"
-PASS testRunner.pageProperty('margin-left', 1) is "200"
-PASS testRunner.pageProperty('margin-left', 2) is "200"
-PASS testRunner.pageProperty('margin-left', 3) is "200"
-PASS testRunner.pageProperty('margin-left', 4) is "200"
+PASS internals.pageProperty('margin-left', 0) is "200"
+PASS internals.pageProperty('margin-left', 1) is "200"
+PASS internals.pageProperty('margin-left', 2) is "200"
+PASS internals.pageProperty('margin-left', 3) is "200"
+PASS internals.pageProperty('margin-left', 4) is "200"
 Set margin to 100px.
-PASS testRunner.pageProperty('margin-left', 0) is "100"
-PASS testRunner.pageProperty('margin-left', 1) is "100"
-PASS testRunner.pageProperty('margin-left', 2) is "100"
-PASS testRunner.pageProperty('margin-left', 3) is "100"
-PASS testRunner.pageProperty('margin-left', 4) is "100"
+PASS internals.pageProperty('margin-left', 0) is "100"
+PASS internals.pageProperty('margin-left', 1) is "100"
+PASS internals.pageProperty('margin-left', 2) is "100"
+PASS internals.pageProperty('margin-left', 3) is "100"
+PASS internals.pageProperty('margin-left', 4) is "100"
 Set margin for right pages to 50px. The first page is a right page.
-PASS testRunner.pageProperty('margin-left', 0) is "50"
-PASS testRunner.pageProperty('margin-left', 1) is "100"
-PASS testRunner.pageProperty('margin-left', 2) is "50"
-PASS testRunner.pageProperty('margin-left', 3) is "100"
-PASS testRunner.pageProperty('margin-left', 4) is "50"
+PASS internals.pageProperty('margin-left', 0) is "50"
+PASS internals.pageProperty('margin-left', 1) is "100"
+PASS internals.pageProperty('margin-left', 2) is "50"
+PASS internals.pageProperty('margin-left', 3) is "100"
+PASS internals.pageProperty('margin-left', 4) is "50"
 Set margin for left pages to 75px.
-PASS testRunner.pageProperty('margin-left', 0) is "50"
-PASS testRunner.pageProperty('margin-left', 1) is "75"
-PASS testRunner.pageProperty('margin-left', 2) is "50"
-PASS testRunner.pageProperty('margin-left', 3) is "75"
-PASS testRunner.pageProperty('margin-left', 4) is "50"
+PASS internals.pageProperty('margin-left', 0) is "50"
+PASS internals.pageProperty('margin-left', 1) is "75"
+PASS internals.pageProperty('margin-left', 2) is "50"
+PASS internals.pageProperty('margin-left', 3) is "75"
+PASS internals.pageProperty('margin-left', 4) is "50"
 Set margin for the first page to 125px.
-PASS testRunner.pageProperty('margin-left', 0) is "125"
-PASS testRunner.pageProperty('margin-left', 1) is "75"
-PASS testRunner.pageProperty('margin-left', 2) is "50"
-PASS testRunner.pageProperty('margin-left', 3) is "75"
-PASS testRunner.pageProperty('margin-left', 4) is "50"
+PASS internals.pageProperty('margin-left', 0) is "125"
+PASS internals.pageProperty('margin-left', 1) is "75"
+PASS internals.pageProperty('margin-left', 2) is "50"
+PASS internals.pageProperty('margin-left', 3) is "75"
+PASS internals.pageProperty('margin-left', 4) is "50"
 Change the document direction to RTL. This makes the first page a left page.
-PASS testRunner.pageProperty('margin-left', 0) is "125"
-PASS testRunner.pageProperty('margin-left', 1) is "50"
-PASS testRunner.pageProperty('margin-left', 2) is "75"
-PASS testRunner.pageProperty('margin-left', 3) is "50"
-PASS testRunner.pageProperty('margin-left', 4) is "75"
+PASS internals.pageProperty('margin-left', 0) is "125"
+PASS internals.pageProperty('margin-left', 1) is "50"
+PASS internals.pageProperty('margin-left', 2) is "75"
+PASS internals.pageProperty('margin-left', 3) is "50"
+PASS internals.pageProperty('margin-left', 4) is "75"
 Check font update and line height override.
-PASS testRunner.pageProperty('line-height', 0) is "180"
-PASS testRunner.pageProperty('font-family', 0) is "Arial"
-PASS testRunner.pageProperty('font-size', 0) is "30"
+PASS internals.pageProperty('line-height', 0) is "180"
+PASS internals.pageProperty('font-family', 0) is "Arial"
+PASS internals.pageProperty('font-size', 0) is "30"
 Check @media rules.
-PASS testRunner.pageProperty('size', 0) is "100 200"
-PASS testRunner.pageProperty('size', 0) is "150 250"
-PASS testRunner.pageProperty('size', 0) is "150 250"
+PASS internals.pageProperty('size', 0) is "100 200"
+PASS internals.pageProperty('size', 0) is "150 250"
+PASS internals.pageProperty('size', 0) is "150 250"
 PASS successfullyParsed is true
 
 TEST COMPLETE
index 7f6d406..727e27f 100644 (file)
         testRunner.dumpAsText();
 
         debug("Default margin is auto.");
-        shouldBeEqualToString("testRunner.pageProperty('margin-left', 0)", "auto");
-        shouldBeEqualToString("testRunner.pageProperty('margin-left', 1)", "auto");
-        shouldBeEqualToString("testRunner.pageProperty('margin-left', 2)", "auto");
-        shouldBeEqualToString("testRunner.pageProperty('margin-left', 3)", "auto");
-        shouldBeEqualToString("testRunner.pageProperty('margin-left', 4)", "auto");
+        shouldBeEqualToString("internals.pageProperty('margin-left', 0)", "auto");
+        shouldBeEqualToString("internals.pageProperty('margin-left', 1)", "auto");
+        shouldBeEqualToString("internals.pageProperty('margin-left', 2)", "auto");
+        shouldBeEqualToString("internals.pageProperty('margin-left', 3)", "auto");
+        shouldBeEqualToString("internals.pageProperty('margin-left', 4)", "auto");
 
         debug("Set margin to 200px via user style sheet.");
         testRunner.addUserStyleSheet("@page { margin:200px; }", true);
-        shouldBeEqualToString("testRunner.pageProperty('margin-left', 0)", "200");
-        shouldBeEqualToString("testRunner.pageProperty('margin-left', 1)", "200");
-        shouldBeEqualToString("testRunner.pageProperty('margin-left', 2)", "200");
-        shouldBeEqualToString("testRunner.pageProperty('margin-left', 3)", "200");
-        shouldBeEqualToString("testRunner.pageProperty('margin-left', 4)", "200");
+        shouldBeEqualToString("internals.pageProperty('margin-left', 0)", "200");
+        shouldBeEqualToString("internals.pageProperty('margin-left', 1)", "200");
+        shouldBeEqualToString("internals.pageProperty('margin-left', 2)", "200");
+        shouldBeEqualToString("internals.pageProperty('margin-left', 3)", "200");
+        shouldBeEqualToString("internals.pageProperty('margin-left', 4)", "200");
 
         debug("Set margin to 100px.");
         appendStyle("@page { margin:100px; }");
-        shouldBeEqualToString("testRunner.pageProperty('margin-left', 0)", "100");
-        shouldBeEqualToString("testRunner.pageProperty('margin-left', 1)", "100");
-        shouldBeEqualToString("testRunner.pageProperty('margin-left', 2)", "100");
-        shouldBeEqualToString("testRunner.pageProperty('margin-left', 3)", "100");
-        shouldBeEqualToString("testRunner.pageProperty('margin-left', 4)", "100");
+        shouldBeEqualToString("internals.pageProperty('margin-left', 0)", "100");
+        shouldBeEqualToString("internals.pageProperty('margin-left', 1)", "100");
+        shouldBeEqualToString("internals.pageProperty('margin-left', 2)", "100");
+        shouldBeEqualToString("internals.pageProperty('margin-left', 3)", "100");
+        shouldBeEqualToString("internals.pageProperty('margin-left', 4)", "100");
 
         debug("Set margin for right pages to 50px. The first page is a right page.");
         appendStyle("@page :right { margin:50px; }");
-        shouldBeEqualToString("testRunner.pageProperty('margin-left', 0)", "50");
-        shouldBeEqualToString("testRunner.pageProperty('margin-left', 1)", "100");
-        shouldBeEqualToString("testRunner.pageProperty('margin-left', 2)", "50");
-        shouldBeEqualToString("testRunner.pageProperty('margin-left', 3)", "100");
-        shouldBeEqualToString("testRunner.pageProperty('margin-left', 4)", "50");
+        shouldBeEqualToString("internals.pageProperty('margin-left', 0)", "50");
+        shouldBeEqualToString("internals.pageProperty('margin-left', 1)", "100");
+        shouldBeEqualToString("internals.pageProperty('margin-left', 2)", "50");
+        shouldBeEqualToString("internals.pageProperty('margin-left', 3)", "100");
+        shouldBeEqualToString("internals.pageProperty('margin-left', 4)", "50");
 
         debug("Set margin for left pages to 75px.");
         appendStyle("@page :left { margin:75px; }");
-        shouldBeEqualToString("testRunner.pageProperty('margin-left', 0)", "50");
-        shouldBeEqualToString("testRunner.pageProperty('margin-left', 1)", "75");
-        shouldBeEqualToString("testRunner.pageProperty('margin-left', 2)", "50");
-        shouldBeEqualToString("testRunner.pageProperty('margin-left', 3)", "75");
-        shouldBeEqualToString("testRunner.pageProperty('margin-left', 4)", "50");
+        shouldBeEqualToString("internals.pageProperty('margin-left', 0)", "50");
+        shouldBeEqualToString("internals.pageProperty('margin-left', 1)", "75");
+        shouldBeEqualToString("internals.pageProperty('margin-left', 2)", "50");
+        shouldBeEqualToString("internals.pageProperty('margin-left', 3)", "75");
+        shouldBeEqualToString("internals.pageProperty('margin-left', 4)", "50");
 
         debug("Set margin for the first page to 125px.");
         appendStyle("@page :first { margin:125px; }");
-        shouldBeEqualToString("testRunner.pageProperty('margin-left', 0)", "125");
-        shouldBeEqualToString("testRunner.pageProperty('margin-left', 1)", "75");
-        shouldBeEqualToString("testRunner.pageProperty('margin-left', 2)", "50");
-        shouldBeEqualToString("testRunner.pageProperty('margin-left', 3)", "75");
-        shouldBeEqualToString("testRunner.pageProperty('margin-left', 4)", "50");
+        shouldBeEqualToString("internals.pageProperty('margin-left', 0)", "125");
+        shouldBeEqualToString("internals.pageProperty('margin-left', 1)", "75");
+        shouldBeEqualToString("internals.pageProperty('margin-left', 2)", "50");
+        shouldBeEqualToString("internals.pageProperty('margin-left', 3)", "75");
+        shouldBeEqualToString("internals.pageProperty('margin-left', 4)", "50");
 
         debug("Change the document direction to RTL. This makes the first page a left page.");
         document.getElementById("html_element").setAttribute("dir", "rtl");
-        shouldBeEqualToString("testRunner.pageProperty('margin-left', 0)", "125");
-        shouldBeEqualToString("testRunner.pageProperty('margin-left', 1)", "50");
-        shouldBeEqualToString("testRunner.pageProperty('margin-left', 2)", "75");
-        shouldBeEqualToString("testRunner.pageProperty('margin-left', 3)", "50");
-        shouldBeEqualToString("testRunner.pageProperty('margin-left', 4)", "75");
+        shouldBeEqualToString("internals.pageProperty('margin-left', 0)", "125");
+        shouldBeEqualToString("internals.pageProperty('margin-left', 1)", "50");
+        shouldBeEqualToString("internals.pageProperty('margin-left', 2)", "75");
+        shouldBeEqualToString("internals.pageProperty('margin-left', 3)", "50");
+        shouldBeEqualToString("internals.pageProperty('margin-left', 4)", "75");
 
         debug("Check font update and line height override.");
         appendStyle("@page { line-height:1em; }");
         appendStyle("@page { font-family:Futura; }");
         appendStyle("@page { font:15px/6em Arial; }");
         appendStyle("@page { zoom:200%; }");
-        shouldBeEqualToString("testRunner.pageProperty('line-height', 0)", "180");
-        shouldBeEqualToString("testRunner.pageProperty('font-family', 0)", "Arial");
-        shouldBeEqualToString("testRunner.pageProperty('font-size', 0)", "30");
+        shouldBeEqualToString("internals.pageProperty('line-height', 0)", "180");
+        shouldBeEqualToString("internals.pageProperty('font-family', 0)", "Arial");
+        shouldBeEqualToString("internals.pageProperty('font-size', 0)", "30");
         appendStyle("@page { zoom:100%; }");
 
         debug("Check @media rules.");
         appendStyle("@page { size:100px 200px; }");
-        shouldBeEqualToString("testRunner.pageProperty('size', 0)", "100 200");
+        shouldBeEqualToString("internals.pageProperty('size', 0)", "100 200");
         appendStyle("@media print { @page { size:150px 250px; } }");
-        shouldBeEqualToString("testRunner.pageProperty('size', 0)", "150 250");
+        shouldBeEqualToString("internals.pageProperty('size', 0)", "150 250");
         appendStyle("@media screen { @page { size:300px 500px; } }");
-        shouldBeEqualToString("testRunner.pageProperty('size', 0)", "150 250");
+        shouldBeEqualToString("internals.pageProperty('size', 0)", "150 250");
 
     } else {
         testFailed("This test can be run only with window.testRunner");
index de8b84f..a2d5deb 100644 (file)
@@ -26,9 +26,9 @@ function numberOfPagesShouldBe(expectedNumberOfPages, pageWidthInPixels, pageHei
     // pageWidthInPixels and pageHeightInPixels can be omitted. If omitted, 800x600 is used.
     var actualNumberOfPages;
     if (pageWidthInPixels && pageHeightInPixels)
-        actualNumberOfPages = testRunner.numberOfPages(pageWidthInPixels, pageHeightInPixels);
+        actualNumberOfPages = internals.numberOfPages(pageWidthInPixels, pageHeightInPixels);
     else
-        actualNumberOfPages = testRunner.numberOfPages();
+        actualNumberOfPages = internals.numberOfPages();
 
     if (actualNumberOfPages == expectedNumberOfPages)
         appendResult('PASS: number of pages is ' + actualNumberOfPages);
index ac62f0e..55efa87 100644 (file)
@@ -1,4 +1,4 @@
-description("Test for testRunner.numberOfPages()");
+description("Test for internals.numberOfPages()");
 
 function test()
 {
index e2ffe70..28e589c 100644 (file)
@@ -13,7 +13,7 @@ if (window.testRunner) {
     // Determine minium page size height where text fits on a single page.
     var height;
     for (var i = 10; i < 100; i += 5) {
-        if (testRunner.numberOfPages(100, i) == 1) {
+        if (internals.numberOfPages(100, i) == 1) {
             height = i;
             break;
         }
@@ -21,7 +21,7 @@ if (window.testRunner) {
     
     // Zoom in and verify that page count does not change.
     for (var i = 0; i < 10; i++) {
-        var numberOfPages = testRunner.numberOfPages(100, height);
+        var numberOfPages = internals.numberOfPages(100, height);
         if (numberOfPages != 1) {
             pass = false;
             msg += 'Got ' + numberOfPages + ' pages, expected 1 page.\n';
index c769219..56a8ea4 100644 (file)
@@ -1,3 +1,23 @@
+2012-09-05  Kaustubh Atrawalkar  <kaustubh@motorola.com>
+
+        [DRT] LTC:: Move printing related APIs from LayoutTestController to Internals
+        https://bugs.webkit.org/show_bug.cgi?id=92735
+
+        Reviewed by Hajime Morita.
+
+        Move numberOfPages, pageProperty & pageSizeAndMarginsInPixels in Internals and remove duplicated code from DumprenderTree & WebkitTestRunner.
+
+        Covered by existing test cases.
+
+        * testing/Internals.cpp:
+        (WebCore::Internals::numberOfPages):
+        (WebCore):
+        (WebCore::Internals::pageProperty):
+        (WebCore::Internals::pageSizeAndMarginsInPixels):
+        * testing/Internals.h:
+        (Internals):
+        * testing/Internals.idl:
+
 2012-09-05  Simon Hausmann  <simon.hausmann@nokia.com>
 
         [Qt] Automate the use of AllInOne files in the Qt build
index d03f03c..3694814 100644 (file)
@@ -1170,6 +1170,34 @@ PassRefPtr<DOMStringList> Internals::iconURLs(Document* document) const
     return stringList.release();
 }
 
+int Internals::numberOfPages(float pageWidth, float pageHeight)
+{
+    if (!frame())
+        return -1;
+
+    return PrintContext::numberOfPages(frame(), FloatSize(pageWidth, pageHeight));
+}
+
+String Internals::pageProperty(String propertyName, int pageNumber, ExceptionCode& ec) const
+{
+    if (!frame()) {
+        ec = INVALID_ACCESS_ERR;
+        return String();
+    }
+
+    return PrintContext::pageProperty(frame(), propertyName.utf8().data(), pageNumber);
+}
+
+String Internals::pageSizeAndMarginsInPixels(int pageNumber, int width, int height, int marginTop, int marginRight, int marginBottom, int marginLeft, ExceptionCode& ec) const
+{
+    if (!frame()) {
+        ec = INVALID_ACCESS_ERR;
+        return String();
+    }
+
+    return PrintContext::pageSizeAndMarginsInPixels(frame(), pageNumber, width, height, marginTop, marginRight, marginBottom, marginLeft);
+}
+
 #if ENABLE(FULLSCREEN_API)
 void Internals::webkitWillEnterFullScreenForElement(Document* document, Element* element)
 {
index 2c7c860..ba84c6d 100644 (file)
@@ -27,6 +27,7 @@
 #define Internals_h
 
 #include "ContextDestructionObserver.h"
+#include "ExceptionCodePlaceholder.h"
 #include "NodeList.h"
 #include "PlatformString.h"
 #include <wtf/PassRefPtr.h>
@@ -199,6 +200,10 @@ public:
     int pageNumber(Element*, float pageWidth = 800, float pageHeight = 600);
     PassRefPtr<DOMStringList> iconURLs(Document*) const;
 
+    int numberOfPages(float pageWidthInPixels = 800, float pageHeightInPixels = 600);
+    String pageProperty(String, int, ExceptionCode& = ASSERT_NO_EXCEPTION) const;
+    String pageSizeAndMarginsInPixels(int, int, int, int, int, int, int, ExceptionCode& = ASSERT_NO_EXCEPTION) const;
+
 #if ENABLE(FULLSCREEN_API)
     void webkitWillEnterFullScreenForElement(Document*, Element*);
     void webkitDidEnterFullScreenForElement(Document*, Element*);
index 7049ec8..f62ceac 100644 (file)
@@ -175,6 +175,10 @@ module window {
         DOMString counterValue(in Element element);
         long pageNumber(in Element element, in [Optional] float pageWidth, in [Optional] float pageHeight);
         DOMString[] iconURLs(in Document document);
+        long numberOfPages(in [Optional] double pageWidthInPixels, in [Optional] double pageHeightInPixels);
+        DOMString pageProperty(in DOMString propertyName, in long pageNumber) raises (DOMException);
+        DOMString pageSizeAndMarginsInPixels(in long pageIndex, in long width, in long height, in long marginTop, in long marginRight, in long marginBottom, in long marginLeft) raises (DOMException);
+
 
 #if defined(ENABLE_FULLSCREEN_API) && ENABLE_FULLSCREEN_API
         void webkitWillEnterFullScreenForElement(in Document document, in Element element);
index 354476e..f68fde8 100644 (file)
@@ -1,3 +1,15 @@
+2012-09-05  Kaustubh Atrawalkar  <kaustubh@motorola.com>
+
+        [DRT] LTC:: Move printing related APIs from LayoutTestController to Internals
+        https://bugs.webkit.org/show_bug.cgi?id=92735
+
+        Reviewed by Hajime Morita.
+
+        Move numberOfPages, pageProperty & pageSizeAndMarginsInPixels in Internals and remove duplicated code from DumprenderTree & WebkitTestRunner.
+
+        * WebCoreSupport/DumpRenderTreeSupportEfl.cpp:
+        * WebCoreSupport/DumpRenderTreeSupportEfl.h:
+
 2012-09-04  Michał Pakuła vel Rutka  <m.pakula@samsung.com>
 
         [EFL] Context menu restore. 
index 9a53d7a..1c98126 100644 (file)
@@ -155,27 +155,6 @@ void DumpRenderTreeSupportEfl::layoutFrame(Evas_Object* ewkFrame)
     frame->view()->layout();
 }
 
-int DumpRenderTreeSupportEfl::numberOfPages(const Evas_Object* ewkFrame, float pageWidth, float pageHeight)
-{
-    DRT_SUPPORT_FRAME_GET_OR_RETURN(ewkFrame, frame, 0);
-
-    return WebCore::PrintContext::numberOfPages(frame, WebCore::FloatSize(pageWidth, pageHeight));
-}
-
-String DumpRenderTreeSupportEfl::pageSizeAndMarginsInPixels(const Evas_Object* ewkFrame, int pageNumber, int width, int height, int marginTop, int marginRight, int marginBottom, int marginLeft)
-{
-    DRT_SUPPORT_FRAME_GET_OR_RETURN(ewkFrame, frame, String());
-
-    return WebCore::PrintContext::pageSizeAndMarginsInPixels(frame, pageNumber, width, height, marginTop, marginRight, marginBottom, marginLeft);
-}
-
-String DumpRenderTreeSupportEfl::pageProperty(const Evas_Object* ewkFrame, const char* propertyName, int pageNumber)
-{
-    DRT_SUPPORT_FRAME_GET_OR_RETURN(ewkFrame, frame, String());
-
-    return WebCore::PrintContext::pageProperty(frame, propertyName, pageNumber);
-}
-
 bool DumpRenderTreeSupportEfl::pauseAnimation(Evas_Object* ewkFrame, const char* name, const char* elementId, double time)
 {
     DRT_SUPPORT_FRAME_GET_OR_RETURN(ewkFrame, frame, false);
index eaf11d3..ed456ab 100644 (file)
@@ -57,10 +57,6 @@ public:
     static Eina_List* frameChildren(const Evas_Object* ewkFrame);
     static WebCore::Frame* frameParent(const Evas_Object* ewkFrame);
     static void layoutFrame(Evas_Object* ewkFrame);
-    static int numberOfPages(const Evas_Object* ewkFrame, float pageWidth, float pageHeight);
-    static int numberOfPagesForElementId(const Evas_Object* ewkFrame, const char* elementId, float pageWidth, float pageHeight);
-    static String pageProperty(const Evas_Object* ewkFrame, const char* propertyName, int pageNumber);
-    static String pageSizeAndMarginsInPixels(const Evas_Object* ewkFrame, int pageNumber, int width, int height, int marginTop, int marginRight, int marginBottom, int marginLeft);
     static bool pauseAnimation(Evas_Object* ewkFrame, const char* name, const char* elementId, double time);
     static bool pauseTransition(Evas_Object* ewkFrame, const char* name, const char* elementId, double time);
     static unsigned pendingUnloadEventCount(const Evas_Object* ewkFrame);
index a74279a..e7e2379 100644 (file)
@@ -1,3 +1,16 @@
+2012-09-05  Kaustubh Atrawalkar  <kaustubh@motorola.com>
+
+        [DRT] LTC:: Move printing related APIs from LayoutTestController to Internals
+        https://bugs.webkit.org/show_bug.cgi?id=92735
+
+        Reviewed by Hajime Morita.
+
+        Move numberOfPages, pageProperty & pageSizeAndMarginsInPixels in Internals and remove duplicated code from DumprenderTree & WebkitTestRunner.
+
+        * WebCoreSupport/DumpRenderTreeSupportGtk.cpp:
+        * WebCoreSupport/DumpRenderTreeSupportGtk.h:
+        (DumpRenderTreeSupportGtk):
+
 2012-09-04  Joanmarie Diggs  <jdiggs@igalia.com>
 
         [GTK] Crash in AccessibilityObject::accessibilityPlatformIncludesObject()
index 582cbff..502501f 100644 (file)
@@ -190,68 +190,6 @@ CString DumpRenderTreeSupportGtk::dumpRenderTree(WebKitWebFrame* frame)
 }
 
 /**
- * numberOfPagesForFrame
- * @frame: a #WebKitWebFrame
- * @pageWidth: width of a page
- * @pageHeight: height of a page
- *
- * Return value: The number of pages to be printed.
- */
-int DumpRenderTreeSupportGtk::numberOfPagesForFrame(WebKitWebFrame* frame, float pageWidth, float pageHeight)
-{
-    g_return_val_if_fail(WEBKIT_IS_WEB_FRAME(frame), 0);
-
-    Frame* coreFrame = core(frame);
-    if (!coreFrame)
-        return -1;
-
-    return PrintContext::numberOfPages(coreFrame, FloatSize(pageWidth, pageHeight));
-}
-
-/**
- * pageProperty
- * @frame: a #WebKitWebFrame
- * @propertyName: name of a property
- * @pageNumber: number of a page 
- *
- * Return value: The value of the given property name.
- */
-CString DumpRenderTreeSupportGtk::pageProperty(WebKitWebFrame* frame, const char* propertyName, int pageNumber)
-{
-    g_return_val_if_fail(WEBKIT_IS_WEB_FRAME(frame), CString());
-
-    Frame* coreFrame = core(frame);
-    if (!coreFrame)
-        return CString();
-
-    return PrintContext::pageProperty(coreFrame, propertyName, pageNumber).utf8();
-}
-
-/**
- * pageSizeAndMarginsInPixels
- * @frame: a #WebKitWebFrame
- * @pageNumber: number of a page 
- * @width: width of a page
- * @height: height of a page
- * @marginTop: top margin of a page
- * @marginRight: right margin of a page
- * @marginBottom: bottom margin of a page
- * @marginLeft: left margin of a page
- *
- * Return value: The value of page size and margin.
- */
-CString DumpRenderTreeSupportGtk::pageSizeAndMarginsInPixels(WebKitWebFrame* frame, int pageNumber, int width, int height, int marginTop, int marginRight, int marginBottom, int marginLeft)
-{
-    g_return_val_if_fail(WEBKIT_IS_WEB_FRAME(frame), CString());
-
-    Frame* coreFrame = core(frame);
-    if (!coreFrame)
-        return CString();
-
-    return PrintContext::pageSizeAndMarginsInPixels(coreFrame, pageNumber, width, height, marginTop, marginRight, marginBottom, marginLeft).utf8();
-}
-
-/**
  * addUserStyleSheet
  * @frame: a #WebKitWebFrame
  * @sourceCode: code of a user stylesheet
index 91df039..7e6e1fa 100644 (file)
@@ -59,9 +59,6 @@ public:
     static GSList* getFrameChildren(WebKitWebFrame*);
     static WTF::CString getInnerText(WebKitWebFrame*);
     static WTF::CString dumpRenderTree(WebKitWebFrame*);
-    static int numberOfPagesForFrame(WebKitWebFrame*, float pageWidth, float pageHeight);
-    static WTF::CString pageProperty(WebKitWebFrame*, const char* propertyName, int pageNumber);
-    static WTF::CString pageSizeAndMarginsInPixels(WebKitWebFrame*, int pageNumber, int width, int height, int marginTop, int marginRight, int marginBottom, int marginLeft);
     static void addUserStyleSheet(WebKitWebFrame*, const char* sourceCode, bool allFrames);
     static guint getPendingUnloadEventCount(WebKitWebFrame*);
     static bool pauseAnimation(WebKitWebFrame*, const char* name, double time, const char* element);
index 803408c..014dcbb 100644 (file)
@@ -1,3 +1,16 @@
+2012-09-05  Kaustubh Atrawalkar  <kaustubh@motorola.com>
+
+        [DRT] LTC:: Move printing related APIs from LayoutTestController to Internals
+        https://bugs.webkit.org/show_bug.cgi?id=92735
+
+        Reviewed by Hajime Morita.
+
+        Move numberOfPages, pageProperty & pageSizeAndMarginsInPixels in Internals and remove duplicated code from DumprenderTree & WebkitTestRunner.
+
+        * WebCoreSupport/DumpRenderTreeSupportQt.cpp:
+        (DumpRenderTreeSupportQt::evaluateScriptInIsolatedWorld):
+        * WebCoreSupport/DumpRenderTreeSupportQt.h:
+
 2012-09-04  Jon Lee  <jonlee@apple.com>
 
         [Qt] REGRESSION(r127321): It made 3 notification test timeout
index 6b178c5..ac1654e 100644 (file)
@@ -386,15 +386,6 @@ void DumpRenderTreeSupportQt::garbageCollectorCollectOnAlternateThread(bool wait
     gcController().garbageCollectOnAlternateThreadForDebugging(waitUntilDone);
 }
 
-int DumpRenderTreeSupportQt::numberOfPages(QWebFrame* frame, float width, float height)
-{
-    Frame* coreFrame = QWebFramePrivate::core(frame);
-    if (!coreFrame)
-        return -1;
-
-    return PrintContext::numberOfPages(coreFrame, FloatSize(width, height));
-}
-
 // Suspend active DOM objects in this frame.
 void DumpRenderTreeSupportQt::suspendActiveDOMObjects(QWebFrame* frame)
 {
@@ -858,19 +849,6 @@ void DumpRenderTreeSupportQt::evaluateScriptInIsolatedWorld(QWebFrame* frame, in
     proxy->executeScriptInWorld(scriptWorld->world(), script, true);
 }
 
-QString DumpRenderTreeSupportQt::pageSizeAndMarginsInPixels(QWebFrame* frame, int pageIndex, int width, int height, int marginTop, int marginRight, int marginBottom, int marginLeft)
-{
-    WebCore::Frame* coreFrame = QWebFramePrivate::core(frame);
-    return PrintContext::pageSizeAndMarginsInPixels(coreFrame, pageIndex, width, height,
-                                                    marginTop, marginRight, marginBottom, marginLeft);
-}
-
-QString DumpRenderTreeSupportQt::pageProperty(QWebFrame* frame, const QString& propertyName, int pageNumber)
-{
-    WebCore::Frame* coreFrame = QWebFramePrivate::core(frame);
-    return PrintContext::pageProperty(coreFrame, propertyName.toUtf8().constData(), pageNumber);
-}
-
 void DumpRenderTreeSupportQt::addUserStyleSheet(QWebPage* page, const QString& sourceCode)
 {
     page->handle()->page->group().addUserStyleSheetToWorld(mainThreadNormalWorld(), sourceCode, QUrl(), nullptr, nullptr, WebCore::InjectInAllFrames);
index 5ed2492..92759ac 100644 (file)
@@ -127,7 +127,6 @@ public:
     static void clearFrameName(QWebFrame* frame);
     static void overwritePluginDirectories();
     static int numberOfActiveAnimations(QWebFrame*);
-    static int numberOfPages(QWebFrame* frame, float width, float height);
     static bool hasDocumentElement(QWebFrame* frame);
     static bool elementDoesAutoCompleteForElementWithId(QWebFrame* frame, const QString& elementId);
     static void setWindowsBehaviorAsEditingBehavior(QWebPage*);
@@ -179,8 +178,6 @@ public:
 
     static void setCustomPolicyDelegate(bool enabled, bool permissive);
 
-    static QString pageSizeAndMarginsInPixels(QWebFrame* frame, int pageIndex, int width, int height, int marginTop, int marginRight, int marginBottom, int marginLeft);
-    static QString pageProperty(QWebFrame* frame, const QString& propertyName, int pageNumber);
     static void addUserStyleSheet(QWebPage* page, const QString& sourceCode);
     static void removeUserStyleSheets(QWebPage*);
     static void simulateDesktopNotificationClick(const QString& title);
index 07e4a6e..2afe260 100644 (file)
@@ -1,3 +1,16 @@
+2012-09-05  Kaustubh Atrawalkar  <kaustubh@motorola.com>
+
+        [DRT] LTC:: Move printing related APIs from LayoutTestController to Internals
+        https://bugs.webkit.org/show_bug.cgi?id=92735
+
+        Reviewed by Hajime Morita.
+
+        Move numberOfPages, pageProperty & pageSizeAndMarginsInPixels in Internals and remove duplicated code from DumprenderTree & WebkitTestRunner.
+        Kept the function defination and declaration to keep binary compatibility for IWebFramePrivate.idl
+
+        * WebFrame.cpp:
+        (WebFrame::numberOfPages):
+
 2012-08-30  Benjamin Poulain  <bpoulain@apple.com>
 
         Replace JSC::UString by WTF::String
index 062cf14..8a0a82c 100644 (file)
@@ -894,15 +894,10 @@ HRESULT STDMETHODCALLTYPE WebFrame::numberOfPages(
     /* [in] */ float pageHeightInPixels,
     /* [retval][out] */ int* result)
 {
-    if (!result)
-        return E_POINTER;
-
-    Frame* coreFrame = core(this);
-    if (!coreFrame)
-        return E_FAIL;
-
-    *result = PrintContext::numberOfPages(coreFrame, FloatSize(pageWidthInPixels, pageHeightInPixels));
-    return S_OK;
+    // TODO: Please remove this function if not needed as this is LTC specific function
+    // and has been moved to Internals.
+    notImplemented();
+    return E_FAIL;
 }
 
 HRESULT STDMETHODCALLTYPE WebFrame::scrollOffset(
index 5588e17..fa80381 100644 (file)
@@ -1,3 +1,16 @@
+2012-09-05  Kaustubh Atrawalkar  <kaustubh@motorola.com>
+
+        [DRT] LTC:: Move printing related APIs from LayoutTestController to Internals
+        https://bugs.webkit.org/show_bug.cgi?id=92735
+
+        Reviewed by Hajime Morita.
+
+        Move numberOfPages, pageProperty & pageSizeAndMarginsInPixels in Internals and remove duplicated code from DumprenderTree & WebkitTestRunner.
+        Add symbols for Mac Win builds for corresponding newly added APIs.
+
+        * win/WebKit2.def:
+        * win/WebKit2CFLite.def:
+
 2012-09-05  Balazs Kelemen  <kbalazs@webkit.org>
 
         [Qt] QRawWebViewPrivate does not initialize some members
index 651c98f..975a73d 100644 (file)
@@ -276,7 +276,10 @@ EXPORTS
         ?setAllowsRoundingHacks@TextRun@WebCore@@SAX_N@Z
         ?profilerEnabled@InspectorController@WebCore@@QAE_NXZ
         ?setProfilerEnabled@InspectorController@WebCore@@QAEX_N@Z
-        ?jsStringWithCacheSlowCase@WebCore@@YA?AVJSValue@JSC@@PAVExecState@3@AAV?$HashMap@PAVStringImpl@WTF@@V?$Weak@VJSString@JSC@@@JSC@@U?$PtrHash@PAVStringImpl@WTF@@@2@U?$HashTraits@PAVStringImpl@WTF@@@2@U?$HashTraits@V?$Weak@VJSString@JSC@@@JSC@@@2@@WTF@@PAVStringImpl@6@@Z        
+        ?jsStringWithCacheSlowCase@WebCore@@YA?AVJSValue@JSC@@PAVExecState@3@AAV?$HashMap@PAVStringImpl@WTF@@V?$Weak@VJSString@JSC@@@JSC@@U?$PtrHash@PAVStringImpl@WTF@@@2@U?$HashTraits@PAVStringImpl@WTF@@@2@U?$HashTraits@V?$Weak@VJSString@JSC@@@JSC@@@2@@WTF@@PAVStringImpl@6@@Z
         ?registerURLSchemeAsBypassingContentSecurityPolicy@SchemeRegistry@WebCore@@SAXABVString@WTF@@@Z
         ?removeURLSchemeRegisteredAsBypassingContentSecurityPolicy@SchemeRegistry@WebCore@@SAXABVString@WTF@@@Z
         ?iconURLs@Document@WebCore@@QAEABV?$Vector@UIconURL@WebCore@@$0A@@WTF@@XZ
+        ?numberOfPages@PrintContext@WebCore@@SAHPAVFrame@2@ABVFloatSize@2@@Z
+        ?pageProperty@PrintContext@WebCore@@SA?AVString@WTF@@PAVFrame@2@PBDH@Z
+        ?pageSizeAndMarginsInPixels@PrintContext@WebCore@@SA?AVString@WTF@@PAVFrame@2@HHHHHHH@Z
index d3bef46..33e8deb 100644 (file)
@@ -270,3 +270,6 @@ EXPORTS
         ?jsStringWithCacheSlowCase@WebCore@@YA?AVJSValue@JSC@@PAVExecState@3@AAV?$HashMap@PAVStringImpl@WTF@@V?$Weak@VJSString@JSC@@@JSC@@U?$PtrHash@PAVStringImpl@WTF@@@2@U?$HashTraits@PAVStringImpl@WTF@@@2@U?$HashTraits@V?$Weak@VJSString@JSC@@@JSC@@@2@@WTF@@PAVStringImpl@6@@Z
         ?registerURLSchemeAsBypassingContentSecurityPolicy@SchemeRegistry@WebCore@@SAXABVString@WTF@@@Z
         ?removeURLSchemeRegisteredAsBypassingContentSecurityPolicy@SchemeRegistry@WebCore@@SAXABVString@WTF@@@Z
+        ?numberOfPages@PrintContext@WebCore@@SAHPAVFrame@2@ABVFloatSize@2@@Z
+        ?pageProperty@PrintContext@WebCore@@SA?AVString@WTF@@PAVFrame@2@PBDH@Z
+        ?pageSizeAndMarginsInPixels@PrintContext@WebCore@@SA?AVString@WTF@@PAVFrame@2@HHHHHHH@Z
index b63b7c1..1a21ef9 100644 (file)
@@ -175,6 +175,9 @@ _ZNK7WebCore14InsertionPoint8isActiveEv;
 _ZN7WebCore26ContextDestructionObserverD2Ev;
 _ZN7WebCore26ContextDestructionObserverC2EPNS_22ScriptExecutionContextE;
 _ZN7WebCore26ContextDestructionObserver16contextDestroyedEv;
+_ZN7WebCore12PrintContext13numberOfPagesEPNS_5FrameERKNS_9FloatSizeE;
+_ZN7WebCore12PrintContext12pagePropertyEPNS_5FrameEPKci;
+_ZN7WebCore12PrintContext26pageSizeAndMarginsInPixelsEPNS_5FrameEiiiiiii;
 
 local:
 _Z*;
index 11a1c67..7a2cd17 100644 (file)
@@ -1,3 +1,39 @@
+2012-09-05  Kaustubh Atrawalkar  <kaustubh@motorola.com>
+
+        [DRT] LTC:: Move printing related APIs from LayoutTestController to Internals
+        https://bugs.webkit.org/show_bug.cgi?id=92735
+
+        Reviewed by Hajime Morita.
+
+        Move numberOfPages, pageProperty & pageSizeAndMarginsInPixels in Internals and remove duplicated code from DumprenderTree & WebkitTestRunner.
+
+        * DumpRenderTree/TestRunner.cpp:
+        (TestRunner::staticFunctions):
+        * DumpRenderTree/TestRunner.h:
+        (TestRunner):
+        * DumpRenderTree/blackberry/TestRunnerBlackBerry.cpp:
+        (TestRunner::addUserStyleSheet):
+        * DumpRenderTree/chromium/DRTTestRunner.cpp:
+        (DRTTestRunner::DRTTestRunner):
+        (DRTTestRunner::hasCustomPageSizeStyle):
+        * DumpRenderTree/chromium/DRTTestRunner.h:
+        (DRTTestRunner):
+        * DumpRenderTree/efl/TestRunnerEfl.cpp:
+        * DumpRenderTree/gtk/TestRunnerGtk.cpp:
+        * DumpRenderTree/mac/TestRunnerMac.mm:
+        * DumpRenderTree/qt/TestRunnerQt.cpp:
+        (TestRunner::evaluateScriptInIsolatedWorld):
+        * DumpRenderTree/qt/TestRunnerQt.h:
+        (TestRunner):
+        * DumpRenderTree/win/TestRunnerWin.cpp:
+        * DumpRenderTree/wx/TestRunnerWx.cpp:
+        (TestRunner::abortModal):
+        * WebKitTestRunner/InjectedBundle/Bindings/TestRunner.idl:
+        * WebKitTestRunner/InjectedBundle/TestRunner.cpp:
+        (WTR):
+        * WebKitTestRunner/InjectedBundle/TestRunner.h:
+        (TestRunner):
+
 2012-09-05  Simon Hausmann  <simon.hausmann@nokia.com>
 
         [Qt] Automate the use of AllInOne files in the Qt build
index 794eb19..1d00c50 100644 (file)
@@ -715,143 +715,12 @@ static JSValueRef notifyDoneCallback(JSContextRef context, JSObjectRef function,
     return JSValueMakeUndefined(context);
 }
 
-static bool parsePageParameters(JSContextRef context, int argumentCount, const JSValueRef* arguments, JSValueRef* exception, float& pageWidthInPixels, float& pageHeightInPixels)
-{
-    pageWidthInPixels = TestRunner::maxViewWidth;
-    pageHeightInPixels = TestRunner::maxViewHeight;
-    switch (argumentCount) {
-    case 2:
-        pageWidthInPixels = static_cast<float>(JSValueToNumber(context, arguments[0], exception));
-        if (*exception)
-            return false;
-        pageHeightInPixels = static_cast<float>(JSValueToNumber(context, arguments[1], exception));
-        if (*exception)
-            return false;
-    case 0: // Fall through.
-        break;
-    default:
-        return false;
-    }
-    return true;
-}
-
-// Caller needs to delete[] propertyName.
-static bool parsePagePropertyParameters(JSContextRef context, int argumentCount, const JSValueRef* arguments, JSValueRef* exception, char*& propertyName, int& pageNumber)
-{
-    pageNumber = 0;
-    switch (argumentCount) {
-    case 2:
-        pageNumber = static_cast<float>(JSValueToNumber(context, arguments[1], exception));
-        if (*exception)
-            return false;
-        // Fall through.
-    case 1: {
-        JSRetainPtr<JSStringRef> propertyNameString(Adopt, JSValueToStringCopy(context, arguments[0], exception));
-        if (*exception)
-            return false;
-
-        size_t maxLength = JSStringGetMaximumUTF8CStringSize(propertyNameString.get());
-        propertyName = new char[maxLength + 1];
-        JSStringGetUTF8CString(propertyNameString.get(), propertyName, maxLength + 1);
-        return true;
-    }
-    case 0:
-    default:
-        return false;
-    }
-}
-
-static bool parsePageNumberSizeMarings(JSContextRef context, int argumentCount, const JSValueRef* arguments, JSValueRef* exception, int& pageNumber, int& width, int& height, int& marginTop, int& marginRight, int& marginBottom, int& marginLeft)
-{
-    pageNumber = 0;
-    width = height = 0;
-    marginTop = marginRight = marginBottom = marginLeft = 0;
-
-    switch (argumentCount) {
-    case 7:
-        marginLeft = static_cast<int>(JSValueToNumber(context, arguments[6], exception));
-        if (*exception)
-            return false;
-        // Fall through.
-    case 6:
-        marginBottom = static_cast<int>(JSValueToNumber(context, arguments[5], exception));
-        if (*exception)
-            return false;
-        // Fall through.
-    case 5:
-        marginRight = static_cast<int>(JSValueToNumber(context, arguments[4], exception));
-        if (*exception)
-            return false;
-        // Fall through.
-    case 4:
-        marginTop = static_cast<int>(JSValueToNumber(context, arguments[3], exception));
-        if (*exception)
-            return false;
-        // Fall through.
-    case 3:
-        height = static_cast<int>(JSValueToNumber(context, arguments[2], exception));
-        if (*exception)
-            return false;
-        // Fall through.
-    case 2:
-        width = static_cast<int>(JSValueToNumber(context, arguments[1], exception));
-        if (*exception)
-            return false;
-        // Fall through.
-    case 1:
-        pageNumber = static_cast<int>(JSValueToNumber(context, arguments[0], exception));
-        if (*exception)
-            return false;
-        // Fall through.
-        return true;
-    default:
-        return false;
-    }
-}
-
-static JSValueRef numberOfPagesCallback(JSContextRef context, JSObjectRef function, JSObjectRef thisObject, size_t argumentCount, const JSValueRef arguments[], JSValueRef* exception)
-{
-    float pageWidthInPixels = 0;
-    float pageHeightInPixels = 0;
-    if (!parsePageParameters(context, argumentCount, arguments, exception, pageWidthInPixels, pageHeightInPixels))
-        return JSValueMakeUndefined(context);
-
-    TestRunner* controller = static_cast<TestRunner*>(JSObjectGetPrivate(thisObject));
-    return JSValueMakeNumber(context, controller->numberOfPages(pageWidthInPixels, pageHeightInPixels));
-}
-
 static JSValueRef numberOfPendingGeolocationPermissionRequestsCallback(JSContextRef context, JSObjectRef function, JSObjectRef thisObject, size_t argumentCount, const JSValueRef arguments[], JSValueRef* exception)
 {
     TestRunner* controller = static_cast<TestRunner*>(JSObjectGetPrivate(thisObject));
     return JSValueMakeNumber(context, controller->numberOfPendingGeolocationPermissionRequests());
 }
 
-static JSValueRef pagePropertyCallback(JSContextRef context, JSObjectRef function, JSObjectRef thisObject, size_t argumentCount, const JSValueRef arguments[], JSValueRef* exception)
-{
-    char* propertyName = 0;
-    int pageNumber = 0;
-    if (!parsePagePropertyParameters(context, argumentCount, arguments, exception, propertyName, pageNumber))
-        return JSValueMakeUndefined(context);
-
-    TestRunner* controller = static_cast<TestRunner*>(JSObjectGetPrivate(thisObject));
-    JSValueRef value = JSValueMakeString(context, controller->pageProperty(propertyName, pageNumber).get());
-
-    delete[] propertyName;
-    return value;
-}
-
-static JSValueRef pageSizeAndMarginsInPixelsCallback(JSContextRef context, JSObjectRef function, JSObjectRef thisObject, size_t argumentCount, const JSValueRef arguments[], JSValueRef* exception)
-{
-    int pageNumber = 0;
-    int width = 0, height = 0;
-    int marginTop = 0, marginRight = 0, marginBottom = 0, marginLeft = 0;
-    if (!parsePageNumberSizeMarings(context, argumentCount, arguments, exception, pageNumber, width, height, marginTop, marginRight, marginBottom, marginLeft))
-        return JSValueMakeUndefined(context);
-
-    TestRunner* controller = static_cast<TestRunner*>(JSObjectGetPrivate(thisObject));
-    return JSValueMakeString(context, controller->pageSizeAndMarginsInPixels(pageNumber, width, height, marginTop, marginRight, marginBottom, marginLeft).get());
-}
-
 static JSValueRef queueBackNavigationCallback(JSContextRef context, JSObjectRef function, JSObjectRef thisObject, size_t argumentCount, const JSValueRef arguments[], JSValueRef* exception)
 {
     // Has mac & windows implementation
@@ -2347,14 +2216,11 @@ JSStaticFunction* TestRunner::staticFunctions()
         { "isCommandEnabled", isCommandEnabledCallback, kJSPropertyAttributeReadOnly | kJSPropertyAttributeDontDelete },
         { "keepWebHistory", keepWebHistoryCallback, kJSPropertyAttributeReadOnly | kJSPropertyAttributeDontDelete },
         { "layerTreeAsText", layerTreeAsTextCallback, kJSPropertyAttributeReadOnly | kJSPropertyAttributeDontDelete },
-        { "numberOfPages", numberOfPagesCallback, kJSPropertyAttributeReadOnly | kJSPropertyAttributeDontDelete },
         { "numberOfPendingGeolocationPermissionRequests", numberOfPendingGeolocationPermissionRequestsCallback, kJSPropertyAttributeReadOnly | kJSPropertyAttributeDontDelete },
         { "markerTextForListItem", markerTextForListItemCallback, kJSPropertyAttributeReadOnly | kJSPropertyAttributeDontDelete },
         { "notifyDone", notifyDoneCallback, kJSPropertyAttributeReadOnly | kJSPropertyAttributeDontDelete },
         { "numberOfActiveAnimations", numberOfActiveAnimationsCallback, kJSPropertyAttributeReadOnly | kJSPropertyAttributeDontDelete },
         { "overridePreference", overridePreferenceCallback, kJSPropertyAttributeReadOnly | kJSPropertyAttributeDontDelete },
-        { "pageSizeAndMarginsInPixels", pageSizeAndMarginsInPixelsCallback, kJSPropertyAttributeReadOnly | kJSPropertyAttributeDontDelete },
-        { "pageProperty", pagePropertyCallback, kJSPropertyAttributeReadOnly | kJSPropertyAttributeDontDelete },
         { "pathToLocalResource", pathToLocalResourceCallback, kJSPropertyAttributeReadOnly | kJSPropertyAttributeDontDelete },
         { "pauseAnimationAtTimeOnElementWithId", pauseAnimationAtTimeOnElementWithIdCallback, kJSPropertyAttributeReadOnly | kJSPropertyAttributeDontDelete },
         { "pauseTransitionAtTimeOnElementWithId", pauseTransitionAtTimeOnElementWithIdCallback, kJSPropertyAttributeReadOnly | kJSPropertyAttributeDontDelete },
index f93685c..9bcc95b 100644 (file)
@@ -70,11 +70,8 @@ public:
     void keepWebHistory();
     JSValueRef computedStyleIncludingVisitedInfo(JSContextRef, JSValueRef);
     void notifyDone();
-    int numberOfPages(float pageWidthInPixels, float pageHeightInPixels);
     int numberOfPendingGeolocationPermissionRequests();
     void overridePreference(JSStringRef key, JSStringRef value);
-    JSRetainPtr<JSStringRef> pageProperty(const char* propertyName, int pageNumber) const;
-    JSRetainPtr<JSStringRef> pageSizeAndMarginsInPixels(int pageNumber, int width, int height, int marginTop, int marginRight, int marginBottom, int marginLeft) const;
     JSStringRef pathToLocalResource(JSContextRef, JSStringRef url);
     void queueBackNavigation(int howFarBackward);
     void queueForwardNavigation(int howFarForward);
index 6074614..8986900 100644 (file)
@@ -325,14 +325,6 @@ bool TestRunner::elementDoesAutoCompleteForElementWithId(JSStringRef id)
     return false;
 }
 
-JSRetainPtr<JSStringRef> TestRunner::pageProperty(const char* propertyName, int pageNumber) const
-{
-    UNUSED_PARAM(propertyName);
-    UNUSED_PARAM(pageNumber);
-    notImplemented();
-    return 0;
-}
-
 void TestRunner::setWaitToDump(bool waitToDump)
 {
     // Change from 30s to 35s because some test cases in multipart need 30 seconds,
@@ -509,18 +501,6 @@ void TestRunner::addUserStyleSheet(JSStringRef, bool)
     notImplemented();
 }
 
-JSRetainPtr<JSStringRef> TestRunner::pageSizeAndMarginsInPixels(int, int, int, int, int, int, int) const
-{
-    notImplemented();
-    return 0;
-}
-
-int TestRunner::numberOfPages(float, float)
-{
-    notImplemented();
-    return -1;
-}
-
 void TestRunner::setScrollbarPolicy(JSStringRef, JSStringRef)
 {
     notImplemented();
index 0bee8a9..0e5a8b3 100644 (file)
@@ -170,12 +170,9 @@ DRTTestRunner::DRTTestRunner(TestShell* shell)
     bindMethod("markerTextForListItem", &DRTTestRunner::markerTextForListItem);
     bindMethod("notifyDone", &DRTTestRunner::notifyDone);
     bindMethod("numberOfActiveAnimations", &DRTTestRunner::numberOfActiveAnimations);
-    bindMethod("numberOfPages", &DRTTestRunner::numberOfPages);
     bindMethod("numberOfPendingGeolocationPermissionRequests", &DRTTestRunner:: numberOfPendingGeolocationPermissionRequests);
     bindMethod("objCIdentityIsEqual", &DRTTestRunner::objCIdentityIsEqual);
     bindMethod("overridePreference", &DRTTestRunner::overridePreference);
-    bindMethod("pageProperty", &DRTTestRunner::pageProperty);
-    bindMethod("pageSizeAndMarginsInPixels", &DRTTestRunner::pageSizeAndMarginsInPixels);
     bindMethod("pathToLocalResource", &DRTTestRunner::pathToLocalResource);
     bindMethod("pauseAnimationAtTimeOnElementWithId", &DRTTestRunner::pauseAnimationAtTimeOnElementWithId);
     bindMethod("pauseTransitionAtTimeOnElementWithId", &DRTTestRunner::pauseTransitionAtTimeOnElementWithId);
@@ -1694,23 +1691,6 @@ static bool parseCppArgumentInt32(const CppArgumentList& arguments, int argIndex
     return true;
 }
 
-static bool parsePageSizeParameters(const CppArgumentList& arguments,
-                                    int argOffset,
-                                    int* pageWidthInPixels,
-                                    int* pageHeightInPixels)
-{
-    // WebKit is using the window width/height of DumpRenderTree as the
-    // default value of the page size.
-    // FIXME: share the default values with other ports.
-    int argCount = static_cast<int>(arguments.size()) - argOffset;
-    if (argCount && argCount != 2)
-        return false;
-    if (!parseCppArgumentInt32(arguments, argOffset, pageWidthInPixels, 800)
-        || !parseCppArgumentInt32(arguments, argOffset + 1, pageHeightInPixels, 600))
-        return false;
-    return true;
-}
-
 static bool parsePageNumber(const CppArgumentList& arguments, int argOffset, int* pageNumber)
 {
     if (static_cast<int>(arguments.size()) > argOffset + 1)
@@ -1720,55 +1700,12 @@ static bool parsePageNumber(const CppArgumentList& arguments, int argOffset, int
     return true;
 }
 
-static bool parsePageNumberSizeMargins(const CppArgumentList& arguments, int argOffset,
-                                       int* pageNumber, int* width, int* height,
-                                       int* marginTop, int* marginRight, int* marginBottom, int* marginLeft)
-{
-    int argCount = static_cast<int>(arguments.size()) - argOffset;
-    if (argCount && argCount != 7)
-        return false;
-    if (!parseCppArgumentInt32(arguments, argOffset, pageNumber, 0)
-        || !parseCppArgumentInt32(arguments, argOffset + 1, width, 0)
-        || !parseCppArgumentInt32(arguments, argOffset + 2, height, 0)
-        || !parseCppArgumentInt32(arguments, argOffset + 3, marginTop, 0)
-        || !parseCppArgumentInt32(arguments, argOffset + 4, marginRight, 0)
-        || !parseCppArgumentInt32(arguments, argOffset + 5, marginBottom, 0)
-        || !parseCppArgumentInt32(arguments, argOffset + 6, marginLeft, 0))
-        return false;
-    return true;
-}
-
 void DRTTestRunner::setPrinting(const CppArgumentList& arguments, CppVariant* result)
 {
     setIsPrinting(true);
     result->setNull();
 }
 
-void DRTTestRunner::pageSizeAndMarginsInPixels(const CppArgumentList& arguments, CppVariant* result)
-{
-    result->set("");
-    int pageNumber = 0;
-    int width = 0;
-    int height = 0;
-    int marginTop = 0;
-    int marginRight = 0;
-    int marginBottom = 0;
-    int marginLeft = 0;
-    if (!parsePageNumberSizeMargins(arguments, 0, &pageNumber, &width, &height, &marginTop, &marginRight, &marginBottom,
-                                    &marginLeft))
-        return;
-
-    WebFrame* frame = m_shell->webView()->mainFrame();
-    if (!frame)
-        return;
-    WebSize pageSize(width, height);
-    frame->pageSizeAndMarginsInPixels(pageNumber, pageSize, marginTop, marginRight, marginBottom, marginLeft);
-    stringstream resultString;
-    resultString << "(" << pageSize.width << ", " << pageSize.height << ") " << marginTop << " " << marginRight << " "
-                 << marginBottom << " " << marginLeft;
-    result->set(resultString.str());
-}
-
 void DRTTestRunner::hasCustomPageSizeStyle(const CppArgumentList& arguments, CppVariant* result)
 {
     result->set(false);
@@ -1781,40 +1718,6 @@ void DRTTestRunner::hasCustomPageSizeStyle(const CppArgumentList& arguments, Cpp
     result->set(frame->hasCustomPageSizeStyle(pageIndex));
 }
 
-void DRTTestRunner::pageProperty(const CppArgumentList& arguments, CppVariant* result)
-{
-    result->set("");
-    int pageNumber = 0;
-    if (!parsePageNumber(arguments, 1, &pageNumber))
-        return;
-    if (!arguments[0].isString())
-        return;
-    WebFrame* frame = m_shell->webView()->mainFrame();
-    if (!frame)
-        return;
-    WebSize pageSize(800, 800);
-    frame->printBegin(pageSize);
-    result->set(frame->pageProperty(cppVariantToWebString(arguments[0]), pageNumber).utf8());
-    frame->printEnd();
-}
-
-void DRTTestRunner::numberOfPages(const CppArgumentList& arguments, CppVariant* result)
-{
-    result->setNull();
-    int pageWidthInPixels = 0;
-    int pageHeightInPixels = 0;
-    if (!parsePageSizeParameters(arguments, 0, &pageWidthInPixels, &pageHeightInPixels))
-        return;
-
-    WebFrame* frame = m_shell->webView()->mainFrame();
-    if (!frame)
-        return;
-    WebPrintParams printParams(WebSize(pageWidthInPixels, pageHeightInPixels));
-    int numberOfPages = frame->printBegin(printParams);
-    frame->printEnd();
-    result->set(numberOfPages);
-}
-
 void DRTTestRunner::numberOfPendingGeolocationPermissionRequests(const CppArgumentList& arguments, CppVariant* result)
 {
     result->setNull();
index f8a5867..b6c907e 100644 (file)
@@ -314,19 +314,10 @@ public:
     // Causes layout to happen as if targetted to printed pages.
     void setPrinting(const CppArgumentList&, CppVariant*);
 
-    // Gets the page size and margins for a printed page.
-    void pageSizeAndMarginsInPixels(const CppArgumentList&, CppVariant*);
-
     // Returns true if the current page box has custom page size style for
     // printing.
     void hasCustomPageSizeStyle(const CppArgumentList&, CppVariant*);
 
-    // Gets the page-related property for printed content
-    void pageProperty(const CppArgumentList&, CppVariant*);
-
-    // Gets the number of pages to be printed.
-    void numberOfPages(const CppArgumentList&, CppVariant*);
-
     // Gets the number of geolocation permissions requests pending.
     void numberOfPendingGeolocationPermissionRequests(const CppArgumentList&, CppVariant*);
 
index b2f332e..8a04e34 100644 (file)
@@ -123,32 +123,6 @@ JSRetainPtr<JSStringRef> TestRunner::layerTreeAsText() const
     return JSRetainPtr<JSStringRef>(Adopt, JSStringCreateWithUTF8CString(""));
 }
 
-int TestRunner::numberOfPages(float pageWidth, float pageHeight)
-{
-    return DumpRenderTreeSupportEfl::numberOfPages(browser->mainFrame(), pageWidth, pageHeight);
-}
-
-JSRetainPtr<JSStringRef> TestRunner::pageProperty(const char* propertyName, int pageNumber) const
-{
-    const String property = DumpRenderTreeSupportEfl::pageProperty(browser->mainFrame(), propertyName, pageNumber);
-    if (property.isEmpty())
-        return 0;
-
-    JSRetainPtr<JSStringRef> propertyValue(Adopt, JSStringCreateWithUTF8CString(property.utf8().data()));
-    return propertyValue;
-}
-
-JSRetainPtr<JSStringRef> TestRunner::pageSizeAndMarginsInPixels(int pageNumber, int width, int height, int marginTop, int marginRight, int marginBottom, int marginLeft) const
-{
-    String pageSizeAndMargins = DumpRenderTreeSupportEfl::pageSizeAndMarginsInPixels(browser->mainFrame(), pageNumber, width, height, marginTop, marginRight, marginBottom, marginLeft);
-
-    if (pageSizeAndMargins.isEmpty())
-        return 0;
-
-    JSRetainPtr<JSStringRef> returnValue(Adopt, JSStringCreateWithUTF8CString(pageSizeAndMargins.utf8().data()));
-    return returnValue;
-}
-
 size_t TestRunner::webHistoryItemCount()
 {
     const Ewk_History* history = ewk_view_history_get(browser->mainView());
index 848f78c..34d2cc8 100644 (file)
@@ -119,23 +119,6 @@ JSRetainPtr<JSStringRef> TestRunner::layerTreeAsText() const
     return string;
 }
 
-int TestRunner::numberOfPages(float pageWidth, float pageHeight)
-{
-    return DumpRenderTreeSupportGtk::numberOfPagesForFrame(mainFrame, pageWidth, pageHeight);
-}
-
-JSRetainPtr<JSStringRef> TestRunner::pageProperty(const char* propertyName, int pageNumber) const
-{
-    JSRetainPtr<JSStringRef> propertyValue(Adopt, JSStringCreateWithUTF8CString(DumpRenderTreeSupportGtk::pageProperty(mainFrame, propertyName, pageNumber).data()));
-    return propertyValue;
-}
-
-JSRetainPtr<JSStringRef> TestRunner::pageSizeAndMarginsInPixels(int pageNumber, int width, int height, int marginTop, int marginRight, int marginBottom, int marginLeft) const
-{
-    JSRetainPtr<JSStringRef> propertyValue(Adopt, JSStringCreateWithUTF8CString(DumpRenderTreeSupportGtk::pageSizeAndMarginsInPixels(mainFrame, pageNumber, width, height, marginTop, marginRight, marginBottom, marginLeft).data()));
-    return propertyValue;
-}
-
 size_t TestRunner::webHistoryItemCount()
 {
     WebKitWebView* webView = webkit_web_frame_get_web_view(mainFrame);
index b376913..a21ed80 100644 (file)
@@ -285,23 +285,6 @@ JSRetainPtr<JSStringRef> TestRunner::markerTextForListItem(JSContextRef context,
     return markerText;
 }
 
-JSRetainPtr<JSStringRef> TestRunner::pageProperty(const char* propertyName, int pageNumber) const
-{
-    JSRetainPtr<JSStringRef> propertyValue(Adopt, JSStringCreateWithCFString((CFStringRef)[mainFrame pageProperty:propertyName:pageNumber]));
-    return propertyValue;
-}
-
-JSRetainPtr<JSStringRef> TestRunner::pageSizeAndMarginsInPixels(int pageNumber, int width, int height, int marginTop, int marginRight, int marginBottom, int marginLeft) const
-{
-    JSRetainPtr<JSStringRef> propertyValue(Adopt, JSStringCreateWithCFString((CFStringRef)[mainFrame pageSizeAndMarginsInPixels:pageNumber:width:height:marginTop:marginRight:marginBottom:marginLeft]));
-    return propertyValue;
-}
-
-int TestRunner::numberOfPages(float pageWidthInPixels, float pageHeightInPixels)
-{
-    return [mainFrame numberOfPages:pageWidthInPixels:pageHeightInPixels];
-}
-
 int TestRunner::numberOfPendingGeolocationPermissionRequests()
 {
     return [[[mainFrame webView] UIDelegate] numberOfPendingGeolocationPermissionRequests];
index 5e3624f..4f8d070 100644 (file)
@@ -726,11 +726,6 @@ int TestRunner::workerThreadCount()
     return DumpRenderTreeSupportQt::workerThreadCount();
 }
 
-int TestRunner::numberOfPages(float width, float height)
-{
-    return DumpRenderTreeSupportQt::numberOfPages(m_drt->webPage()->mainFrame(), width, height);
-}
-
 bool TestRunner::callShouldCloseOnWebView()
 {
     return DumpRenderTreeSupportQt::shouldClose(m_drt->webPage()->mainFrame());
@@ -886,17 +881,6 @@ void TestRunner::evaluateScriptInIsolatedWorld(int worldID, const QString& scrip
     DumpRenderTreeSupportQt::evaluateScriptInIsolatedWorld(m_drt->webPage()->mainFrame(), worldID, script);
 }
 
-QString TestRunner::pageSizeAndMarginsInPixels(int pageIndex, int width, int height, int marginTop, int marginRight, int marginBottom, int marginLeft)
-{
-    return DumpRenderTreeSupportQt::pageSizeAndMarginsInPixels(m_drt->webPage()->mainFrame(), pageIndex,
-                                                               width, height, marginTop, marginRight, marginBottom, marginLeft);
-}
-
-QString TestRunner::pageProperty(const QString& propertyName, int pageNumber)
-{
-    return DumpRenderTreeSupportQt::pageProperty(m_drt->webPage()->mainFrame(), propertyName, pageNumber);
-}
-
 void TestRunner::addUserStyleSheet(const QString& sourceCode)
 {
     DumpRenderTreeSupportQt::addUserStyleSheet(m_drt->webPage(), sourceCode);
index 36d9d1b..486b9fd 100644 (file)
@@ -214,7 +214,6 @@ public Q_SLOTS:
     void setUserStyleSheetEnabled(bool);
     void setDomainRelaxationForbiddenForURLScheme(bool forbidden, const QString& scheme);
     int workerThreadCount();
-    int numberOfPages(float width = maxViewWidth, float height = maxViewHeight);
     bool callShouldCloseOnWebView();
     // For now, this is a no-op. This may change depending on outcome of
     // https://bugs.webkit.org/show_bug.cgi?id=33333
@@ -261,8 +260,6 @@ public Q_SLOTS:
 
     void evaluateScriptInIsolatedWorldAndReturnValue(int worldID, const QString& script);
     void evaluateScriptInIsolatedWorld(int worldID, const QString& script);
-    QString pageSizeAndMarginsInPixels(int pageIndex, int width, int height, int marginTop, int marginRight, int marginBottom, int marginLeft);
-    QString pageProperty(const QString& propertyName, int pageNumber);
     void addUserStyleSheet(const QString& sourceCode);
 
     void setMinimumTimerInterval(double);
index 530260b..d33651b 100644 (file)
@@ -1300,35 +1300,11 @@ void TestRunner::removeAllVisitedLinks()
     sharedHistoryPrivate->removeAllVisitedLinks();
 }
 
-int TestRunner::numberOfPages(float pageWidthInPixels, float pageHeightInPixels)
-{
-    COMPtr<IWebFramePrivate> framePrivate(Query, frame);
-    if (!framePrivate)
-        return 0;
-
-    int pageNumber = -1;
-    if (FAILED(framePrivate->numberOfPages(pageWidthInPixels, pageHeightInPixels, &pageNumber)))
-        pageNumber = -1;
-    return pageNumber;
-}
-
-JSRetainPtr<JSStringRef> TestRunner::pageProperty(const char* propertyName, int pageNumber) const
-{
-    // FIXME: Implement this.
-    return JSRetainPtr<JSStringRef>();
-}
-
 void TestRunner::apiTestNewWindowDataLoadBaseURL(JSStringRef utf8Data, JSStringRef baseURL)
 {
 
 }
 
-JSRetainPtr<JSStringRef> TestRunner::pageSizeAndMarginsInPixels(int pageNumber, int width, int height, int marginTop, int marginRight, int marginBottom, int marginLeft) const
-{
-    // FIXME: implement
-    return JSRetainPtr<JSStringRef>();
-}
-
 void TestRunner::apiTestGoToCurrentBackForwardItem()
 {
     COMPtr<IWebView> webView;
index ee09cb0..d7a5252 100644 (file)
@@ -472,12 +472,6 @@ void TestRunner::setScrollbarPolicy(JSStringRef orientation, JSStringRef policy)
     // FIXME: implement
 }
 
-int TestRunner::numberOfPages(float, float)
-{
-    // FIXME: implement
-    return -1;
-}
-
 void TestRunner::apiTestNewWindowDataLoadBaseURL(JSStringRef utf8Data, JSStringRef baseURL)
 {
 
@@ -525,18 +519,6 @@ void TestRunner::abortModal()
 {
 }
 
-JSRetainPtr<JSStringRef> TestRunner::pageProperty(const char* propertyName, int pageNumber) const
-{
-    // FIXME: Implement
-    return 0;
-}
-
-JSRetainPtr<JSStringRef> TestRunner::pageSizeAndMarginsInPixels(int pageNumber, int width, int height, int marginTop, int marginRight, int marginBottom, int marginLeft) const
-{
-    // FIXME: Implement
-    return 0;
-}
-
 void TestRunner::setAsynchronousSpellCheckingEnabled(bool)
 {
     // FIXME: Implement this.
index 07a35a2..393dd7c 100644 (file)
@@ -84,8 +84,6 @@ module WTR {
         void display();
 
         // Printing
-        int numberOfPages(in double pageWidthInPixels, in double pageHeightInPixels);
-        DOMString pageSizeAndMarginsInPixels(in int pageIndex, in int width, in int height, in int marginTop, in int marginRight, in int marginBottom, in int marginLeft);
         boolean isPageBoxVisible(in int pageIndex);
 
         [PassContext] void setValueForUser(in object element, in DOMString value);
index 1adcdac..c824c38 100644 (file)
@@ -378,18 +378,6 @@ void TestRunner::removeOriginAccessWhitelistEntry(JSStringRef sourceOrigin, JSSt
     WKBundleRemoveOriginAccessWhitelistEntry(InjectedBundle::shared().bundle(), toWK(sourceOrigin).get(), toWK(destinationProtocol).get(), toWK(destinationHost).get(), allowDestinationSubdomains);
 }
 
-int TestRunner::numberOfPages(double pageWidthInPixels, double pageHeightInPixels)
-{
-    WKBundleFrameRef mainFrame = WKBundlePageGetMainFrame(InjectedBundle::shared().page()->page());
-    return WKBundleNumberOfPages(InjectedBundle::shared().bundle(), mainFrame, pageWidthInPixels, pageHeightInPixels);
-}
-
-JSRetainPtr<JSStringRef> TestRunner::pageSizeAndMarginsInPixels(int pageIndex, int width, int height, int marginTop, int marginRight, int marginBottom, int marginLeft)
-{
-    WKBundleFrameRef mainFrame = WKBundlePageGetMainFrame(InjectedBundle::shared().page()->page());
-    return toJS(WKBundlePageSizeAndMarginsInPixels(InjectedBundle::shared().bundle(), mainFrame, pageIndex, width, height, marginTop, marginRight, marginBottom, marginLeft));
-}
-
 bool TestRunner::isPageBoxVisible(int pageIndex)
 {
     WKBundleFrameRef mainFrame = WKBundlePageGetMainFrame(InjectedBundle::shared().page()->page());
index 8c4042e..bd11e6a 100644 (file)
@@ -145,8 +145,6 @@ public:
     void setAppCacheMaximumSize(uint64_t);
 
     // Printing
-    int numberOfPages(double pageWidthInPixels, double pageHeightInPixels);
-    JSRetainPtr<JSStringRef> pageSizeAndMarginsInPixels(int pageIndex, int width, int height, int marginTop, int marginRight, int marginBottom, int marginLeft);
     bool isPageBoxVisible(int pageIndex);
 
     void setValueForUser(JSContextRef, JSValueRef element, JSStringRef value);