WebKit-https.git
8 years ago2011-06-28 Hans Wennborg <hans@chromium.org>
hans@chromium.org [Thu, 30 Jun 2011 18:32:30 +0000 (18:32 +0000)]
2011-06-28  Hans Wennborg  <hans@chromium.org>

        Reviewed by Tony Chang.

        IndexedDB: Prepare for running layout tests with LevelDB
        https://bugs.webkit.org/show_bug.cgi?id=63593

        Migration of data from SQLite to LevelDB must be done before the
        m_backingStore map is checked for an open LevelDB, because if a
        previous layout test has used LevelDB, it will be in the
        m_backingStore map.

        Covered by existing layout tests.

        * storage/IDBFactoryBackendImpl.cpp:
        (WebCore::IDBFactoryBackendImpl::open):
2011-06-28  Hans Wennborg  <hans@chromium.org>

        Reviewed by Tony Chang.

        IndexedDB: Prepare for running layout tests with LevelDB
        https://bugs.webkit.org/show_bug.cgi?id=63593

        WebIDBFactory should decide what DefaultBackingStore means.
        Also fix handling of layout test and incognito mode properly.

        * src/WebIDBFactoryImpl.cpp:
        (WebKit::WebIDBFactoryImpl::open):
2011-06-28  Hans Wennborg  <hans@chromium.org>

        Reviewed by Tony Chang.

        IndexedDB: Prepare for running layout tests with LevelDB
        https://bugs.webkit.org/show_bug.cgi?id=63593

        The TestShell must always provide a temporary folder for LevelDB,
        not just when the backing store type is overridden.

        * DumpRenderTree/chromium/LayoutTestController.cpp:
        (LayoutTestController::setOverrideIndexedDBBackingStore):
        (LayoutTestController::clearAllDatabases):
        * DumpRenderTree/chromium/LayoutTestController.h:
        * DumpRenderTree/chromium/TestShell.cpp:
        (TestShell::TestShell):
        * DumpRenderTree/chromium/TestShell.h:

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

8 years ago2011-06-30 Pavel Feldman <pfeldman@chromium.org>
pfeldman@chromium.org [Thu, 30 Jun 2011 18:22:00 +0000 (18:22 +0000)]
2011-06-30  Pavel Feldman  <pfeldman@chromium.org>

        Reviewed by Yury Semikhatsky.

        Web Inspector: introduce "filter" icons for timeline panel and elements sidebar.
        https://bugs.webkit.org/show_bug.cgi?id=63731

        * inspector/front-end/EventListenersSidebarPane.js:
        * inspector/front-end/Images/paneFilterButtons.png: Added.
        * inspector/front-end/Images/statusbarButtonGlyphs.png:
        * inspector/front-end/StylesSidebarPane.js:
        * inspector/front-end/inspector.css:
        (.pane > .title > select):
        (.pane > .title > select.select-settings):
        (.pane > .title > select.select-filter):
        (.timeline-filter-status-bar-item .glyph):

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

8 years ago2011-06-28 Hans Wennborg <hans@chromium.org>
hans@chromium.org [Thu, 30 Jun 2011 18:16:31 +0000 (18:16 +0000)]
2011-06-28  Hans Wennborg  <hans@chromium.org>

        Reviewed by Tony Chang.

        IndexedDB: Support for deleteIndex with LevelDB backing store
        https://bugs.webkit.org/show_bug.cgi?id=63539

        Covered by storage/indexeddb/mozilla/remove-index.html.

        * storage/IDBLevelDBBackingStore.cpp:
        (WebCore::IDBLevelDBBackingStore::deleteIndex):

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

8 years ago2011-06-30 Nate Chapin <japhet@chromium.org>
japhet@chromium.org [Thu, 30 Jun 2011 17:57:47 +0000 (17:57 +0000)]
2011-06-30  Nate Chapin  <japhet@chromium.org>

        Unreviewed, remove passing tests from chromium test expectations.

        * platform/chromium/test_expectations.txt:

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

8 years agoReinstate WKSecurityOriginGetHost and WKSecurityOriginGetProtocol as wrappers around...
mrowe@apple.com [Thu, 30 Jun 2011 17:38:49 +0000 (17:38 +0000)]
Reinstate WKSecurityOriginGetHost and WKSecurityOriginGetProtocol as wrappers around the correctly-named functions.

Reviewed by Anders Carlsson.

They're needed for ABI compatibility.

* Shared/API/c/WKSecurityOrigin.cpp:
(WKSecurityOriginGetHost):
(WKSecurityOriginGetProtocol):

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

8 years ago2011-06-30 Adam Barth <abarth@webkit.org>
abarth@webkit.org [Thu, 30 Jun 2011 17:24:26 +0000 (17:24 +0000)]
2011-06-30  Adam Barth  <abarth@webkit.org>

        Reviewed by Eric Seidel.

        [reviewtool] Expanding lines "below" does not remove purple context line
        https://bugs.webkit.org/show_bug.cgi?id=63695

        Honestly, this is the last bug in the review tool that bothers me.

        * PrettyPatch/PrettyPatch.rb:
            - Jump straight to version 42, because, well, you know.
        * code-review.js:
            - In addition to removing the context bar when we expand all, we
              now remove it when we expand below (but not when we expand
              above).

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

8 years agoANGLE shouldn't try to use internal SDKs. It doesn't need them!
mrowe@apple.com [Thu, 30 Jun 2011 17:04:46 +0000 (17:04 +0000)]
ANGLE shouldn't try to use internal SDKs. It doesn't need them!

Rubber-stamped by Dan Bernstein.

* Configurations/Base.xcconfig:

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

8 years ago2011-06-30 Abhishek Arya <inferno@chromium.org>
inferno@chromium.org [Thu, 30 Jun 2011 17:00:48 +0000 (17:00 +0000)]
2011-06-30  Abhishek Arya  <inferno@chromium.org>

        Reviewed by Ryosuke Niwa.

        Crash when calling DOMSubtreeModified event when extracting range
        contents.
        https://bugs.webkit.org/show_bug.cgi?id=63650

        Convert a few nodes to RefPtrs and add commonRoot verification checks
        for Range::processContents.

        Tests: fast/dom/Range/range-extract-contents-event-fire-crash.html
               fast/dom/Range/range-extract-contents-event-fire-crash2.html

        * dom/Range.cpp:
        (WebCore::childOfCommonRootBeforeOffset):
        (WebCore::Range::processContents):
        (WebCore::Range::processContentsBetweenOffsets):
        (WebCore::Range::processAncestorsAndTheirSiblings):
2011-06-29  Abhishek Arya  <inferno@chromium.org>

        Reviewed by Ryosuke Niwa.

        Crash when calling DOMSubtreeModified event when extracting range
        contents.
        https://bugs.webkit.org/show_bug.cgi?id=63650

        * fast/dom/Range/range-extract-contents-event-fire-crash-expected.txt: Added.
        * fast/dom/Range/range-extract-contents-event-fire-crash.html: Added.
        * fast/dom/Range/range-extract-contents-event-fire-crash2-expected.txt: Added.
        * fast/dom/Range/range-extract-contents-event-fire-crash2.html: Added.
        * fast/dom/Range/range-extractContents.html: remove the appending of fragment
        in this crasher test since we now refptr the nodes and leftContents will be visible.
        This crasher test does not need to show the extractContents fragment.

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

8 years agoAdd +[WebView _allowsRoundingHacks]
mitz@apple.com [Thu, 30 Jun 2011 16:53:52 +0000 (16:53 +0000)]
Add +[WebView _allowsRoundingHacks]
https://bugs.webkit.org/show_bug.cgi?id=63734

Reviewed by Adele Peterson.

Source/WebCore:

* WebCore.exp.in: Export TextRun::allowsRoundingHacks().
* platform/graphics/TextRun.cpp:
(WebCore::TextRun::allowsRoundingHacks): Added this accessor.
* platform/graphics/TextRun.h:

Source/WebKit/mac:

* WebView/WebView.mm:
(+[WebView _allowsRoundingHacks]): Added this accessor.
* WebView/WebViewPrivate.h:

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

8 years ago2011-06-30 Nate Chapin <japhet@chromium.org>
japhet@chromium.org [Thu, 30 Jun 2011 16:52:27 +0000 (16:52 +0000)]
2011-06-30  Nate Chapin  <japhet@chromium.org>

        Unreviewed, chromium test expectations update after r90101.

        * platform/chromium/test_expectations.txt:

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

8 years ago2011-06-30 Patrick Gansterer <paroga@webkit.org>
paroga@webkit.org [Thu, 30 Jun 2011 16:48:48 +0000 (16:48 +0000)]
2011-06-30  Patrick Gansterer  <paroga@webkit.org>

        Reviewed by Adam Roben.

        [WIN] Don't redefine the keyboard codes
        https://bugs.webkit.org/show_bug.cgi?id=63726

        Only define the codes when undefined.
        r64957 fixed this problem for COMPILER(MINGW) but not for OS(WINCE).

        * platform/WindowsKeyboardCodes.h:

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

8 years ago2011-06-30 Yury Semikhatsky <yurys@chromium.org>
yurys@chromium.org [Thu, 30 Jun 2011 16:39:21 +0000 (16:39 +0000)]
2011-06-30  Yury Semikhatsky  <yurys@chromium.org>

        Reviewed by Pavel Feldman.

        Web Inspector: hide "Enabled background events collection" option in context menu until the setting is persisted
        https://bugs.webkit.org/show_bug.cgi?id=63708

        The option has no much sense if it is not preserved between browser restarts.

        * inspector/front-end/NetworkPanel.js:
        (WebInspector.NetworkPanel.prototype._contextMenu):

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

8 years agoTools: [WK2] Create a general wk2 specific LayoutTests/platform/wk2 platform
ossy@webkit.org [Thu, 30 Jun 2011 16:28:10 +0000 (16:28 +0000)]
Tools: [WK2] Create a general wk2 specific LayoutTests/platform/wk2 platform
qt-wk2, win-wk2, mac-wk2 use this common Skipped list too
https://bugs.webkit.org/show_bug.cgi?id=61785

Patch by Zsolt Fehér <h490267@stud.u-szeged.hu> on 2011-06-30
Reviewed by Csaba Osztrogonác.

* Scripts/old-run-webkit-tests:

LayoutTests: [WK2] Create a general wk2 specific LayoutTests/platform/wk2 platform
Common Skipped list copied from LayoutTests/platform/mac-wk2/Skipped.
https://bugs.webkit.org/show_bug.cgi?id=61785

Patch by Zsolt Fehér <h490267@stud.u-szeged.hu> on 2011-06-30
Reviewed by Csaba Osztrogonác.

* platform/mac-wk2/Skipped:
* platform/wk2/Skipped: Copied from LayoutTests/platform/mac-wk2/Skipped.

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

8 years ago2011-06-30 Patrick Gansterer <paroga@webkit.org>
paroga@webkit.org [Thu, 30 Jun 2011 16:05:11 +0000 (16:05 +0000)]
2011-06-30  Patrick Gansterer  <paroga@webkit.org>

        Reviewed by Darin Adler.

        Remove dependnecy on IDL generation for disabled features in JSC bindings
        https://bugs.webkit.org/show_bug.cgi?id=63718

        Move the #if ENABLE() line directly after #include "config.h".
        This enables us to compile all files even when we don't generate the IDL output.

        * bindings/js/IDBBindingUtilities.cpp:
        * bindings/js/JSConsoleCustom.cpp:
        * bindings/js/JSCustomSQLStatementErrorCallback.cpp:
        * bindings/js/JSCustomXPathNSResolver.cpp:
        * bindings/js/JSDOMApplicationCacheCustom.cpp:
        * bindings/js/JSDeviceMotionEventCustom.cpp:
        * bindings/js/JSDeviceOrientationEventCustom.cpp:
        * bindings/js/JSInjectedScriptHostCustom.cpp:
        * bindings/js/JSInjectedScriptManager.cpp:
        * bindings/js/JSInspectorFrontendHostCustom.cpp:
        * bindings/js/JSJavaScriptCallFrameCustom.cpp:
        * bindings/js/JSSQLResultSetRowListCustom.cpp:
        * bindings/js/JSSQLTransactionCustom.cpp:
        * bindings/js/JSSQLTransactionSyncCustom.cpp:
        * bindings/js/JSSVGElementInstanceCustom.cpp:
        * bindings/js/JSStorageCustom.cpp:
        * bindings/js/JSTouchCustom.cpp:
        * bindings/js/JSTouchListCustom.cpp:
        * bindings/js/JSXPathResultCustom.cpp:
        * bindings/js/JavaScriptCallFrame.cpp:
        * bindings/js/PageScriptDebugServer.cpp:
        * bindings/js/ScriptDebugServer.cpp:
        * bindings/js/ScriptGCEvent.cpp:
        * bindings/js/ScriptProfile.cpp:
        * bindings/js/WorkerScriptDebugServer.cpp:

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

8 years agoUse a colon to separate test names from failure info on TestFailures
aroben@apple.com [Thu, 30 Jun 2011 15:30:52 +0000 (15:30 +0000)]
Use a colon to separate test names from failure info on TestFailures

This matches how the "Possibly Flaky Tests" list works, and is much prettier when the
failure info itself includes parentheses.

Fixes <http://webkit.org/b/63722> Crashing symbols on TestFailures page look ugly due to
double parentheses

Reviewed by Darin Adler.

* BuildSlaveSupport/build.webkit.org-config/public_html/TestFailures/ViewController.js:
(ViewController.prototype._domForFailedTest): Use a colon instead of surrounding the failure
info in parentheses.

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

8 years agoRobustify PersistentCache against cached data that contains the string ': '
aroben@apple.com [Thu, 30 Jun 2011 15:30:43 +0000 (15:30 +0000)]
Robustify PersistentCache against cached data that contains the string ': '

Fixes <http://webkit.org/b/63721> Exceptions thrown while loading TestFailures page due to
passing a bad string to JSON.parse

Reviewed by Darin Adler.

* BuildSlaveSupport/build.webkit.org-config/public_html/TestFailures/PersistentCache.js:
(PersistentCache._parseDateAndJSONFromString): Manually split the string on the separator
using indexOf and substring. (String.prototype.split always splits on all instances of the
separator.)

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

8 years agoInclude the crashing symbol in crash logs links on TestFailures
aroben@apple.com [Thu, 30 Jun 2011 15:30:32 +0000 (15:30 +0000)]
Include the crashing symbol in crash logs links on TestFailures

Fixes <http://webkit.org/b/63465> Links to crash logs on TestFailures page should include
the crashing symbol (like links in results.html do)

Reviewed by David Kilzer.

* BuildSlaveSupport/build.webkit.org-config/public_html/TestFailures/LayoutTestResultsLoader.js:
(LayoutTestResultsLoader.prototype.start): Bumped the cache version because we now store
crashing symbols for crashing tests. Renamed testsForResultTable to parseResultTable because
it now returns more than just the test names. Specifically, it now looks for crash log links
and extracts the crashing symbol name from them. Updated callers of parseResultTable to
match its new behavior. Changed to store the crashing symbol along with the failure type in
the data we pass to the callback for tests which crashed.

* BuildSlaveSupport/build.webkit.org-config/public_html/TestFailures/TestFailures.css:
(code): Make <code> elements a little smaller because their contents can be quite long.

* BuildSlaveSupport/build.webkit.org-config/public_html/TestFailures/ViewController.js:
(ViewController.prototype._domForFailureDiagnosis): Include the crashing symbol inside a
<code> element in the link, if there is a crashing symbol.

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

8 years agoUse objects instead of strings to represent a test result in TestFailures code
aroben@apple.com [Thu, 30 Jun 2011 15:30:18 +0000 (15:30 +0000)]
Use objects instead of strings to represent a test result in TestFailures code

This will eventually allow us to store more than just the type of failure for each test.
(E.g., we can store the name of the crashing symbol for tests which crashed.)

Prep work for <http://webkit.org/b/63465> Links to crash logs on TestFailures page should
include the crashing symbol (like links in results.html do)

Reviewed by David Kilzer.

* BuildSlaveSupport/build.webkit.org-config/public_html/TestFailures/Builder.js:
(Builder.prototype.failureDiagnosisTextAndURL): Changed to expect a testResult object
instead of just a failureType string.

* BuildSlaveSupport/build.webkit.org-config/public_html/TestFailures/FlakyLayoutTestDetector.js:
(FlakyLayoutTestDetector.prototype.incorporateTestResults): Changed to store a
testResult-like object for passing tests.
(FlakyLayoutTestDetector.prototype.flakinessExamples): Changed to expect testResult-like
objects.

* BuildSlaveSupport/build.webkit.org-config/public_html/TestFailures/LayoutTestResultsLoader.js:
(LayoutTestResultsLoader.prototype.start): Store a version number along with the cached data
so we can throw away cached data that's in an old format. Store a testResult object for each
test instead of just its failure type.

* BuildSlaveSupport/build.webkit.org-config/public_html/TestFailures/ViewController.js:
(ViewController.prototype._domForFailedTest):
(ViewController.prototype._domForFailureDiagnosis):
Changed to expect testResult objects instead of failureType strings.

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

8 years ago2011-06-30 Alexander Pavlov <apavlov@chromium.org>
apavlov@chromium.org [Thu, 30 Jun 2011 14:45:21 +0000 (14:45 +0000)]
2011-06-30  Alexander Pavlov  <apavlov@chromium.org>

        Reviewed by Kent Tamura.

        [Chromium] Autofill suggestions appear in upper left corner after input change
        https://bugs.webkit.org/show_bug.cgi?id=63702

        This change fixes a few popup layout issues, some of which have existed for quite a while:
        - the absence of the ChromeClientImpl::screenToWindow() implementation (the method used to always return (0, 0));
        - the confusion of window and screen coordinates passed into the autofill popup client's setWindowRect() method;
        - the use of the current frameRect() width as the target element's width (which was wrong when refreshing
          a popup resized during the initial layout (e.g. made wider than the target element)).

        No new tests, as this is a change to platform-specific widget code. The test case is provided in the bug description.

        * platform/chromium/PopupMenuChromium.cpp:
        (WebCore::PopupContainer::layoutAndGetRTLOffset): Use original target element width, not frameRect().width(),
        since the latter gets updated if the popup is not the same width as the target element.
        (WebCore::PopupContainer::refresh): Restore only the popup bounds rather than its original rectangle,
        since it sometimes breaks the invalidation region and layout.
2011-06-30  Alexander Pavlov  <apavlov@chromium.org>

        Reviewed by Kent Tamura.

        [Chromium] Autofill suggestions appear in upper left corner after input change
        https://bugs.webkit.org/show_bug.cgi?id=63702

        * src/ChromeClientImpl.cpp:
        (WebKit::ChromeClientImpl::screenToWindow): Implemented.
        * src/WebViewImpl.cpp:
        (WebKit::WebViewImpl::refreshAutoFillPopup): Pass screen (not window) coordinates into setWindowRect().

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

8 years agoShow full commit logs when visiting Trac from TestFailures page
aroben@apple.com [Thu, 30 Jun 2011 14:30:19 +0000 (14:30 +0000)]
Show full commit logs when visiting Trac from TestFailures page

Fixes <http://webkit.org/b/63715> Links to Trac on TestFailures page
should show full commit logs

Reviewed by David Kilzer.

* BuildSlaveSupport/build.webkit.org-config/public_html/TestFailures/Trac.js:
(Trac.prototype.logURL): Added new 'showFullCommitLogs' parameter. When specified, we add
verbose=on to the log URL to tell Trac to show full commit logs.

* BuildSlaveSupport/build.webkit.org-config/public_html/TestFailures/ViewController.js:
(ViewController.prototype._domForRegressionRange): Specify true for showFullCommitLogs.

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

8 years ago2011-06-30 Sheriff Bot <webkit.review.bot@gmail.com>
commit-queue@webkit.org [Thu, 30 Jun 2011 14:11:21 +0000 (14:11 +0000)]
2011-06-30  Sheriff Bot  <webkit.review.bot@gmail.com>

        Unreviewed, rolling out r90102.
        http://trac.webkit.org/changeset/90102
        https://bugs.webkit.org/show_bug.cgi?id=63714

        Lots of tests asserting beneath
        SVGSMILElement::findInstanceTime (Requested by aroben on
        #webkit).

        * wtf/StdLibExtras.h:
        (WTF::binarySearch):
2011-06-30  Sheriff Bot  <webkit.review.bot@gmail.com>

        Unreviewed, rolling out r90102.
        http://trac.webkit.org/changeset/90102
        https://bugs.webkit.org/show_bug.cgi?id=63714

        Lots of tests asserting beneath
        SVGSMILElement::findInstanceTime (Requested by aroben on
        #webkit).

        * svg/animation/SVGSMILElement.cpp:
        (WebCore::SVGSMILElement::findInstanceTime):

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

8 years ago2011-06-30 Andrey Kosyakov <caseq@chromium.org>
caseq@chromium.org [Thu, 30 Jun 2011 14:03:29 +0000 (14:03 +0000)]
2011-06-30  Andrey Kosyakov  <caseq@chromium.org>

        Reviewed by Yury Semikhatsky.

        Web Inspector: remove ExtensionAPISchema.json
        https://bugs.webkit.org/show_bug.cgi?id=63687

        * inspector/front-end/ExtensionAPISchema.json: Removed.

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

8 years agoAdd a link to file a bug about TestFailures to the TestFailures page
aroben@apple.com [Thu, 30 Jun 2011 14:00:30 +0000 (14:00 +0000)]
Add a link to file a bug about TestFailures to the TestFailures page

Fixes <http://webkit.org/b/63466> TestFailures page should include a link to file a bug
about TestFailures itself

Reviewed by Anders Carlsson.

* BuildSlaveSupport/build.webkit.org-config/public_html/TestFailures/TestFailures.css:
(aside): Added some styles for the new link. It is positioned in the upper-right and
hopefully not to glaringly obvious.

* BuildSlaveSupport/build.webkit.org-config/public_html/TestFailures/ViewController.js:
(ViewController.prototype.loaded): Set up our main content element and auxiliary UI.

(ViewController.prototype._displayBuilder):
(ViewController.prototype._displayTesters):
Modify the main content element instead of the entire body, as there are other things in the
body that we want to preserve.

(ViewController.prototype._domForAuxiliaryUIElements): Create and return an <aside> element
that contains a link to file a bug about the TestFailures page.

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

8 years agoIgnore callbacks that correspond to old navigations of the TestFailures page
aroben@apple.com [Thu, 30 Jun 2011 14:00:16 +0000 (14:00 +0000)]
Ignore callbacks that correspond to old navigations of the TestFailures page

Fixes <http://webkit.org/b/63711> TestFailures page sometimes ignores back/forward
navigations

Reviewed by Anders Carlsson.

* BuildSlaveSupport/build.webkit.org-config/public_html/TestFailures/LayoutTestHistoryAnalyzer.js:
(LayoutTestHistoryAnalyzer.prototype.start): Check the return value of the callback. If the
callback returns false, stop fetching more data.

* BuildSlaveSupport/build.webkit.org-config/public_html/TestFailures/ViewController.js:
(ViewController): Initialize new _navigationID member. This member is used to assign each
navigation by the user a unique ID.
(ViewController.prototype.parseHash): Increment the navigation ID.

(ViewController.prototype._displayBuilder):
(ViewController.prototype._displayTesters):
At the start of these functions, record the current navigation ID. Bail out from callbacks
if the navigation ID has changed, indicating that the user has navigated elsewhere.

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

8 years ago2011-06-30 Fabrizio Machado <fabrizio.machado@nokia.com>
commit-queue@webkit.org [Thu, 30 Jun 2011 13:59:41 +0000 (13:59 +0000)]
2011-06-30  Fabrizio Machado  <fabrizio.machado@nokia.com>

        Reviewed by Benjamin Poulain.

        [Qt] QML Webview causes performance drops
        https://bugs.webkit.org/show_bug.cgi?id=57554

        Remove tiledBackingStoreEnabled from QML WebView.

        * declarative/qdeclarativewebview.cpp:
        (QDeclarativeWebView::setPage):

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

8 years ago2011-06-30 Ilya Tikhonovsky <loislo@chromium.org>
loislo@chromium.org [Thu, 30 Jun 2011 13:43:59 +0000 (13:43 +0000)]
2011-06-30  Ilya Tikhonovsky  <loislo@chromium.org>

        Unreviewed build fix for chromium.

        * inspector/CodeGeneratorInspector.pm:

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

8 years ago2011-06-30 Ilya Tikhonovsky <loislo@chromium.org>
loislo@chromium.org [Thu, 30 Jun 2011 13:38:00 +0000 (13:38 +0000)]
2011-06-30  Ilya Tikhonovsky  <loislo@chromium.org>

        Unreviewed build fix for mac.

        * inspector/CodeGeneratorInspector.pm:

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

8 years ago2011-06-30 Ilya Tikhonovsky <loislo@chromium.org>
loislo@chromium.org [Thu, 30 Jun 2011 13:22:29 +0000 (13:22 +0000)]
2011-06-30  Ilya Tikhonovsky  <loislo@chromium.org>

        Reviewed by Pavel Feldman.

        Web Inspector: Protocol: error object needs some improvement.
        https://bugs.webkit.org/show_bug.cgi?id=63707

        Current implementation is not useful in the code.
        We need to have toString implementation with nice formatting
        I'd like to use error.message property as the main place for the error's text.

        * inspector/elements/set-attribute.html:
        * inspector/report-protocol-errors-expected.txt:
        * inspector/report-protocol-errors.html:
2011-06-30  Ilya Tikhonovsky  <loislo@chromium.org>

        Reviewed by Pavel Feldman.

        Web Inspector: Protocol: error object needs some improvement.
        https://bugs.webkit.org/show_bug.cgi?id=63707

        Current implementation is not useful in the code.
        We need to have toString implementation with nice formatting
        I'd like to use error.message property as the main place for the error's text.

        * inspector/CodeGeneratorInspector.pm:

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

8 years ago2011-06-30 Carlos Garcia Campos <cgarcia@igalia.com>
carlosgc@webkit.org [Thu, 30 Jun 2011 13:02:47 +0000 (13:02 +0000)]
2011-06-30  Carlos Garcia Campos  <cgarcia@igalia.com>

        Reviewed by Martin Robinson.

        [GTK] Add back/forward menu to MiniBrowser toolbar
        https://bugs.webkit.org/show_bug.cgi?id=63445

        * MiniBrowser/gtk/BrowserWindow.c:
        (browser_window_init): Use a GtkMenuToolButton for navigation
        widgets.
        (browserWindowConstructed): Get the page BackForwardList.
        (browserWindowHistoryItemActivated): Go to activated history item.
        (browserWindowHistoryItemSelected): Show the url of currently
        selected history item in the statusbar.
        (createGtkActionFromBackForwardItem): Create a GtkAction from a
        WKBackForwardListItemRef.
        (browserWindowCreateMenuItemFromBackForwardItem): Create a
        GtkMenuItem from a WKBackForwardListItemRef.
        (browserWindowCreateBackForwardMenu): Create a GtkMenu with the
        given history items.
        (browserWindowUpdateNavigationActions): Update navigation widgets
        sensitivity and history menus.
        (didChangeBackForwardList): Call
        browserWindowUpdateNavigationActions().
        (browserWindowLoaderClientInit): Add didChangeBackForwardList
        callback.

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

8 years ago2011-06-30 Dmitry Lomov <dslomov@google.com>
commit-queue@webkit.org [Thu, 30 Jun 2011 12:55:39 +0000 (12:55 +0000)]
2011-06-30  Dmitry Lomov  <dslomov@google.com>

        Reviewed by Adam Roben.

        https://bugs.webkit.org/show_bug.cgi?id=63660
        This enables unit-tests step on Windows and MacOS SnowLeopard.

        * BuildSlaveSupport/build.webkit.org-config/master.cfg:

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

8 years ago2011-06-30 Mikhail Naganov <mnaganov@chromium.org>
mnaganov@chromium.org [Thu, 30 Jun 2011 12:10:13 +0000 (12:10 +0000)]
2011-06-30  Mikhail Naganov  <mnaganov@chromium.org>

        Reviewed by Yury Semikhatsky.

        Web Inspector: [Chromium] Support annotations for arrays in heap profiler.
        https://bugs.webkit.org/show_bug.cgi?id=63690

        * inspector/front-end/DetailedHeapshotGridNodes.js:
        (WebInspector.HeapSnapshotGenericObjectNode.prototype.get data):

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

8 years ago2011-06-30 Patrick Gansterer <paroga@webkit.org>
paroga@webkit.org [Thu, 30 Jun 2011 11:41:14 +0000 (11:41 +0000)]
2011-06-30  Patrick Gansterer  <paroga@webkit.org>

        Unreviewed build fix for !ENABLE(SVG_ANIMATION) after r88663, r89187, r89220, r89431 and r89661.

        * svg/SVGAnimatedAngle.h:
        * svg/SVGAnimatedLength.h:
        * svg/SVGAnimatedNumber.h:
        * svg/SVGAnimatedRect.h:
        * svg/SVGAnimatedString.h:

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

8 years ago2011-06-30 Gabor Loki <loki@webkit.org>
loki@webkit.org [Thu, 30 Jun 2011 11:19:40 +0000 (11:19 +0000)]
2011-06-30 Gabor Loki <loki@webkit.org>

        Reviewed by Csaba Osztrogonác.

        Fixed several style and typo issues after r90091.
        https://bugs.webkit.org/show_bug.cgi?id=63692

        Rename yStart to extremaStartY during computing extremas

        * platform/graphics/filters/FEMorphology.cpp:
        (WebCore::FEMorphology::platformApplyGeneric):
        * platform/graphics/filters/FEMorphology.h:

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

8 years ago Unreviewed, skip a failing media test on GTK because of
philn@webkit.org [Thu, 30 Jun 2011 11:08:57 +0000 (11:08 +0000)]
    Unreviewed, skip a failing media test on GTK because of
        https://bugs.webkit.org/show_bug.cgi?id=63699

        * platform/gtk/Skipped: skip media/media-blocked-by-willsendrequest.html

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

8 years ago2011-06-30 Kent Tamura <tkent@chromium.org>
tkent@chromium.org [Thu, 30 Jun 2011 11:00:40 +0000 (11:00 +0000)]
2011-06-30  Kent Tamura  <tkent@chromium.org>

        [Chromium] Resolve a conflict.

        * platform/chromium/test_expectations.txt:

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

8 years ago2011-06-30 Oliver Varga <Varga.Oliver@stud.u-szeged.hu>
commit-queue@webkit.org [Thu, 30 Jun 2011 10:56:49 +0000 (10:56 +0000)]
2011-06-30  Oliver Varga  <Varga.Oliver@stud.u-szeged.hu>

        Reviewed by Nikolas Zimmermann.

        Speed up SVGSMILElement::findInstanceTime.
        https://bugs.webkit.org/show_bug.cgi?id=61025

        Add a new parameter to StdlibExtras.h::binarySerarch function
        to also handle cases when the array does not contain the key value.
        This is needed for an svg function.

        * wtf/StdLibExtras.h:
        (WTF::binarySearch):
2011-06-30  Oliver Varga  <Varga.Oliver@stud.u-szeged.hu>

        Reviewed by Nikolas Zimmermann.

        Speed up SVGSMILElement::findInstanceTime.
        https://bugs.webkit.org/show_bug.cgi?id=61025

        Replace the linear search to binary search on ordered list because
        the previous searches from the beginning was not efficient.

        No new tests this is only a performance tweak.

        * svg/animation/SVGSMILElement.cpp:
        (WebCore::extractTimeFromVector):
        (WebCore::SVGSMILElement::findInstanceTime):

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

8 years ago2011-06-30 Kentaro Hara <haraken@google.com>
commit-queue@webkit.org [Thu, 30 Jun 2011 10:40:32 +0000 (10:40 +0000)]
2011-06-30  Kentaro Hara  <haraken@google.com>

        Reviewed by Kent Tamura.

        Change the label of an HTML5 file chooser button to "Choose Files"
        https://bugs.webkit.org/show_bug.cgi?id=49245

        We should notify capability of multiple files to users.
        input-file-label.html is a test that checks a label of the file chooser
        with/without 'multiple' attribute. input-file-label.html will fail in Chromium
        until a planned Chromium patch is landed. input-file-re-render.html will
        also fail in Chromium, gtk and MacLeopard until we update the expected images
        for the test.

        * fast/forms/input-file-label-expected.txt: Added.
        * fast/forms/input-file-label.html: Added.
        * platform/chromium/test_expectations.txt:
        * platform/mac/fast/forms/input-file-re-render-expected.png:
        * platform/mac/fast/forms/input-file-re-render-expected.txt:
2011-06-30  Kentaro Hara  <haraken@google.com>

        Reviewed by Kent Tamura.

        Change the label of an HTML5 file chooser button to "Choose Files"
        https://bugs.webkit.org/show_bug.cgi?id=49245

        We should notify capability of multiple files to users.

        Test: fast/forms/input-file-label.html

        * English.lproj/Localizable.strings:
        * html/FileInputType.cpp:
        (WebCore::UploadButtonElement::createForMultiple): Constructs a multiple file chooser button.
        (WebCore::FileInputType::createShadowSubtree): Constructs a single/multiple file chooser button depending on 'multiple' attribute.
        (WebCore::FileInputType::multipleAttributeChanged): Updates the label of the file chooser button.
        * html/FileInputType.h:
        * html/HTMLInputElement.cpp:
        (WebCore::HTMLInputElement::parseMappedAttribute): Updates the shadow tree of the file chooser button when 'multiple' attribute is set or unset.
        * html/InputType.cpp:
        (WebCore::InputType::multipleAttributeChanged): Virtual method to be overridden by FileInputType.
        * html/InputType.h:
        * platform/DefaultLocalizationStrategy.cpp:
        (WebCore::DefaultLocalizationStrategy::fileButtonChooseFileLabel): Updated the description for "Choose File".
        (WebCore::DefaultLocalizationStrategy::fileButtonChooseMultipleFilesLabel): Returns a string "Choose Files".
        * platform/DefaultLocalizationStrategy.h:
        * platform/LocalizationStrategy.h:
        * platform/LocalizedStrings.cpp:
        (WebCore::fileButtonChooseMultipleFilesLabel): Wrapper method for getting the "Choose Files" label.
        * platform/LocalizedStrings.h:
        * platform/brew/LocalizedStringsBrew.cpp:
        (WebCore::fileButtonChooseMultipleFilesLabel): Returns a "Choose Files" label.
        * platform/efl/LocalizedStringsEfl.cpp:
        (WebCore::fileButtonChooseMultipleFilesLabel): Returns a "Choose Files" label.
        * platform/gtk/LocalizedStringsGtk.cpp:
        (WebCore::fileButtonChooseMultipleFilesLabel): Returns a "Choose Files" label.
        * platform/haiku/LocalizedStringsHaiku.cpp:
        (WebCore::fileButtonChooseMultipleFilesLabel): Returns a "Choose Files" label.
        * platform/wx/LocalizedStringsWx.cpp:
        (WebCore::fileButtonChooseMultipleFilesLabel): Returns a "Choose Files" label.
2011-06-30  Kentaro Hara  <haraken@google.com>

        Reviewed by Kent Tamura.

        Change the label of an HTML5 file chooser button to "Choose Files"
        https://bugs.webkit.org/show_bug.cgi?id=49245

        We should notify capability of multiple files to users.

        Test: fast/forms/input-file-label.html

        * DEPS:
        * public/WebLocalizedString.h: Depends on chromium_rev>=91051.
        * src/LocalizedStrings.cpp:
        (WebCore::fileButtonChooseMultipleFilesLabel): Returns a "Choose Files" label.
2011-06-30  Kentaro Hara  <haraken@google.com>

        Reviewed by Kent Tamura.

        Change the label of an HTML5 file chooser button to "Choose Files"
        https://bugs.webkit.org/show_bug.cgi?id=49245

        We should notify capability of multiple files to users.

        Test: fast/forms/input-file-label.html

        * WebCoreSupport/WebPlatformStrategies.cpp:
        (WebPlatformStrategies::fileButtonChooseFileLabel): Updated the description for "Choose File".
        (WebPlatformStrategies::fileButtonChooseMultipleFilesLabel):  Returns a "Choose Files" label.
        * WebCoreSupport/WebPlatformStrategies.h:
2011-06-30  Kentaro Hara  <haraken@google.com>

        Reviewed by Kent Tamura.

        Change the label of an HTML5 file chooser button to "Choose Files"
        https://bugs.webkit.org/show_bug.cgi?id=49245

        We should notify capability of multiple files to users.

        Test: fast/forms/input-file-label.html

        * WebCoreSupport/PlatformStrategiesWinCE.cpp:
        (PlatformStrategiesWinCE::fileButtonChooseFileLabel): Updated the description for "Choose File".
        (PlatformStrategiesWinCE::fileButtonChooseMultipleFilesLabel): Returns a "Choose Files" label.
        * WebCoreSupport/PlatformStrategiesWinCE.h:

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

8 years ago2011-06-30 Yuta Kitamura <yutak@chromium.org>
yutak@chromium.org [Thu, 30 Jun 2011 09:31:01 +0000 (09:31 +0000)]
2011-06-30  Yuta Kitamura  <yutak@chromium.org>

        Reviewed by Kent Tamura.

        WebSocket: Don't use script-tests for WebSocket tests
        https://bugs.webkit.org/show_bug.cgi?id=63691

        Move content of script-tests/XXX.js into XXX.html.

        No changes were made to script code. Hence, these tests are not using
        window.jsTestAsync and finishJSTest() for now. I will fix this later.

        * http/tests/websocket/tests/alert-in-event-handler.html:
        * http/tests/websocket/tests/bad-handshake-crash.html:
        * http/tests/websocket/tests/bad-sub-protocol-control-chars.html:
        * http/tests/websocket/tests/bad-sub-protocol-empty.html:
        * http/tests/websocket/tests/bad-sub-protocol-non-ascii.html:
        * http/tests/websocket/tests/bufferedAmount-after-close.html:
        * http/tests/websocket/tests/cross-origin.html:
        * http/tests/websocket/tests/error-detect.html:
        * http/tests/websocket/tests/frame-length-longer-than-buffer.html:
        * http/tests/websocket/tests/frame-length-skip.html:
        * http/tests/websocket/tests/handshake-challenge-randomness.html:
        * http/tests/websocket/tests/handshake-error.html:
        * http/tests/websocket/tests/handshake-fail-by-cross-origin.html:
        * http/tests/websocket/tests/handshake-fail-by-no-cr.html:
        * http/tests/websocket/tests/handshake-fail-by-sub-protocol-mismatch.html:
        * http/tests/websocket/tests/long-invalid-header.html:
        * http/tests/websocket/tests/null-character.html:
        * http/tests/websocket/tests/script-tests/TEMPLATE.html: Removed.
        * http/tests/websocket/tests/script-tests/alert-in-event-handler.js: Removed.
        * http/tests/websocket/tests/script-tests/bad-handshake-crash.js: Removed.
        * http/tests/websocket/tests/script-tests/bad-sub-protocol-control-chars.js: Removed.
        * http/tests/websocket/tests/script-tests/bad-sub-protocol-empty.js: Removed.
        * http/tests/websocket/tests/script-tests/bad-sub-protocol-non-ascii.js: Removed.
        * http/tests/websocket/tests/script-tests/bufferedAmount-after-close.js: Removed.
        * http/tests/websocket/tests/script-tests/cross-origin.js: Removed.
        * http/tests/websocket/tests/script-tests/error-detect.js: Removed.
        * http/tests/websocket/tests/script-tests/frame-length-longer-than-buffer.js: Removed.
        * http/tests/websocket/tests/script-tests/frame-length-skip.js: Removed.
        * http/tests/websocket/tests/script-tests/handshake-challenge-randomness.js: Removed.
        * http/tests/websocket/tests/script-tests/handshake-error.js: Removed.
        * http/tests/websocket/tests/script-tests/handshake-fail-by-cross-origin.js: Removed.
        * http/tests/websocket/tests/script-tests/handshake-fail-by-no-cr.js: Removed.
        * http/tests/websocket/tests/script-tests/handshake-fail-by-sub-protocol-mismatch.js: Removed.
        * http/tests/websocket/tests/script-tests/long-invalid-header.js: Removed.
        * http/tests/websocket/tests/script-tests/null-character.js: Removed.
        * http/tests/websocket/tests/script-tests/send-throw.js: Removed.
        * http/tests/websocket/tests/script-tests/send.js: Removed.
        * http/tests/websocket/tests/script-tests/simple.js: Removed.
        * http/tests/websocket/tests/script-tests/sub-protocol-with-space.js: Removed.
        * http/tests/websocket/tests/script-tests/sub-protocol.js: Removed.
        * http/tests/websocket/tests/script-tests/unicode.js: Removed.
        * http/tests/websocket/tests/script-tests/url-no-trailing-slash.js: Removed.
        * http/tests/websocket/tests/script-tests/url-parsing.js: Removed.
        * http/tests/websocket/tests/script-tests/url-with-credential.js: Removed.
        * http/tests/websocket/tests/script-tests/url-with-empty-query.js: Removed.
        * http/tests/websocket/tests/script-tests/url-with-query-for-no-query.js: Removed.
        * http/tests/websocket/tests/script-tests/url-with-query.js: Removed.
        * http/tests/websocket/tests/script-tests/websocket-event-target.js: Removed.
        * http/tests/websocket/tests/script-tests/websocket-protocol-ignored.js: Removed.
        * http/tests/websocket/tests/send-throw.html:
        * http/tests/websocket/tests/send.html:
        * http/tests/websocket/tests/simple.html:
        * http/tests/websocket/tests/sub-protocol-with-space.html:
        * http/tests/websocket/tests/sub-protocol.html:
        * http/tests/websocket/tests/unicode.html:
        * http/tests/websocket/tests/url-no-trailing-slash.html:
        * http/tests/websocket/tests/url-parsing.html:
        * http/tests/websocket/tests/url-with-credential.html:
        * http/tests/websocket/tests/url-with-empty-query.html:
        * http/tests/websocket/tests/url-with-query-for-no-query.html:
        * http/tests/websocket/tests/url-with-query.html:
        * http/tests/websocket/tests/websocket-event-target.html:
        * http/tests/websocket/tests/websocket-protocol-ignored.html:

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

8 years ago2011-06-30 Yuta Kitamura <yutak@chromium.org>
yutak@chromium.org [Thu, 30 Jun 2011 09:22:49 +0000 (09:22 +0000)]
2011-06-30  Yuta Kitamura  <yutak@chromium.org>

        Reviewed by Kent Tamura.

        WebSocket: Fix style issues of pywebsocket handlers
        https://bugs.webkit.org/show_bug.cgi?id=63689

        Fix minor style errors such as wrong indentation or wrong number of blank lines.

        * http/tests/websocket/tests/bad-handshake-crash_wsh.py:
        * http/tests/websocket/tests/close-on-unload_wsh.py:
        * http/tests/websocket/tests/echo-challenge_wsh.py:
        * http/tests/websocket/tests/echo-cookie_wsh.py:
        * http/tests/websocket/tests/frame-length-longer-than-buffer_wsh.py:
        * http/tests/websocket/tests/frame-length-overflow_wsh.py:
        * http/tests/websocket/tests/frame-length-skip_wsh.py:
        * http/tests/websocket/tests/handshake-fail-by-maxlength_wsh.py:
        Use sequence repetition pattern instead of a loop.
        * http/tests/websocket/tests/handshake-fail-by-no-connection-header_wsh.py:
        * http/tests/websocket/tests/handshake-fail-by-no-cr_wsh.py:
        * http/tests/websocket/tests/handshake-fail-by-no-upgrade-header_wsh.py:
        * http/tests/websocket/tests/handshake-fail-by-prepended-null_wsh.py:
        * http/tests/websocket/tests/hanging-handshake_wsh.py:
        * http/tests/websocket/tests/long-invalid-header_wsh.py:
        * http/tests/websocket/tests/send2_wsh.py:
        * http/tests/websocket/tests/unknown-frame-type_wsh.py:
        * http/tests/websocket/tests/workers/resources/echo-challenge_wsh.py:

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

8 years ago2011-06-30 Ryosuke Niwa <rniwa@webkit.org>
rniwa@webkit.org [Thu, 30 Jun 2011 09:14:17 +0000 (09:14 +0000)]
2011-06-30  Ryosuke Niwa  <rniwa@webkit.org>

        Reviewed by Kent Tamura.

        Replace isNodeInTextFormControl(Node*) by enclosingTextFormControl(const Position&)
        https://bugs.webkit.org/show_bug.cgi?id=63672

        Replaced isNodeInTextFormControl(Node*) by enclosingTextFormControl(const Position&).
        Also added a safe toTextFormControl to HTMLFormControlElement.h.

        * editing/DeleteSelectionCommand.cpp:
        (WebCore::DeleteSelectionCommand::doApply): Calls enclosingTextFormControl instead of manually
        checking the tag name of shadowAncestorNode.
        * editing/Editor.cpp:
        (WebCore::Editor::selectionForCommand): Rewritten using enclosingTextFormControl and toTextFormControl.
        (WebCore::Editor::cut): Calls enclosingTextFormControl instead of isNodeInTextFormControl.
        (WebCore::Editor::copy): Ditto.
        (WebCore::Editor::setBaseWritingDirection): Calls toTextFormControl instead of manually checking tag name.
        * editing/FrameSelection.cpp:
        (WebCore::FrameSelection::isInPasswordField): Calls enclosingTextFormControl.
        * editing/ReplaceSelectionCommand.cpp:
        (WebCore::ReplaceSelectionCommand::doApply): Ditto.
        * editing/htmlediting.cpp:
        (WebCore::enclosingTextFormControl): Added.
        * editing/htmlediting.h:
        * html/HTMLFormControlElement.h:
        (WebCore::toTextFormControl): Added.
        * page/DragController.cpp:
        (WebCore::DragController::startDrag): Calls enclosingTextFormControl instead of isNodeInTextFormControl.

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

8 years ago2011-06-30 Kent Tamura <tkent@chromium.org>
tkent@chromium.org [Thu, 30 Jun 2011 09:08:07 +0000 (09:08 +0000)]
2011-06-30  Kent Tamura  <tkent@chromium.org>

        [Mac-wk2] Update expectation files for r90089.
        https://bugs.webkit.org/show_bug.cgi?id=61415

        * platform/mac-wk2/fast/forms/textfield-overflow-expected.txt:

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

8 years ago2011-06-27 Pavel Podivilov <podivilov@chromium.org>
podivilov@chromium.org [Thu, 30 Jun 2011 08:18:28 +0000 (08:18 +0000)]
2011-06-27  Pavel Podivilov  <podivilov@chromium.org>

        Reviewed by Pavel Feldman.

        Web Inspector: source code is not highlighted after drawer height animations.
        https://bugs.webkit.org/show_bug.cgi?id=62860

        * inspector/front-end/Drawer.js:
        (WebInspector.Drawer.prototype._animateDrawerHeight.animationFinished):
        (WebInspector.Drawer.prototype._animateDrawerHeight):

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

8 years ago2011-06-30 Balazs Kelemen <kbalazs@webkit.org>
kbalazs@webkit.org [Thu, 30 Jun 2011 08:00:31 +0000 (08:00 +0000)]
2011-06-30  Balazs Kelemen  <kbalazs@webkit.org>

        Reviewed by Kenneth Rohde Christiansen.

        [Qt][WK2] Implement the rest of WTR::PlatformWebView
        https://bugs.webkit.org/show_bug.cgi?id=63630

        * WebKitTestRunner/qt/PlatformWebViewQt.cpp:
        (WTR::PlatformWebView::windowFrame):
        (WTR::PlatformWebView::setWindowFrame):

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

8 years ago2011-06-30 MORITA Hajime <morrita@google.com>
morrita@google.com [Thu, 30 Jun 2011 07:43:39 +0000 (07:43 +0000)]
2011-06-30  MORITA Hajime  <morrita@google.com>

        Reviewed by Dimitri Glazkov.

        Crash if ShadowRoot has a text node.
        https://bugs.webkit.org/show_bug.cgi?id=63607

        * fast/dom/shadow/shadow-root-text-child-expected.txt: Added.
        * fast/dom/shadow/shadow-root-text-child.html: Added.
2011-06-30  MORITA Hajime  <morrita@google.com>

        Reviewed by Dimitri Glazkov.

        Crash if ShadowRoot has a text node.
        https://bugs.webkit.org/show_bug.cgi?id=63607

        Node::styleForRenderer() for Text node wasn't shadow-aware.
        This change allow styleForRenderer() to know its visual parent by
        passing NodeRenderingContext.

        Changes other than NodeRenderingContext and Element is just a
        follow up for change above.

        * dom/Element.cpp: use NodeRenderingContext to resolve parent node.
        (WebCore::Element::recalcStyle):
        * dom/Node.cpp:
        (WebCore::Node::styleForRenderer): Added NodeRenderingContext as a parameter.
        * dom/Node.h:
        * dom/NodeRenderingContext.cpp:
        (WebCore::NodeRendererFactory::createRendererAndStyle):
        * html/HTMLOptGroupElement.cpp:
        (WebCore::HTMLOptGroupElement::attach):
        * html/HTMLOptionElement.cpp:
        (WebCore::HTMLOptionElement::attach):
        * html/HTMLTitleElement.cpp:
        (WebCore::HTMLTitleElement::textWithDirection):
        * html/shadow/TextControlInnerElements.cpp:
        (WebCore::TextControlInnerElement::styleForRenderer):
        (WebCore::TextControlInnerTextElement::styleForRenderer):
        * html/shadow/TextControlInnerElements.h:

        Test: fast/dom/shadow/shadow-root-text-child.html

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

8 years ago2011-06-30 Leandro Pereira <leandro@profusion.mobi>
commit-queue@webkit.org [Thu, 30 Jun 2011 07:42:09 +0000 (07:42 +0000)]
2011-06-30  Leandro Pereira  <leandro@profusion.mobi>

        Reviewed by Simon Hausmann.

        generate-webkitversion.pl should not "use Switch"
        https://bugs.webkit.org/show_bug.cgi?id=63628

        Switch got deprecated on recent Perl versions and the script wasn't
        using the switch statement.

        * scripts/generate-webkitversion.pl: Remove "use Switch;" line.

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

8 years ago2011-06-30 Kent Tamura <tkent@chromium.org>
tkent@chromium.org [Thu, 30 Jun 2011 07:42:07 +0000 (07:42 +0000)]
2011-06-30  Kent Tamura  <tkent@chromium.org>

        [GTK][Mac][Win] Update expectation files for r90089.

        * platform/gtk/fast/block/float/032-expected.txt:
        * platform/gtk/fast/css/input-search-padding-expected.txt:
        * platform/gtk/fast/css/pseudo-cache-stale-expected.txt:
        * platform/gtk/fast/css/text-input-with-webkit-border-radius-expected.txt:
        * platform/gtk/fast/forms/box-shadow-override-expected.txt:
        * platform/gtk/fast/forms/control-restrict-line-height-expected.txt:
        * platform/gtk/fast/forms/input-appearance-height-expected.txt:
        * platform/gtk/fast/forms/input-appearance-spinbutton-disabled-readonly-expected.txt:
        * platform/gtk/fast/forms/input-appearance-spinbutton-layer-expected.txt:
        * platform/gtk/fast/forms/input-appearance-spinbutton-visibility-expected.txt:
        * platform/gtk/fast/forms/input-text-word-wrap-expected.txt:
        * platform/gtk/fast/forms/placeholder-position-expected.txt:
        * platform/gtk/fast/forms/placeholder-pseudo-style-expected.txt:
        * platform/gtk/fast/forms/placeholder-set-value-expected.txt:
        * platform/gtk/fast/forms/search-cancel-button-style-sharing-expected.txt:
        * platform/gtk/fast/forms/search-display-none-cancel-button-expected.txt:
        * platform/gtk/fast/forms/search-placeholder-value-changed-expected.txt:
        * platform/gtk/fast/forms/search-rtl-expected.txt:
        * platform/gtk/fast/forms/search-styled-expected.txt:
        * platform/gtk/fast/forms/search-vertical-alignment-expected.txt:
        * platform/gtk/fast/forms/searchfield-heights-expected.txt:
        * platform/gtk/fast/forms/textfield-overflow-expected.txt:
        * platform/gtk/fast/repaint/search-field-cancel-expected.txt:
        * platform/gtk/fast/replaced/width100percent-searchfield-expected.txt:
        * platform/gtk/fast/table/colspanMinWidth-vertical-expected.txt:
        * platform/mac/fast/block/float/032-expected.txt:
        * platform/win/fast/forms/search-cancel-button-style-sharing-expected.txt:
        * platform/win/fast/forms/search-display-none-cancel-button-expected.txt:
        * platform/win/fast/forms/search-placeholder-value-changed-expected.txt:

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

8 years ago2011-06-30 Piroska András <Piroska.Andras@stud.u-szeged.hu>
commit-queue@webkit.org [Thu, 30 Jun 2011 07:20:50 +0000 (07:20 +0000)]
2011-06-30  Piroska András  <Piroska.Andras@stud.u-szeged.hu>

        Reviewed by Dirk Schulze.

        Apply the ParallelJobs support to FEMorphology
        https://bugs.webkit.org/show_bug.cgi?id=63064

        The FEMorphology filter of SVG can consume lots of resources if it is
        applied to a large area. The computation can be distributed to multiple
        cores if the architecture supports.
        The average performance progression is 20-25% on dual-core machines.

        * platform/graphics/filters/FEMorphology.cpp:
        (WebCore::FEMorphology::platformApplyGeneric): Apply the filter between yStart and yEnd interval
        (WebCore::FEMorphology::platformApplyWorker): Call the generic apply from a parallel worker
        (WebCore::FEMorphology::platformApply):
        (WebCore::FEMorphology::apply): Move the functionality into platformApply
        (WebCore::FEMorphology::externalRepresentation): Remove whitespaces
        * platform/graphics/filters/FEMorphology.h:

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

8 years ago[Qt] Unreviewed. Update platform specific expected files after r90089.
ossy@webkit.org [Thu, 30 Jun 2011 07:14:45 +0000 (07:14 +0000)]
[Qt] Unreviewed. Update platform specific expected files after r90089.

* platform/qt/fast/css/pseudo-cache-stale-expected.png: Added.
* platform/qt/fast/css/pseudo-cache-stale-expected.txt:
* platform/qt/fast/css/text-input-with-webkit-border-radius-expected.png: Added.
* platform/qt/fast/css/text-input-with-webkit-border-radius-expected.txt:
* platform/qt/fast/forms/control-restrict-line-height-expected.png:
* platform/qt/fast/forms/control-restrict-line-height-expected.txt:
* platform/qt/fast/forms/placeholder-set-value-expected.png: Added.
* platform/qt/fast/forms/placeholder-set-value-expected.txt:
* platform/qt/fast/forms/search-cancel-button-style-sharing-expected.png: Added.
* platform/qt/fast/forms/search-cancel-button-style-sharing-expected.txt:
* platform/qt/fast/forms/search-display-none-cancel-button-expected.png: Added.
* platform/qt/fast/forms/search-display-none-cancel-button-expected.txt:
* platform/qt/fast/forms/search-placeholder-value-changed-expected.png: Added.
* platform/qt/fast/forms/search-placeholder-value-changed-expected.txt:
* platform/qt/fast/forms/search-rtl-expected.png: Added.
* platform/qt/fast/forms/search-rtl-expected.txt:
* platform/qt/fast/forms/search-styled-expected.png: Added.
* platform/qt/fast/forms/search-styled-expected.txt:
* platform/qt/fast/forms/search-vertical-alignment-expected.png: Added.
* platform/qt/fast/forms/search-vertical-alignment-expected.txt:
* platform/qt/fast/replaced/width100percent-searchfield-expected.png:
* platform/qt/fast/replaced/width100percent-searchfield-expected.txt:

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

8 years ago2011-06-29 Kent Tamura <tkent@chromium.org>
tkent@chromium.org [Thu, 30 Jun 2011 06:30:23 +0000 (06:30 +0000)]
2011-06-29  Kent Tamura  <tkent@chromium.org>

        Reviewed by Dimitri Glazkov.

        Avoid custom layout code of RenderTextControlSingleLine
        https://bugs.webkit.org/show_bug.cgi?id=61415

        Update tests including <input type=search> or <input type=number>. Their
        internal structures are changed.
        Slight position changes for search result buttons and search cancel
        buttons are expected.

        * fast/css/text-input-with-webkit-border-radius-expected.txt:
        * platform/chromium/test_expectations.txt:
        * platform/mac/fast/css/input-search-padding-expected.png:
        * platform/mac/fast/css/input-search-padding-expected.txt:
        * platform/mac/fast/css/pseudo-cache-stale-expected.txt:
        * platform/mac/fast/forms/box-shadow-override-expected.txt:
        * platform/mac/fast/forms/control-restrict-line-height-expected.txt:
        * platform/mac/fast/forms/input-appearance-height-expected.txt:
        * platform/mac/fast/forms/input-appearance-spinbutton-disabled-readonly-expected.txt:
        * platform/mac/fast/forms/input-appearance-spinbutton-expected.txt:
        * platform/mac/fast/forms/input-appearance-spinbutton-layer-expected.txt:
        * platform/mac/fast/forms/input-appearance-spinbutton-up-expected.txt:
        * platform/mac/fast/forms/input-appearance-spinbutton-visibility-expected.txt:
        * platform/mac/fast/forms/placeholder-position-expected.txt:
        * platform/mac/fast/forms/placeholder-pseudo-style-expected.txt:
        * platform/mac/fast/forms/placeholder-set-value-expected.txt:
        * platform/mac/fast/forms/search-cancel-button-style-sharing-expected.txt:
        * platform/mac/fast/forms/search-display-none-cancel-button-expected.txt:
        * platform/mac/fast/forms/search-placeholder-value-changed-expected.txt:
        * platform/mac/fast/forms/search-rtl-expected.txt:
        * platform/mac/fast/forms/search-styled-expected.txt:
        * platform/mac/fast/forms/search-vertical-alignment-expected.png:
        * platform/mac/fast/forms/search-vertical-alignment-expected.txt:
        * platform/mac/fast/forms/searchfield-heights-expected.txt:
        * platform/mac/fast/repaint/search-field-cancel-expected.png:
        * platform/mac/fast/repaint/search-field-cancel-expected.txt:
        * platform/mac/fast/replaced/width100percent-searchfield-expected.txt:
        * platform/mac/fast/table/colspanMinWidth-vertical-expected.png:
        * platform/mac/fast/table/colspanMinWidth-vertical-expected.txt:
2011-06-29  Kent Tamura  <tkent@chromium.org>

        Reviewed by Dimitri Glazkov.

        Avoid custom layout code of RenderTextControlSingleLine
        https://bugs.webkit.org/show_bug.cgi?id=61415

        If a text field has a search results button, a search cancel
        button, a speech button, or a spin button, the editable inner text
        element and the buttons are wrapped by a flexible box container,
        and the editable inner text element has flexibility. As the result
        of applying the flexible box, we don't need any horizontal layout
        code in RenderTextControlSingleLine except the inner spin button.

        Also, we remove custom hit-testing code in nodeAtPoint(). If a
        user clicks on a position left of the inner editable text but not
        on the search results button, the search results button doesn't
        work though it worked.

        * css/html.css:
        (input::-webkit-textfield-decoration-container):
          New style definition for the "container" element.
        (input[type="search"]::-webkit-search-cancel-button):
          It should be a block element because it's a child of the flexible box.
          Also, this doesn't have flexibility.
        (input[type="search"]::-webkit-search-decoration): ditto.
        (input[type="search"]::-webkit-search-results-decoration): ditto.
        (input[type="search"]::-webkit-search-results-button): ditto.
        (input::-webkit-inner-spin-button): ditto.
        (input::-webkit-input-speech-button): ditto.
        * html/HTMLInputElement.cpp:
        (WebCore::HTMLInputElement::containerElement):
          Add an accessor of the flexible box container.
          It just calls InputType::containerElement().
        * html/HTMLInputElement.h: Declare it.
        * html/InputType.h:
        (WebCore::InputType::containerElement): New accessor for a shadow element.
        * html/SearchInputType.cpp:
        (WebCore::SearchInputType::SearchInputType):
          m_innerBlock is moved to TextFieldInputType.
        (WebCore::SearchInputType::needsContainer):
          A search field always needs the flexible box container because
          of the results button and the cancel button.
        (WebCore::SearchInputType::createShadowSubtree):
          Use TextFieldInputType::createShadowSubtree(), then modify the tree.
        (WebCore::SearchInputType::destroyShadowSubtree):
          m_innerBlock is moved to TextFieldInputType.
        * html/SearchInputType.h:
        * html/TextFieldInputType.cpp:
        (WebCore::TextFieldInputType::TextFieldInputType):
          Initialize the new members.
        (WebCore::TextFieldInputType::needsContainer):
          Returns true if x-webkit-speech. We check a spin button separately.
        (WebCore::TextFieldInputType::createShadowSubtree):
          Wrap everything with the flexible box container if needsContainer() or we need a spin button.
        (WebCore::TextFieldInputType::containerElement): Added an accessor.
        (WebCore::TextFieldInputType::innerBlockElement): ditto.
        (WebCore::TextFieldInputType::innerTextElement): ditto.
        (WebCore::TextFieldInputType::innerSpinButtonElement): ditto.
        (WebCore::TextFieldInputType::speechButtonElement): ditto.
        (WebCore::TextFieldInputType::destroyShadowSubtree):
          Clear references to new shadow nodes.
        * html/TextFieldInputType.h:
        * rendering/RenderTextControlSingleLine.cpp:
        (WebCore::RenderTextControlSingleLine::containerElement): Added an accessor.
        (WebCore::RenderTextControlSingleLine::innerTextElement):
        (WebCore::RenderTextControlSingleLine::layout):
         - Correct heights check
          Comparing with height(), not contentHeigth(), was incorrect.
         - Adjust the container height like the editable inner text.
         - Remove width adjustment
         - Remove the speech button layout code
          The flexible box layout is enough.
        (WebCore::RenderTextControlSingleLine::nodeAtPoint):
          Remove custom code for results/cancel/speech buttons, and
          Simplify comments.
        (WebCore::RenderTextControlSingleLine::forwardEvent):
          Remove custom forwarding code for results/cancel buttons.
        (WebCore::RenderTextControlSingleLine::styleDidChange):
          Clear the style of the container because we update it in layout().
        (WebCore::RenderTextControlSingleLine::hasControlClip):
          Decoration buttons should be clipped by the container.
        (WebCore::RenderTextControlSingleLine::controlClipRect): ditto.
        (WebCore::RenderTextControlSingleLine::createInnerTextStyle):
          We don't need to use display:inline-block any more.
        (WebCore::RenderTextControlSingleLine::createInnerBlockStyle):
          The text wrapper element should have flexibility.
        (WebCore::RenderTextControlSingleLine::textBlockInsetTop):
          Updated for the tree structure change.
        * rendering/RenderTextControlSingleLine.h:

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

8 years ago2011-06-29 Kent Tamura <tkent@chromium.org>
tkent@chromium.org [Thu, 30 Jun 2011 05:39:40 +0000 (05:39 +0000)]
2011-06-29  Kent Tamura  <tkent@chromium.org>

        [Mac] Attempt to fix Leopard build.

        * platform/mac/CursorMac.mm:
        (WebCore::Cursor::ensurePlatformCursor):

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

8 years ago2011-06-29 Darin Adler <darin@apple.com>
darin@apple.com [Thu, 30 Jun 2011 05:12:28 +0000 (05:12 +0000)]
2011-06-29  Darin Adler  <darin@apple.com>

        Reviewed by Dan Bernstein.

        [Mac] Use system cursors instead of custom bitmaps where possible
        https://bugs.webkit.org/show_bug.cgi?id=63679

        * WebCoreSupport/WebSystemInterface.mm:
        (InitWebCoreSystemInterface): Added WKCursor.
2011-06-29  Darin Adler  <darin@apple.com>

        Reviewed by Dan Bernstein.

        [Mac] Use system cursors instead of custom bitmaps where possible
        https://bugs.webkit.org/show_bug.cgi?id=63679

        * Resources/crossHairCursor.png: Removed.
        * Resources/notAllowedCursor.png: Removed.
        * WebCore.exp.in: Added wkCursor.
        * WebCore/WebCore.gyp/WebCore.gyp: Removed the two now-unused PNG files.
        * WebCore.xcodeproj/project.pbxproj: Ditto.

        * platform/mac/CursorMac.mm:
        (WebCore::Cursor::ensurePlatformCursor): Use NSCursor API for Cross, Hand,
        VerticalText, ContextMenu, Copy, and NotAllowed. Use wkCursor for Move,
        Alias, Progress, and the resize and panning cursors.

        * platform/mac/WebCoreSystemInterface.h: Added wkCursor.
        * platform/mac/WebCoreSystemInterface.mm: Ditto.
2011-06-29  Darin Adler  <darin@apple.com>

        Reviewed by Dan Bernstein.

        [Mac] Use system cursors instead of custom bitmaps where possible
        https://bugs.webkit.org/show_bug.cgi?id=63679

        * WebProcess/WebCoreSupport/mac/WebSystemInterface.mm:
        (InitWebCoreSystemInterface): Added WKCursor.

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

8 years ago2011-06-29 Dirk Pranke <dpranke@chromium.org>
dpranke@chromium.org [Thu, 30 Jun 2011 04:16:18 +0000 (04:16 +0000)]
2011-06-29  Dirk Pranke  <dpranke@chromium.org>

        Reviewed by Adam Barth.

        test-webkitpy is spamming lots of messages about updating test expectations
        https://bugs.webkit.org/show_bug.cgi?id=63680

        This patch changes the passing_run() function to capture the
        output to stdout and stderr and assert that it is empty, and
        also specifies --no-new-test-results by default to avoid getting
        "generating baseline" messages for the expected-missing results.

        Also check that we are logging messages when we generate new
        baselines.

        Also fix a minor bug in the way test_tolerance() was
        checking for whether the tests were passing or not.

        * Scripts/webkitpy/layout_tests/run_webkit_tests_integrationtest.py:

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

8 years ago2011-06-29 Yuta Kitamura <yutak@chromium.org>
yutak@chromium.org [Thu, 30 Jun 2011 03:40:01 +0000 (03:40 +0000)]
2011-06-29  Yuta Kitamura  <yutak@chromium.org>

        Reviewed by Kent Tamura.

        DRT: Make Hixie76WebSocketProtocolEnabled preference flag configurable from LayoutTestController
        https://bugs.webkit.org/show_bug.cgi?id=63532

        LayoutTestController::overridePreference() will be used to change the value of
        the Hixie76WebSocketProtocolEnabled preference flag from layout tests. In this way,
        we can avoid writing a lot of boilerplate code in each port's DumpRenderTree, but
        at least we need to reset preference values in DumpRenderTree every time we start
        a new test because preference values changed in some test should not be leaked
        to the next test.

        * DumpRenderTree/chromium/LayoutTestController.cpp:
        (LayoutTestController::overridePreference):
        The name of preference key is defined in WebKit/{mac/WebView,win}/WebPreferenceKeysPrivate.h.
        * DumpRenderTree/chromium/WebPreferences.cpp:
        (WebPreferences::reset):
        Set the default value. This will be called from TestShell::resetTestController().
        (WebPreferences::applyTo):
        * DumpRenderTree/chromium/WebPreferences.h:
        * DumpRenderTree/mac/DumpRenderTree.mm:
        (resetDefaultsToConsistentValues): Set the default value.
        * DumpRenderTree/win/DumpRenderTree.cpp:
        (resetDefaultsToConsistentValues): Ditto.

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

8 years ago2011-06-29 Simon Fraser <simon.fraser@apple.com>
simon.fraser@apple.com [Thu, 30 Jun 2011 03:34:18 +0000 (03:34 +0000)]
2011-06-29  Simon Fraser  <simon.fraser@apple.com>

        Fix the test description.

        * fast/layers/negative-scroll-positions-expected.txt:
        * fast/layers/script-tests/negative-scroll-positions.js:

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

8 years ago2011-06-29 Kenichi Ishibashi <bashi@chromium.org>
bashi@chromium.org [Thu, 30 Jun 2011 03:07:40 +0000 (03:07 +0000)]
2011-06-29  Kenichi Ishibashi  <bashi@chromium.org>

        Reviewed by Adam Barth.

        check-webkit-style fails to detect https://webkit.org/b/XXXXX as a bug number
        https://webkit.org/b/57360

        Modify the regular expression to match https scheme.

        * Scripts/webkitpy/common/checkout/changelog_unittest.py: Added a test case for https://webkit.org/b/XXXXX.
        * Scripts/webkitpy/common/config/urls.py: Modified bug_url_short to match https scheme.

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

8 years ago2011-06-29 Keishi Hattori <keishi@webkit.org>
keishi@webkit.org [Thu, 30 Jun 2011 03:05:03 +0000 (03:05 +0000)]
2011-06-29  Keishi Hattori  <keishi@webkit.org>

        Reviewed by Adam Barth.

        Remove extra space at EOL in view source mode
        https://bugs.webkit.org/show_bug.cgi?id=63620

        * fast/frames/resources/viewsource-frame-5.html: Added.
        * fast/frames/viewsource-empty-attribute-value-expected.txt: Regenerated.
        * fast/frames/viewsource-linebreak-expected.txt: Added.
        * fast/frames/viewsource-linebreak.html: Added. Test that there are no extra spaces at EOL.
        And that the multiple consecutive line breaks are maintained in the frame.
2011-06-29  Keishi Hattori  <keishi@webkit.org>

        Reviewed by Adam Barth.

        Remove extra space at EOL in view source mode
        https://bugs.webkit.org/show_bug.cgi?id=63620

        Test: fast/frames/viewsource-linebreak.html

        * css/view-source.css:
        (tbody:last-child .webkit-line-content:empty:before): If we don't do this
        the last line will be empty and the height will be shorter than the rest.
        * html/HTMLViewSourceDocument.cpp:
        (WebCore::HTMLViewSourceDocument::addSource): If source file is empty, add an empty line.
        (WebCore::HTMLViewSourceDocument::finishLine): Call when you are done with the current line.
        Adds a BR element if the line is empty and sets current to tbody.
        (WebCore::HTMLViewSourceDocument::addText):
        * html/HTMLViewSourceDocument.h:

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

8 years ago2011-06-29 Adam Barth <abarth@webkit.org>
abarth@webkit.org [Thu, 30 Jun 2011 02:30:02 +0000 (02:30 +0000)]
2011-06-29  Adam Barth  <abarth@webkit.org>

        Reviewed by Eric Seidel.

        new-run-webkit-tests: images page switch actual and expected images before they're loaded
        https://bugs.webkit.org/show_bug.cgi?id=63199

        We now preload the images and keep the DOM nodes in memory instead of
        going back to the MemoryCache all the time.  (Also, remove some nutty
        </img> HTML.)

        * Scripts/webkitpy/layout_tests/layout_package/test_result_writer.py:

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

8 years ago2011-06-29 Kangil Han <kangil.han@samsung.com>
commit-queue@webkit.org [Thu, 30 Jun 2011 02:06:39 +0000 (02:06 +0000)]
2011-06-29  Kangil Han  <kangil.han@samsung.com>

        Reviewed by Kenneth Rohde Christiansen.

        [EFL] Add APIs to get/set view mode
        https://bugs.webkit.org/show_bug.cgi?id=63229

        Implement public API for getting/setting the view-mode media feature as specified in W3C.

        * ewk/ewk_view.cpp:
        (ewk_view_mode_set):
        (ewk_view_mode_get):
        * ewk/ewk_view.h:

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

8 years ago2011-06-29 Adam Barth <abarth@webkit.org>
abarth@webkit.org [Thu, 30 Jun 2011 02:02:04 +0000 (02:02 +0000)]
2011-06-29  Adam Barth  <abarth@webkit.org>

        Reviewed by Eric Seidel.

        new-run-webkit-tests should tell you when it creates new expectations
        https://bugs.webkit.org/show_bug.cgi?id=63675

        Unforunately, this patch creates a bunch of test-webkitpy spam because
        we don't understand how Python logging works well enough to stop the
        spam.  :(

        * Scripts/webkitpy/layout_tests/layout_package/single_test_runner.py:

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

8 years ago2011-06-29 Eric Seidel <eric@webkit.org>
eric@webkit.org [Thu, 30 Jun 2011 01:48:44 +0000 (01:48 +0000)]
2011-06-29  Eric Seidel  <eric@webkit.org>

        Reviewed by Adam Barth.

        Refactor WebKitDriver.run_test into smaller pieces
        https://bugs.webkit.org/show_bug.cgi?id=63673

        There is no functional change here.  Just moving code around.
        This is in preparation for adding support for #CRASHED and #CRASHED - WebProcess

        * Scripts/webkitpy/layout_tests/port/webkit.py:

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

8 years ago2011-06-29 Eric Seidel <eric@webkit.org>
eric@webkit.org [Thu, 30 Jun 2011 01:36:03 +0000 (01:36 +0000)]
2011-06-29  Eric Seidel  <eric@webkit.org>

        Reviewed by Adam Barth.

        Simplify skipped file finding in preparation for adding wk2 skipped list fallback
        https://bugs.webkit.org/show_bug.cgi?id=63501

        Re-landing after fixing a bug in WebKitPort.baseline_search_path which broke
        the rebaselineserver unit tests.  Also fixed nits brought up by Dirk after previous landing.

        The goal was to add support for wk2 skipped lists.
        However, I found that our skipped list computation was a
        manual hack (which only worked for the mac port).

        So I fixed a FIXME to move the skipped list finding
        into WebKitPort instead of MacPort.
        Doing so required the concept of a "port_name", since previously
        the only name accessible from a port object was name()
        which includes many things beyond "mac" or "chromium", etc.

        Eventually I believe we'll want to re-think the way that we pass
        in a port_name argument to Port subclasses and expect them to parse
        it.  But for now I just added a cls.port_name variable which contains
        the static information needed to compute wk2 names as well as
        compute Skipped list fallback which works for Mac/Win/Qt and Gtk.

        In order to test my new _skipped_file_search_paths method, I
        fixed another FIXME by making it return relative paths.

        I also fixed the test_expectations_path code in WebKitPort to use port_name.
        It was using name() which would return PORT-VERSION so MacPort was overriding
        it to use just PORT.  After fixing test_expectations_path to use port_name
        (and making it aware of webkit2) I was able to remove the MacPort implementation.

        * Scripts/webkitpy/layout_tests/port/base.py:
         - Add port_name() to access "mac", since name() returns "mac-leopard" etc.
         - Document that real_name() seems to have no purpose.
        * Scripts/webkitpy/layout_tests/port/chromium_linux.py:
         - Add _parse_port_name(), eventually we might call this from WebKitPort directly.
        * Scripts/webkitpy/layout_tests/port/chromium_mac.py:
         - Add _parse_port_name.
        * Scripts/webkitpy/layout_tests/port/chromium_win.py:
         - Add _parse_port_name.
        * Scripts/webkitpy/layout_tests/port/gtk.py:
        * Scripts/webkitpy/layout_tests/port/mac.py:
         - Move Skipped-file finding code down to WebKitPort
        * Scripts/webkitpy/layout_tests/port/mac_unittest.py:
        * Scripts/webkitpy/layout_tests/port/qt.py:
        * Scripts/webkitpy/layout_tests/port/webkit.py:
        * Scripts/webkitpy/layout_tests/port/webkit_unittest.py:

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

8 years ago2011-06-29 Sheriff Bot <webkit.review.bot@gmail.com>
commit-queue@webkit.org [Thu, 30 Jun 2011 01:11:42 +0000 (01:11 +0000)]
2011-06-29  Sheriff Bot  <webkit.review.bot@gmail.com>

        Unreviewed, rolling out r90070.
        http://trac.webkit.org/changeset/90070
        https://bugs.webkit.org/show_bug.cgi?id=63676

        Broke test-webkitpy rebaseline tests (Requested by eseidel2 on
        #webkit).

        * Scripts/webkitpy/layout_tests/port/base.py:
        * Scripts/webkitpy/layout_tests/port/chromium.py:
        * Scripts/webkitpy/layout_tests/port/chromium_linux.py:
        * Scripts/webkitpy/layout_tests/port/chromium_mac.py:
        * Scripts/webkitpy/layout_tests/port/chromium_win.py:
        * Scripts/webkitpy/layout_tests/port/gtk.py:
        * Scripts/webkitpy/layout_tests/port/mac.py:
        * Scripts/webkitpy/layout_tests/port/mac_unittest.py:
        * Scripts/webkitpy/layout_tests/port/qt.py:
        * Scripts/webkitpy/layout_tests/port/webkit.py:
        * Scripts/webkitpy/layout_tests/port/webkit_unittest.py:
        * Scripts/webkitpy/layout_tests/port/win.py:
        * Scripts/webkitpy/tool/commands/rebaselineserver_unittest.py:

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

8 years ago[WinCE] Correct logical state of WINCE test.
bfulgham@webkit.org [Thu, 30 Jun 2011 00:46:42 +0000 (00:46 +0000)]
[WinCE] Correct logical state of WINCE test.

* platform/win/ScrollbarThemeWin.cpp: The #if OS(WINCE)
  should have been #if !OS(WINCE)
(WebCore::ScrollbarThemeWin::paintTrackPiece):
(WebCore::ScrollbarThemeWin::paintButton):
(WebCore::ScrollbarThemeWin::paintThumb):
* rendering/RenderThemeWin.cpp: ditto.
(WebCore::drawControl):

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

8 years ago2011-06-29 Adam Barth <abarth@webkit.org>
abarth@webkit.org [Thu, 30 Jun 2011 00:42:39 +0000 (00:42 +0000)]
2011-06-29  Adam Barth  <abarth@webkit.org>

        Reviewed by Eric Seidel.

        new-run-webkit-tests doesn't support sample-on-timeout
        https://bugs.webkit.org/show_bug.cgi?id=56731

        * Scripts/webkitpy/layout_tests/port/server_process.py:
        * Scripts/webkitpy/layout_tests/port/server_process_unittest.py:
        * Scripts/webkitpy/layout_tests/run_webkit_tests.py:

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

8 years ago2011-06-29 Justin Garcia <justin.garcia@apple.com>
justin.garcia@apple.com [Thu, 30 Jun 2011 00:30:49 +0000 (00:30 +0000)]
2011-06-29  Justin Garcia  <justin.garcia@apple.com>

        Reviewed by Enrica Casucci.

        I forgot to include these (reviewed) style fix-ups in my previous check-in.

        * editing/htmlediting.cpp:
        (WebCore::indexForVisiblePosition):
        (WebCore::visiblePositionForIndex):

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

8 years ago2011-06-29 Justin Garcia <justin.garcia@apple.com>
justin.garcia@apple.com [Thu, 30 Jun 2011 00:10:53 +0000 (00:10 +0000)]
2011-06-29  Justin Garcia  <justin.garcia@apple.com>

        Reviewed by Enrica Casucci.

        https://bugs.webkit.org/show_bug.cgi?id=62922
        indexForVisiblePosition(const VisiblePosition& visiblePosition) does not consider shadow content

        VisiblePositions can be inside web form text regions, which use shadow trees. Made indexForVisiblePosition
        aware of this, and added a new parameter to obtain the scope for a VisiblePosition, in addition to its index.

        Added visiblePositionForIndex to go in the opposite direction, taking into account the scope
        used to compute the index.

        These two functions use TextIterators to convert between VisiblePositions and indices. But
        TextIterator iteration using TextIteratorEmitsCharactersBetweenAllVisiblePositions does not
        exactly match VisiblePosition iteration, so using them to preserve a selection during an
        editing operation is unreliable. This can be seen in the expected results for:

        editing/execCommand/indent-pre-list.html
        editing/execCommand/crash-indenting-list-item.html

        TextIterator's TextIteratorEmitsCharactersBetweenAllVisiblePositions mode needs to be fixed, or
        these functions need to be changed to iterate using actual VisiblePositions. See:

        https://bugs.webkit.org/show_bug.cgi?id=63590
        TextIterators in TextIteratorEmitsCharactersBetweenAllVisiblePositions do not exactly match VisiblePositions

        Also:

        https://bugs.webkit.org/show_bug.cgi?id=63592
        Use visiblePositionForIndex and indexForVisiblePosition everywhere that TextIterators are used to convert between VisiblePositions and indices

        No new tests added because indexForVisiblePosition is currently only used for editing operations
        that cannot be performed inside web form fields.

        * editing/ApplyBlockElementCommand.cpp:
        (WebCore::ApplyBlockElementCommand::doApply):
        * editing/InsertListCommand.cpp:
        (WebCore::InsertListCommand::doApply):
        * editing/htmlediting.cpp:
        (WebCore::indexForVisiblePosition):
        (WebCore::visiblePositionForIndex):
        * editing/htmlediting.h:

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

8 years ago2011-06-29 Dimitri Glazkov <dglazkov@chromium.org>
dglazkov@chromium.org [Thu, 30 Jun 2011 00:03:36 +0000 (00:03 +0000)]
2011-06-29  Dimitri Glazkov  <dglazkov@chromium.org>

        Reviewed by Kent Tamura.

        Remove ElementWithPseudoId, since now an Element can store it all by itself.
        https://bugs.webkit.org/show_bug.cgi?id=63657

        Refactoring, covered by existing tests.

        * CMakeLists.txt: Removed ElementWithPseudoId from build.
        * GNUmakefile.list.am: Ditto.
        * WebCore.gypi: Ditto.
        * WebCore.order: Ditto.
        * WebCore.pro: Ditto.
        * WebCore.vcproj/WebCore.vcproj: Ditto.
        * WebCore.xcodeproj/project.pbxproj: Ditto.
        * html/ColorInputType.cpp:
        (WebCore::ColorInputType::createShadowSubtree): Changed to use plain-old HTMLDivElement.
        * html/RangeInputType.cpp:
        (WebCore::RangeInputType::createShadowSubtree): Ditto.
        * html/ValidationMessage.cpp:
        (WebCore::ValidationMessage::buildBubbleTree): Ditto.
        * html/shadow/ElementWithPseudoId.cpp: Removed.
        * html/shadow/ElementWithPseudoId.h: Removed.

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

8 years ago2011-06-29 Eric Seidel <eric@webkit.org>
eric@webkit.org [Thu, 30 Jun 2011 00:02:16 +0000 (00:02 +0000)]
2011-06-29  Eric Seidel  <eric@webkit.org>

        Reviewed by Adam Barth.

        Simplify skipped file finding in preparation for adding wk2 skipped list fallback
        https://bugs.webkit.org/show_bug.cgi?id=63501

        The goal was to add support for wk2 skipped lists.
        However, I found that our skipped list computation was a
        manual hack (which only worked for the mac port).

        So I fixed a FIXME to move the skipped list finding
        into WebKitPort instead of MacPort.
        Doing so required the concept of a "port_name", since previously
        the only name accessible from a port object was name()
        which includes many things beyond "mac" or "chromium", etc.

        Eventually I believe we'll want to re-think the way that we pass
        in a port_name argument to Port subclasses and expect them to parse
        it.  But for now I just added a cls.port_name variable which contains
        the static information needed to compute wk2 names as well as
        compute Skipped list fallback which works for Mac/Win/Qt and Gtk.

        In order to test my new _skipped_file_search_paths method, I
        fixed another FIXME by making it return relative paths.

        I also fixed the test_expectations_path code in WebKitPort to use port_name.
        It was using name() which would return PORT-VERSION so MacPort was overriding
        it to use just PORT.  After fixing test_expectations_path to use port_name
        (and making it aware of webkit2) I was able to remove the MacPort implementation.

        * Scripts/webkitpy/layout_tests/port/base.py:
         - Add port_name() to access "mac", since name() returns "mac-leopard" etc.
         - Document that real_name() seems to have no purpose.
        * Scripts/webkitpy/layout_tests/port/chromium_linux.py:
         - Add _parse_port_name(), eventually we might call this from WebKitPort directly.
        * Scripts/webkitpy/layout_tests/port/chromium_mac.py:
         - Add _parse_port_name.
        * Scripts/webkitpy/layout_tests/port/chromium_win.py:
         - Add _parse_port_name.
        * Scripts/webkitpy/layout_tests/port/gtk.py:
        * Scripts/webkitpy/layout_tests/port/mac.py:
         - Move Skipped-file finding code down to WebKitPort
        * Scripts/webkitpy/layout_tests/port/mac_unittest.py:
        * Scripts/webkitpy/layout_tests/port/qt.py:
        * Scripts/webkitpy/layout_tests/port/webkit.py:
        * Scripts/webkitpy/layout_tests/port/webkit_unittest.py:

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

8 years ago2011-06-29 Levi Weintraub <leviw@chromium.org>
leviw@chromium.org [Wed, 29 Jun 2011 23:59:49 +0000 (23:59 +0000)]
2011-06-29  Levi Weintraub  <leviw@chromium.org>

        Reviewed by Eric Seidel.

        Switch computeWidth and computeHeight functions to new layout types
        https://bugs.webkit.org/show_bug.cgi?id=63655

        Switch computeWidth and computeHeight functions over to the new layout unit abstraction.

        Making a point to mark rounding that should eventually be removed by linking to a new
        tracking bug: 63645.

        No new tests, no functionality changes.

        * rendering/AutoTableLayout.cpp:
        (WebCore::AutoTableLayout::computePreferredLogicalWidths):
        * rendering/AutoTableLayout.h:
        * rendering/FixedTableLayout.cpp:
        (WebCore::FixedTableLayout::computePreferredLogicalWidths):
        * rendering/FixedTableLayout.h:
        * rendering/LayoutTypes.h:
        * rendering/RenderBlock.cpp:
        (WebCore::RenderBlock::computePreferredLogicalWidths):
        (WebCore::RenderBlock::computeInlinePreferredLogicalWidths):
        (WebCore::RenderBlock::computeBlockPreferredLogicalWidths):
        * rendering/RenderBox.cpp:
        (WebCore::RenderBox::computeLogicalWidthUsing):
        (WebCore::RenderBox::computeLogicalHeight):
        (WebCore::RenderBox::computeLogicalHeightUsing):
        (WebCore::RenderBox::computePercentageLogicalHeight):
        (WebCore::RenderBox::computeReplacedLogicalWidth):
        (WebCore::RenderBox::computeReplacedLogicalWidthRespectingMinMaxWidth):
        (WebCore::RenderBox::computeReplacedLogicalWidthUsing):
        (WebCore::RenderBox::computeReplacedLogicalHeight):
        (WebCore::RenderBox::computeReplacedLogicalHeightRespectingMinMaxHeight):
        (WebCore::RenderBox::computeReplacedLogicalHeightUsing):
        * rendering/RenderBox.h:
        * rendering/RenderIFrame.cpp:
        (WebCore::RenderIFrame::computeLogicalWidth):
        * rendering/RenderImage.cpp:
        (WebCore::RenderImage::computeReplacedLogicalWidth):
        (WebCore::RenderImage::computeReplacedLogicalHeight):
        * rendering/RenderImage.h:
        * rendering/RenderListBox.cpp:
        (WebCore::RenderListBox::computePreferredLogicalWidths):
        * rendering/RenderListMarker.cpp:
        (WebCore::RenderListMarker::computePreferredLogicalWidths):
        * rendering/RenderMenuList.cpp:
        (WebCore::RenderMenuList::computePreferredLogicalWidths):
        * rendering/RenderReplaced.cpp:
        (WebCore::RenderReplaced::computeReplacedLogicalWidth):
        (WebCore::RenderReplaced::computeReplacedLogicalHeight):
        (WebCore::RenderReplaced::computePreferredLogicalWidths):
        * rendering/RenderReplaced.h:
        * rendering/RenderSlider.cpp:
        (WebCore::RenderSlider::computePreferredLogicalWidths):
        * rendering/RenderTable.cpp:
        (WebCore::RenderTable::computeLogicalWidth):
        * rendering/RenderTextControl.cpp:
        (WebCore::RenderTextControl::computePreferredLogicalWidths):
        * rendering/RenderVideo.cpp:
        (WebCore::RenderVideo::computeReplacedLogicalWidth):
        (WebCore::RenderVideo::computeReplacedLogicalHeight):
        * rendering/RenderVideo.h:
        * rendering/svg/RenderSVGForeignObject.cpp:
        (WebCore::RenderSVGForeignObject::computeLogicalWidth):
        (WebCore::RenderSVGForeignObject::computeLogicalHeight):
        * rendering/svg/RenderSVGRoot.cpp:
        (WebCore::RenderSVGRoot::computePreferredLogicalWidths):
        (WebCore::RenderSVGRoot::computeIntrinsicWidth):
        (WebCore::RenderSVGRoot::computeIntrinsicHeight):
        (WebCore::RenderSVGRoot::computeReplacedLogicalWidth):
        (WebCore::RenderSVGRoot::computeReplacedLogicalHeight):
        (WebCore::RenderSVGRoot::layout):
        * rendering/svg/RenderSVGRoot.h:

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

8 years ago2011-06-23 Abhishek Arya <inferno@chromium.org>
inferno@chromium.org [Wed, 29 Jun 2011 23:58:25 +0000 (23:58 +0000)]
2011-06-23  Abhishek Arya  <inferno@chromium.org>

        Reviewed by James Robinson.

        In RenderBlock, RenderWidget and RenderReplaced destroy functions,
        call dirtyLinesFromChangedChild to tell our parent that we are going away.
        https://bugs.webkit.org/show_bug.cgi?id=60307

        Test: fast/block/child-not-removed-from-parent-lineboxes-crash.html
              fast/block/block-not-removed-from-parent-lineboxes-crash.html

        * rendering/RenderBlock.cpp:
        (WebCore::RenderBlock::destroy):
        * rendering/RenderReplaced.cpp:
        (WebCore::RenderReplaced::destroy):
        * rendering/RenderReplaced.h:
        * rendering/RenderWidget.cpp:
        (WebCore::RenderWidget::destroy):
2011-06-23  Abhishek Arya  <inferno@chromium.org>

        Reviewed by James Robinson.

        Tests that we do not crash when failing to remove the linebreak
        object(widget, block, replaced, etc) from parent block lineboxes. The
        testcase tries to fuzz over the list of html and svg tags.
        https://bugs.webkit.org/show_bug.cgi?id=60307

        * fast/block/block-not-removed-from-parent-lineboxes-crash-expected.txt: Added.
        * fast/block/block-not-removed-from-parent-lineboxes-crash.html: Added.
        * fast/block/child-not-removed-from-parent-lineboxes-crash-expected.txt: Added.
        * fast/block/child-not-removed-from-parent-lineboxes-crash.html: Added.

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

8 years ago2011-06-29 Alan Stearns <stearns@adobe.com>
commit-queue@webkit.org [Wed, 29 Jun 2011 23:54:37 +0000 (23:54 +0000)]
2011-06-29  Alan Stearns  <stearns@adobe.com>

        Reviewed by Tony Chang.

        Modify Skipped files so that future css regions layout tests are
        skipped by default.

        [CSS Regions] Add support for skipped tests
        https://bugs.webkit.org/show_bug.cgi?id=63632

        * fast/regions/flow-content-basic.html: Added.
        * platform/chromium/test_expectations.txt:
        * platform/gtk/Skipped:
        * platform/mac/Skipped:
        * platform/qt/Skipped:
        * platform/win/Skipped:

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

8 years agoSpeculative WinCE build correction.
bfulgham@webkit.org [Wed, 29 Jun 2011 23:52:08 +0000 (23:52 +0000)]
Speculative WinCE build correction.

* rendering/RenderThemeWin.cpp:
(WebCore::drawControl): Don't call setRGBABitmapAlpha
  for WinCE case.

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

8 years agohttps://bugs.webkit.org/show_bug.cgi?id=63669
barraclough@apple.com [Wed, 29 Jun 2011 23:50:12 +0000 (23:50 +0000)]
https://bugs.webkit.org/show_bug.cgi?id=63669
DFG JIT - fix spectral-norm regression

Reviewed by Geoff Garen.

The problem is a mis-speculation leading to us falling off the speculative path.
Make the speculation logic slightly smarter, don't predict int if one of the
operands is already loaded as a double (we use this logic already for compares).

* dfg/DFGSpeculativeJIT.cpp:
(JSC::DFG::SpeculativeJIT::compile):
* dfg/DFGSpeculativeJIT.h:
(JSC::DFG::SpeculativeJIT::shouldSpeculateInteger):

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

8 years ago2011-06-29 Ryosuke Niwa <rniwa@webkit.org>
rniwa@webkit.org [Wed, 29 Jun 2011 23:45:52 +0000 (23:45 +0000)]
2011-06-29  Ryosuke Niwa  <rniwa@webkit.org>

        Reviewed by Darin Adler.

        REGRESSION(r89952): r89952 added a wrong assertion in visible_units.cpp
        https://bugs.webkit.org/show_bug.cgi?id=63668

        We need to assert the offset to be zero only when the position is NOT offset in node.

        * editing/visible_units.cpp:
        (WebCore::startOfParagraph):

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

8 years ago2011-06-29 Filip Pizlo <fpizlo@apple.com>
commit-queue@webkit.org [Wed, 29 Jun 2011 23:45:28 +0000 (23:45 +0000)]
2011-06-29  Filip Pizlo  <fpizlo@apple.com>

        Reviewed by Gavin Barraclough.

        DFG JIT does not do put_by_id transition caching.
        https://bugs.webkit.org/show_bug.cgi?id=63662

        * dfg/DFGJITCodeGenerator.cpp:
        (JSC::DFG::JITCodeGenerator::cachedPutById):
        * dfg/DFGJITCompiler.h:
        (JSC::DFG::JITCompiler::addPropertyAccess):
        * dfg/DFGRepatch.cpp:
        (JSC::DFG::testPrototype):
        (JSC::DFG::tryCachePutByID):

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

8 years agoSpeculative WinCE build correction after r90057.
bfulgham@webkit.org [Wed, 29 Jun 2011 23:42:03 +0000 (23:42 +0000)]
Speculative WinCE build correction after r90057.

* CMakeListsWinCE.txt: Roll back r90061, since GetWorldTransform
  is not available in WinCE.
* platform/win/ScrollbarThemeWin.cpp: Conditionally exclude calls
  to DIBPixelData::setRGBABitmapAlpha for WinCE.
(WebCore::ScrollbarThemeWin::paintTrackPiece):
(WebCore::ScrollbarThemeWin::paintButton):
(WebCore::ScrollbarThemeWin::paintThumb):

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

8 years agoSpeculative WinCE build correction after r90057.
bfulgham@webkit.org [Wed, 29 Jun 2011 23:27:25 +0000 (23:27 +0000)]
Speculative WinCE build correction after r90057.

* CMakeListsWinCE.txt: Add missing DIBPixelData.cpp to
WinCE build to support setting bitmap image alpha value.

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

8 years ago2011-06-29 Geoffrey Garen <ggaren@apple.com>
ggaren@apple.com [Wed, 29 Jun 2011 23:11:59 +0000 (23:11 +0000)]
2011-06-29  Geoffrey Garen  <ggaren@apple.com>

        Reviewed by Oliver Hunt.

        Added a dummy write barrier emitting function in all the right places in the old JIT
        https://bugs.webkit.org/show_bug.cgi?id=63667

        SunSpider reports no change.

        * jit/JIT.h:
        * jit/JITPropertyAccess.cpp:
        (JSC::JIT::emit_op_put_by_id):
        (JSC::JIT::emit_op_put_scoped_var): Do it.

        (JSC::JIT::emit_op_put_global_var): Global object needs to be in a register
        for the sake of the write barrier.

        (JSC::JIT::emitWriteBarrier): Empty for now. Not for long!

        * jit/JITPropertyAccess32_64.cpp:
        (JSC::JIT::emit_op_put_by_val):
        (JSC::JIT::emit_op_put_by_id):
        (JSC::JIT::emit_op_put_scoped_var): Do it.

        (JSC::JIT::emit_op_put_global_var): Global object needs to be in a register
        for the sake of the write barrier.

        (JSC::JIT::emitWriteBarrier): Empty for now. Not for long!

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

8 years ago2011-06-29 Simon Fraser <simon.fraser@apple.com>
simon.fraser@apple.com [Wed, 29 Jun 2011 22:56:54 +0000 (22:56 +0000)]
2011-06-29  Simon Fraser  <simon.fraser@apple.com>

        Reviewed by Beth Dakin.

        Lay some groundwork for rubber-banding in overflow:scroll divs
        https://bugs.webkit.org/show_bug.cgi?id=63654

        Make it possible for scrollTop and scrollLeft on a RenderLayer to be
        set to negative values, which we will do during rubber-banding in
        future. Continue to clamp programmatic scrolls, and scroll positions
        set by layout between 0 and the max value.

        Test: fast/layers/negative-scroll-positions.html

        * rendering/RenderBox.cpp:
        (WebCore::RenderBox::setScrollLeft):
        (WebCore::RenderBox::setScrollTop):
        * rendering/RenderLayer.cpp:
        (WebCore::RenderLayer::scrollToOffset):
        (WebCore::RenderLayer::scrollTo):
        * rendering/RenderLayer.h:
        (WebCore::RenderLayer::scrollToXOffset):
        (WebCore::RenderLayer::scrollToYOffset):
        * rendering/RenderTextControlSingleLine.cpp:
        (WebCore::RenderTextControlSingleLine::forwardEvent):

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

8 years ago2011-06-29 Anders Carlsson <andersca@apple.com>
andersca@apple.com [Wed, 29 Jun 2011 22:53:15 +0000 (22:53 +0000)]
2011-06-29  Anders Carlsson  <andersca@apple.com>

        Reviewed by Simon Fraser.

        RenderLayer::maximumScrollPosition() is wrong
        https://bugs.webkit.org/show_bug.cgi?id=62667

        * platform/ScrollableArea.h:
        (WebCore::ScrollableArea::isVerticalScrollerPinnedToMinimumPosition):
        (WebCore::ScrollableArea::isVerticalScrollerPinnedToMaximumPosition):
        Add these for good measure; they will be needed in an upcoming patch.

        * rendering/RenderLayer.cpp:
        (WebCore::RenderLayer::maximumScrollPosition):
        Subtract the visible content rect, including the scroll bars.

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

8 years agoSpeculative WinCE build correction after r90053.
bfulgham@webkit.org [Wed, 29 Jun 2011 22:52:10 +0000 (22:52 +0000)]
Speculative WinCE build correction after r90053.

* platform/graphics/GraphicsContext.h:
(WebCore::GraphicsContext::inTransparencyLayer): Provide
stub implementation for 'inTransparencyLayer'.

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

8 years agoeventSender.keyDown is unimplemented (WKTR)
jberlin@webkit.org [Wed, 29 Jun 2011 22:42:04 +0000 (22:42 +0000)]
eventSender.keyDown is unimplemented (WKTR)
https://bugs.webkit.org/show_bug.cgi?id=57515

Add another test that uses eventSender.keyDown to the mac-wk2 Skipped list.

* platform/mac-wk2/Skipped:

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

8 years ago2011-06-29 Adam Barth <abarth@webkit.org>
abarth@webkit.org [Wed, 29 Jun 2011 22:25:58 +0000 (22:25 +0000)]
2011-06-29  Adam Barth  <abarth@webkit.org>

        Reviewed by Dirk Pranke.

        new-run-webkit-tests complains about missing pixel results instead of plopping down new expectations
        https://bugs.webkit.org/show_bug.cgi?id=38063

        This patch changes new-run-webkit-tests to match old-run-webkit-tests
        in generating new expected results when the expected results are
        missing.

        There are still a couple details that are different:

        1) Image baselines aren't generated unless you run with -p.
        2) Render tree dumps are places in the cross-platform directory instead
        of the platform-specific directory.

        I'm inclined to deal with both of these issues in follow-up patches.

        * Scripts/webkitpy/common/net/layouttestresults.py:
        * Scripts/webkitpy/layout_tests/layout_package/manager.py:
        * Scripts/webkitpy/layout_tests/layout_package/single_test_runner.py:
        * Scripts/webkitpy/layout_tests/layout_package/test_results.py:
        * Scripts/webkitpy/layout_tests/port/test.py:
        * Scripts/webkitpy/layout_tests/run_webkit_tests.py:
        * Scripts/webkitpy/layout_tests/run_webkit_tests_integrationtest.py:

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

8 years agoTeach TestFailures to detect possibly flaky tests and list them separately
aroben@apple.com [Wed, 29 Jun 2011 22:24:48 +0000 (22:24 +0000)]
Teach TestFailures to detect possibly flaky tests and list them separately

Fixes <http://webkit.org/b/61061> <rdar://problem/9452796> TestFailures page blames
arbitrary revisions for breaking flaky tests

Reviewed by Dan Bates.

* BuildSlaveSupport/build.webkit.org-config/public_html/TestFailures/FlakyLayoutTestDetector.js: Added.
(FlakyLayoutTestDetector): This class identifies flaky tests when given the test results
from various builds (in reverse-chronological order).
(FlakyLayoutTestDetector.prototype.incorporateTestResults): Detects flaky tests. Tests move
monotonically through three states: LastSeenFailing, LastSeenPassing, and PossiblyFlaky.
(FlakyLayoutTestDetector.prototype.flakinessExamples): Finds examples of flakiness for the
given test. Essentially, finds all the transitions from passing to failing (or vice-versa)
and puts them in an array in reverse-chronological order.
(FlakyLayoutTestDetector.prototype.get possiblyFlakyTests): Returns all tests we've detected
to be possibly flaky.

* BuildSlaveSupport/build.webkit.org-config/public_html/TestFailures/LayoutTestHistoryAnalyzer.js:
(LayoutTestHistoryAnalyzer): Initialize new members.
(LayoutTestHistoryAnalyzer.prototype.start): Now passes the callback an object with two
properties: history and possiblyFlaky. history holds the data this function used to pass to
the callback, while possiblyFlaky lists all tests that might be flaky and examples of their
flakiness. Updated documentation comment to match.
(LayoutTestHistoryAnalyzer.prototype._incorporateBuildHistory): Now uses a
FlakyLayoutTestDetector to identify possibly flaky tests. Any possibly flaky tests are
removed from the failure history, since when they started failing is no longer meaningful.
We tell our caller to keep calling until all current failures have been explained and we've
gone through 5 builds without any new flaky tests being identified.

* BuildSlaveSupport/build.webkit.org-config/public_html/TestFailures/Utilities.js:
(sorted): New helper function to return a sorted copy of an array.
(Array.prototype.findLast): New helper function. Like findFirst, but finds the last item
that matches the predicate.

* BuildSlaveSupport/build.webkit.org-config/public_html/TestFailures/ViewController.js:
(ViewController.prototype._displayBuilder): Updated for change in the object passed to us by
the analyzer. Now puts the list of possibly flaky tests after the failure history.
(ViewController.prototype._domForFailedTest): Moved some code from here...
(ViewController.prototype._domForFailureDiagnosis): ...to here.
(ViewController.prototype._domForPossiblyFlakyTests): New function, builds up a list of
possibly flaky tests and examples of their flakiness and returns it.

* BuildSlaveSupport/build.webkit.org-config/public_html/TestFailures/index.html: Pull in
FlakyLayoutTestDetector.js.

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

8 years ago2011-06-29 Xu Fan <qifuren1985@gmail.com>
bfulgham@webkit.org [Wed, 29 Jun 2011 22:11:47 +0000 (22:11 +0000)]
2011-06-29  Xu Fan  <qifuren1985@gmail.com>

        Reviewed by Brent Fulgham.

        Fix Scrollbars Transparent in Windows XP if WebKit is using Layered Window
        https://bugs.webkit.org/show_bug.cgi?id=61136

        * platform/graphics/win/DIBPixelData.h: add a static method setRGBABitmapAlpha
        * platform/graphics/win/DIBPixelData.cpp: add implementation
        * platform/win/ScrollbarThemeWin.cpp: fix scroolbar transparent
        * rendering/RenderThemeWin.cpp: fix textbox and button transparent

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

8 years ago2011-06-29 Eric Seidel <eric@webkit.org>
eric@webkit.org [Wed, 29 Jun 2011 22:04:29 +0000 (22:04 +0000)]
2011-06-29  Eric Seidel  <eric@webkit.org>

        Adam says cowboys don't review (or unit test).

        * Scripts/webkitpy/tool/commands/download.py:

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

8 years ago2011-06-29 Eric Seidel <eric@webkit.org>
eric@webkit.org [Wed, 29 Jun 2011 21:56:50 +0000 (21:56 +0000)]
2011-06-29  Eric Seidel  <eric@webkit.org>

        Reviewed by Adam Barth.

        Make port/config.py mockable for easier unit testing
        https://bugs.webkit.org/show_bug.cgi?id=63661

        Fix an exception from the unit tests.

        * Scripts/webkitpy/layout_tests/port/config.py:

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

8 years ago2011-06-29 Brent Fulgham <bfulgham@webkit.org>
bfulgham@webkit.org [Wed, 29 Jun 2011 21:48:00 +0000 (21:48 +0000)]
2011-06-29  Brent Fulgham  <bfulgham@webkit.org>

        Unreviewed build correction.

        * DumpRenderTree/win/ImageDiffWinCairoCommon.vsprops:
        Link against zlib1.dll, not the static zlib.lib.

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

8 years ago2011-06-29 Emil A Eklund <eae@chromium.org>
eae@chromium.org [Wed, 29 Jun 2011 21:41:41 +0000 (21:41 +0000)]
2011-06-29  Emil A Eklund  <eae@chromium.org>

        Reviewed by Eric Seidel.

        Switch RenderBoxModelObject::margin* and setMargin* to to new layout types
        https://bugs.webkit.org/show_bug.cgi?id=63652

        Switch margin methods for render tree over to the new layout unit abstraction.

        No new tests, no functionality changes.

        * rendering/InlineFlowBox.h:
        * rendering/RenderBlock.cpp:
        * rendering/RenderBlock.h:
        * rendering/RenderBox.cpp:
        * rendering/RenderBox.h:
        * rendering/RenderBoxModelObject.h:
        * rendering/RenderInline.cpp:
        * rendering/RenderInline.h:
        * rendering/RenderText.h:

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

8 years ago2011-06-29 Levi Weintraub <leviw@chromium.org>
leviw@chromium.org [Wed, 29 Jun 2011 21:33:11 +0000 (21:33 +0000)]
2011-06-29  Levi Weintraub  <leviw@chromium.org>

        Reviewed by Eric Seidel.

        Switch paint to new layout types
        https://bugs.webkit.org/show_bug.cgi?id=63645

        Switch vanilla paint function over to the new layout unit abstraction.

        No new tests, no functionality changes.

        * rendering/RenderBlock.cpp:
        (WebCore::RenderBlock::paint):
        * rendering/RenderBlock.h:
        * rendering/RenderBox.cpp:
        (WebCore::RenderBox::paint):
        * rendering/RenderBox.h:
        * rendering/RenderDetailsMarker.cpp:
        (WebCore::RenderDetailsMarker::paint):
        * rendering/RenderDetailsMarker.h:
        * rendering/RenderEmbeddedObject.cpp:
        (WebCore::RenderEmbeddedObject::paint):
        * rendering/RenderEmbeddedObject.h:
        * rendering/RenderFrameSet.cpp:
        (WebCore::RenderFrameSet::paint):
        * rendering/RenderFrameSet.h:
        * rendering/RenderImage.cpp:
        (WebCore::RenderImage::paint):
        * rendering/RenderImage.h:
        * rendering/RenderInline.cpp:
        (WebCore::RenderInline::paint):
        * rendering/RenderInline.h:
        * rendering/RenderLineBoxList.h:
        * rendering/RenderListItem.cpp:
        (WebCore::RenderListItem::paint):
        * rendering/RenderListItem.h:
        * rendering/RenderListMarker.cpp:
        (WebCore::RenderListMarker::paint):
        * rendering/RenderListMarker.h:
        * rendering/RenderObject.h:
        * rendering/RenderReplaced.cpp:
        (WebCore::RenderReplaced::paint):
        * rendering/RenderReplaced.h:
        * rendering/RenderReplica.cpp:
        (WebCore::RenderReplica::paint):
        * rendering/RenderReplica.h:
        * rendering/RenderTable.cpp:
        (WebCore::RenderTable::paint):
        * rendering/RenderTable.h:
        * rendering/RenderTableCell.cpp:
        (WebCore::RenderTableCell::paint):
        * rendering/RenderTableCell.h:
        * rendering/RenderTableRow.cpp:
        (WebCore::RenderTableRow::paint):
        * rendering/RenderTableRow.h:
        * rendering/RenderTableSection.cpp:
        (WebCore::RenderTableSection::paint):
        * rendering/RenderTableSection.h:
        * rendering/RenderTextControlSingleLine.cpp:
        (WebCore::RenderTextControlSingleLine::paint):
        * rendering/RenderTextControlSingleLine.h:
        * rendering/RenderTheme.h:
        * rendering/RenderView.cpp:
        (WebCore::RenderView::paint):
        * rendering/RenderView.h:
        * rendering/RenderWidget.cpp:
        (WebCore::RenderWidget::paint):
        * rendering/RenderWidget.h:
        * rendering/mathml/RenderMathMLBlock.h:
        * rendering/mathml/RenderMathMLFraction.h:
        * rendering/mathml/RenderMathMLRoot.h:
        * rendering/mathml/RenderMathMLSquareRoot.h:
        * rendering/svg/RenderSVGContainer.h:
        * rendering/svg/RenderSVGForeignObject.h:
        * rendering/svg/RenderSVGHiddenContainer.h:
        * rendering/svg/RenderSVGImage.h:
        * rendering/svg/RenderSVGPath.h:
        * rendering/svg/RenderSVGRoot.cpp:
        (WebCore::RenderSVGRoot::paint):
        * rendering/svg/RenderSVGRoot.h:
        * rendering/svg/RenderSVGText.h:

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

8 years agoUnreviewed build correction.
bfulgham@webkit.org [Wed, 29 Jun 2011 21:29:58 +0000 (21:29 +0000)]
Unreviewed build correction.

* DumpRenderTree/win/ImageDiffWinCairoCommon.vsprops:
Link against zlib1.dll, not the static zlib.lib.

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

8 years agoWebGL test failing on SL bots: fast/canvas/webgl/data-view-test.html
jianli@chromium.org [Wed, 29 Jun 2011 21:28:55 +0000 (21:28 +0000)]
WebGL test failing on SL bots: fast/canvas/webgl/data-view-test.html
https://bugs.webkit.org/show_bug.cgi?id=63151

Reviewed by Kenneth Russell.

Source/WebCore:

The fix is to change to throw TypeError in custom JS/V8 bindings.

* bindings/js/JSDataViewCustom.cpp:
(WebCore::getDataViewMember):
(WebCore::setDataViewMember):
* bindings/v8/custom/V8DataViewCustom.cpp:
(WebCore::V8DataView::constructorCallback):
(WebCore::V8DataView::getInt8Callback):

LayoutTests:

Updated the expected result and removed the rebaselined files that are not needed.

* fast/canvas/webgl/data-view-test-expected.txt:
* platform/chromium-mac/fast/canvas/webgl/data-view-test-expected.txt: Removed.
* platform/chromium-win/fast/canvas/webgl/data-view-test-expected.txt: Removed.
* platform/mac/fast/canvas/webgl/data-view-test-expected.txt: Removed.

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

8 years ago2011-06-29 Luke Macpherson <macpherson@chromium.org>
macpherson@chromium.org [Wed, 29 Jun 2011 21:25:59 +0000 (21:25 +0000)]
2011-06-29  Luke Macpherson   <macpherson@chromium.org>

        Reviewed by Darin Adler.

        Convert uses of computeLengthIntForLength() to computeLength<Length>()
        https://bugs.webkit.org/show_bug.cgi?id=63641

        No new tests / refactoring only.

        * css/CSSStyleSelector.cpp:
        (WebCore::convertToLength):
        (WebCore::CSSStyleSelector::applyProperty):
        (WebCore::CSSStyleSelector::applyPageSizeProperty):
        (WebCore::CSSStyleSelector::mmLength):
        (WebCore::CSSStyleSelector::inchLength):
        (WebCore::CSSStyleSelector::mapFillSize):
        (WebCore::CSSStyleSelector::mapFillXPosition):
        (WebCore::CSSStyleSelector::mapFillYPosition):

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

8 years ago2011-06-29 Jeff Timanus <twiz@chromium.org>
commit-queue@webkit.org [Wed, 29 Jun 2011 21:25:12 +0000 (21:25 +0000)]
2011-06-29  Jeff Timanus  <twiz@chromium.org>

        Reviewed by Stephen White.

        [Chromium] Remove flakiness marker from fast/canvas/canvas-longlived-context.html
        https://bugs.webkit.org/show_bug.cgi?id=63649

        * platform/chromium/test_expectations.txt:

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

8 years ago2011-06-29 Eric Seidel <eric@webkit.org>
eric@webkit.org [Wed, 29 Jun 2011 21:22:52 +0000 (21:22 +0000)]
2011-06-29  Eric Seidel  <eric@webkit.org>

        Reviewed by Adam Barth.

        Remove duplicate methods in filesystem.py
        https://bugs.webkit.org/show_bug.cgi?id=63658

        Looks like there was a bad merge at some point.

        I also removed a bunch of redundant docstrings.

        * Scripts/webkitpy/common/system/filesystem.py:

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

8 years ago2011-06-29 Chris Rogers <crogers@google.com>
crogers@google.com [Wed, 29 Jun 2011 21:20:08 +0000 (21:20 +0000)]
2011-06-29  Chris Rogers  <crogers@google.com>

        Reviewed by Kenneth Russell.

        Simplify AudioBufferSourceNode rendering
        https://bugs.webkit.org/show_bug.cgi?id=63586

        No new tests since this doesn't change API.

        * webaudio/AudioBufferSourceNode.cpp:
        (WebCore::AudioBufferSourceNode::AudioBufferSourceNode):
        (WebCore::AudioBufferSourceNode::process):
        (WebCore::AudioBufferSourceNode::renderFromBuffer):
        (WebCore::AudioBufferSourceNode::reset):
        (WebCore::AudioBufferSourceNode::setBuffer):
        (WebCore::AudioBufferSourceNode::noteOn):
        (WebCore::AudioBufferSourceNode::noteGrainOn):
        (WebCore::AudioBufferSourceNode::totalPitchRate):
        * webaudio/AudioBufferSourceNode.h:

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

8 years ago2011-06-29 Martin Robinson <mrobinson@igalia.com>
commit-queue@webkit.org [Wed, 29 Jun 2011 21:03:13 +0000 (21:03 +0000)]
2011-06-29  Martin Robinson  <mrobinson@igalia.com>

        Reviewed by Xan Lopez.

        [Freetype] Many tests report ERROR: Failed to get glyph page zero.
        https://bugs.webkit.org/show_bug.cgi?id=63498

        Instead of conditionally instantiating the m_scaledFont member of FontPlatformData,
        always instantiate it. For times when the size is 0 and the instantiation would lead
        to a Cairo error, we use size 1 and just make operations which would fail conditional
        on the m_size > 0.

        No new tests. This change is covered by existing tests. We should
        observe no errors on the bots after landing.

        * platform/graphics/cairo/FontCairo.cpp:
        (WebCore::Font::drawGlyphs): Instead of looking at whether or not there is
        a scaled font to determine whether to draw, just look at the size.
        * platform/graphics/freetype/FontPlatformData.h: Remove the m_font member.
        * platform/graphics/freetype/FontPlatformDataFreeType.cpp: Always instantiate
        a scaled font member and remove references to m_font. If we are instantiating
        a scaled font for a 0 size font, instantiate a size 1 font, so that we can
        still use Cairo to query font properties.
        * platform/graphics/freetype/GlyphPageTreeNodeFreeType.cpp:
        (WebCore::GlyphPage::fill): We can assume there is always a scaled font now.
        * platform/graphics/freetype/SimpleFontDataFreeType.cpp:
        (WebCore::SimpleFontData::platformInit): Ditto.
        (WebCore::SimpleFontData::createScaledFontData): Ditto.
        (WebCore::SimpleFontData::containsCharacters): Ditto.
        (WebCore::SimpleFontData::platformWidthForGlyph): Ditto.

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

8 years ago2011-06-29 Adam Barth <abarth@webkit.org>
abarth@webkit.org [Wed, 29 Jun 2011 20:55:24 +0000 (20:55 +0000)]
2011-06-29  Adam Barth  <abarth@webkit.org>

        Reviewed by Eric Seidel.

        Pass --no-new-test-results to NRWT on build.webkit.org
        https://bugs.webkit.org/show_bug.cgi?id=63653

        This is in preparation for https://bugs.webkit.org/show_bug.cgi?id=38063.

        * BuildSlaveSupport/build.webkit.org-config/master.cfg:

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

8 years ago2011-06-29 Eric Carlson <eric.carlson@apple.com>
eric.carlson@apple.com [Wed, 29 Jun 2011 20:52:35 +0000 (20:52 +0000)]
2011-06-29  Eric Carlson  <eric.carlson@apple.com>

        Reviewed by Darin Adler.

        Media element loads blocked by a resource load delegate do not generate an error event
        https://bugs.webkit.org/show_bug.cgi?id=63558

        Tests: media/media-blocked-by-beforeload.html
               media/media-blocked-by-willsendrequest.html

        * html/HTMLMediaElement.cpp:
        (WebCore::HTMLMediaElement::selectMediaResource): Call mediaLoadingFailed instead of noneSupported
            so all failures follow the same code path.
        (WebCore::HTMLMediaElement::loadResource): Call mediaLoadingFailed when there is no frame, when
            the loader delegate rejects the url, or when MediaPlayer doesn't find a media engine that
            supports the type.
        (WebCore::HTMLMediaElement::selectNextSourceChild): Fix incorrect logging.
        (WebCore::HTMLMediaElement::mediaLoadingFailed): Change name of parameter from "state" to
            "error" to make its purpose obvious.

        * platform/graphics/MediaPlayer.cpp:
        (WebCore::MediaPlayer::load): Return a bool indicating whether or not the url was given to
            a media engine.
        (WebCore::MediaPlayer::loadWithNextMediaEngine): Add some logging.
        * platform/graphics/MediaPlayer.h:

2011-06-29  Eric Carlson  <eric.carlson@apple.com>

        Reviewed by Darin Adler.

        Media element loads blocked by a resource load delegate do not generate an error event
        https://bugs.webkit.org/show_bug.cgi?id=63558

        * media/media-blocked-by-beforeload-expected.txt: Added.
        * media/media-blocked-by-beforeload.html: Added.
        * media/media-blocked-by-willsendrequest-expected.txt: Added.
        * media/media-blocked-by-willsendrequest.html: Added.

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

8 years ago2011-06-29 Darin Adler <darin@apple.com>
darin@apple.com [Wed, 29 Jun 2011 20:46:33 +0000 (20:46 +0000)]
2011-06-29  Darin Adler  <darin@apple.com>

        Reviewed by Anders Carlsson.

        [WebKit2] Crash loading page that adds/removes frame in DOMContentLoaded/loaded
        https://bugs.webkit.org/show_bug.cgi?id=63483

        * fast/loader/create-frame-in-DOMContentLoaded-expected.txt: Added.
        * fast/loader/create-frame-in-DOMContentLoaded.html: Added.
2011-06-29  Darin Adler  <darin@apple.com>

        Reviewed by Anders Carlsson.

        [WebKit2] Crash loading page that adds/removes frame in DOMContentLoaded/loaded
        https://bugs.webkit.org/show_bug.cgi?id=63483

        Test: fast/loader/create-frame-in-DOMContentLoaded.html

        * loader/FrameLoader.cpp:
        (WebCore::FrameLoader::init): Added an assertion.
        (WebCore::FrameLoader::finishedLoadingDocument): Removed a non-helpful #if
        statement. The rule that we do not call the client when creating the initial
        empty document was nominally specific to Windows and Chromium but is needed
        for all platforms.
2011-06-29  Darin Adler  <darin@apple.com>

        Reviewed by Anders Carlsson.

        [WebKit2] Crash loading page that adds/removes frame in DOMContentLoaded/loaded
        https://bugs.webkit.org/show_bug.cgi?id=63483

        * WebProcess/WebCoreSupport/WebFrameLoaderClient.cpp:
        (WebKit::WebFrameLoaderClient::createFrame): Added a check for null when creating
        a frame. Also added a check that was in the WebKit1 code, but not here. Strangely,
        the comment from the second check was still here, but not the code.

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