| Commit message (Collapse) | Author | Age | Files | Lines |
|
|
|
|
|
|
|
| |
and copy Purify's list of leaky net tests for Valgrind's benefit.
Review URL: http://codereview.chromium.org/42309
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@12042 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
| |
was that the wrong pid was being sent to RDH from RMF. It needs to be the fake pid for the in-process renderer.
BUG=8890
Review URL: http://codereview.chromium.org/42366
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@12040 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
| |
Review URL: http://codereview.chromium.org/42311
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@12038 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
builds. In debug builds, the method will DCHECK if the
current thread does not hold the lock.
I also removed a class that was not being used (AutoLockImpl) which seems like someone just copied the Lock.h file and blindly added
an Impl for the AutoLock class.
Thre are places in my code where I want to use the AutoUnlock class. I can't see why it was restricted to use by the condition variable
class only, so I just made it into a regular class with no restrictions.
I noticed that JamesR posted a CL about a month ago that made AutoUnlock a public class, but it was never committed, so I added him to this CL for review too.
Review URL: http://codereview.chromium.org/48109
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@12037 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
| |
(found by Coverity)
Review URL: http://codereview.chromium.org/42370
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@12035 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
minimized
browser window.
BUG=4557 ( http://crbug.com/4557 )
Original patch by Mohamed Mansour <m0.interactive@gmail.com> in
http://codereview.chromium.org/42208/show
Review URL: http://codereview.chromium.org/48167
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@12034 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
| |
Review URL: http://codereview.chromium.org/42344
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@12033 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
| |
There are cases where the renderer asks for the root rectangle of its
window before we have a window for it. Since we still have the
NativeViewId hack in place, this previously caused a crash.
Review URL: http://codereview.chromium.org/42356
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@12032 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
LayoutTests/http/tests/xmlhttprequest/web-apps/013.html. We're now
failing this because of differences in how lighthttpd behaves vs
apache. I'ved filed
http://code.google.com/p/chromium/issues/detail?id=8941 to track it.
BUG=none
TEST=none
TBR=finnur
Review URL: http://codereview.chromium.org/48159
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@12031 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
| |
(found by Coverity)
Review URL: http://codereview.chromium.org/42369
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@12030 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
| |
new_name_count_ is unsigned, so checking for >= 0 is useless.
(Found by Coverity)
Review URL: http://codereview.chromium.org/48152
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@12029 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
| |
Review URL: http://codereview.chromium.org/48155
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@12028 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
| |
(found by Coverity)
Review URL: http://codereview.chromium.org/42367
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@12027 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
| |
Review URL: http://codereview.chromium.org/48154
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@12026 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
| |
(Coverity discovered it.)
Review URL: http://codereview.chromium.org/49002
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@12025 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
| |
Review URL: http://codereview.chromium.org/42363
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@12024 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
| |
TBR=jeremy
Review URL: http://codereview.chromium.org/48165
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@12023 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
chrome/browser/renderer_host/resource_dispatcher_host_unittest.cc
is not updated when |request_context| is added to
ViewHostMsg_Resource_Request.
In r11816 CreateResourceRequest() is used to prepare
ViewHostMsg_Resource_Request with |request_context| uninitialized,
giving the purify bot red. Initializing the variable should fix the
UMR error.
Review URL: http://codereview.chromium.org/42360
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@12021 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
| |
TBR=ojan
Review URL: http://codereview.chromium.org/48158
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@12019 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
| |
TBR=ojan
Review URL: http://codereview.chromium.org/42365
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@12018 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
| |
TBR=ojan
Review URL: http://codereview.chromium.org/42364
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@12017 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
| |
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@12016 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
| |
TBR=finnur
Review URL: http://codereview.chromium.org/48153
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@12015 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
(tab) to change Search Engine. Not a good first impression.
BUG=4688
TEST=Open Chrome using --first-run, press Cancel on the
import dialog and watch the First Run bubble come up. Use Tab
and then Enter and the Change Search Engine should appear.
Then try it again, but this time click outside the bubble to
dismiss it (corner case that could lead to a crash).
Review URL: http://codereview.chromium.org/48138
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@12014 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
| |
R=ojan
Review URL: http://codereview.chromium.org/42352
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@12013 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
| |
I had assuming that we always scroll the whole window; I was wrong.
This patch fixes issues with 'ghost' scrollbars appearing when the
window looses focus and with huge misrendering when scrolling iframes.
Review URL: http://codereview.chromium.org/42357
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@12012 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
| |
The test fails in webkit and the expected.txt contains failure
output. We work fine so just rebaseline till it is fixed upstream.
Review URL: http://codereview.chromium.org/42353
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@12010 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
for DownloadDatabase, InMemoryDatabase, and RedirectUiTest. Except for in those
files, we now only use std::wstring for text strings.
This CL also enables two more unittests on Mac.
patch by Rohit Rao.
http://codereview.chromium.org/48018
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@12009 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
| |
http://dev.chromium.org/developers/design-documents/native-controls
Review URL: http://codereview.chromium.org/48094
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@12008 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
| |
- share more code between different platforms
- put more code inside anonymous namespace
- add more DISALLOW_COPY_AND_ASSIGNs
- small #include cleanup
Review URL: http://codereview.chromium.org/42343
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@12007 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
I don't understand how the test could have worked because
if you don't specify a CODEBASE for the Java Applet the
default is the url. In the Java console I was seeing it
try to load the class by appending the TestApplet to the
path to the html file (the layout test file). By
specifying "." as the CODEBASE it is able to find and
load the class that resides in the same directory and
all is well.
Review URL: http://codereview.chromium.org/48148
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@12005 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
| |
Review URL: http://codereview.chromium.org/48146
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@12004 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
| |
Review URL: http://codereview.chromium.org/42350
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@12003 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
| |
Review URL: http://codereview.chromium.org/42338
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@12002 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
| |
- Add G_DEBUG=fatal_warnings to abort on GTK assertions.
- Find the suppressions file no matter where you run the script from.
- Add +x on the script.
Review URL: http://codereview.chromium.org/45011
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@12001 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
The test tries to load a file:/// URL from a data: URL. This isn't allowed
in Chromium but it is in Safari. The test really doesn't have anything to
do with security checks so we should fix the test upstream instead.
I'm deleting the existing pixel baseline because they're wrong (missing the image because of the security check mentioned above).
Review URL: http://codereview.chromium.org/42349
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@12000 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Fixes
244 bytes in 5 blocks are possibly lost in loss record 57 of 97
at operator new(unsigned int) m_replacemalloc/vg_replace_malloc.c:230
by std::string::_Rep::_S_create(unsigned int, unsigned int, std::allocator<char> const&) (in libstdc++.so.6.0.9)
by (within libstdc++.so.6.0.9)
by std::basic_string<char, std::char_traits<char>, std::allocator<char> >::basic_string(char const*, std::allocator<char> const&) (in libstdc++.so.6.0.9)
by CommandLine::CommandLine(int, char const* const*) src/base/command_line.cc:96
by CommandLine::Init(int, char const* const*) src/base/command_line.cc:177
by TestSuite::TestSuite(int, char**) src/base/test_suite.h:34
Review URL: http://codereview.chromium.org/42341
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@11999 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
| |
Also ensure that Chromium.app quits properly between the tests.
http://crbug.com/8391
Review URL: http://codereview.chromium.org/42306
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@11998 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
| |
other fixes related to the new eviction code. As before,
almost everything is disabled by default.
Review URL: http://codereview.chromium.org/48112
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@11995 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
| |
This contains the new compiler infrastructure which gives a nice performance improvement on the V8 benchmark suite.
Review URL: http://codereview.chromium.org/48143
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@11994 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
| |
flips page. Rebaseline on linux which matches win in metrics but
the image differs due to anti-aliased fonts and the scrollbar.
Review URL: http://codereview.chromium.org/48145
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@11993 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
| |
TBR=jkhawkins
Review URL: http://codereview.chromium.org/42347
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@11992 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
| |
These are used for <select> html items, for example.
Review URL: http://codereview.chromium.org/48054
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@11991 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
| |
tbr=avi
Review URL: http://codereview.chromium.org/48139
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@11988 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
| |
perform autocomplete.
This matches FF behavior.
BUG=6310
TEST=See bug.
Review URL: http://codereview.chromium.org/48033
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@11987 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
| |
whitespaces before checking.
Review URL: http://codereview.chromium.org/42316
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@11986 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
| |
hermetic cygwin in setup_env.bat
Review URL: http://codereview.chromium.org/48137
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@11985 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
| |
list. This test hasn't failed in a long time.
BUG=none
TEST=none
TBR=tc
Review URL: http://codereview.chromium.org/42342
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@11984 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
| |
loop if we were ever to get a "chrome_config" node back in metrics response data.
Review URL: http://codereview.chromium.org/42335
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@11983 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
| |
This belongs as a private method of BrowserView. Neither mac nor linux uses it.
Review URL: http://codereview.chromium.org/42298
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@11982 0039d316-1c4b-4281-b951-d872f2087c98
|