| Commit message (Collapse) | Author | Age | Files | Lines |
|
|
|
|
|
|
|
|
|
| |
Obvious result of Rohit's addition of paste-and-go and my refactor to
add AutocompleteTextFieldObserver. Bonus prize of unit tests for
cocoa/ part of Rohit's change.
Review URL: http://codereview.chromium.org/207047
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@26993 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
| |
easier to navigate.
Review URL: http://codereview.chromium.org/221001
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@26992 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
| |
BUG=22067
TEST=No more random strings in crash client GUIDs.
Review URL: http://codereview.chromium.org/213038
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@26991 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
| |
TBR=jennb
BUG=none
TEST=none
Review URL: http://codereview.chromium.org/222012
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@26990 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
| |
and LayoutTests/http/tests/misc/BOM-override-script.html on Mac to the flaky test list
BUG=9797,9798
TEST=Webkit test passes reliably on Mac.
TBR=mark
Review URL: http://codereview.chromium.org/224008
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@26989 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
| |
This should be OK since http://src.chromium.org/viewvc/chrome?view=rev&revision=23306
This can speed up some tests by 10%...20%
I was running Valgrind & TSAN with both patches locally for more than a month and everything was fine.
Review URL: http://codereview.chromium.org/220013
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@26988 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
| |
TBR=jennb
BUG=none
TEST=none
Review URL: http://codereview.chromium.org/218008
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@26987 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
| |
BUG=none
TEST=none
Review URL: http://codereview.chromium.org/209058
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@26986 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
| |
CID=3970
BUG=none
TEST=none
Review URL: http://codereview.chromium.org/233001
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@26985 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
| |
CID=4402
BUG=none
TEST=none
Review URL: http://codereview.chromium.org/227007
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@26984 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
| |
state of the save type before the user has selected a save type. Initialize save_type_ to this value in the constructor.
CID=3809
BUG=none
TEST=none
Review URL: http://codereview.chromium.org/220016
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@26983 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
| |
TEST=none
BUG=none
Review URL: http://codereview.chromium.org/220015
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@26982 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
| |
It's most probably a bug in the code being tested, or a bad assumption
in the test.
TEST=none
BUG=19876
Review URL: http://codereview.chromium.org/233002
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@26981 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
| |
Review URL: http://codereview.chromium.org/222011
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@26979 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Windows-only DEPS inclusions.
Is this safe, given the vcproj files that are in the repository? I believe so. The gitignore documentation states:
"Note that all the gitignore files really
concern only files that are not already
tracked by git; in order to ignore
uncommitted changes in already tracked
files, please refer to the git update-index
--assume-unchanged documentation."
TEST=ran "git status" on Windows, saw no generated files
BUG=none
Review URL: http://codereview.chromium.org/219004
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@26978 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
| |
This patch sync's us with Mozilla's upstream version of the eTLD list.
BUG=19957
TEST=Enter test.local into the omnibar. The default action should be to load http://test.local, just like test.com
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@26976 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
| |
and image_operations.* can be moved because they are not used by WebKit code.
This also fixes the spelling of "Convolusion" to "Convolution" and updates some copyrights.
BUG=none
TEST=none
Review URL: http://codereview.chromium.org/207059
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@26975 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
| |
BUG=22563
Review URL: http://codereview.chromium.org/229001
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@26974 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
| |
instead of its current size (different when the source is maximized)
BUG=22154
TEST=open a window with two tabs, maximize it, then drag one tab out - it should be the original size of the original window
Review URL: http://codereview.chromium.org/216053
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@26972 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
| |
BUG= http://crbug.com/22801
TEST= none
Review URL: http://codereview.chromium.org/218006
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@26971 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
| |
Failures are rebaselines.
R=levin
TEST=none
BUG=none
Review URL: http://codereview.chromium.org/232004
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@26970 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
"...may be suffering from what we in the soft sciences call 'Obsessive
Compulsive Disorder,' or the 'The O.C. Disorder.'"
Here's what we're doing:
mkdir OCMock
svn add OCMock
for i in * ; do
if [ "${i}" != "OCMock" ] && [ "${i}" != "README.chromium" ] ; then
svn move "${i}" OCMock
fi
done
and make the SVN URL in README.chromium a tiny bit more specific.
Review URL: http://codereview.chromium.org/220017
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@26969 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
| |
Patch by Fred Akalin <akalin@gmail.com>
Review URL: http://codereview.chromium.org/205023
Review URL: http://codereview.chromium.org/209070
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@26968 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
| |
(oops)
Also say which valgrind is being used
Review URL: http://codereview.chromium.org/233003
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@26967 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
This CL also removes two unnecessary methods on WebViewDelegate:
UserMetricsRecordAction and ShowSpellingUI were both never called.
R=dglazkov
BUG=10033
TEST=none
Review URL: http://codereview.chromium.org/227006
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@26965 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
http://codereview.chromium.org/177024 enabled about:memory for Linux,
but it went from not-implemented to crashing for Mac, due to an
out-of-bounds vector access. This returns it to simply not-implemented.
http://crbug.com/21830
TEST=Browse to about:memory, don't crash, show a template with
obviously null data in it.
Review URL: http://codereview.chromium.org/219008
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@26964 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
| |
TEST=none
BUG=none
Review URL: http://codereview.chromium.org/220012
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@26963 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
This mostly reverts commit e461ca0cebce93375b81a14af7caf5d95a10158c (r25080) and adds a comment about why I reverted it.
This breaks on NX, probably because we're shaping a toplevel window
from within the expose event. I don't see a different point when we
can shape the window reliably (configure-event doesn't appear to work),
so just revert rounded corners on the popup for now.
BUG=22015
Review URL: http://codereview.chromium.org/224004
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@26962 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
- tab menus
- page and tool menus
- menubar sub menus
TEST=the above menus should show up in Title case on Mac/GTK. these are new strings, so they will need new translations for the languages.
BUG=22254
BUG=22255
BUG=22256
Review URL: http://codereview.chromium.org/214056
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@26961 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
| |
in the constructor.
CID=5442
BUG=none
TEST=none
Review URL: http://codereview.chromium.org/222008
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@26960 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
browsing data"""
This reverts commit r26956 and re-applies the keyboard bindings for clearing
browsing data.
_ph says that the test is just flaky.
TBR=estade
Review URL: http://codereview.chromium.org/232005
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@26959 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
| |
BUG=none
TEST=none
Review URL: http://codereview.chromium.org/227005
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@26958 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
| |
This reverts commit 4e0de697be80a0a91fbcceda63d3c85055f1afcc.
TBR=davemoore
Review URL: http://codereview.chromium.org/219013
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@26957 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
| |
This reverts commit r26947.
TBR=estade
Review URL: http://codereview.chromium.org/225015
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@26956 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
| |
TEST=none
BUG=none
Review URL: http://codereview.chromium.org/221002
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@26955 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
| |
CID=5040
BUG=none
TEST=none
Review URL: http://codereview.chromium.org/225013
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@26954 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
crash while dereferencing an invalid
delegate_ (HtmlDialogUIDelegate*) member. I could not reproduce this consistently though. The inference is
as below:-
1. When the dialog is closed the HtmlDialogView::OnDialogClosed member function is invoked which calls the
OnDialogClosed function on the delegate. This in turn causes the delegate to be destroyed.
2. It then sets the delegate to NULL and attempts to close the window.
3. Before the Close method is dispatched if the view attempts to Paint it causes a crash in the
HtmlDialogView::GetWindowTitle function because of dereferencing a NULL delegate_.
Fix is to add corresponding NULL checks in the relevant functions.
This fixes http://b/issue?id=2138035, which was reported with ChromeFrame.
Bug=2138035
Review URL: http://codereview.chromium.org/220011
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@26953 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
A focus related asserion was hit when closing Chrome.
It was caused by the top level widget being destroyed and deleting itself
before its children. When deleted it would delete the focus manager without
giving child widgets a chance to remove focus change listener causing the
assert.
We now delay the deletion of the WidgetGtk so child widget get the destroy
message when the top WidgetGtk is still alive.
BUG=None
TEST=Close Chrome toolkit views. It should not assert because of the focus
manager.
Review URL: http://codereview.chromium.org/231003
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@26952 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
| |
BUG=20669
Review URL: http://codereview.chromium.org/208032
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@26950 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
| |
Review URL: http://codereview.chromium.org/218004
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@26949 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
| |
TBR=dumi
TEST=none
BUG=4359
Review URL: http://codereview.chromium.org/224006
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@26948 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
| |
dialog on linux. When we removed the menu item, we lost the keyboard
binding as well.
Review URL: http://codereview.chromium.org/216056
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@26947 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
| |
* Allow IPC::Listeners to send a message on OnChannelConnected.
* Fix a bug in MessagePumpLibevent::WatchFileDescriptor causing a read-after-free.
BUG=22451
Review URL: http://codereview.chromium.org/209061
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@26946 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
| |
TBR=paul
TEST=none
BUG=none
Review URL: http://codereview.chromium.org/219011
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@26945 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
| |
tests. Unlike on Linux, it doesn't crash on Mac. Also, it's only observed on Mac OS 10.5 debug bot. Because there's no way to mark OS 10.5 separately from other Mac OS versions. I'm just using Mac per Mark.
BUG=9798
TBR=dglazkov
Review URL: http://codereview.chromium.org/231002
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@26944 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
| |
the comment I added to the code.
http://crbug.com/21849
TEST=covered by unit tests
Review URL: http://codereview.chromium.org/210043
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@26941 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
| |
BUG=19773
TEST=none
Review URL: http://codereview.chromium.org/212014
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@26939 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
InProcessBrowserTest that let tests do
what is normally done by overriding SetUp/TearDown methods. This special method is needed because
of the way SetUp winds up at the bottom of the stack that invokes your test fixture methods.
Make the LiveBookmarksSyncTest class allow direct lookup for google.com domain as part of setup,
which these tests need to access GAIA and sync servers.
Review URL: http://codereview.chromium.org/214047
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@26938 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
| |
MessageBox, TabbedPane and RadioButton
BUG=none
TEST=none
Review URL: http://codereview.chromium.org/214053
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@26937 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
| |
R=levin
TEST=none
BUG=none
Review URL: http://codereview.chromium.org/225010
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@26936 0039d316-1c4b-4281-b951-d872f2087c98
|