- Dec 10, 2010
-
-
evan@chromium.org authored
BUG=23581 Review URL: http://codereview.chromium.org/5742006 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@68875 0039d316-1c4b-4281-b951-d872f2087c98
-
cbentzel@chromium.org authored
BUG=65993 TEST=unit_tests --gtest_filter="*Prerender*" Review URL: http://codereview.chromium.org/5693001 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@68874 0039d316-1c4b-4281-b951-d872f2087c98
-
thakis@chromium.org authored
Defined PNG_NO_READ_SWAP, PNG_NO_WRITE_SWAP, PNG_NO_READ_oFFs, PNG_NO_WRITE_oFFs, PNG_NO_USER_LIMITS, PNG_NO_SET_USER_LIMITS, PNG_NO_TIME_RFC1123. Clarified that PNG_NO_INFO_IMAGE, PNG_NO_PROGRESSIVE_READ are undefined. Used PNG_NO_SEQUENTIAL_READ instead of PNG_NO_SEQUENTIAL_READ_SUPPORTED. Defined PNG_NO_WRITE_gAMA, PNG_NO_WRITE_sRGB. Clarified that PNG_NO_READ_gAMA and PNG_NO_READ_sRGB are undefined (i.e. gAMA and sRGB are retrieved by libpng, along with iCCP). Fixed typo in the mangling of png_set_text_2. Renamed the prefix webkit_ to wk_ in mangled names. Restored the full function name in mangling; e.g., png_create_read_struct is mangled as wk_png_create_read_struct instead of wk_png_cr_read_str. Added mangling for all symbols with external linkage that are never macros. Added explicit #undef for symbols that may sometimes be macros in png.h: png_benign_error, png_chunk_benign_error, png_get_int_32, png_get_uint_16, png_get_uint_32, png_info_init, png_read_init, png_write_init. BUG=none TEST=none Review URL: http://codereview.chromium.org/5689001 Patch from Cosmin Truta <ctruta@chromium.org>. git-svn-id: svn://svn.chromium.org/chrome/trunk/src@68873 0039d316-1c4b-4281-b951-d872f2087c98
-
mirandac@chromium.org authored
BUG=http://code.google.com/p/chromium-os/issues/detail?id=10031 TEST=cros sync works Review URL: http://codereview.chromium.org/5685004 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@68872 0039d316-1c4b-4281-b951-d872f2087c98
-
shess@chromium.org authored
Previously, the profile-locking code was conservative in the face of errors. Histograms indicate that the error rate is acceptable (no errors seen), so flip locking errors to me failure-to-lock. BUG=58986 TEST=none Review URL: http://codereview.chromium.org/5717002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@68871 0039d316-1c4b-4281-b951-d872f2087c98
-
jam@chromium.org authored
Review URL: http://codereview.chromium.org/5541005 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@68870 0039d316-1c4b-4281-b951-d872f2087c98
-
jam@chromium.org authored
BUG=65395 Review URL: http://codereview.chromium.org/5774001 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@68868 0039d316-1c4b-4281-b951-d872f2087c98
-
cdn@chromium.org authored
Committing for tsepez. Original code review is here http://codereview.chromium.org/5682002/ BUG=49747 TEST=WordIteratorTest.* Review URL: http://codereview.chromium.org/5718003 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@68867 0039d316-1c4b-4281-b951-d872f2087c98
-
cmasone@chromium.org authored
Finish taking new SignedSettings::ReturnCode values all the way out to consumers of the SignedSettingsHelper API BUG=chromium-os:9666 TEST=unit tests, install on device, ensure that Guest mode setting can be updated and is honored. Review URL: http://codereview.chromium.org/5671003 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@68866 0039d316-1c4b-4281-b951-d872f2087c98
-
phajdan.jr@chromium.org authored
TBR=bulach BUG=66400 TEST=browser_tests Review URL: http://codereview.chromium.org/5739002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@68863 0039d316-1c4b-4281-b951-d872f2087c98
-
phajdan.jr@chromium.org authored
TBR=jorlow BUG=66394 TEST=browser_tests Review URL: http://codereview.chromium.org/5730002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@68862 0039d316-1c4b-4281-b951-d872f2087c98
-
lzheng@chromium.org authored
BUG=64089 TEST=none Review URL: http://codereview.chromium.org/5750001 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@68861 0039d316-1c4b-4281-b951-d872f2087c98
-
phajdan.jr@chromium.org authored
Some FTP servers send MLSD listings with no facts, just the file names. This is a valid RFC-3659 listing, but is useless for web browsers. In theory we could check whether a server advertises MLSD support in FEAT response, or use LIST when MLSD only returns file names, but the extra complexity is not worth it. All servers supporting MLSD must also support LIST, so this should not regress compatibility. This change also removes the "HPRC" directory listing parser. Actually, it was just MLSD listing with no facts. BUG=65328, 65584 TEST=net_unittests, also see the bugs Review URL: http://codereview.chromium.org/5669001 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@68859 0039d316-1c4b-4281-b951-d872f2087c98
-
rsesek@chromium.org authored
BUG=24255 TEST=none Review URL: http://codereview.chromium.org/5695006 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@68858 0039d316-1c4b-4281-b951-d872f2087c98
-
pfeldman@chromium.org authored
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@68854 0039d316-1c4b-4281-b951-d872f2087c98
-
bauerb@chromium.org authored
BUG=65859 TEST=yes please Review URL: http://codereview.chromium.org/5519015 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@68853 0039d316-1c4b-4281-b951-d872f2087c98
-
cbentzel@chromium.org authored
BUG=66102 TEST=None TBR=danno@chromium.org Review URL: http://codereview.chromium.org/5731004 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@68852 0039d316-1c4b-4281-b951-d872f2087c98
-
hans@chromium.org authored
Depends on WebKit r73697. BUG=65619 TEST=browser_tests Review URL: http://codereview.chromium.org/5695002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@68851 0039d316-1c4b-4281-b951-d872f2087c98
-
cbentzel@chromium.org authored
BUG=66227 TEST=None TBR=panayiotis@google.com Review URL: http://codereview.chromium.org/5771002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@68850 0039d316-1c4b-4281-b951-d872f2087c98
-
pfeldman@chromium.org authored
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@68845 0039d316-1c4b-4281-b951-d872f2087c98
-
danno@chromium.org authored
BUG=66102, 66054 TEST=valgrind on linux and mac Review URL: http://codereview.chromium.org/5748002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@68844 0039d316-1c4b-4281-b951-d872f2087c98
-
jochen@chromium.org authored
BUG=66277 TEST=less redness on memory bots Review URL: http://codereview.chromium.org/5704002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@68843 0039d316-1c4b-4281-b951-d872f2087c98
-
altimofeev@chromium.org authored
1. Do not fade out the background and text at the same time. 2. Fixed username shape on the screen lock. 3. Fixed corners drawing for the user image. 4. Do not use rounded corners for 'add user' pod (it cased the white frame blinking) 5. Do not use rounded corners for the unselected label, since the user image is scaled and the frame almost disappears. BUG=chromium-os:9721 TEST=manual Review URL: http://codereview.chromium.org/5552003 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@68842 0039d316-1c4b-4281-b951-d872f2087c98
-
chrome-release@google.com authored
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@68838 0039d316-1c4b-4281-b951-d872f2087c98
-
joth@chromium.org authored
- this allows keygen created keys to be stored in a platform specific way, and looked up in client auth - still needs additional work in CertDatabase to fully implement client auth BUG=64917 TEST=Keygen unittest Review URL: http://codereview.chromium.org/5594009 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@68836 0039d316-1c4b-4281-b951-d872f2087c98
-
https://bugs.webkit.org/show_bug.cgi?id=50674hans@chromium.org authored
WebIDBKey::number() will be changed to return a double. Cast this to int explicitly to avoid warnings in the mean time (esp. on Visual Studio). BUG=65619 TEST=none Review URL: http://codereview.chromium.org/5727002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@68835 0039d316-1c4b-4281-b951-d872f2087c98
-
jochen@chromium.org authored
BUG=64753 TEST=*ContentSettings*.* Review URL: http://codereview.chromium.org/5528010 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@68834 0039d316-1c4b-4281-b951-d872f2087c98
-
isherman@chromium.org authored
BUG=none TEST=none Review URL: http://codereview.chromium.org/5740003 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@68833 0039d316-1c4b-4281-b951-d872f2087c98
-
pfeldman@chromium.org authored
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@68832 0039d316-1c4b-4281-b951-d872f2087c98
-
hbono@chromium.org authored
It seems these leaks are caused by a reference-count mismatch for a ChannelProxy::Context object. (This class expects to increase its reference count in OnChannelOpened() and to decrease it in OnChannelClosed(). On the other hand, we cannot call OnChannelClosed() after calling ClearIPCMessageLoop() because it detaches the message loop of a ChannelProxy::Context object. That is, calling ChannelProxy::Close() in ChannelProxy::~ChannelProxy() cannot decrease the reference count to its context.) This change just explicitly calls ChannelProxy::Close() before calling ChannelProxy::ClearIPCMessageLoop(). BUG=65940 TEST=make the valgrind bot green without suppressions. Review URL: http://codereview.chromium.org/5706001 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@68831 0039d316-1c4b-4281-b951-d872f2087c98
-
estade@chromium.org authored
BUG=65963 TEST=manual (see bug) Review URL: http://codereview.chromium.org/5760001 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@68830 0039d316-1c4b-4281-b951-d872f2087c98
-
rsleevi@chromium.org authored
When performing a SSL renegotiation handshake, do not send Certificate/CertificateVerify messages unless the peer sends a CertificateRequest, requesting client auth. This would happen if the following conditions were true: - In the initial/previous handshake, the peer requests client authentication. - The client chooses a certificate, versus declining to provide one. - A (EC-)DHE cipher suite is negotiated. - The peer requests (secure) renegotiation. - The peer does NOT request a client certificate during the renegotiated handshake. R=wtc BUG=62027 TEST=none Review URL: http://codereview.chromium.org/5611005 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@68829 0039d316-1c4b-4281-b951-d872f2087c98
-
panayiotis@google.com authored
Send malware reports when a user opts-in from the safe browsing interstitial page. In this first iteration, the reports are minimal, but I created a new class because I would like to expand them. Workflow: A SafeBrowsingBlockingPage is shown. A new MalwareReport object is created. The blocking page goes away, either when the user clicks proceed, go back or closed the tab. We read the user's preference, and if we have a pending report, we pass it on to the SafeBrowsingService so it can send it. Depends on: http://codereview.chromium.org/4827001/ Also relevant: http://codereview.chromium.org/5102001/ Design doc: https://docs.google.com/document/edit?id=1s-7qMjm23onV68SyqO6yi-xsfFzz4OBSOyHoV3cY8mQ&authkey=CMCF-MID BUG=60831 TEST=unit_tests,relevant browser_test Review URL: http://codereview.chromium.org/4822002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@68828 0039d316-1c4b-4281-b951-d872f2087c98
-
willchan@chromium.org authored
Test to see if this stops memory corruption crashes, will revert after a canary release. BUG=65948 TEST=none Review URL: http://codereview.chromium.org/5733003 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@68827 0039d316-1c4b-4281-b951-d872f2087c98
-
thakis@chromium.org authored
Elliot: Please review GTK changes, referencing model changes. Nico: Please review MAC changes, referencing model changes. Remove horizontal divider in plug-in content setting bubble (WIN,GTK) Add separator after blocked geolocation domain list. Refactor model and layout for simplicity (WIN,GTK,MAC) clear_link, info_link, load_plugins_link_title are now just custom_link. Make proper cookies subclass of ContentSettingTitleLinkAndCustomModel Update unit test to reflect changes and test slightly more. NOTE: Removes an extra vertical space that was between the popups list and the radio group (I can add more space here if desired). NOTE: I would have liked to refactor some more mac code, but that would require modifying the .xibs, which is tough without my own mac dev box... I'll file a bug on this later. NOTE: I addressed all of Peter's earlier comments, except using an enabled bit field. This change supersedes 5592004 and 5563003. BUG=http://code.google.com/p/chromium/issues/detail?id=62188 TEST=Check layout and functionality of content settings bubbles, especially around verticals spacing, horizontal separators, and the buttons "Run all plug-ins this time", "Clear these settings for future visits" (Geolocation), and "Show cookies and other site data". Review URL: http://codereview.chromium.org/5564007 Patch from Michael Wasserman <msw@chromium.org>. git-svn-id: svn://svn.chromium.org/chrome/trunk/src@68824 0039d316-1c4b-4281-b951-d872f2087c98
-
brettw@chromium.org authored
flakyness. This also ups the timeout time for actually running the test. We don't know what the tests do (some of the file ones do a lot of I/O), so the short timeout is somewhat dangerous. We have not historically had many problems with hanging tests other than problems with startup (which has a shorter timeout), so I think the longer test timeout is unlikely to cause problems. TEST=this is BUG=none Review URL: http://codereview.chromium.org/5743002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@68819 0039d316-1c4b-4281-b951-d872f2087c98
-
sky@chromium.org authored
search engine supported instant but it doesn't. BUG=66133 TEST=covered by interactive ui test. You can also manually test Review URL: http://codereview.chromium.org/5771001 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@68816 0039d316-1c4b-4281-b951-d872f2087c98
-
chase@chromium.org authored
These changed after the v8 and ref build updates. linux-release-hardy/dhtml/times/t linux-release-hardy/dhtml/total_byte_b/IO_b linux-release-hardy/dhtml/total_byte_r/IO_r linux-release-hardy/dhtml/total_op_b/IO_op_b linux-release-hardy/dhtml/total_op_r/IO_op_r linux-release-hardy/dhtml/vm_rss_final_b/vm_rss_f_b linux-release-hardy/dhtml/vm_rss_final_r/vm_rss_f_r linux-release-hardy/dhtml/vm_rss_final_t/vm_rss_f_t linux-release-hardy/dhtml/vm_size_final_b/vm_size_f_b linux-release-hardy/dhtml/vm_size_final_r/vm_size_f_r linux-release-hardy/dhtml/vm_size_final_t/vm_size_f_t linux-release-hardy/intl1/times/t linux-release-hardy/intl2/times/t linux-release-hardy/morejs/times/t linux-release-hardy/moz/times/t linux-release-hardy/v8_benchmark/score/score BUG=none TEST=linux steps are green TBR=nsylvain@chromium.org Review URL: http://codereview.chromium.org/5731002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@68815 0039d316-1c4b-4281-b951-d872f2087c98
-
twiz@google.com authored
Simple fix preventing a potential crash location when shifting focus from an experimental extension popup view. BUG=NONE TEST=ExtensionApiTest.Popup Review URL: http://codereview.chromium.org/5733002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@68814 0039d316-1c4b-4281-b951-d872f2087c98
-
nirnimesh@chromium.org authored
Review URL: http://codereview.chromium.org/5666001 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@68813 0039d316-1c4b-4281-b951-d872f2087c98
-