- Dec 06, 2010
-
-
jshin@chromium.org authored
With -webkit-{margin,padding}-{start,end}, a lot of dir=rtl pseudo-selectors can be removed. While I'm at it, I fixed one glitch in the history page where 'text-align: right' is hard-coded instead of 'text-align: end' BUG=NONE TEST=extension/download/plugin/history and DOM UI pages are laid out as before both in LTR CHrome and RTL Chrome. Review URL: http://codereview.chromium.org/5217005 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@68364 0039d316-1c4b-4281-b951-d872f2087c98
-
scherkus@chromium.org authored
Previously we were using it as a DCHECK but we would fail gracefully. Now that stopped_on_render_loop_ is set via Abort() (i.e., catastrophic failure), we need to use it as a singal to make sure all work on the render loop does indeed stop. BUG=16751 TEST=http/tests/media should stop crashing Review URL: http://codereview.chromium.org/5603004 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@68363 0039d316-1c4b-4281-b951-d872f2087c98
-
nduca@chromium.org authored
If compositor window already exists, then simply return it rather than complaining. May affect how bug 65194 manifests. BUG=65194 TEST=Verify against WebGL demos, poster circle content with tab opening and closing, as well as killing of GPU process. Review URL: http://codereview.chromium.org/5528007 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@68362 0039d316-1c4b-4281-b951-d872f2087c98
-
isherman@chromium.org authored
BUG=65129 TEST=unit_tests --gtest_filter=WebDatabaseTest.Update* Review URL: http://codereview.chromium.org/5610002 TBR=isherman@chromium.org Review URL: http://codereview.chromium.org/5515006 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@68360 0039d316-1c4b-4281-b951-d872f2087c98
-
ananta@chromium.org authored
TBR=amit BUG=none TEST=none Review URL: http://codereview.chromium.org/5515005 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@68359 0039d316-1c4b-4281-b951-d872f2087c98
-
isherman@chromium.org authored
BUG=65129 TEST=unit_tests --gtest_filter=WebDatabaseTest.Update* Review URL: http://codereview.chromium.org/5610002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@68358 0039d316-1c4b-4281-b951-d872f2087c98
-
victorw@chromium.org authored
TBR=yutak TESt=none BUG=65404 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@68357 0039d316-1c4b-4281-b951-d872f2087c98
-
gene@google.com authored
Make http connection for cupsGetPPD non-blocking to prevent that. BUG=none TEST=Make sure GettPPD does not get stuck. Review URL: http://codereview.chromium.org/5609004 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@68356 0039d316-1c4b-4281-b951-d872f2087c98
-
bauerb@chromium.org authored
This is a redesign for http://codereview.chromium.org/4852002/ following discussions. The goal of this patch is to ensure that Chrome maintains the order in which extensions apply their preferences between restarts. This may be an issue if two extensions overwrite each others preferences. Furthermore, it ensures that preferences are persisted to disk between browser restarts. Therefore, previous settings are immediately available when the browser is restarted. A description of the design has been published https://docs.google.com/a/google.com/document/d/1E_HX_cUpET1gH2gDunGIU1EOywMM6FEOuVU6TlpnSwo/edit?hl=en for review and comments. - Sorry, accessible Google internally only. This CL requires http://codereview.chromium.org/5204006/ Contributed by battre@google.com BUG=50726 TEST=none Review URL: http://codereview.chromium.org/5213002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@68355 0039d316-1c4b-4281-b951-d872f2087c98
-
sky@chromium.org authored
instant. BUG=65206 TEST=see bug Review URL: http://codereview.chromium.org/5631001 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@68354 0039d316-1c4b-4281-b951-d872f2087c98
-
victorw@chromium.org authored
TBR=yutak TEST=none BUG=none git-svn-id: svn://svn.chromium.org/chrome/trunk/src@68353 0039d316-1c4b-4281-b951-d872f2087c98
-
kbr@google.com authored
implementation via new glGetRequestableExtensionsCHROMIUM and glRequestExtensionCHROMIUM entry points. These entry points are needed in order to allow WebGL to both query the available extensions and enable them individually. Added these entry points to WebGraphicsContext3DCommandBufferImpl. A subsequent WebKit checkin under https://bugs.webkit.org/show_bug.cgi?id=40316 will utilize them and implement the OES_texture_float extension for WebGL. BUG=none TEST=none (ran with new oes-texture-float.html WebGL conformance test) Review URL: http://codereview.chromium.org/5626008 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@68352 0039d316-1c4b-4281-b951-d872f2087c98
-
joth@chromium.org authored
I missed some comments from wtc on patch http://codereview.chromium.org/5195001 before landing, so following up here. BUG=None TEST=None Review URL: http://codereview.chromium.org/5587001 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@68351 0039d316-1c4b-4281-b951-d872f2087c98
-
bauerb@chromium.org authored
contributed by battre@google.com BUG=none TEST=none Review URL: http://codereview.chromium.org/5204006 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@68350 0039d316-1c4b-4281-b951-d872f2087c98
-
bauerb@chromium.org authored
Fix for the following bug: [11725:11725:1206/035953:10306309386:ERROR:base/process_util_posix.cc(105)] Received signal 11 base::debug::StackTrace::StackTrace() [0x1f68261] base::(anonymous namespace)::StackDumpSignalHandler() [0x1fa2853] 0x2ba8f9b0f100 __pthread_mutex_lock_internal [0x2ba8f6dce423] LockImpl::Lock() [0x1f7a9f6] Lock::Acquire() [0x58a947] AutoLock::AutoLock() [0x58a979] MessageLoop::PostTask_Helper() [0x1f7e2cc] MessageLoop::PostDelayedTask() [0x1f7e40d] WebResourceService::ScheduleNotification() [0x158db6f] WebResourceService::UnpackPromoSignal() [0x158e1db] WebResourceServiceTest_UnpackPromoSignal_Test::TestBody() [0xcbb723] testing::HandleExceptionsInMethodIfSupported<>() [0x1b6ff6b] testing::Test::Run() [0x1b72cb9] testing::TestInfo::Run() [0x1b76548] testing::TestCase::Run() [0x1b7669b] testing::internal::UnitTestImpl::RunAllTests() [0x1b771f0] testing::HandleExceptionsInMethodIfSupported<>() [0x1b6fd4f] testing::UnitTest::Run() [0x1b6fdb7] base::TestSuite::Run() [0x33ed0a8] main [0x1a4f56d] 0x2ba8f9afb1c4 0x433829 Program received signal SIGSEGV, Segmentation fault. __pthread_mutex_lock (mutex=0x170) at pthread_mutex_lock.c:50 50 pthread_mutex_lock.c: No such file or directory. in pthread_mutex_lock.c (gdb) bt #0 __pthread_mutex_lock (mutex=0x170) at pthread_mutex_lock.c:50 #1 0x00007ffff5ac327d in LockImpl::Lock (this=0x170) at base/lock_impl_posix.cc:41 #2 0x00007ffff5a85dee in Lock::Acquire (this=0x168) at ./base/lock.h:40 #3 0x00007ffff5a85e50 in AutoLock::AutoLock (this=0x7fffffffd450, lock=...) at ./base/lock.h:95 #4 0x00007ffff5ac7a16 in MessageLoop::PostTask_Helper (this=0x0, from_here=..., task=0x7fffe29a8e00, delay_ms=1974154931, nestable=true) at base/message_loop.cc:378 #5 0x00007ffff5ac7816 in MessageLoop::PostDelayedTask (this=0x0, from_here=..., task=0x7fffe29a8e00, delay_ms=1974154931) at base/message_loop.cc:318 #6 0x00007ffff77009e1 in WebResourceService::ScheduleNotification (this=0x7fffe41ed4e0, promo_start=1264899600, promo_end=1327971600) at chrome/browser/web_resource/web_resource_service.cc:285 Note how this is NULL in MessageLoop::PostDelayedTask. Patch from Dominic Battré (battre@google.com) BUG=65462 TEST=none TBR=mirandac Review URL: http://codereview.chromium.org/5559007 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@68349 0039d316-1c4b-4281-b951-d872f2087c98
-
gfeher@chromium.org authored
Don't test the killbit in each ExternalExtensionProvider, but test it in ExternalExtensionsService. BUG=65107 TEST=ExtensionManagementTest.ExternalUpdate Review URL: http://codereview.chromium.org/5535002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@68348 0039d316-1c4b-4281-b951-d872f2087c98
-
finnur@chromium.org authored
Address review comments from Nico and add a separating line as requested by Glen. BUG=51105 TEST=None Review URL: http://codereview.chromium.org/5640001 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@68346 0039d316-1c4b-4281-b951-d872f2087c98
-
yutak@chromium.org authored
TBR=victorw BUG=none TEST=none Review URL: http://codereview.chromium.org/5653001 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@68345 0039d316-1c4b-4281-b951-d872f2087c98
-
avayvod@chromium.org authored
BUG=chromium-os:7295 TEST=Login with a new user, lock screen on profile picture step, sign out, login as the same user, verify there's no crash infobar shown. Review URL: http://codereview.chromium.org/5600001 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@68344 0039d316-1c4b-4281-b951-d872f2087c98
-
pastarmovj@chromium.org authored
Changed the static call of SQLitePersistantCookieStore::ClearLocalState to a call of a virtual method PersistantCookieStore::ClearLocalState from inside the destruction sequence of the owning UrlRequestContext. Thus the code will now work with any other persistancy implementation and allow for better parallelism. To test. Enable deleting of cookies and user data on shutdown and check if the Cookies file in the profile directory has been deleted. BUG=64920 TEST=Manual. Review URL: http://codereview.chromium.org/5430004 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@68343 0039d316-1c4b-4281-b951-d872f2087c98
-
chrome-release@google.com authored
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@68339 0039d316-1c4b-4281-b951-d872f2087c98
-
kochi@chromium.org authored
The description for IDS_ABOUT_PAGE_TITLE (Title used for about options panel) was ambiguous enough to confuse translators: 1. Title used for <about> <options panel> (original intention) 2. Title used for <about options> panel (misunderstanding) And unfortunately for Japanese, 2 was translated. 2 ways to fix this, clarify the description or use IDS_ABOUT. As title on the wrench menu use IDS_ABOUT ("About &PRODUCT_NAME"), using IDS_ABOUT for the title for the options panel makes sense. BUG=none TEST=manually open chrome://settings/about (or wrench menu -> about) and see tht title displayed properly. Review URL: http://codereview.chromium.org/5567001 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@68336 0039d316-1c4b-4281-b951-d872f2087c98
-
thestig@chromium.org authored
BUG=51822 TEST=Webkit Linux (valgrind layout) goes green. TBR=evan Review URL: http://codereview.chromium.org/5586005 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@68335 0039d316-1c4b-4281-b951-d872f2087c98
-
yutak@chromium.org authored
TBR=victorw BUG=none TEST=none Review URL: http://codereview.chromium.org/5606006 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@68334 0039d316-1c4b-4281-b951-d872f2087c98
-
rsleevi@chromium.org authored
When viewing a certificate on Linux, if the certificate has a CRL Distribution Point extension, display all of the URIs contained, rather than only the first. BUG=64565 TEST=none Review URL: http://codereview.chromium.org/5372003 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@68333 0039d316-1c4b-4281-b951-d872f2087c98
-
rsleevi@chromium.org authored
When using NSS for certificate verification, add a check when fetching CRLs/OCSP responses to prevent redirects to non-HTTP URLs. This matches the initial check when first called from NSS to create the URLRequest. In particular, fetching a CRL/OCSP response over HTTPS is troublesome, as the certificate sent by the responder may also need revocation checking, potentially causing revocation checking loops. The existing check only considered the initial URL scheme supplied by NSS. However, if the server issues a redirect, the new URL scheme scheme was not filtered. BUG=64521 TEST=none Review URL: http://codereview.chromium.org/5372002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@68329 0039d316-1c4b-4281-b951-d872f2087c98
-
- Dec 05, 2010
-
-
creis@google.com authored
BUG=31526 BUG=60840 TEST=TaskManagerBrowserTest.NoticeExtensionTabs TEST=TaskManagerBrowserTest.NoticeAppTabs Review URL: http://codereview.chromium.org/5526002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@68324 0039d316-1c4b-4281-b951-d872f2087c98
-
thestig@chromium.org authored
BUG=none TEST=none Review URL: http://codereview.chromium.org/5512009 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@68323 0039d316-1c4b-4281-b951-d872f2087c98
-
rsleevi@chromium.org authored
Cleanup X509CertificateMac style nits and introduce a crypto library agnostic means of parsing certificate dates. In the process of addressing style issues, a bug was found in the date/time parsing routines used on Mac. Though not currently hit, it could lead to wild misrepresentation of the validity period for a certificate. Switch to using the routine used when using OpenSSL, which does things correctly. BUG=none TEST=existing Review URL: http://codereview.chromium.org/4653002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@68322 0039d316-1c4b-4281-b951-d872f2087c98
-
thakis@chromium.org authored
It's now no longer allowed to do network requests and can't access most files. Here are the stacks that it prints if I patch in http://codereview.chromium.org/1765005/show: http://codepad.org/6zrJfnlB BUG=48607 TEST=GPU stuff still works. When you run with --enable-sandbox-logging, quite a bunch of stuff is logged as "denied". Review URL: http://codereview.chromium.org/5580002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@68321 0039d316-1c4b-4281-b951-d872f2087c98
-
jhawkins@chromium.org authored
TBR=thestig BUG=none TEST=none Review URL: http://codereview.chromium.org/5637003 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@68320 0039d316-1c4b-4281-b951-d872f2087c98
-
mad@google.com authored
Yet another attempt at landing http://src.chromium.org/viewvc/chrome?view=rev&revision=68298 : "Make sure we don't keep tab id - handles mapping when a dread^Wthread dies." And hopefully won't hit http://code.google.com/p/chromium/issues/detail?id=63479 this time... TBR=joi TEST=None BUG=65153 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@68319 0039d316-1c4b-4281-b951-d872f2087c98
-
mad@google.com authored
TBR=Joi BUG=None TEST=We need to fix http://crbug.com/63479 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@68318 0039d316-1c4b-4281-b951-d872f2087c98
-
http://src.chromium.org/viewvc/chrome?view=rev&revision=68298mad@google.com authored
Make sure we don't keep tab id - handles mapping when a dread^Wthread dies. TBR=joi BUG=65153 TEST=Just make sure it builds... git-svn-id: svn://svn.chromium.org/chrome/trunk/src@68315 0039d316-1c4b-4281-b951-d872f2087c98
-
jhawkins@chromium.org authored
Also fix up some style issues. CID=13912,13971,13977 BUG=none TEST=none Review URL: http://codereview.chromium.org/5618002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@68313 0039d316-1c4b-4281-b951-d872f2087c98
-
petermayo@chromium.org authored
directory in the root. TEST=Locally on a bot like machine (no prior fix). Try bots (with chmod /var/run). BUG=none Review URL: http://codereview.chromium.org/5617004 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@68312 0039d316-1c4b-4281-b951-d872f2087c98
-
mad@google.com authored
As described in more details in the bug description, if we use COM for these and RPC for the other events, then they could arrive out of order. BUG=65316 TEST=Create as many tabs as quickly as possible and watch the logs (sawbuck) for errors. Review URL: http://codereview.chromium.org/5647001 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@68311 0039d316-1c4b-4281-b951-d872f2087c98
-
sadrul@chromium.org authored
For the menus to work correctly, the X messge pump also needs updating to allow nested event dispatching. BUG=None TEST=On a touchui build, non-GTK context menus in web-pages should work correctly. Review URL: http://codereview.chromium.org/5110011 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@68309 0039d316-1c4b-4281-b951-d872f2087c98
-
agl@chromium.org authored
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@68303 0039d316-1c4b-4281-b951-d872f2087c98
-
- Dec 04, 2010
-
-
tfarina@chromium.org authored
- Remove 'Select trusted SSL certificates' text. BUG=63841 TEST=visual Review URL: http://codereview.chromium.org/5573008 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@68301 0039d316-1c4b-4281-b951-d872f2087c98
-