summaryrefslogtreecommitdiffstats
path: root/content
Commit message (Collapse)AuthorAgeFilesLines
* Only dlopen() OMX/VAAPI libs in the GPU process, and only lazily dlsym() ↵fischman@chromium.org2012-08-049-71/+186
| | | | | | | | | | | | | | their symbols. This reinstates r150007, which was reverted in r150009. TBR=piman@chromium.org BUG=140412 Review URL: https://chromiumcodereview.appspot.com/10827173 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@150024 0039d316-1c4b-4281-b951-d872f2087c98
* Update media test binaries for new audio codecs.ihf@chromium.org2012-08-043-0/+0
| | | | | | | | BUG=chromium-os:31955 TEST=Played on linux CrOS build. Review URL: https://chromiumcodereview.appspot.com/10835028 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@150016 0039d316-1c4b-4281-b951-d872f2087c98
* VAVDA: Cleanups and fixes.marcheu@chromium.org2012-08-042-29/+67
| | | | | | | | | | | | | | | | | | | | This CL does some cleanups and fixes: - Destroy vaapi buffers after we're done drawing.If we do this before we're done drawing, the memory can be reclaimed by other threads in the meantime and result in interesting use-after-free bugs. - Don't leak buffers when we go through the error path. - Don't leak buffers when we Reset(). - Fix a small indentation. BUG=chromium-os:33170,chromium:139755 TEST=by hand Change-Id: Icd44bd52c5b6c8f32d904ad2980d19a090f01e38 Review URL: https://chromiumcodereview.appspot.com/10843058 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@150010 0039d316-1c4b-4281-b951-d872f2087c98
* Revert 150007 - Only dlopen() OMX/VAAPI libs in the GPU process, and only ↵fischman@chromium.org2012-08-048-176/+71
| | | | | | | | | | | | | | | | | | | | lazily dlsym() their symbols. Broke several cros bots: http://build.chromium.org/p/chromium.chromiumos/builders/ChromiumOS%20%28daisy%29/builds/1752/steps/cbuildbot/logs/stdio http://build.chromium.org/p/chromium.chromiumos/builders/ChromiumOS%20%28tegra2%29/builds/8653/steps/cbuildbot/logs/stdio http://build.chromium.org/p/chromium.memory/builders/Chromium%20OS%20ASAN%20Builder/builds/1212/steps/compile/logs/stdio http://build.chromium.org/p/chromium.chrome/builders/Google%20Chrome%20ChromeOS/builds/38398/steps/compile/logs/stdio BUG=140412 Review URL: https://chromiumcodereview.appspot.com/10829182 TBR=fischman@chromium.org Review URL: https://chromiumcodereview.appspot.com/10836098 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@150009 0039d316-1c4b-4281-b951-d872f2087c98
* Only dlopen() OMX/VAAPI libs in the GPU process, and only lazily dlsym() ↵fischman@chromium.org2012-08-048-71/+176
| | | | | | | | | | | their symbols. BUG=140412 Review URL: https://chromiumcodereview.appspot.com/10829182 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@150007 0039d316-1c4b-4281-b951-d872f2087c98
* [RDS] Reloads a page using the original request URLdfalcantara@chromium.org2012-08-0410-11/+151
| | | | | | | | | | | | | | | | | | | | | | | | | When toggling between different versions of a website by changing user agents, we often need to reload using an earlier URL in a redirect chain to avoid redirects caused by user-agent detection. * Adds a new type of reload that hooks into WebFrame::reloadWithOverrideURL, added in https://bugs.webkit.org/show_bug.cgi?id=89788 * Changes the original request URL we were saving before to be the URL sent back as part of WebKit's original request. This URL often differs from the URL stored at the beginning at the redirect chain. It's sent back as a new field when the renderer navigates. Part of upstreaming Chrome for Android. Internal BUG=6272286 BUG=112923 TEST= Review URL: https://chromiumcodereview.appspot.com/10704048 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@150003 0039d316-1c4b-4281-b951-d872f2087c98
* Fix lost context recreation race in BrowserGpuChannelHostFactorypiman@chromium.org2012-08-042-17/+41
| | | | | | | | | | | | | | | An existing GPU process may have died between the last message we sent on its channel and the next time we try to establish a new GPU channel. In that case, we would re-use the GpuProcessHost but fail on the Send. When that happens, we want to retry with a fresh GPU process. BUG=129067 Review URL: https://chromiumcodereview.appspot.com/10852012 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@149993 0039d316-1c4b-4281-b951-d872f2087c98
* Clarify ownership of GpuVideoDecodeAcceleratorHost and avoid crash on ↵fischman@chromium.org2012-08-044-8/+15
| | | | | | | | | | | | | | | | | | | | context loss. GpuVideoDecoder owns GpuVideoDecodeAcceleratorHost; it was a bug that ~CommandBufferProxyImpl() called GpuVideoDecodeAcceleratorHost::Destroy() (since the latter deletes |this|). This bug was uncovered due to an unrelated GPU-process crashing bug, which triggered context loss and the subsequent stack in the linked bug. With this change, killing the GPU process mid-playback fires a JS error on the <video> tag (correctly), but the renderer keeps on just fine. BUG=140138 Review URL: https://chromiumcodereview.appspot.com/10852009 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@149985 0039d316-1c4b-4281-b951-d872f2087c98
* Don't reuse a GpuProcessHost that failed to send a messagepiman@chromium.org2012-08-042-3/+11
| | | | | | | | | | | | | | | When the GPU process dies/gets killed, the Send() can fail (e.g. EPIPE), triggering lost context recreation, before OnChannelError gets called (happening when we return to the message loop) which ends up deleting the GpuProcessHost. So, in the mean time, make sure we don't try to reuse that GpuProcessHost to recreate a channel. BUG=129067 Review URL: https://chromiumcodereview.appspot.com/10834165 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@149984 0039d316-1c4b-4281-b951-d872f2087c98
* Fix channel lost race in GpuChannelHostpiman@chromium.org2012-08-031-4/+7
| | | | | | | | | | | | | | | | | | | | | | | The problem this fixes is, depending on the exact timing, a context would receive a lost context callback (task 1), and would then post a task to recreate the context (task 2). In parallel a task would be posted to signal the GpuChannelHost that the channel is lost (task 3). Because task 3 is posted from the IO thread, whereas task 2 is posted from another thread (but they both are posted to the "main" thread), the race in posting results in a race in execution, and task 2 may try to recreate a context using a channel that doesn't know yet that it's non-functional, resulting in failing to send the message and failing to create the context. This patch changes the post order - task 3 is posted before task 1 - ensuring task 2 is posted after task 3 (since task 2 is posted after task 1 executes). BUG=129067 Review URL: https://chromiumcodereview.appspot.com/10837104 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@149976 0039d316-1c4b-4281-b951-d872f2087c98
* VAVDA: Fix locking.marcheu@chromium.org2012-08-032-11/+30
| | | | | | | | | | | | | | Ensure proper locking around all uses of curr_input_buffer_ and input_buffers_. TEST=by hand BUG=none Change-Id: I4cc92a1b0be47ce95bc0def580a88ac3a9f7deec Review URL: https://chromiumcodereview.appspot.com/10853005 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@149973 0039d316-1c4b-4281-b951-d872f2087c98
* Implement "Key Presence" step in "Encrypted Block Encounted" algorithm in EME.xhwang@chromium.org2012-08-031-0/+1
| | | | | | | | | | | | | | | See related spec here: http://dvcs.w3.org/hg/html-media/raw-file/tip/encrypted-media/encrypted-media.html#algorithms-enrypted-block If the AesDecryptor cannot find the decryption for the encrypted buffer, it returns a kNoKey. In ProxyDecryptor, if the input cannot be decrypted due to kNoKey, the ProxyDecryptor will cache the encrypted buffers and callbacks and fire a needkey event. When some key is added (AddKey called) later, it will try to decrypt these buffers again. The callbacks will be fired on the same thread where Decrypt() was called originally. BUG=125401, 125753 TEST=media_unittest, media layout_tests Review URL: https://chromiumcodereview.appspot.com/10822026 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@149952 0039d316-1c4b-4281-b951-d872f2087c98
* Set the default locale for ICU in the PPAPI process.brettw@chromium.org2012-08-031-0/+10
| | | | | | | | BUG=140432 Review URL: https://chromiumcodereview.appspot.com/10853007 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@149937 0039d316-1c4b-4281-b951-d872f2087c98
* Pass tap count (1 or 2) with tap gesture eventsrbyers@chromium.org2012-08-032-4/+25
| | | | | | | | | | | | | | | | | This is necessary to support the generation of the correct dblclick event sequence in WebKit. Remove evil generic_x/generic_y from GestureEventDetails (some event types have data other than floats in that position of the union, this wasn't typesafe). Adds missing double tap gesture recognizer unit tests. This makes ET_GESTURE_DOUBLE_TAP pretty pointless now. I've filed crbug.com/140382 to track removing it. BUG=115486 Review URL: https://chromiumcodereview.appspot.com/10824158 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@149935 0039d316-1c4b-4281-b951-d872f2087c98
* Added DCHECK() to prevent webwidget_->isInputThrottled() from being called ↵khorimoto@chromium.org2012-08-031-1/+1
| | | | | | | | | | | | | when webwidget_ is NULL. BUG= TEST= CID=103685 TBR=jamesr Review URL: https://chromiumcodereview.appspot.com/10830024 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@149918 0039d316-1c4b-4281-b951-d872f2087c98
* Block Avast! AV DLLs in sandboxed processes.jschuh@chromium.org2012-08-031-0/+2
| | | | | | | BUG=140140 Review URL: https://chromiumcodereview.appspot.com/10852008 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@149909 0039d316-1c4b-4281-b951-d872f2087c98
* Linux: add BPF sandbox status in about:sandboxjln@chromium.org2012-08-033-3/+20
| | | | | | | | | | | | | | | This adds the status of the BPF sandbox in about:sandbox. This is of course pretty limited because the real status depends on the process type. We assume renderers. BUG= TBR=joi@chromium.org NOTRY=true Review URL: https://chromiumcodereview.appspot.com/10829156 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@149896 0039d316-1c4b-4281-b951-d872f2087c98
* Change return type of FaviconTabHelper::GetFavicon() to gfx::Image.thakis@chromium.org2012-08-032-7/+0
| | | | | | | | | | | | | No intended behavior change for now. Depends on http://codereview.chromium.org/10832128/ Part of supporting hidpi favicons in the tab strip. BUG=138550 TBR=joi Review URL: https://chromiumcodereview.appspot.com/10837090 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@149891 0039d316-1c4b-4281-b951-d872f2087c98
* Convert FaviconStatus::bitmap from SkBitmap to gfx::Image.thakis@chromium.org2012-08-032-3/+11
| | | | | | | | | | | | Also rename it to "image". No intended behavior change. Part of supporting hidpi favicons in the tab strip. BUG=138550 TBR=joi Review URL: https://chromiumcodereview.appspot.com/10832128 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@149874 0039d316-1c4b-4281-b951-d872f2087c98
* Refactors DeviceOrientation to make it more extensibleaousterh@chromium.org2012-08-0320-472/+844
| | | | | | | | | | | | | | | | | | This refactors the ProviderImpl in DeviceOrientation to make it more general. This will make it easy to add other types of device data (such as DeviceMotion) in the future and to have them use the same Provider/ProviderImpl. This is an alternative to http://codereview.chromium.org/10689106. TBR=avi BUG=none TEST=browser_tests:DeviceOrientationBrowserTest.BasicTest, content_unittests --gtest_filter=DeviceOrientation* Review URL: https://chromiumcodereview.appspot.com/10755002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@149864 0039d316-1c4b-4281-b951-d872f2087c98
* Correct const accessors in base/values.(h|cc), Part II (ListValue)vabr@chromium.org2012-08-033-6/+6
| | | | | | | | | | | | | | For problem description and other info please see the BUG page. This is for ListValue. BUG=138946 TEST=N/A (no fix & no new feature) TBR=jar,zelidrag,scottbyer,mpcomplete,darin,achuith,sky,estade,atwilson,grt,thakis,jamesr,hans,sadrul,pastarmovj Review URL: https://chromiumcodereview.appspot.com/10837044 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@149819 0039d316-1c4b-4281-b951-d872f2087c98
* We should grant read permission to the file returned by CreateFileSnapshot ↵kinuko@chromium.org2012-08-031-0/+18
| | | | | | | | | | | since the file must have given for reading BUG=none Review URL: https://chromiumcodereview.appspot.com/10834138 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@149798 0039d316-1c4b-4281-b951-d872f2087c98
* DownloadItem::Observer::OnDownloadDestroyed() replaces DownloadItem::REMOVINGbenjhayden@chromium.org2012-08-039-53/+84
| | | | | | | | | | | | | | | | | | Reviewers: Ben Goodger: chrome/browser/ui/views/download/download_item_view.*, chrome/browser/plugin_installer.*, chrome/browser/automation/automation_provider.cc rdsmith, asanka: all Nico (thakis): chrome/browser/ui/cocoa/download/download_item_mac.* Achuith: chrome/browser/chromeos/gdata/gdata_download_observer.* Evan (estade): chrome/browser/ui/gtk/download/download_item_gtk.*, chrome/browser/ui/webui/downloads_dom_handler.* Scott (sky): chrome/browser/history/history_unittest.cc Aaron (aa): chrome/browser/extensions/webstore_installer.* Add DownloadItem::Observer::OnDownloadRemoved() to signal when a download is being removed from history. Make chrome.downloads.onErased trigger from DownloadItem::Observer::OnDownloadRemoved() so that extensions don't think that all downloads are being erased just because the browser is closing. Review URL: https://chromiumcodereview.appspot.com/10704026 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@149794 0039d316-1c4b-4281-b951-d872f2087c98
* Rolling libjingle r164.mallinath@chromium.org2012-08-038-91/+1
| | | | | | Review URL: https://chromiumcodereview.appspot.com/10824143 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@149781 0039d316-1c4b-4281-b951-d872f2087c98
* Add a function to layout.h for getting ScaleFactor of the native view.mazda@chromium.org2012-08-031-15/+2
| | | | | | | | BUG=125043 Review URL: https://chromiumcodereview.appspot.com/10823067 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@149775 0039d316-1c4b-4281-b951-d872f2087c98
* Coverity: Fix a sizeof mismatch.jhawkins@chromium.org2012-08-031-1/+1
| | | | | | | | | | | | | | | | | | | It appears the sizeof change was to make the code more future proof: sizeof(float) -> sizeof(audio_data[0]) However, audio_data[0] is a float*, not the intended float. Adding the deref to fix this. CID_COUNT=1 CID=104561 BUG=none R=scherkus Review URL: https://chromiumcodereview.appspot.com/10832123 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@149770 0039d316-1c4b-4281-b951-d872f2087c98
* Safely handle uninitialized load_timing info in ResourceResponseHead.simonjam@chromium.org2012-08-032-34/+121
| | | | | | | | | | | | This is often uninitialized in UI and browser tests. This blocks issue 10829031 from landing. BUG=139093 Review URL: https://chromiumcodereview.appspot.com/10831104 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@149761 0039d316-1c4b-4281-b951-d872f2087c98
* Use empty string as the mid before it's ready from WebKit. Label is not the ↵ronghuawu@chromium.org2012-08-031-3/+10
| | | | | | | | | | | correct mid, so we shouldn't use it, otherwise there will be a mismatch of mid and mline_index. BUG= http://code.google.com/p/webrtc/issues/detail?id=717 TEST=Test page from http://code.google.com/p/webrtc/issues/detail?id=717. Review URL: https://chromiumcodereview.appspot.com/10834137 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@149754 0039d316-1c4b-4281-b951-d872f2087c98
* Disable DumpAccessibilityTreeTest.AccessibilityContenteditableDescendants on ↵dimich@chromium.org2012-08-031-1/+9
| | | | | | | | | | | | | WIN. Started to fail in Chrome r149732 TBR=lambroslambrou BUG=140397 Review URL: https://chromiumcodereview.appspot.com/10836089 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@149753 0039d316-1c4b-4281-b951-d872f2087c98
* Adjust the audio frame count.brettw@chromium.org2012-08-021-1/+1
| | | | | | | | | | | | This sets 2048 to select the high-latency path rather than the low one. This allows us to make Flash use power-of-two buffers and still get the high latency path, avoiding bugs in the current low-latency path. BUG= Review URL: https://chromiumcodereview.appspot.com/10830153 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@149750 0039d316-1c4b-4281-b951-d872f2087c98
* Create a class for seccomp-bpf sandboxing in content.jln@chromium.org2012-08-027-581/+677
| | | | | | | | | | | | | | | This adds a SandboxSeccompBpf class to centralize Seccomp BPF sandbox policies inside of content/ LinuxSandbox is the only user of this class and Linux sandboxing is now fully unified through LinuxSandbox. BUG= NOTRY=true Review URL: https://chromiumcodereview.appspot.com/10843042 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@149738 0039d316-1c4b-4281-b951-d872f2087c98
* Create a LinuxSandbox class.jln@chromium.org2012-08-0214-159/+314
| | | | | | | | | | | | | | | | | | The LinuxSandbox class aims to become the central place for Linux sandboxing inside content/. For now, this refactors mostly code from the Zygote. (Note: this is a re-land of https://chromiumcodereview.appspot.com/10826093/ with a trivial fix for ARM architectures). BUG= TBR=piman@chromium.org NOTRY=true Review URL: https://chromiumcodereview.appspot.com/10843059 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@149734 0039d316-1c4b-4281-b951-d872f2087c98
* Fix a bug that H264BitReader::ReadBits cannot read more than 16 bits.xiaomings@google.com2012-08-021-1/+1
| | | | | | | | TEST=H264BitReaderTest::ReadStreamWithoutEscapeAndTrailingZeroBytes in h264_parser_unittest. Review URL: https://chromiumcodereview.appspot.com/10843057 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@149733 0039d316-1c4b-4281-b951-d872f2087c98
* Remove a ton of IPC from landing putWithIndexKeysalecflett@chromium.org2012-08-0212-146/+11
| | | | | | | | | | | | | | This removes a whole bunch of IPC stubs for methods that don't exist anymore: IDBCursorBackendInterface::update IDBObjectStoreBackendInterface::put IDBTransaction::mode BUG=129471 Review URL: https://chromiumcodereview.appspot.com/10831138 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@149731 0039d316-1c4b-4281-b951-d872f2087c98
* Remove VideoDecoder::natural_size() & added VideoFrame::natural_size().acolwell@chromium.org2012-08-026-72/+84
| | | | | | | | BUG=122913 Review URL: https://chromiumcodereview.appspot.com/10824141 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@149728 0039d316-1c4b-4281-b951-d872f2087c98
* Speculative resource prefetching - fixing missing navigations due to server ↵shishir@chromium.org2012-08-024-2/+6
| | | | | | | | | | side redirects. BUG=140335 Review URL: https://chromiumcodereview.appspot.com/10827103 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@149727 0039d316-1c4b-4281-b951-d872f2087c98
* Add PP_FlashLSORestrictions to the list of settings supported by the ↵bauerb@chromium.org2012-08-027-0/+75
| | | | | | | | | | Flash_GetSetting Pepper API. BUG=132410 Review URL: https://chromiumcodereview.appspot.com/10831045 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@149711 0039d316-1c4b-4281-b951-d872f2087c98
* Wire up the deleteFileSystem operation.nhiroki@google.com2012-08-027-0/+70
| | | | | | | | | BUG=139366 TEST="Will be tested from WebKit inspector test" Review URL: https://chromiumcodereview.appspot.com/10828043 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@149707 0039d316-1c4b-4281-b951-d872f2087c98
* Revert 149692 - Create a LinuxSandbox class.mazda@chromium.org2012-08-0214-311/+159
| | | | | | | | | | | | | | | | | | | | r149692 broke ChromiumOS (tegra2) build. The LinuxSandbox class aims to become the central place for Linux sandboxing inside content/. For now, this refactors mostly code from the Zygote. BUG= NOTRY=true Review URL: https://chromiumcodereview.appspot.com/10826093 TBR=jln@chromium.org Review URL: https://chromiumcodereview.appspot.com/10837081 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@149705 0039d316-1c4b-4281-b951-d872f2087c98
* Revert 149508 - Always use non-low-latency (wave out) audio for Pepper ↵viettrungluu@chromium.org2012-08-021-9/+0
| | | | | | | | | | | | | | | | | | | | | plugins on Windows. [Because the non-low-latency path is so awful, especially on Vista.] (In particular, for Pepper Flash.) The intent is to try this out on Canary for M21, merge it to M21 if it looks good, and then revert this change. BUG=139953 TBR=cpu,brettw Review URL: https://chromiumcodereview.appspot.com/10828124 TBR=viettrungluu@chromium.org Review URL: https://chromiumcodereview.appspot.com/10823148 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@149699 0039d316-1c4b-4281-b951-d872f2087c98
* Create a LinuxSandbox class.jln@chromium.org2012-08-0214-159/+311
| | | | | | | | | | | | | | | The LinuxSandbox class aims to become the central place for Linux sandboxing inside content/. For now, this refactors mostly code from the Zygote. BUG= NOTRY=true Review URL: https://chromiumcodereview.appspot.com/10826093 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@149692 0039d316-1c4b-4281-b951-d872f2087c98
* Android: replace temporary test runner with the instrumentation runner.bulach@chromium.org2012-08-021-55/+0
| | | | | | | | | | | | Follow up from http://codereview.chromium.org/10826071/ BUG=139365 TEST=Android FYI tester runs instrumentation tests. Review URL: https://chromiumcodereview.appspot.com/10828094 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@149679 0039d316-1c4b-4281-b951-d872f2087c98
* Start upstreaming accessibility.dtrainor@chromium.org2012-08-027-10/+602
| | | | | | | | | BUG=http://crbug.com/138218 Review URL: https://chromiumcodereview.appspot.com/10832104 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@149667 0039d316-1c4b-4281-b951-d872f2087c98
* Changes ProviderImpl to use a single DataFetcherFactory instead of aaousterh@chromium.org2012-08-024-29/+19
| | | | | | | | | | | | | | | vector of DataFetcherFactories In the current implementation, any OS can only use at most one DataFetcher, so there will always be 0 or 1 DataFetcherFactories in the vector. Therefore it is simpler to use a single DataFetcherFactory instead of an array. Since DataFetcherFactory is a pointer itself, it can be NULL when there are no available DataFetchers. BUG=none Review URL: https://chromiumcodereview.appspot.com/10837055 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@149614 0039d316-1c4b-4281-b951-d872f2087c98
* SpeechRecognitionDispatcher: break out translation of error codes to ↵hans@chromium.org2012-08-022-26/+25
| | | | | | | | | | | | | separate function Also remove stale comment from speech_recognition_error.h. BUG=116954 Review URL: https://chromiumcodereview.appspot.com/10831110 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@149612 0039d316-1c4b-4281-b951-d872f2087c98
* Reland 149510 - Allow filters in accessibility tests to specify which ↵pkasting@chromium.org2012-08-0222-234/+448
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | attributes to check. Rolled this out in hopes of stopping some crashes, but it didn't help, so relanding. With this change, a DumpAccessibilityTree test can specify filters to control which attributes are printed. This keeps each test expectation file small and readable, while making it easy to test for all sorts of attributes, including obscure ones. Each platform will have a few attributes that print by default, like the role and name. HTML files add rules of this form to override the defaults: @MAC-DENY:subrole* @MAC-DENY:value* @MAC-ALLOW:description* You can also specify @MAC-ALLOW:* or @WIN-ALLOW:* if you want to dump everything - helpful during development sometimes. BUG=124314 Review URL: https://chromiumcodereview.appspot.com/10662003 TBR=dmazzoni@chromium.org Review URL: https://chromiumcodereview.appspot.com/10837065 TBR=pkasting@chromium.org Review URL: https://chromiumcodereview.appspot.com/10830133 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@149573 0039d316-1c4b-4281-b951-d872f2087c98
* Use JavaObjectWeakGlobalRef in ContentViewCore.nileshagrawal@chromium.org2012-08-024-43/+62
| | | | | | | | | | | | | | Java ContentViewCore object may be garbage collected without first destroying the native object. Because of this, all C++->Java calls to ContentViewCore must be guarded with a null check. BUG=139386,138237 Review URL: https://chromiumcodereview.appspot.com/10827125 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@149570 0039d316-1c4b-4281-b951-d872f2087c98
* Implement media path filtertzik@chromium.org2012-08-021-0/+1
| | | | | | | | | | BUG=137670 TEST='NativeMediaFileUtilTest.*' Review URL: https://chromiumcodereview.appspot.com/10825042 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@149566 0039d316-1c4b-4281-b951-d872f2087c98
* Revert 149510 - Allow filters in accessibility tests to specify which ↵pkasting@chromium.org2012-08-0222-448/+234
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | attributes to check. This may have caused the Win 7 WebKit canary bot to begin crashing; see http://test-results.appspot.com/dashboards/flakiness_dashboard.html#tests=accessibility%2Floading-iframe-updates-axtree.html If rolling this out does not fix the bot, I will put it back in later tonight. With this change, a DumpAccessibilityTree test can specify filters to control which attributes are printed. This keeps each test expectation file small and readable, while making it easy to test for all sorts of attributes, including obscure ones. Each platform will have a few attributes that print by default, like the role and name. HTML files add rules of this form to override the defaults: @MAC-DENY:subrole* @MAC-DENY:value* @MAC-ALLOW:description* You can also specify @MAC-ALLOW:* or @WIN-ALLOW:* if you want to dump everything - helpful during development sometimes. BUG=124314 Review URL: https://chromiumcodereview.appspot.com/10662003 TBR=dmazzoni@chromium.org Review URL: https://chromiumcodereview.appspot.com/10837065 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@149550 0039d316-1c4b-4281-b951-d872f2087c98
* Make RenderProcessImpl::CreateTransportDIB and ↵fsamuel@chromium.org2012-08-025-9/+16
| | | | | | | | | | | | RenderProcessImpl::FreeTransportDIB public This code allows for platform-specific ways to allocate and free TransportDIBs. This is needed by the BrowserPlugin. Make them public so that they can be accessed by the BrowserPlugin BUG=none Review URL: https://chromiumcodereview.appspot.com/10826100 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@149538 0039d316-1c4b-4281-b951-d872f2087c98