summaryrefslogtreecommitdiffstats
path: root/webkit/port
Commit message (Collapse)AuthorAgeFilesLines
* Chrome side of webkit merge from 42547 to 42580.tc@google.com2009-04-164-24/+2
| | | | | | | | | | | | | | | | In http://trac.webkit.org/changeset/42579 ScriptController::createInlineEventListener was changed to take 4 params instead of 3, so I updated the V8 ScriptController to do the same. We don't seem to need the extra arg because both v8_proxy::createSVGEventHandler and v8_proxy::createInlineEventListener are identical. Also one new .cpp/.h file from http://trac.webkit.org/changeset/42580 Review URL: http://codereview.chromium.org/79008 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@13884 0039d316-1c4b-4281-b951-d872f2087c98
* Make HTMLSelectElement and HTMLOptionsCollection return collections of optionsdavemoore@chromium.org2009-04-163-9/+35
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | elements if there are multiple options that share a name. Currently Safari and chromium differ in their behavior in this area. Safari returns multiple options in a collection if necessary, but only for HTMLOptionsCollection. For HTMLSelectElement it always returns null. Chromium returns a single option no matter what, but for both HTMLSelectElement and HTMLOptionsCollection. It's also very slow at this, as it bypasses a webkit cache. The HTML5 doc states that a collection should be returned if necessary for both. The relevant parts of the spec are: http://dev.w3.org/html5/spec/Overview.html#the-select-element and http://dev.w3.org/html5/spec/Overview.html#htmloptionscollection-0 So if you have: <select id="sl1"><option value="Value" name="test" /></select> and in JS code you do: var test = document.getElementById("sl1").test test should be a single HTMLOptionElement. If instead you have: <select id="sl2"><option value="Value1" name="test" /><option value="Value2" name="test" /></select> test should be a collection with 2 HTMLOptionElements. This bug: https://bugs.webkit.org/show_bug.cgi?id=25191 has been filed against webkit, with a new layout test to confirm the behavior Review URL: http://codereview.chromium.org/69014 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@13814 0039d316-1c4b-4281-b951-d872f2087c98
* Chromium side of webkit merge 42440:42523.ericroman@google.com2009-04-151-3/+3
| | | | | | Review URL: http://codereview.chromium.org/73066 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@13774 0039d316-1c4b-4281-b951-d872f2087c98
* Reverting 13760.davemoore@chromium.org2009-04-153-34/+9
| | | | | | Review URL: http://codereview.chromium.org/75002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@13762 0039d316-1c4b-4281-b951-d872f2087c98
* Make HTMLSelectElement and HTMLOptionsCollection return collections of optionsdavemoore@chromium.org2009-04-153-9/+34
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | elements if there are multiple options that share a name. Currently Safari and chromium differ in their behavior in this area. Safari returns multiple options in a collection if necessary, but only for HTMLOptionsCollection. For HTMLSelectElement it always returns null. Chromium returns a single option no matter what, but for both HTMLSelectElement and HTMLOptionsCollection. It's also very slow at this, as it bypasses a webkit cache. The HTML5 doc states that a collection should be returned if necessary for both. The relevant parts of the spec are: http://dev.w3.org/html5/spec/Overview.html#the-select-element and http://dev.w3.org/html5/spec/Overview.html#htmloptionscollection-0 So if you have: <select id="sl1"><option value="Value" name="test" /></select> and in JS code you do: var test = document.getElementById("sl1").test test should be a single HTMLOptionElement. If instead you have: <select id="sl2"><option value="Value1" name="test" /><option value="Value2" name="test" /></select> test should be a collection with 2 HTMLOptionElements. This bug: https://bugs.webkit.org/show_bug.cgi?id=25191 has been filed against webkit, with a new layout test to confirm the behavior Review URL: http://codereview.chromium.org/67148 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@13760 0039d316-1c4b-4281-b951-d872f2087c98
* Revert changes causing memory_test problems. asargent@chromium.org2009-04-141-11/+2
| | | | | | | | | | | | This undoes changes from: http://codereview.chromium.org/66045 Review URL: http://codereview.chromium.org/67114 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@13639 0039d316-1c4b-4281-b951-d872f2087c98
* Fix problems in event listener teardownasargent@chromium.org2009-04-131-2/+11
| | | | | | | | | | | The problem is sometimes the functions to remove event listeners get called outside of a ContextScope (even though we still have a valid context). BUG=9775 Review URL: http://codereview.chromium.org/66045 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@13622 0039d316-1c4b-4281-b951-d872f2087c98
* Fix worker constructor so that nested worker can be created.jianli@chromium.org2009-04-131-6/+13
| | | | | | Review URL: http://codereview.chromium.org/67093 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@13619 0039d316-1c4b-4281-b951-d872f2087c98
* Chromium side of merge 42364:42403.pam@chromium.org2009-04-111-1/+1
| | | | | | | | Includes change to v8_proxy.cpp to account for addition of MessagePortProxy class. Review URL: http://codereview.chromium.org/66062 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@13553 0039d316-1c4b-4281-b951-d872f2087c98
* Remove unneeded ASSERT -- this code path is valid and should not be ASSERTing.dglazkov@google.com2009-04-101-1/+0
| | | | | | | | R=mbelshe Review URL: http://codereview.chromium.org/67040 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@13523 0039d316-1c4b-4281-b951-d872f2087c98
* Chrome side of WEbKit merge 42244:42287.brettw@chromium.org2009-04-081-2/+10
| | | | | | Review URL: http://codereview.chromium.org/63110 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@13337 0039d316-1c4b-4281-b951-d872f2087c98
* Chrome side change to use V8EventListenerList from upstream. This involves ↵jianli@chromium.org2009-04-032-114/+1
| | | | | | | | removing V8EventListenerList from v8_proxy and change the project files to compile V8EventListenerList. Review URL: http://codereview.chromium.org/60102 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@13107 0039d316-1c4b-4281-b951-d872f2087c98
* Implement custom methods for some Node dom callsmbelshe@google.com2009-04-011-2/+5
| | | | | | | | | | | | | so that we can take advantage of lazy wrapper initialization. Dependent on the webkit fix. will roll the deps for webkit as well. Review URL: http://codereview.chromium.org/57086 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@12958 0039d316-1c4b-4281-b951-d872f2087c98
* Keep JS wrappers for non-node objects associated with a DOMWindowager@chromium.org2009-04-012-7/+64
| | | | | | | | | | alive across GCs. This change needs a WebKit change to go in first which adds frame and type accessors to these objects. Review URL: http://codereview.chromium.org/49038 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@12942 0039d316-1c4b-4281-b951-d872f2087c98
* Reland changes to fix the problem that DOM object is deleted in the GC ↵jianli@chromium.org2009-03-294-382/+218
| | | | | | | | | | | | | thread, not owning thread. This patch contains the fix to the Purify failure: memory leaks of thread specific data for main thread. The fix is to switch to using the static maps for main thread. Compared with previous patch being reviewed, the main change in this patch is in V8DOMMap.cpp: I introduce NonMainThreadSpecificDOMData and MainThreadSpecificDOMData that are both derived from ThreadSpecificDOMData. Review URL: http://codereview.chromium.org/49044 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@12757 0039d316-1c4b-4281-b951-d872f2087c98
* Search box support, Chromium side.BUG=9210pkasting@chromium.org2009-03-271-1/+1
| | | | | | Review URL: http://codereview.chromium.org/42678 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@12680 0039d316-1c4b-4281-b951-d872f2087c98
* Start using the upstream versions of V8NodeFilter and V8XML*.levin@chromium.org2009-03-278-1083/+1
| | | | | | Review URL: http://codereview.chromium.org/42673 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@12626 0039d316-1c4b-4281-b951-d872f2087c98
* Speedup event performance by caching V8EventListener on the wrapped object. asargent@chromium.org2009-03-262-27/+120
| | | | | | Review URL: http://codereview.chromium.org/49026 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@12574 0039d316-1c4b-4281-b951-d872f2087c98
* Fix window.history GC issue. Since the history object is not part ofager@chromium.org2009-03-262-14/+27
| | | | | | | | the DOM, we need to explicitly keep the history wrapper object alive for the lifetime of the DOMWindow object with which it is associated. Review URL: http://codereview.chromium.org/53092 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@12543 0039d316-1c4b-4281-b951-d872f2087c98
* Revert change 12507 and 12532 because it breaks the nsylvain@chromium.org2009-03-265-787/+388
| | | | | | | tree and the purify tests. Review URL: http://codereview.chromium.org/42634 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@12534 0039d316-1c4b-4281-b951-d872f2087c98
* Fix the problem that DOM object is deleted in the GC thread, not owning thread.jianli@chromium.org2009-03-255-388/+787
| | | | | | Review URL: http://codereview.chromium.org/40088 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@12507 0039d316-1c4b-4281-b951-d872f2087c98
* Fix data url security tests.ager@chromium.org2009-03-251-4/+7
| | | | | | | | | | | | | | The problem was that the security token in V8 was set to the string "null" for all data urls. Therefore, the access check succeeded. The cause of the regression was that securityOrigin's toString implementation was changed in WebKit to return "null" instead of String(). R=kasperl@chromium.org TBR=scherkus@chromium.org Review URL: http://codereview.chromium.org/53049 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@12444 0039d316-1c4b-4281-b951-d872f2087c98
* Start using upstreamed worker files and remove the old ones from webkit/port.dimich@google.com2009-03-244-642/+0
| | | | | | | Before submitting, I'll add DEPS with corresponding WebKit rev (see http://codereview.chromium.org/52007) Review URL: http://codereview.chromium.org/45035 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@12377 0039d316-1c4b-4281-b951-d872f2087c98
* Add basic infrastructure for sending async browser API request and receiving ↵aa@chromium.org2009-03-242-12/+28
| | | | | | | | results. Review URL: http://codereview.chromium.org/42262 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@12347 0039d316-1c4b-4281-b951-d872f2087c98
* Chrome side of webkit merge from r41860 to 41906.tc@google.com2009-03-232-2/+2
| | | | | | | | | | | | | The encoding parameter got dropped from Workers (r41871) so drop it through the whole stack. The upstream WorkerContextExecutionProxy has trackEvent with a lower case t, but we pull in the wrong header in some places so make both headers match. Review URL: http://codereview.chromium.org/42538 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@12326 0039d316-1c4b-4281-b951-d872f2087c98
* Back out string morphing.mbelshe@google.com2009-03-231-5/+10
| | | | | | Review URL: http://codereview.chromium.org/52027 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@12320 0039d316-1c4b-4281-b951-d872f2087c98
* Rework the v8 binding string handling to be more efficient.mbelshe@google.com2009-03-232-79/+128
| | | | | | | | | | | Break out ToAtomicWebCoreString in addition to ToWebCoreString. Re-enables the v8 string morphing; there is no longer degradation on intl1 page cycler. Review URL: http://codereview.chromium.org/42445 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@12309 0039d316-1c4b-4281-b951-d872f2087c98
* Change chromium to use the V8Collection from WebKit/WebCore/bindings/v8.levin@chromium.org2009-03-232-192/+1
| | | | | | Review URL: http://codereview.chromium.org/52007 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@12293 0039d316-1c4b-4281-b951-d872f2087c98
* Hook up V8 Database Bindings.dglazkov@google.com2009-03-225-3/+29
| | | | | | | | R=darin Review URL: http://codereview.chromium.org/50071 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@12264 0039d316-1c4b-4281-b951-d872f2087c98
* Remove the DOM constructor cache and instead use an object wrapper prototypembelshe@google.com2009-03-202-63/+76
| | | | | | | | | | which we just clone. This fix cannot land until we get the latest v8. Review URL: http://codereview.chromium.org/50056 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@12225 0039d316-1c4b-4281-b951-d872f2087c98
* Preupstreaming cleanup for xhr (part 2): levin@chromium.org2009-03-205-325/+492
| | | | | | | | Split V8XMLHttpRequestCustom into appropriate files. Review URL: http://codereview.chromium.org/45018 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@12216 0039d316-1c4b-4281-b951-d872f2087c98
* Pre-upstreaming style changes to match webkit style.levin@chromium.org2009-03-193-761/+743
| | | | | | Review URL: http://codereview.chromium.org/48169 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@12169 0039d316-1c4b-4281-b951-d872f2087c98
* Enable IDL attribute [V8ReadOnly] to have the same mbelshe@google.com2009-03-191-3/+4
| | | | | | | | | | | | | semantics as having specified an attribute as readonly. This allows us to unfork IDL differences where V8 wants readonly and JSC does not by using the V8ReadOnly tag. Review URL: http://codereview.chromium.org/50037 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@12165 0039d316-1c4b-4281-b951-d872f2087c98
* Linux: fix include filename caseagl@chromium.org2009-03-191-2/+2
| | | | | | | Review URL: http://codereview.chromium.org/42383 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@12075 0039d316-1c4b-4281-b951-d872f2087c98
* Pre-upstream style changes: rename enableWebWorkers to ↵dimich@google.com2009-03-192-5/+5
| | | | | | | | | setIsWebWorkersEnabled to follow isFoo/setIsFoo pattern. Also pull DEPS to bring in the WebKit/chromium/src part. Review URL: http://codereview.chromium.org/48157 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@12066 0039d316-1c4b-4281-b951-d872f2087c98
* Unfork WebInspector frontend take 2: New DEPS and webkit/...pfeldman@chromium.org2009-03-181-0/+9
| | | | | | Review URL: http://codereview.chromium.org/48132 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@11967 0039d316-1c4b-4281-b951-d872f2087c98
* Add a handle scope to V8Proxy::initContextIfNeeded.ager@chromium.org2009-03-181-1/+4
| | | | | | | | | | | | initContextIfNeeded can be called without a handle scope in place which will lead to a renderer crash when creating the first handle. Also V8Proxy::initContextIfNeeded creates a number of local handles that should be deallocated on exit. BUG=8922 Review URL: http://codereview.chromium.org/48131 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@11965 0039d316-1c4b-4281-b951-d872f2087c98
* Revert the src/webkit part of r11958 since it broke the build.maruel@chromium.org2009-03-181-9/+0
| | | | | | | TBR=pfeldman Review URL: http://codereview.chromium.org/42332 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@11960 0039d316-1c4b-4281-b951-d872f2087c98
* Unfork WebInspector frontend.pfeldman@chromium.org2009-03-181-0/+9
| | | | | | Review URL: http://codereview.chromium.org/48063 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@11958 0039d316-1c4b-4281-b951-d872f2087c98
* Make worker constructor return error if it is enabled by the flag.jianli@chromium.org2009-03-183-0/+22
| | | | | | | This is to move corresponding files out of http://codereview.chromium.org/48106/ so that this can be checked in before the change to enable worker. The reason for doing this is to avoid build break since the already checked-in third_party/WebKit/WebKit/chromium/src/WebKit.cpp has the ifdef guarded code depend on this. Review URL: http://codereview.chromium.org/42313 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@11952 0039d316-1c4b-4281-b951-d872f2087c98
* Pre-upstream cleaning of files - changing them to follow WebKit coding style.dimich@google.com2009-03-174-448/+416
| | | | | | Review URL: http://codereview.chromium.org/42308 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@11935 0039d316-1c4b-4281-b951-d872f2087c98
* Get v8_collection.h to match webkit coding standards inlevin@chromium.org2009-03-172-237/+205
| | | | | | | | preparation for upstreaming. Review URL: http://codereview.chromium.org/42294 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@11930 0039d316-1c4b-4281-b951-d872f2087c98
* Revert "Enable building worker and use command line switch to turn on ..."agl@chromium.org2009-03-173-20/+0
| | | | | | | This reverts commit 11870. git-svn-id: svn://svn.chromium.org/chrome/trunk/src@11871 0039d316-1c4b-4281-b951-d872f2087c98
* Enable building worker and use command line switch to turn on this feature.jianli@chromium.org2009-03-173-0/+20
| | | | | | Review URL: http://codereview.chromium.org/41029 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@11870 0039d316-1c4b-4281-b951-d872f2087c98
* WebKit merge 41613:41660 (Chromium side).levin@chromium.org2009-03-151-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Layout tests changes (and baselines): * LayoutTests/fast/css/font-face-default-font.html Fixed by aa upstream. The change was in the test itself. * LayoutTests/fast/replaced/width100percent-searchfield.html Fixed by levin upstream -- the themeChromiumWin.css change. * LayoutTests/fast/forms/input-appearance-height.html * LayoutTests/fast/repaint/search-field-cancel.html Rebase needed due to themeChromiumWin.css change * LayoutTests/fast/layers/normal-flow-hit-test.html A slightly different font seems to be used in chromium which casued some pixel differences in this test. * LayoutTests/fast/forms/placeholder-pseudo-style.html * LayoutTests/fast/forms/search-placeholder-value-changed.html New tests which needed chromium baselines due to differences in the search input box display. * LayoutTests/fast/forms/placeholder-set-value.html Differences in fonts and search input boxes were the cause for the chromium baseline. Review URL: http://codereview.chromium.org/46071 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@11712 0039d316-1c4b-4281-b951-d872f2087c98
* Commit 40144. I had to move to to a separate CL to use gcl'saa@chromium.org2009-03-135-48/+127
| | | | | | | | "try multiple commits" feature. Review URL: http://codereview.chromium.org/46062 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@11683 0039d316-1c4b-4281-b951-d872f2087c98
* Fix build break to worker feature due to latest webkit merge.jianli@chromium.org2009-03-134-19/+26
| | | | | | Review URL: http://codereview.chromium.org/46051 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@11663 0039d316-1c4b-4281-b951-d872f2087c98
* Move HTMLDocument functions from v8_custom to V8HTMLDocumentCustom.mbelshe@google.com2009-03-133-80/+4
| | | | | | | | Remove NAMED_PROPERTY_SETTER interceptor for HTMLDocument. Review URL: http://codereview.chromium.org/42157 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@11635 0039d316-1c4b-4281-b951-d872f2087c98
* WebKit merge 41588:41613 (Chromium side).levin@chromium.org2009-03-137-815/+45
| | | | | | | | | | | | | | | | | | webkit\port\bindings\v8, webkit\build files These are all adjustments to use the recently upstreamed V8Events files. percent-top-value-with-relative-position-expected This is a new test that has the same result in chromium but uses a slightly different font to display it, so pixels and coordinates were also different. search-field-cancel-expected This is a new test that verifies the visual aspects of the search field. Since chromium doesn't implement this field in a special way, chromium's results are expected to be different. Review URL: http://codereview.chromium.org/42163 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@11606 0039d316-1c4b-4281-b951-d872f2087c98
* Add v8::Locker in worker related code in order to run multiple workers in a ↵jianli@chromium.org2009-03-122-0/+3
| | | | | | | | worker process. Review URL: http://codereview.chromium.org/44003 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@11570 0039d316-1c4b-4281-b951-d872f2087c98