⚠ Archived content — this site is no longer maintained.   Current WebKit documentation is at docs.webkit.org.

Timeline



Nov 17, 2009:

10:51 PM Changeset in webkit [51102] by darin@chromium.org
  • 2 edits in trunk/WebCore

2009-11-17 Nicolas Weber <​thakis@chromium.org>

Reviewed by Darin Fisher.

Fix crash in Chromium/Mac where dropdowns weren't dismissed correctly
after navigation.
​https://bugs.webkit.org/show_bug.cgi?id=31609

  • platform/chromium/PopupMenuChromium.cpp: (WebCore::PopupContainer::showExternal): Set parent for external dropdowns, so that |PopupListBox::hidePopup()| can successfully notify its parent's client.
9:19 PM Changeset in webkit [51101] by tkent@chromium.org
  • 4 edits
    3 adds in trunk

2009-11-17 Hayato Ito <​hayato@google.com>

Reviewed by Darin Adler.

Avoid infinite mutual recursion when deeply nested tags are loaded
​https://bugs.webkit.org/show_bug.cgi?id=30651

  • fast/parser/block-nesting-cap-table-expected.txt: Added.
  • fast/parser/block-nesting-cap-table.html: Added.
  • fast/parser/script-tests/block-nesting-cap-table.js: Added.

2009-11-17 Hayato Ito <​hayato@google.com>

Reviewed by Darin Adler.

Avoid infinite mutual recursion when deeply nested tags are loaded
​https://bugs.webkit.org/show_bug.cgi?id=30651

Test: fast/parser/block-nesting-cap-table.html

  • html/HTMLParser.cpp: (WebCore::HTMLParser::parseToken): (WebCore::tagPriorityOfNode): (WebCore::HTMLParser::limitBlockDepth): (WebCore::HTMLParser::insertNodeAfterLimitBlockDepth): (WebCore::HTMLParser::insertNode):
  • html/HTMLParser.h:
8:18 PM Changeset in webkit [51100] by jhoneycutt@apple.com
  • 2 edits in trunk/JavaScriptCore

Add JSContextRefPrivate.h to list of copied files.

Reviewed by Mark Rowe.

5:12 PM Changeset in webkit [51099] by mrowe@apple.com
  • 2 edits in trunk/WebKitTools

<​http://webkit.org/b/31603> WebSocket server is confused if WebKit tests run from within /tmp on Mac OS X

Reviewed by Alexey Proskuryakov.

  • pywebsocket/mod_pywebsocket/dispatch.py: Use os.path.realpath as it returns the canonical path of a file.

This prevents symlinks from confusing the descendant check.

5:08 PM Changeset in webkit [51098] by bfulgham@webkit.org
  • 2 edits in trunk/WebCore

Rubber-stamped by Alexey Proskuryakov.

Final clean-ups for minor coding standard violations.
​https://bugs.webkit.org/show_bug.cgi?id=26102.

  • platform/graphics/cairo/GraphicsContextCairo.cpp:
4:35 PM Changeset in webkit [51097] by dimich@chromium.org
  • 5 edits
    2 adds in trunk

WebCore: In all valid script tags for JavaScript, the event handler in <script...for> should not get executed.
​https://bugs.webkit.org/show_bug.cgi?id=31567

Patch by Johnny Ding <​jnd@chromium.org> on 2009-11-17
Reviewed by Darin Adler.

  • dom/ScriptElement.cpp:

(WebCore::ScriptElementData::shouldExecuteAsJavaScript):

LayoutTests: In all valid script tags for JavaScript, the event handler in <script...for> should not get executed.
Change the original test to pure js test, and test the situation about script tags which have text or language attribute.
​https://bugs.webkit.org/show_bug.cgi?id=31567

Patch by Johnny Ding <​jnd@chromium.org> on 2009-11-17
Reviewed by Darin Adler.

  • fast/dom/HTMLScriptElement/script-for-attribute-unexpected-execution-expected.txt:
  • fast/dom/HTMLScriptElement/script-for-attribute-unexpected-execution.html:
  • fast/dom/HTMLScriptElement/script-tests: Added.
  • fast/dom/HTMLScriptElement/script-tests/script-for-attribute-unexpected-execution.js: Added.

(ScriptForAttributeExecute):

4:31 PM Changeset in webkit [51096] by mrowe@apple.com
  • 3 edits
    2 adds in trunk

<​http://webkit.org/b/31602> Failing to start the WebSocket server shouldn’t terminate entire test run

Reviewed by Alexey Proskuryakov.

WebKitTools:

If the WebSocket server fails to start have DRT load an error page in place of tests that require the
server to be up rather than having run-webkit-tests abort immediately.

  • Scripts/run-webkit-tests:

LayoutTests:

  • websocket/resources/server-failed-to-start.html: Error page that will be loaded in place of websocket

tests when the server fails to start.

4:24 PM Changeset in webkit [51095] by dimich@chromium.org
  • 2 edits in trunk/WebKit/chromium

Revert r50887, it regressed handling of cmd-left when a text box is focused.
​https://bugs.webkit.org/show_bug.cgi?id=31566

Patch by Nicolas Weber <​thakis@chromium.org> on 2009-11-17
Reviewed by Darin Fisher, Dmitry Titov.

  • src/mac/WebInputEventFactory.mm:

(WebKit::WebInputEventFactory::keyboardEvent):

4:08 PM CommitQueue edited by thakis@chromium.org
(diff)
3:38 PM Changeset in webkit [51094] by jhoneycutt@apple.com
  • 2 edits in trunk/WebKit/win

Implement DOMHTMLInputElement::replaceCharactersInRange().

​https://bugs.webkit.org/show_bug.cgi?id=31492

Reviewed by Dan Bernstein.

  • DOMHTMLClasses.cpp:

(DOMHTMLInputElement::replaceCharactersInRange):
Get the text of the input field. Replace the given range with the
replacement text, and set this new string as the input element's value.
Select from index to the end of the field. This matches the
implementation in the Obj-C bindings.

3:37 PM Changeset in webkit [51093] by jhoneycutt@apple.com
  • 2 edits in trunk/WebKit/win

DOMHTMLOptionElement is missing some functionality.

​https://bugs.webkit.org/show_bug.cgi?id=31491

Reviewed by Dan Bernstein.

  • DOMHTMLClasses.cpp:

(DOMHTMLOptionElement::text):
Cast m_element to an HTMLOptionElement, and call its text() function.
(DOMHTMLOptionElement::label):
Ditto, for label().

3:37 PM Changeset in webkit [51092] by jhoneycutt@apple.com
  • 2 edits in trunk/WebKit/win

DOMHTMLSelectElement is missing some implementation.

​https://bugs.webkit.org/show_bug.cgi?id=31489

Reviewed by Dan Bernstein.

  • DOMHTMLClasses.cpp:

(DOMHTMLSelectElement::options):
Cast m_element to an HTMLSelectElement. If it has no options, return
E_FAIL. Otherwise, create a DOMHTMLOptionsCollection to wrap the
options, and return it.
(DOMHTMLSelectElement::activateItemAtIndex):
If the index is out of bounds, return E_FAIL. Otherwise, select the
item.

3:37 PM Changeset in webkit [51091] by jhoneycutt@apple.com
  • 3 edits in trunk/WebKit/win

DOMHTMLOptionsCollection is missing some implementation.

​https://bugs.webkit.org/show_bug.cgi?id=31488

Reviewed by Dan Bernstein.

  • DOMHTMLClasses.cpp:

(DOMHTMLOptionsCollection::DOMHTMLOptionsCollection):
Initialize m_collection.
(DOMHTMLOptionsCollection::createInstance):
Create a DOMHTMLOptionsCollection. If we fail to query for
IDOMHTMLOptionsCollection, delete it, and return 0. Otherwise, return
the result.
(DOMHTMLOptionsCollection::length):
(DOMHTMLOptionsCollection::item):
Create a DOMNode for the WebCore Node. If this is 0, return E_FAIL.
(DOMHTMLOptionsCollection::namedItem):
Correct the signature of this function.

  • DOMHTMLClasses.h:

Declare DOMHTMLOptionsCollection::createInstance(). Correct the
signature of namedItem() to match IDOMHTMLOptionsCollection. Add a
member to DOMHTMLOptionsCollection to hold the WebCore object.

3:37 PM Changeset in webkit [51090] by jhoneycutt@apple.com
  • 2 edits in trunk/WebKit/win

DOMHTMLInputElement::rectOnScreen() returns the wrong rect

​https://bugs.webkit.org/show_bug.cgi?id=31487

Reviewed by Darin Adler.

  • DOMHTMLClasses.cpp:

(DOMHTMLInputElement::rectOnScreen):
Return the rect on screen, not the rect in the window.

2:07 PM Changeset in webkit [51089] by bweinstein@apple.com
  • 2 edits in trunk/LayoutTests

Rubber-stamped by Pavel Feldman.

Skip inspector tests that are flaky on Windows Release - this is being tracked by:
​https://bugs.webkit.org/show_bug.cgi?id=31596.

  • platform/win/Skipped:
1:40 PM Changeset in webkit [51088] by jparent@chromium.org
  • 4 edits in trunk/LayoutTests

Eliminate flakiness in char-encoding tests by removing setTimeout,
that was working around an old issue.
​https://bugs.webkit.org/show_bug.cgi?id=31571

Reviewed by Darin Adler.

  • fast/encoding/char-encoding-mac.html:
  • fast/encoding/char-encoding.html:
  • fast/encoding/resources/char-encoding-utils.js:
1:30 PM Changeset in webkit [51087] by kenneth@webkit.org
  • 2 edits in trunk/WebKitTools

Unreviewed buildbot fix.

Revert part of earlier patch and add comment, as it
was causing timeouts on the buildbot.

  • DumpRenderTree/qt/LayoutTestControllerQt.cpp:

(LayoutTestController::notifyDone):

12:38 PM Changeset in webkit [51086] by eric@webkit.org
  • 3 edits in trunk/JavaScriptCore

2009-11-17 Martin Robinson <​martin.james.robinson@gmail.com>

Reviewed by Adam Barth.

[GTK] Style cleanup for GOwnPtr
​https://bugs.webkit.org/show_bug.cgi?id=31506

Remove forward declaration in GOwnPtr and do some style cleanup.

  • wtf/GOwnPtr.cpp:
  • wtf/GOwnPtr.h: (WTF::GOwnPtr::GOwnPtr): (WTF::GOwnPtr::~GOwnPtr): (WTF::GOwnPtr::get): (WTF::GOwnPtr::release): (WTF::GOwnPtr::outPtr): (WTF::GOwnPtr::set): (WTF::GOwnPtr::clear): (WTF::GOwnPtr::operator*): (WTF::GOwnPtr::operator->): (WTF::GOwnPtr::operator!): (WTF::GOwnPtr::operator UnspecifiedBoolType): (WTF::GOwnPtr::swap): (WTF::swap): (WTF::operator==): (WTF::operator!=): (WTF::getPtr): (WTF::freeOwnedGPtr):
12:11 PM Changeset in webkit [51085] by eric@webkit.org
  • 3 edits in trunk/WebKitTools

2009-11-17 Eric Seidel <​eric@webkit.org>

Reviewed by Darin Adler.

commit-queue is failing to set reviewer in ChangeLogs
​https://bugs.webkit.org/show_bug.cgi?id=31592

  • Scripts/bugzilla-tool: Clarify the "applying" log message.
  • Scripts/modules/bugzilla.py:
    • Add a new _validate_committer_and_reviewer function as a temporary solution until we can make a real Attachment object which knows how to fill in its committer/reviewer fields automatically.
12:05 PM Changeset in webkit [51084] by kenneth@webkit.org
  • 2 edits in trunk/WebKitTools

Make the timeout 15 sec as for the other DRT's and make
it print out the same output when a test timeout.

Reviewed by Oliver Hunt.

  • DumpRenderTree/qt/LayoutTestControllerQt.cpp:

(LayoutTestController::waitUntilDone):
(LayoutTestController::notifyDone):
(LayoutTestController::timerEvent):

11:56 AM Changeset in webkit [51083] by yael.aharon@nokia.com
  • 2 edits in trunk/WebKit/qt

[Qt] QGLLauncher does not support drag&drop of local files
​https://bugs.webkit.org/show_bug.cgi?id=31057

Reviewed by Kenneth Rohde Christiansen.

Enable accepting files in QGraphicsWebView.

  • Api/qgraphicswebview.cpp:

(QGraphicsWebView::QGraphicsWebView):
(QGraphicsWebView::dragEnterEvent):

11:33 AM Changeset in webkit [51082] by bfulgham@webkit.org
  • 2 edits in trunk/WebKit/win

Correct build error in Debug_Cairo target after @49705.

Reviewed by NOBODY - Build Fix.

  • WebKit.vcproj/WebKit.vcproj: Revise JavaScriptCore.lib dependency to have proper "_debug" suffix needed by the Debug_Cairo target.
11:27 AM Web Inspector edited by timothy@apple.com
(diff)
11:26 AM WebInspector created by timothy@apple.com
Copied from Web Inspector
11:11 AM Changeset in webkit [51081] by bweinstein@apple.com
  • 1 edit in trunk/WebCore/inspector/InspectorController.cpp

Touch InspectorController to force the Inspector HTML and JS files to be
updated.

10:57 AM Changeset in webkit [51080] by Csaba Osztrogonác
  • 2 edits in trunk/LayoutTests

Not reviewed, follow up to r51072: rename skipped entries

  • platform/qt/Skipped:
10:52 AM Changeset in webkit [51079] by Joseph Pecoraro
  • 2 edits in trunk/WebKitTools

2009-11-17 Joseph Pecoraro <Joseph Pecoraro>

Reviewed by Timothy Hatcher.

Fixed typos in comments.

  • Scripts/modules/committers.py:
10:42 AM Changeset in webkit [51078] by Csaba Osztrogonác
  • 2 edits in trunk/LayoutTests

[Qt] Crashing tests after updating to Qt-4.6.0
​https://bugs.webkit.org/show_bug.cgi?id=31591

Reviewed by Kenneth Rohde Christiansen.

  • platform/qt/Skipped: Put tests into skiplist temporarily.
10:37 AM Changeset in webkit [51077] by bweinstein@apple.com
  • 4 edits in trunk/WebKit/win

Touch files to try to fix the build.

Reviewed by NOBODY - Build Fix.

  • Interfaces/IWebInspector.idl:
  • Interfaces/WebKit.idl:
  • WebKit.vcproj/Interfaces.vcproj:
9:55 AM Changeset in webkit [51076] by tonikitoo@webkit.org
  • 2 edits in trunk/WebKit/qt

[Qt] better test coverage for ErrorPageExtension
​https://bugs.webkit.org/show_bug.cgi?id=31583

Reviewed by Kenneth Christiansen.

Improved the coverage of current ErrorPageExtension tests by
adding autotests involving frameset and iframes.

  • tests/qwebpage/tst_qwebpage.cpp:

(ErrorPage::extension): Make the ErrorPageExtension
to work for all frames, not only the main frame.
(tst_QWebPage::errorPageExtension): Stop using
the 'frameset.html' resouce in this method since
an autotest specific for frameset's is being added.
(tst_QWebPage::errorPageExtensionInIFrames): Added.
(tst_QWebPage::errorPageExtensionInFrameset): Added.

6:14 AM Changeset in webkit [51075] by pfeldman@chromium.org
  • 1 edit
    6 moves
    2 adds in trunk/LayoutTests

2009-11-17 Pavel Feldman <​pfeldman@chromium.org>

Not reviewed, tests fix. Follow up to r51072: move
platform-specific results for fast/inspector into
fast/inspector-support.

  • platform/mac/fast/inspector-support/matchedrules-expected.checksum: Renamed from LayoutTests/platform/mac/fast/inspector/matchedrules-expected.checksum.
  • platform/mac/fast/inspector-support/matchedrules-expected.png: Renamed from LayoutTests/platform/mac/fast/inspector/matchedrules-expected.png.
  • platform/mac/fast/inspector-support/matchedrules-expected.txt: Renamed from LayoutTests/platform/mac/fast/inspector/matchedrules-expected.txt.
  • platform/mac/fast/inspector-support/style-expected.txt: Renamed from LayoutTests/platform/mac/fast/inspector/style-expected.txt.
  • platform/qt/fast/inspector-support/matchedrules-expected.txt: Renamed from LayoutTests/platform/qt/fast/inspector/matchedrules-expected.txt.
  • platform/qt/fast/inspector-support/style-expected.txt: Renamed from LayoutTests/platform/qt/fast/inspector/style-expected.txt.
5:54 AM Changeset in webkit [51074] by kov@webkit.org
  • 2 edits in trunk/LayoutTests

Actually skip the inspector console tests while the problem
discussed in ​https://bugs.webkit.org/show_bug.cgi?id=31472 is
fixed.

5:38 AM Changeset in webkit [51073] by pfeldman@chromium.org
  • 5 edits in trunk

2009-11-17 Pavel Feldman <​pfeldman@chromium.org>

Not reviewed, build fix. Follow up to r51072: rename skipped entries
for the tests that were moved. Also skipped console tests on GTK
while investigating and touched Interfaces.vcproj to clobber win bot.


  • platform/gtk/Skipped:
  • platform/qt/Skipped:
5:11 AM Changeset in webkit [51072] by pfeldman@chromium.org
  • 37 edits
    10 moves
    2 adds in trunk

2009-11-17 Pavel Feldman <​pfeldman@chromium.org>

Reviewed by Timothy Hatcher.

Web Inspector: Make DRT show web inspector for tests in inspector/ folder.

  • Updated DRT to show/close inspector for all tests under /inspector
  • Introduced LayoutTestController::setTimelineProfilingEnabled and WebInspector::setTimelineProfilingEnabled beside setJavaScriptProfilingEnabled
  • Removed reload on each inspector test
  • Renamed fast/inspector to fast/inspector-support in order not to trigger inspector for those.
  • Reimplemented timeline tests in order to get rid of reload there.
  • Moved tests that don't require harness into the fast group.

​https://bugs.webkit.org/show_bug.cgi?id=31472

WebCore:

  • WebCore.Inspector.exp:
  • inspector/front-end/TimelinePanel.js: (WebInspector.TimelinePanel.prototype._formatRecord):

WebKitTools:

  • DumpRenderTree/LayoutTestController.cpp: (setTimelineProfilingEnabledCallback): (closeWebInspectorCallback): (LayoutTestController::staticFunctions):
  • DumpRenderTree/LayoutTestController.h:
  • DumpRenderTree/gtk/DumpRenderTree.cpp: (shouldOpenWebInspector): (runTest):
  • DumpRenderTree/gtk/LayoutTestControllerGtk.cpp: (LayoutTestController::setTimelineProfilingEnabled):
  • DumpRenderTree/mac/DumpRenderTree.mm: (shouldOpenWebInspector): (runTest):
  • DumpRenderTree/mac/LayoutTestControllerMac.mm: (LayoutTestController::setTimelineProfilingEnabled):
  • DumpRenderTree/win/DumpRenderTree.cpp: (shouldOpenWebInspector): (runTest):
  • DumpRenderTree/win/LayoutTestControllerWin.cpp: (LayoutTestController::setTimelineProfilingEnabled):

LayoutTests:

  • fast/inspector-support/cssURLQuotes-expected.txt: Renamed from LayoutTests/fast/inspector/cssURLQuotes-expected.txt.
  • fast/inspector-support/cssURLQuotes.html: Renamed from LayoutTests/fast/inspector/cssURLQuotes.html.
  • fast/inspector-support/matchedrules.html: Renamed from LayoutTests/fast/inspector/matchedrules.html.
  • fast/inspector-support/style.html: Renamed from LayoutTests/fast/inspector/style.html.
  • fast/inspector-support/uncaught-dom1-exception-expected.txt: Renamed from LayoutTests/inspector/uncaught-dom1-exception-expected.txt.
  • fast/inspector-support/uncaught-dom1-exception.html: Renamed from LayoutTests/inspector/uncaught-dom1-exception.html.
  • fast/inspector-support/uncaught-dom3-exception-expected.txt: Renamed from LayoutTests/inspector/uncaught-dom3-exception-expected.txt.
  • fast/inspector-support/uncaught-dom3-exception.html: Renamed from LayoutTests/inspector/uncaught-dom3-exception.html.
  • fast/inspector-support/uncaught-dom8-exception-expected.txt: Renamed from LayoutTests/inspector/uncaught-dom8-exception-expected.txt.
  • fast/inspector-support/uncaught-dom8-exception.html: Renamed from LayoutTests/inspector/uncaught-dom8-exception.html.
  • inspector/inspector-test.js: (onload): (evaluateInWebInspector): (notifyDone):
  • inspector/resources/timeline-iframe-data.html: Added.
  • inspector/timeline-layout-expected.txt:
  • inspector/timeline-layout.html:
  • inspector/timeline-mark-timeline.html:
  • inspector/timeline-paint.html:
  • inspector/timeline-parse-html-expected.txt:
  • inspector/timeline-parse-html.html:
  • inspector/timeline-recalculate-styles-expected.txt:
  • inspector/timeline-recalculate-styles.html:
  • inspector/timeline-script-tag-1-expected.txt:
  • inspector/timeline-script-tag-1.html:
  • inspector/timeline-script-tag-2-expected.txt:
  • inspector/timeline-script-tag-2.html:
  • inspector/timeline-script-tag-2.js:
  • inspector/timeline-test.js: (printTimelineRecords): (): (dumpTimelineRecords): (printProps): (frontend_getTimelineResults):

WebKit/mac:

WebKit/win:

  • Interfaces/IWebInspector.idl:
  • WebInspector.cpp: (WebInspector::isTimelineProfilingEnabled): (WebInspector::setTimelineProfilingEnabled):
  • WebInspector.h:

WebKit/gtk:

  • webkit/webkitwebinspector.cpp: (webkit_web_inspector_class_init): (webkit_web_inspector_set_property): (webkit_web_inspector_get_property):
4:40 AM Changeset in webkit [51071] by benm@google.com
  • 1 edit
    4 adds in trunk/WebCore

[Android] Android is missing the implementation of the GeolocationService iface.
​https://bugs.webkit.org/show_bug.cgi?id=31554

Patch by Andrei Popescu <​andreip@google.com> on 2009-11-17
Reviewed by Dimitri Glazkov.

No new tests required as this is platform specific code.

  • platform/android/GeolocationServiceAndroid.cpp: Added.

(WebCore::GeolocationServiceAndroid::create):
(WebCore::GeolocationServiceAndroid::GeolocationServiceAndroid):
(WebCore::GeolocationServiceAndroid::startUpdating):
(WebCore::GeolocationServiceAndroid::stopUpdating):
(WebCore::GeolocationServiceAndroid::suspend):
(WebCore::GeolocationServiceAndroid::resume):
(WebCore::GeolocationServiceAndroid::newPositionAvailable):
(WebCore::GeolocationServiceAndroid::newErrorAvailable):
(WebCore::GeolocationServiceAndroid::timerFired):
(WebCore::GeolocationServiceAndroid::isPositionMovement):
(WebCore::GeolocationServiceAndroid::isPositionMoreAccurate):
(WebCore::GeolocationServiceAndroid::isPositionMoreTimely):

  • platform/android/GeolocationServiceAndroid.h: Added.

(WebCore::GeolocationServiceAndroid::~GeolocationServiceAndroid):
(WebCore::GeolocationServiceAndroid::lastPosition):
(WebCore::GeolocationServiceAndroid::lastError):

  • platform/android/GeolocationServiceBridge.cpp: Added.

(WebCore::):
(WebCore::GeolocationServiceBridge::GeolocationServiceBridge):
(WebCore::GeolocationServiceBridge::~GeolocationServiceBridge):
(WebCore::GeolocationServiceBridge::start):
(WebCore::GeolocationServiceBridge::stop):
(WebCore::GeolocationServiceBridge::setEnableGps):
(WebCore::GeolocationServiceBridge::newLocationAvailable):
(WebCore::GeolocationServiceBridge::newErrorAvailable):
(WebCore::GeolocationServiceBridge::toGeoposition):
(WebCore::GeolocationServiceBridge::startJavaImplementation):
(WebCore::GeolocationServiceBridge::stopJavaImplementation):

  • platform/android/GeolocationServiceBridge.h: Added.
1:36 AM Changeset in webkit [51070] by hamaji@chromium.org
  • 7 edits
    1 add in trunk/LayoutTests

2009-11-17 Yuta Kitamura <​yutak@chromium.org>

Reviewed by Eric Seidel.

Fix http tests so that they do not submit a GET form to about:blank.

Original tests expected that submitting an empty form to <about:blank> navigates
the browser to <about:blank?>, not <about:blank>. However, some other browsers
do not act like Safari, and in fact the layout test runner of Chromium was
failing to run these tests.

Prior to this patch, a test that only checks this behavior was added, so now
we can remove the dependency on it from these tests.

Some HTTP tests submit a GET form to about:blank
​https://bugs.webkit.org/show_bug.cgi?id=30423

  • http/tests/navigation/onload-navigation-iframe-2-expected.txt:
  • http/tests/navigation/onload-navigation-iframe-2.html:
  • http/tests/navigation/onload-navigation-iframe-expected.txt:
  • http/tests/navigation/onload-navigation-iframe-timeout-expected.txt:
  • http/tests/navigation/onload-navigation-iframe-timeout.html:
  • http/tests/navigation/onload-navigation-iframe.html:
  • http/tests/navigation/resources/blank.txt: Added.
1:27 AM Changeset in webkit [51069] by hamaji@chromium.org
  • 1 edit
    2 adds in trunk/LayoutTests

2009-11-17 Yuta Kitamura <​yutak@chromium.org>

Reviewed by Darin Adler.

Add a test to check the behavior of submitting a GET form to <about:blank>.

Currently Safari redirects the browser to <about:blank?>, while some other
browsers do not (including Firefox and Chromium). The objective of this test is
to express the current behavior of Safari and to become ready to remove
unintentional dependency on it from several tests.

Some HTTP tests submit a GET form to about:blank
​https://bugs.webkit.org/show_bug.cgi?id=30423

  • fast/forms/get-forms-to-about-blank-expected.txt: Added.
  • fast/forms/get-forms-to-about-blank.html: Added.
12:41 AM Changeset in webkit [51068] by oliver@apple.com
  • 6 edits in trunk

Incorrect use of JavaScriptCore API in DumpRenderTree
​https://bugs.webkit.org/show_bug.cgi?id=31577

Reviewed by Maciej Stachowiak

Return undefined rather than a literal null.

Nov 16, 2009:

11:16 PM Changeset in webkit [51067] by zoltan@webkit.org
  • 4 edits in trunk/JavaScriptCore

​https://bugs.webkit.org/show_bug.cgi?id=31050

Patch by Zoltan Herczeg <​zherczeg@inf.u-szeged.hu> on 2009-11-16
Reviewed by Gavin Barraclough.

Minor fixes for JSVALUE32_64: branchConvertDoubleToInt32
failed on a CortexA8 CPU, but not on a simulator; and
JITCall.cpp modifications was somehow not committed to mainline.

  • assembler/ARMAssembler.h:

(JSC::ARMAssembler::fmrs_r):

  • assembler/MacroAssemblerARM.h:

(JSC::MacroAssemblerARM::branchConvertDoubleToInt32):

  • jit/JITCall.cpp:

(JSC::JIT::compileOpCall):

10:31 PM Changeset in webkit [51066] by tkent@chromium.org
  • 3 edits in trunk/WebCore

2009-11-16 Kent Tamura <​tkent@chromium.org>

Unreviewd build fix.

  • Fix typo in WebCore.vcproj.
  • Intlude limits.h for INT_MAX.
  • Enclose with parenthesis to suspress warning.
  • WebCore.vcproj/WebCore.vcproj:
  • html/ISODateTime.cpp: (WebCore::ISODateTime::addDay):
9:50 PM Changeset in webkit [51065] by kevino@webkit.org
  • 6 edits in trunk/WebCore

Reviewed by Kevin Ollivier.

Make sure wx scrollbar drawing code factors in transforms when switching backends,
fix calcs for scrollbar length, and tweak the Mac scrollbar tracking rects.

​https://bugs.webkit.org/show_bug.cgi?id=31570

9:39 PM Changeset in webkit [51064] by Girish Ramakrishnan
  • 1 edit in trunk/WebKitTools/Scripts/modules/committers.py

Add myself to commiters.py

9:13 PM Changeset in webkit [51063] by tkent@chromium.org
  • 7 edits
    2 adds in trunk/WebCore

2009-11-16 Kent Tamura <​tkent@chromium.org>

Reviewed by David Levin.

Introduce WebCore::ISODateTime class.
​https://bugs.webkit.org/show_bug.cgi?id=31340

This class represents a value of date/time types of the HTML5 INPUT
element, and has some parsing methods for ISO 8601.

This change has no tests because the class is not used yet.

  • GNUmakefile.am: Add ISODateTime.cpp and ISODateTime.h.
  • WebCore.gypi: ditto.
  • WebCore.pro: ditto.
  • WebCore.vcproj/WebCore.vcproj: ditto.
  • WebCore.xcodeproj/project.pbxproj: ditto.
  • WebCoreSources.bkl: ditto.
  • html/ISODateTime.cpp: Added. Implementation of WebCore::ISODateTime class. (WebCore::isLeapYear): (WebCore::maxDayOfMonth): (WebCore::dayOfWeek): (WebCore::ISODateTime::maxWeekNumberInYear): (WebCore::countDigits): (WebCore::toInt): (WebCore::ISODateTime::parseYear): Private helper for parseDate() and parseWeek(). (WebCore::ISODateTime::addDay): Private helper for parseTimeZone(). (WebCore::ISODateTime::addMinute): ditto. (WebCore::ISODateTime::parseTimeZone): Private helper for parseDateTime(). (WebCore::ISODateTime::parseMonth): Parser for <input type=month>. (WebCore::ISODateTime::parseDate): Parser for <input type=date>. (WebCore::ISODateTime::parseWeek): Parser for <input type=week>. (WebCore::ISODateTime::parseTime): Parser for <input type=time>. (WebCore::ISODateTime::parseDateTimeLocal): Parser for <input type=datetime-local>. (WebCore::ISODateTime::parseDateTime): Parser for <input type=datetime>.
  • html/ISODateTime.h: Added. Declare WebCore::ISODateTime class. (WebCore::ISODateTime::ISODateTime): (WebCore::ISODateTime::millisecond): (WebCore::ISODateTime::second): (WebCore::ISODateTime::minute): (WebCore::ISODateTime::hour): (WebCore::ISODateTime::monthDay): (WebCore::ISODateTime::month): (WebCore::ISODateTime::fullYear): (WebCore::ISODateTime::week):
7:05 PM BuildingQtOnLinux edited by dbates@webkit.org
Added missing "libsqlite3-dev" dependency for Ubuntu (diff)
4:09 PM Changeset in webkit [51062] by ap@apple.com
  • 2 edits in trunk/WebCore

Fix a typo in previous commit.

  • platform/network/cf/SocketStreamHandleCFNet.cpp: #ifdef, not #if.
4:04 PM Changeset in webkit [51061] by ap@apple.com
  • 2 edits in trunk/WebCore

Fix a typo in previous commit.

  • platform/network/cf/SocketStreamHandleCFNet.cpp:
3:57 PM Changeset in webkit [51060] by ap@apple.com
  • 2 edits in trunk/WebCore

Tiger build fix.

  • platform/network/cf/SocketStreamHandleCFNet.cpp: Define CFN_EXPORT, as this macro was named differently in Tiger CFNetwork.
3:32 PM Changeset in webkit [51059] by ap@apple.com
  • 3 edits in trunk/LayoutTests

Disabling WebSocket tests on Tiger and Windows again, as they still time out.

  • platform/mac-tiger/Skipped:
  • platform/win/Skipped:
3:08 PM Changeset in webkit [51058] by Dimitri Glazkov
  • 2 edits in trunk/WebCore

2009-11-16 Dimitri Glazkov <Dimitri Glazkov>

Reviewed by Darin Fisher.

[KURLGoogle] setHostAndPort doesn't handle arguments without port correctly.

Fix a bug in the code that was dormant until ​http://trac.webkit.org/changeset/50784.

Covered by existing test: LayoutTests/fast/dom/HTMLAnchorElement/set-href-attribute-host.html

  • platform/KURLGoogle.cpp: (WebCore::KURL::setHostAndPort): Added handling of arguments without port specified.
2:58 PM Changeset in webkit [51057] by ap@apple.com
  • 2 edits in trunk/WebCore

Rubber-stamped by Jon Honeycutt.

A better Windows build fix

  • platform/network/cf/SocketStreamHandleCFNet.cpp: (WebCore::SocketStreamHandle::createStreams): Use a macro that inserts declspec(dllimport) when appropriate for CFNetwork imports.
2:57 PM Changeset in webkit [51056] by yael.aharon@nokia.com
  • 5 edits in trunk/WebCore

Rename protocolIsValid to isValidProtocol.
​https://bugs.webkit.org/show_bug.cgi?id=31503

Reviewed by Darin Adler.

This name change was suggested in ​https://bugs.webkit.org/show_bug.cgi?id=29972#c19.

No new tests since no new functionality was introduced.

  • html/HTMLAnchorElement.cpp:

(WebCore::HTMLAnchorElement::setProtocol):

  • platform/KURL.cpp:

(WebCore::isValidProtocol):

  • platform/KURL.h:
  • platform/KURLGoogle.cpp:

(WebCore::isValidProtocol):

2:41 PM Changeset in webkit [51055] by Chris Fleizach
  • 8 edits
    2 adds in trunk

AX: aria-labelledby duplicates some of its WAI-ARIA label
​https://bugs.webkit.org/show_bug.cgi?id=31565

Reviewed by Beth Dakin.

WebCore:

Test: accessibility/aria-labelledby-overrides-label.html

  • accessibility/AccessibilityRenderObject.cpp:

(WebCore::AccessibilityRenderObject::hasTextAlternative):
(WebCore::AccessibilityRenderObject::exposesTitleUIElement):
(WebCore::AccessibilityRenderObject::accessibilityIsIgnored):

  • accessibility/AccessibilityRenderObject.h:

WebKitTools:

  • DumpRenderTree/AccessibilityUIElement.cpp:

(titleUIElementCallback):
(getIsValidCallback):
(AccessibilityUIElement::getJSClass):

LayoutTests:

  • accessibility/aria-labelledby-overrides-label-expected.txt: Added.
  • accessibility/aria-labelledby-overrides-label.html: Added.
  • platform/gtk/Skipped:
  • platform/win/Skipped:
2:40 PM Changeset in webkit [51054] by Nate Chapin
  • 2 edits in trunk/WebCore

2009-11-16 Nate Chapin <Nate Chapin>

Reviewed by Darin Fisher.

Handle the case of a null NPObject* in NPN_SetException in
the V8 bindings. This allow out of process plugins calling
NPN_SetException to just send null instead of sending an
NPObject* that would be an address in a different
process's memory space.

​https://bugs.webkit.org/show_bug.cgi?id=31561

  • bindings/v8/NPV8Object.cpp: (_NPN_SetException): Allow null NPObject* and just throw a general error.
2:30 PM Changeset in webkit [51053] by ap@apple.com
  • 2 edits in trunk/WebCore

Windows build fix.

  • platform/network/cf/SocketStreamHandleCFNet.cpp: (WebCore::SocketStreamHandle::createStreams): Disable CONNECT proxies on Windows until WebKitSupportLibrary includes support for those.
2:21 PM Changeset in webkit [51052] by yael.aharon@nokia.com
  • 3 edits in trunk/WebCore

[Qt] ASSERT failure while running DRT
​https://bugs.webkit.org/show_bug.cgi?id=30978

Reviewed by Kenneth Rohde Christiansen.

Add needed Structure typeInfo flags to QtRuntimeObjectImpl and QtRuntimeMethod.
These flags are needed after r49649, where HasDefaultmark was changed to OverrideMarkChildren.

  • bridge/qt/qt_instance.cpp:

(JSC::Bindings::QtRuntimeObjectImp::createStructure):

  • bridge/qt/qt_runtime.h:

(JSC::Bindings::QtRuntimeMethod::createStructure):

1:51 PM Changeset in webkit [51051] by mrowe@apple.com
  • 1 edit
    1 add in trunk/WebCore

Attempt to fix the build. Land a file that was missing from r51049.

  • bindings/js/JSWebGLArrayHelper.h: Copied from WebCore/html/canvas/WebGLByteArray.idl.

(WebCore::setWebGLArrayFromArray):

1:31 PM Changeset in webkit [51050] by kenneth@webkit.org
  • 3 edits in trunk/WebKitTools

r50942 broke output from created windows. Make the
m_enableTextOutput a member of the DRT and not the
WebPage.

Reviewed by Simon Hausmann.

  • DumpRenderTree/qt/DumpRenderTree.cpp:

(WebCore::WebPage::WebPage):
(WebCore::WebPage::javaScriptAlert):
(WebCore::WebPage::javaScriptConsoleMessage):
(WebCore::WebPage::javaScriptConfirm):
(WebCore::WebPage::javaScriptPrompt):
(WebCore::WebPage::acceptNavigationRequest):
(WebCore::DumpRenderTree::DumpRenderTree):
(WebCore::DumpRenderTree::open):
(WebCore::DumpRenderTree::createWindow):

  • DumpRenderTree/qt/DumpRenderTree.h:

(WebCore::DumpRenderTree::setTextOutputEnabled):
(WebCore::DumpRenderTree::isTextOutputEnabled):
(WebCore::WebPage::shouldInterruptJavaScript):
(WebCore::WebPage::isTextOutputEnabled):
(WebCore::WebPage::setViewGeometry):

1:30 PM Changeset in webkit [51049] by cmarrin@apple.com
  • 53 edits in trunk

Update API of WebGLArray and friends
​https://bugs.webkit.org/show_bug.cgi?id=31175

1:28 PM Changeset in webkit [51048] by ap@apple.com
  • 2 edits in trunk/WebCore

Windows build fix.

  • platform/network/cf/SocketStreamHandleCFNet.cpp: Declare constants as extern "C".
1:28 PM Changeset in webkit [51047] by Simon Hausmann
  • 4 edits in trunk/WebKit/qt

API documentation fixes.

Reviewed by Kenneth Rohde Christiansen.

  • Api/qgraphicswebview.cpp: Removed duplicate docs.
  • Api/qwebelement.cpp: Added missing docs.
  • Api/qwebsettings.cpp: Ditto.
1:01 PM Changeset in webkit [51046] by bfulgham@webkit.org
  • 2 edits in trunk/WebCore

Enable support for webkit-box-shadow in Cairo builds.
​https://bugs.webkit.org/show_bug.cgi?id=26102.

Reviewed by Oliver Hunt.

Covered by existing fast/box-shadow tests.

  • platform/graphics/cairo/GraphicsContextCairo.cpp: Add support for fillRect shadows.
12:36 PM Changeset in webkit [51045] by ap@apple.com
  • 2 edits in trunk/WebCore

Windows build fix.

  • platform/network/cf/SocketStreamHandleCFNet.cpp: Don't try to include a file that's not in WebKitSupportLibrary.
12:05 PM Changeset in webkit [51044] by kenneth@webkit.org
  • 3 edits
    4 moves
    4 adds in trunk/WebKitTools

Moved DumpRenderTree/gtk/TestNetscapePlugin to DumpRenderTree/unix/TestNetscapePlugin
as the implementation is being used by at least Qt and Gtk+.

Reviewed by Gustavo Noronha Silva.

Update buildsystems as well.

  • DumpRenderTree/qt/TestNetscapePlugin/TestNetscapePlugin.pro:
  • DumpRenderTree/unix/TestNetscapePlugin/ForwardingHeaders/WebKit/npapi.h:
  • DumpRenderTree/unix/TestNetscapePlugin/ForwardingHeaders/WebKit/npfunctions.h:
  • DumpRenderTree/unix/TestNetscapePlugin/ForwardingHeaders/WebKit/npruntime.h:
  • DumpRenderTree/unix/TestNetscapePlugin/TestNetscapePlugin.cpp:
  • GNUmakefile.am:
11:08 AM Changeset in webkit [51043] by weinig@apple.com
  • 2 edits in trunk/WebKitSite

Update Security Group Members list.

Reviewed by Anders Carlsson.

  • security/security-group-members.html:
10:58 AM Changeset in webkit [51042] by weinig@apple.com
  • 2 edits in trunk/WebKitSite

Update Security Group Members list.

Reviewed by Adele Peterson.

  • security/security-group-members.html:
10:40 AM Changeset in webkit [51041] by ap@apple.com
  • 5 edits in trunk

WebKitTools:
2009-11-16 Yuzo Fujishima <​yuzo@google.com>

Reviewed by Alexey Proskuryakov.

Disable wss until all platforms support pyOpenSSL

​https://bugs.webkit.org/show_bug.cgi?id=31479

  • Scripts/run-webkit-tests:

LayoutTests:

Re-enabling WebSocket tests on Tiger and Windows, now that SSL is disabled.

  • platform/mac-tiger/Skipped:
  • platform/win/Skipped:
9:59 AM Changeset in webkit [51040] by ap@apple.com
  • 2 edits in trunk/WebCore

Reviewed by Darin Adler.

​https://bugs.webkit.org/show_bug.cgi?id=31494
Add unauthenticated proxy support to SocketStreamHandleCFNet

Cannot be tested in DRT.

  • platform/network/cf/SocketStreamHandleCFNet.cpp: (WebCore::SocketStreamHandle::chooseProxy): Fetch proxy information from OS. (WebCore::SocketStreamHandle::createStreams): Apply it to the newly created streams.
8:54 AM WebKit Team edited by keishi@webkit.org
(diff)
8:51 AM Changeset in webkit [51039] by keishi@webkit.org
  • 1 edit in trunk/WebKitTools/Scripts/modules/committers.py

Adding myself to committers.py.

7:38 AM Changeset in webkit [51038] by tonikitoo@webkit.org
  • 8 edits
    2 adds in trunk

[Qt] Broken back/forward after using ErrorPageExtension to set error page
​https://bugs.webkit.org/show_bug.cgi?id=30573

Reviewed by Antti Koivisto.

WebCore:

Make FrameLoader::checkLoadCompleteForThisFrame method
to check for any working DocumentLoader instance (through
activeDocumentLoader()) instead of only checking for
'm_provisionalDocumentLoader' in order to decide to if
it is going to reset of not the back and forward history.
after an error page has been loaded.

Test: LayoutTests/fast/history/back-forward-reset-after-error-handling.html

  • loader/FrameLoader.cpp:

(WebCore::FrameLoader::checkLoadCompleteForThisFrame):

WebKit/qt:

Implemented autotests for covering the back/forward
reset problem involving error pages.

  • tests/qwebpage/tst_qwebpage.cpp:

(tst_QWebPage::errorPageExtension):

LayoutTests:

Skipped fast/history/back-forward-reset-after-error-handling.html
in mac, gtk and win DRT, because their DRT do not support yet
error pages handling.

  • fast/history/back-forward-reset-after-error-handling.html:
  • platform/gtk/Skipped:
  • platform/mac/Skipped:
  • platform/win/Skipped:
7:37 AM Changeset in webkit [51037] by tonikitoo@webkit.org
  • 5 edits in trunk/WebKitTools

[Qt] Implement load error pages support for Qt's DRT.
​https://bugs.webkit.org/show_bug.cgi?id=31509

Reviewed by Kenneth Christiansen.

For now, it will not be a default feature, and layout tests
that want to make use of this have to explicitily call
'handleErrorPages();' for the test source.

Any of the other DumpRenderTree's (mac, win and gtk)
support handling error pages. Qt's will be the first.

  • DumpRenderTree/qt/DumpRenderTree.cpp:

(WebCore::WebPage::supportsExtension):
(WebCore::WebPage::extension):

  • DumpRenderTree/qt/LayoutTestControllerQt.cpp:

(LayoutTestController::reset):

  • DumpRenderTree/qt/LayoutTestControllerQt.h:

(LayoutTestController::shouldHandleErrorPages):
(LayoutTestController::handleErrorPages):

7:05 AM QtWebKitContrib edited by Simon Hausmann
(diff)
6:59 AM QtWebKitJournal edited by Simon Hausmann
(diff)
6:58 AM QtWebKitJournal edited by Simon Hausmann
(diff)
6:30 AM QtWebKitJournal edited by Simon Hausmann
(diff)
6:26 AM QtWebKitJournal edited by Simon Hausmann
(diff)
6:24 AM QtWebKitJournal edited by Simon Hausmann
(diff)
6:18 AM QtWebKitJournal edited by Simon Hausmann
(diff)
6:12 AM QtWebKit edited by kenneth@webkit.org
(diff)
6:06 AM QtWebKitJournal edited by Simon Hausmann
(diff)
6:05 AM QtWebKit edited by kenneth@webkit.org
(diff)
6:04 AM QtWebKit edited by Simon Hausmann
(diff)
5:59 AM QtWebKitJournal edited by Simon Hausmann
(diff)
5:47 AM Changeset in webkit [51036] by eric@webkit.org
  • 2 edits in trunk/WebKitTools

2009-11-16 Eric Seidel <​eric@webkit.org>

Reviewed by Adam Barth.

Rename the --commit-queue flag on land-* now that the commit-queue needs no special treatment
​https://bugs.webkit.org/show_bug.cgi?id=31549

Renamed --commit-queue to --non-interactive in most places
and remove the code in land-patches which is no longer needed.

  • Scripts/bugzilla-tool:
5:43 AM Changeset in webkit [51035] by kenneth@webkit.org
  • 1 edit
    4 adds in trunk/LayoutTests

Rubberstamped by Simon Hausmann.

Added new Qt expected results for animation tests.

  • platform/qt/animations/change-transform-in-end-event-expected.txt: Added.
  • platform/qt/animations/state-at-end-event-expected.txt: Added.
  • platform/qt/animations/state-at-end-event-transform-expected.txt: Added.
5:34 AM Changeset in webkit [51034] by eric@webkit.org
  • 2 edits in trunk/WebKitTools

2009-11-16 Eric Seidel <​eric@webkit.org>

Reviewed by Adam Barth.

The commit-queue should use land-attachment
​https://bugs.webkit.org/show_bug.cgi?id=31548

  • Scripts/bugzilla-tool:
5:25 AM Changeset in webkit [51033] by eric@webkit.org
  • 2 edits in trunk/WebKitTools

2009-11-16 Adam Barth <​abarth@webkit.org>

Reviewed by Eric Seidel.

Convert CommitQueue over to PatchCollection
​https://bugs.webkit.org/show_bug.cgi?id=31547

Also fixes a bug in workqueue and adds a test!

  • Scripts/bugzilla-tool:
  • Scripts/modules/workqueue.py:
  • Scripts/modules/workqueue_unittest.py:
5:20 AM Changeset in webkit [51032] by Simon Hausmann
  • 3 edits in trunk/JavaScriptCore

Fix Qt build on Windows CE 6.

Patch by Joerg Bornemann <​joerg.bornemann@trolltech.com> on 2009-11-16
Reviewed by Simon Hausmann.

  • JavaScriptCore.pri: Add missing include path.
  • wtf/Platform.h: Include ce_time.h for Windows CE 6.
5:05 AM Changeset in webkit [51031] by abarth@webkit.org
  • 4 edits in trunk/WebKitTools

2009-11-16 Adam Barth <​abarth@webkit.org>

Reviewed by Eric Seidel.

Convert CommitQueue over to PatchCollection
​https://bugs.webkit.org/show_bug.cgi?id=31547

Also fixes a bug in workqueue and adds a test!

  • Scripts/bugzilla-tool:
  • Scripts/modules/workqueue.py:
  • Scripts/modules/workqueue_unittest.py:
4:29 AM Changeset in webkit [51030] by abarth@webkit.org
  • 4 edits in trunk/WebKitTools

2009-11-16 Adam Barth <​abarth@webkit.org>

Reviewed by Eric Seidel.

Move StyleQueue over to using PatchCollection
​https://bugs.webkit.org/show_bug.cgi?id=31544

That's what the class it's for.

  • Scripts/bugzilla-tool:
  • Scripts/modules/patchcollection.py:
  • Scripts/modules/patchcollection_unittest.py:
4:11 AM Changeset in webkit [51029] by eric@webkit.org
  • 2 edits in trunk/WebKitTools

2009-11-16 Eric Seidel <​eric@webkit.org>

Reviewed by Adam Barth.

Move more patch-landing code into WebKitLandingScripts in preparation for land-attachment
​https://bugs.webkit.org/show_bug.cgi?id=31543

Just moving code and updating the one caller to use WebKitLandingScripts instead of 'self'.

  • Scripts/bugzilla-tool:
4:04 AM Changeset in webkit [51028] by eric@webkit.org
  • 2 edits in trunk/WebKitTools

2009-11-16 Eric Seidel <​eric@webkit.org>

Reviewed by Adam Barth.

Disable the style queue from posting to the commit queue status page.

  • Scripts/bugzilla-tool:
3:54 AM Changeset in webkit [51027] by eric@webkit.org
  • 2 edits in trunk/WebKitTools

2009-11-16 Adam Barth <​abarth@webkit.org>

Reviewed by Eric Seidel.

Implement PatchCollection
​https://bugs.webkit.org/show_bug.cgi?id=31541

This class holds a set of patches and lets clients iterate through
them. Optionally, clients can install a filter.

  • Scripts/modules/patchcollection.py: Added.
  • Scripts/modules/patchcollection_unittest.py: Added.
  • Scripts/run-webkit-unittests:
3:47 AM Changeset in webkit [51026] by abarth@webkit.org
  • 2 edits
    1 copy
    1 add in trunk/WebKitTools

2009-11-16 Adam Barth <​abarth@webkit.org>

Reviewed by Eric Seidel.

Implement PatchCollection
​https://bugs.webkit.org/show_bug.cgi?id=31541

This class holds a set of patches and lets clients iterate through
them. Optionally, clients can install a filter.

  • Scripts/modules/patchcollection.py: Added.
  • Scripts/modules/patchcollection_unittest.py: Added.
  • Scripts/run-webkit-unittests:
3:31 AM Changeset in webkit [51025] by eric@webkit.org
  • 2 edits in trunk/WebKitTools

2009-11-16 Eric Seidel <​eric@webkit.org>

Reviewed by Adam Barth.

bugzilla-tool check-style should work with attachment ids instead of bug ids
​https://bugs.webkit.org/show_bug.cgi?id=31540

  • Scripts/bugzilla-tool:
3:04 AM Changeset in webkit [51024] by eric@webkit.org
  • 4 edits in trunk/WebKitTools

2009-11-16 Eric Seidel <​eric@webkit.org>

Reviewed by Adam Barth.

WorkQueue is the only place that should know about special exit codes
​https://bugs.webkit.org/show_bug.cgi?id=31534

Move LandPatchesFromBugs.handled_error to WorkQueue.exit_after_handled_error
and add tests for handling exit codes.
I also cleaned up workqueue_unittest.py more.

  • Scripts/bugzilla-tool:
  • Scripts/modules/workqueue.py:
  • Scripts/modules/workqueue_unittest.py:
3:02 AM Changeset in webkit [51023] by eric@webkit.org
  • 2 edits in trunk/WebKitTools

2009-11-16 Eric Seidel <​eric@webkit.org>

Reviewed by Adam Barth.

Re-factor workqueue_unittest to allow for more than one test.
​https://bugs.webkit.org/show_bug.cgi?id=31535

  • Scripts/modules/workqueue_unittest.py:
3:00 AM Changeset in webkit [51022] by eric@webkit.org
  • 2 edits in trunk/WebKitTools

2009-11-16 Eric Seidel <​eric@webkit.org>

Reviewed by Adam Barth.

land-patches needs to be re-factored into smaller chunks
​https://bugs.webkit.org/show_bug.cgi?id=31532

  • Scripts/bugzilla-tool: fix a couple obvious typos.
2:50 AM Changeset in webkit [51021] by eric@webkit.org
  • 2 edits in trunk/WebKitTools

2009-11-16 Eric Seidel <​eric@webkit.org>

Reviewed by Adam Barth.

land-patches needs to be re-factored into smaller chunks
​https://bugs.webkit.org/show_bug.cgi?id=31532

The next patch will move these methods into WebKitLandingScripts.

  • Scripts/bugzilla-tool:
    • Split out _land_patch and _close_bug_if_no_active_patches.
2:45 AM Changeset in webkit [51020] by abarth@webkit.org
  • 2 edits in trunk/WebKitTools

2009-11-16 Adam Barth <​abarth@webkit.org>

Reviewed by Eric Seidel.

Fix silly copy-and-paste code. I am a terrible coder.

  • Scripts/modules/bugzilla.py:
2:34 AM Changeset in webkit [51019] by abarth@webkit.org
  • 3 edits in trunk/WebKitTools

2009-11-16 Adam Barth <​abarth@webkit.org>

Reviewed by Eric Seidel.

Implement a StyleQueue
​https://bugs.webkit.org/show_bug.cgi?id=31537

The first iteration of the style queue only produces output locally.
There is also a limit of 10 patches because it's not that useful to
iterate through the entire review queue at this point. We can remove
the limit later.

  • Scripts/bugzilla-tool:
  • Scripts/modules/bugzilla.py:
1:25 AM Changeset in webkit [51018] by abarth@webkit.org
  • 5 edits
    1 add in trunk/WebKitTools

2009-11-16 Adam Barth <​abarth@webkit.org>

Reviewed by Eric Seidel.

Unit test WorkQueue
​https://bugs.webkit.org/show_bug.cgi?id=31531

Adds basic unit testing for WorkQueue. Just runs through one cycle.

  • Scripts/bugzilla-tool:
  • Scripts/modules/statusbot.py:
  • Scripts/modules/workqueue.py:
  • Scripts/modules/workqueue_unittest.py: Added.
  • Scripts/run-webkit-unittests:
12:49 AM Changeset in webkit [51017] by eric@webkit.org
  • 2 edits in trunk/WebKitTools

2009-11-16 Eric Seidel <​eric@webkit.org>

Reviewed by Adam Barth.

bugzilla-tool land-diff should know how to parse bug ids out of ChangeLogs
​https://bugs.webkit.org/show_bug.cgi?id=31530

  • Scripts/bugzilla-tool:
12:39 AM Changeset in webkit [51016] by eric@webkit.org
  • 4 edits in trunk/WebKitTools

2009-11-16 Eric Seidel <​eric@webkit.org>

Reviewed by Adam Barth.

bugzilla-tool needs apply-attachment
​https://bugs.webkit.org/show_bug.cgi?id=31528

  • Scripts/bugzilla-tool:
    • Add ApplyAttachment command.
    • Abstract applying code into WebKitApplyingScripts.
    • Rename setup_for_landing to prepare_clean_working_directory and make local_commit checking optional.
  • Scripts/modules/bugzilla.py:
    • Add fetch_attachment and bug_id_for_attachment_id.
  • Scripts/modules/bugzilla_unittest.py:
    • Add test for new parsing.
    • Fix previous parsing test which broke with Adam's check-style patch (bug 31515).
12:38 AM Changeset in webkit [51015] by eric@webkit.org
  • 3 edits in trunk/WebKitTools

2009-11-16 Eric Seidel <​eric@webkit.org>

Reviewed by Adam Barth.

Document check-style's use of force_clean.

  • Scripts/bugzilla-tool:
  • Scripts/modules/scm.py:
12:28 AM Changeset in webkit [51014] by abarth@webkit.org
  • 3 edits
    1 add in trunk/WebKitTools

2009-11-16 Adam Barth <​abarth@webkit.org>

Reviewed by Eric Seidel.

Move WorkQueue to its own file
​https://bugs.webkit.org/show_bug.cgi?id=31529

WorkQueue and WorkQueueDelegate are separate concerns from
bugzilla-tool. Also added a missing include to logging.py.

  • Scripts/bugzilla-tool:
  • Scripts/modules/logging.py:
  • Scripts/modules/workqueue.py:
12:11 AM Changeset in webkit [51013] by abarth@webkit.org
  • 3 edits in trunk/WebKitTools

2009-11-16 Adam Barth <​abarth@webkit.org>

Reviewed by Eric Siedel.

Move OutputTee to logging.py.

  • Scripts/bugzilla-tool:
  • Scripts/modules/logging.py:
12:07 AM Changeset in webkit [51012] by abarth@webkit.org
  • 2 edits in trunk/WebKitTools

2009-11-15 Adam Barth <​abarth@webkit.org>

Reviewed by Eric Seidel.

Refactor bugzilla-tool to allow for multiple queues
​https://bugs.webkit.org/show_bug.cgi?id=31513

Divide the commit queue class into three class to make creating
additional queues easier.

  • Scripts/bugzilla-tool:
Note: See TracTimeline for information about the timeline view.