| Commit message (Collapse) | Author | Age | Files | Lines |
|
|
|
|
|
|
|
|
| |
TBR=tc
Review URL: http://codereview.chromium.org/8706
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@4141 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
| |
code in resource message filter, which was initilaizing spellchecker in the filter if it was NULL. This part of the code is not required since the resource message filter should not initialize spellchecker; it is up to the profile to give the spellchecker to the resource message filter.
Issue = 120
Review URL: http://codereview.chromium.org/7935
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@4140 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
| |
Review URL: http://codereview.chromium.org/8874
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@4138 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
| |
Review URL: http://codereview.chromium.org/8000
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@4136 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
in DoReadHeadersComplete.
Add an unused state, STATE_UNUSED, at the old value
of STATE_READ_HEADERS_COMPLETE (15 = 0xF) to see if
that value has any significance in the crash.
TBR=eroman
BUG=3772
Review URL: http://codereview.chromium.org/8701
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@4133 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
| |
had this code in place already, I wouldn't have needed to rev the compatible version number in my upcoming database change >:(
I checked the rest of our sourcebase's INSERT and SELECT statements, but these were the only ones that needed to have fieldnames added (in a few cases SELECT * was appropriate).
Review URL: http://codereview.chromium.org/8684
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@4130 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
| |
this server for more general purposes than only layout tests. (In particular,
I've found it helpful when testing perf plots.)
BUG=none
TEST=none
Review URL: http://codereview.chromium.org/8668
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@4129 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
| |
Patch committed for mendola@gmail.com
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@4128 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
| |
BUG=674
TEST=none
Review URL: http://codereview.chromium.org/8862
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@4124 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
| |
a readonly textarea. This triggered lots of asserts.
Review URL: http://codereview.chromium.org/8860
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@4119 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
word is selected. This is because we use the wrong node when
getting the selection. We want to use the shadow inner node,
but the hit test result provided to the ContextMenu is not
the shadow node. To work around this, we recompute the
hit test result.
BUG=613
TEST=Go to a page with a text area, add text until a scroll bar
appears, scroll the text area and right click on a word. The
word under the cursor should be highlighted.
Review URL: http://codereview.chromium.org/8853
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@4118 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
| |
BUG=1340229
Review URL: http://codereview.chromium.org/8690
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@4117 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
| |
the contents view has been attached to the view hierarchy.
http://crbug.com/2751
Review URL: http://codereview.chromium.org/8681
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@4116 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
as we need to simulate mousemoves here. ui tests run on machines which are locked,
causing this test to fail.
Moved the NPAPITester and NPAPIVisiblePluginTester classes to a common file
npapi_test_helper.cc so it can be used from both tests.
Relanding the SetCursor patch. It has already been approved by John.
Description below:-
Proposed fix for http://b/issue?id=1362948, which is a crash in the rendererwhen we invoke the setCursor call on the parent view in WebPluginImpl::handleEvent.
This crash occurs because the plugin is deleted in the context of a mouse down event. This could occur by invoking a javascript function via NPN_Evaluate. On return from the HandleEvent sync call we attempt to retreive the parent frame, which
returns NULL and hence the crash.
The fix is to retreive the parent frameview at the start of the WebPluginImpl::handleMouseEvent function and use it whereever needed.
Added a unit test which deletes the plugin instance in a mousemove event.
R=jam
Bug=1362948
Review URL: http://codereview.chromium.org/8691
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@4115 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
agent string on the session handle, which is the only
way to specify the User-Agent header for HTTP CONNECT
requests.
This requires some refactoring. Most of the work done
by Session::Init() is moved to the Session constructor.
Session::Init() now only creates the primary session
handle, and the call to Session::Init() is delayed to
the first time we use the primary session handle
(session_->internet()).
R=darin
BUG=552
Review URL: http://codereview.chromium.org/8841
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@4114 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
| |
{basic_auth, digest_auth} x {empty_username, empty_password}.
Review URL: http://codereview.chromium.org/8863
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@4113 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
| |
cleanup timer from 5 seconds to 10 seconds.
http://code.google.com/p/chromium/issues/detail?id=3574
Review URL: http://codereview.chromium.org/8124
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@4112 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
| |
R=eroman
BUG=297
Review URL: http://codereview.chromium.org/8859
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@4109 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
| |
Review URL: http://codereview.chromium.org/8688
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@4108 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
| |
Review URL: http://codereview.chromium.org/8861
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@4107 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
| |
"actual, expected").
This is a by-product of my in-progress cookie fixes.
Review URL: http://codereview.chromium.org/8852
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@4106 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
| |
component directly, whose DEPS file then grabs the needed build stuff out of
the Chrome tree. Since gclient only examines DEPS files at the root of
solutions, this file has no effect on dependencies for Chrome developers.
Review URL: http://codereview.chromium.org/8675
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@4105 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
| |
BUG=2804
Review URL: http://codereview.chromium.org/8196
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@4104 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Change 6053 involved making the PasswordManagerView's table model contain a vector
of stack-allocated PasswordRow that owned a heap allocated PasswordForm, but
std::vector creates temp PasswordRow's internally (on the stack) that were deleting the
PasswordForm in their dtor. This accidental vector-internal delete caused subsequent
operations on saved_signons_ to blow up.
My fix is to make PasswordRows a vector of heap allocated PasswordRow instead, and
have a STLElementDeleter member handle cleanup.
Review URL: http://codereview.chromium.org/8651
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@4103 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
| |
Review URL: http://codereview.chromium.org/8685
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@4100 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
* Use SCons -C to invoke the v8 build from our own build directory,
using the -Y option to pull all source files from the v8 source
directory, to avoid building things under src/v8.
* Make sure the build directory exists before we invoke the subsidiary
SCons that will chdir there.
* Build libv8_snapshot.a in the obj/v8 directory, not src/build.
* Remove the svn:ignore property from src/build (specifically libv8snapshot.a).
* Correct the $LIBS_DIR location of libv8.a.
* Use the actual file names involved instead of local Python variables.
* Don't forcibly add our targets to other component's Aliases.
Review URL: http://codereview.chromium.org/8851
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@4099 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
certificate errors (ERR_CERT_NO_REVOCATION_MECHANISM and
ERR_CERT_UNABLE_TO_CHECK_REVOCATION).
This requires moving the code that automatically denies
subresource requests into only the SSLPolicy subclasses
that need it. To avoid duplicating that code in five classes,
I added convenience methods for handling overridable certificate
errors and fatal errors.
R=jcampan,abarth
Review URL: http://codereview.chromium.org/8085
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@4098 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
| |
Review URL: http://codereview.chromium.org/8402
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@4095 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
rendererwhen we invoke the setCursor call on the parent view in WebPluginImpl::handleEvent.
This crash occurs because the plugin is deleted in the context of a mouse down event. This could occur by invoking a javascript function via NPN_Evaluate. On return from the HandleEvent sync call we attempt to retreive the parent frame, which
returns NULL and hence the crash.
The fix is to retreive the parent frameview at the start of the WebPluginImpl::handleMouseEvent function and use it whereever needed.
Added a unit test which deletes the plugin instance in a mousemove event.R=jamBug=1362948
Review URL: http://codereview.chromium.org/8178
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@4094 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
| |
The Google Bookmarks bookmarklet opens a popup window that allows you to bookmark a page. When the window closes we crash because we dereference a NULL TabContents pointer the we get from GetSelectedTabContents(). This fixes that.
Review URL: http://codereview.chromium.org/8185
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@4093 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
| |
Review URL: http://codereview.chromium.org/8847
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@4092 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
| |
and filter storage so that it can run side by side with the
old implementation without data loss.
Review URL: http://codereview.chromium.org/8844
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@4087 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
| |
http://code.google.com/p/chromium/issues/detail?id=3774
Review URL: http://codereview.chromium.org/8649
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@4086 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
| |
Review URL: http://codereview.chromium.org/8672
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@4085 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
| |
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@4084 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
| |
BUG=3784
TEST=none
Review URL: http://codereview.chromium.org/8837
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@4083 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
| |
Review URL: http://codereview.chromium.org/8842
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@4082 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
| |
typing working.
Review URL: http://codereview.chromium.org/8663
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@4081 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
| |
failure
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@4079 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
| |
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@4075 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
| |
Review URL: http://codereview.chromium.org/8667
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@4074 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
| |
This reverts commit 54dfed3ab933cea7492ae1ed2f35cdbfe822e8a4.
Review URL: http://codereview.chromium.org/8839
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@4072 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
| |
Review URL: http://codereview.chromium.org/8000
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@4071 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
| |
the renderer. Returning as it did before would end up in V8 OS::Abort calling
__debugbreak() which would also crask the renderer.
I am about to change V8 OS::Abort to call abort() instead of __debugbreak()
and Chrome should not depend on how V8 handles aborts anyway.
Review URL: http://codereview.chromium.org/8838
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@4070 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
| |
- Automatically reload scripts when the directory changes.
- Add a unit test for the GreasemonkeyMaster.
Review URL: http://codereview.chromium.org/7472
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@4069 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
| |
OS X, it fails in release mode.
Review URL: http://codereview.chromium.org/8662
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@4067 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
| |
In upstream Webkit r36057, RenderTextControlInnerBlock moved from rendering/RenderTextControl.cpp to rendering/TextControlInnerElements.cpp. Our WebKit snapshot purports to be r36102, but our version of RenderTextControl.cpp does not have this change, or others made in r36057. It looks like our version of that file, unforked at r3114 but previously in pending with the same problem, never had the upstream changes merged into it properly during the r36102 WebKit merge.
DEPS change corresponds to third_party/WebKit r4065
Review URL: http://codereview.chromium.org/8661
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@4066 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
| |
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@4064 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
| |
were using the width of the image, when the width is actually inferred by the
Windows API call. Instead, we were supposed to pass the height.
BUG=100
Review URL: http://codereview.chromium.org/8646
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@4063 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
| |
TBR=kasperl
Review URL: http://codereview.chromium.org/8654
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@4060 0039d316-1c4b-4281-b951-d872f2087c98
|