| Commit message (Collapse) | Author | Age | Files | Lines |
|
|
|
|
|
|
|
| |
This is easy enough to fix, and would be occasionally helpful.
Review URL: http://codereview.chromium.org/524015
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@35405 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
The "execute the command later" workaround, which was in place to avoid OwnedWidget-DCHECKing when selecting menu items that might cause cascading object-destruction effects, caused problems with the Browser::SetBlockCommandExecution() workaround, the reasoning for which can be found in BrowserWindowGtk and is reproduced here:
// This piece of code is based on the fact that calling
// gtk_window_activate_key() method against |window_| may only trigger a
// browser command execution, by matching either a global accelerator
// defined in above |kAcceleratorMap| or the accelerator key of a menu
// item defined in chrome/browser/gtk/standard_menus.cc.
//
// Here we need to retrieve the command id (if any) associated to the
// keyboard event. Instead of looking up the command id in above
// |kAcceleratorMap| table by ourselves, we block the command execution of
// the |browser_| object then send the keyboard event to the |window_| by
// calling gtk_window_activate_key() method, as if we are activating an
// accelerator key. Then we can retrieve the command id from the
// |browser_| object.
//
// Pros of this approach:
// 1. We can handle accelerators defined not only in above
// |kAcceleratorMap| table, but also those in standard_menus.cc.
// 2. We don't need to care about keyboard layout problem, as
// gtk_window_activate_key() method handles it for us.
//
// Cons:
// 1. The logic is a little complicated.
// 2. We should be careful not to introduce any accelerators that trigger
// customized code instead of browser commands.
The easiest and cleanest thing is just to remove the OwnedWidgetGtk.
BUG=none
TEST=interactive ui tests pass again
Review URL: http://codereview.chromium.org/522028
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@35404 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
| |
TEST=none
BUG=30081, 30082
Review URL: http://codereview.chromium.org/515074
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@35403 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
| |
- add a TOOLKIT_USES_GTK #define to catch GTK and Views+GTK
- lots of ifdef twiddling
Review URL: http://codereview.chromium.org/519030
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@35402 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
| |
BUG=30656
TEST=manual
Review URL: http://codereview.chromium.org/515073
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@35401 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
| |
BUG=30656
TEST=manual (right click on browser actions and test the actions)
Review URL: http://codereview.chromium.org/521016
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@35400 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
| |
TBR=ukai
TEST=none
BUG=29157
Review URL: http://codereview.chromium.org/523030
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@35399 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
| |
BUG=27993
TEST=valgrind try runs
Review URL: http://codereview.chromium.org/519029
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@35398 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
| |
Rebaselining LayoutTests/svg/custom/use-on-symbol-inside-pattern-expected.svg.
TBR=dglazkov
BUG=none
TEST=green bots.
Review URL: http://codereview.chromium.org/515070
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@35397 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
| |
creates the rest of the schema.
BUG=none
TEST=existing tests apply
Review URL: http://codereview.chromium.org/518023
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@35396 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
| |
this has been consistently failing
on the chrome frame builder.
TBR=tommi
Review URL: http://codereview.chromium.org/515071
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@35395 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
| |
values. Also handle all the forms of the paths with links resolved.
TEST=page cycles, browser_tests all get back to being fast. normal profiles still exclude the files listed in the first bug.
BUG=25959,30002
Review URL: http://codereview.chromium.org/523026
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@35394 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
| |
BUG=NONE
TEST=NONE
Review URL: http://codereview.chromium.org/522022
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@35393 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
| |
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@35392 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
| |
Rebaseline a bunch of SVG test broken to due webkit r52647.
BUG=25977,31028,29737,31350
TEST=mostly green layout test try runs.
Review URL: http://codereview.chromium.org/515067
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@35391 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
SHM Pixmaps were frequently unavailable (which makes some
sense: you're asking for a shared memory handle into a
graphics-card-side memory buffer). But we can still do
an SHM copy from pixels->pixmap as long as we only go in
write only direction, via the XShmPutImage API.
BUG=31301
Review URL: http://codereview.chromium.org/522023
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@35390 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
| |
On Linux, implement just enough MenuModel support in GtkMenu. On Linux + windows, remove wrapper classes. On Mac, just change a few function calls around.
BUG=none
TEST=unit test + manually clicking around on it in debug mode
Review URL: http://codereview.chromium.org/501168
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@35389 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
| |
BUG=27216
TEST=none
Review URL: http://codereview.chromium.org/523021
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@35388 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
| |
BUG=NONE
TEST=NONE
Review URL: http://codereview.chromium.org/517025
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@35387 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
| |
TEST=none
BUG=26754
TBR=mark
Review URL: http://codereview.chromium.org/524013
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@35386 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
| |
BUG=none
TEST=none
Review URL: http://codereview.chromium.org/515063
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@35385 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
| |
BUG=23069
TEST=Download large file, click pause in download item's menu to pause/unpause.
Review URL: http://codereview.chromium.org/524009
Patch from sig11@reprehensible.net.
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@35384 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
| |
Change arrowLocation to arrow_location, to follow Chromium code standards.
BUG=494008
TEST=None
Review URL: http://codereview.chromium.org/522027
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@35383 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
| |
This isn't the prettiest, but it should get the perf back where we want it. If the file we're asked to exclude is in /var or /private, ignore it. The problem on the bots is the TimeMachine prefs plist is growing unbounded as we keep adding 4 files each run for temporary profiles. The load/modify/write gets slower with each addition. This quick/dirty check to avoid that. Since the original bug for the backup of this data is still open, I figure landing this is the right thing to do until that is fully addressed.
BUG=25959,30002
TEST=page cycles, browser_tests all get back to being fast. normal profiles still exclude the files listed in the first bug.
Review URL: http://codereview.chromium.org/523024
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@35382 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
| |
Fails on Linux
TBR=pfeldman@chromium.org
Review URL: http://codereview.chromium.org/519028
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@35381 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
| |
TBR=pfeldman@chromium.org
BUG=none
TEST=DevToolsSanityTest.TestConsoleLog
Review URL: http://codereview.chromium.org/521015
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@35380 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
| |
from the old request.
BUG=7357
TEST=none
Review URL: http://codereview.chromium.org/470010
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@35379 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
| |
TEST=none
BUG=30582
Review URL: http://codereview.chromium.org/506073
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@35378 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
| |
File->"Create application shortcuts." The \x is converted back to % in
handling the --app switch.
BUG=23693
TEST=None
Review URL: http://codereview.chromium.org/515028
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@35377 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
| |
BUG=none
TEST=files shouldn't be left around.
Review URL: http://codereview.chromium.org/518019
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@35376 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
| |
- Don't set the images in the xib since cocoa can't find them in the other bundle during tests (and maybe not during normal app runtime)
- Make sure the images are setup during view start.
- Create constants for the image names since some are used in more then one place.
BUG=30342
TEST=no longer get the error messages on test runs
Review URL: http://codereview.chromium.org/515061
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@35375 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
| |
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@35374 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
| |
non-debug builds had unused static function
tbr=mbelshe
Review URL: http://codereview.chromium.org/523023
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@35373 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
To make it easier to use the about:object profiling facility,
I put in a quick/dirty way to reset all profile stats to 0 (as if
there were no births, deaths, etc.). This code is only activated
under debug builds (or if a developer inists in a private build).
These stats don't impact semantics of the browser, so the hackish
approach to clearing the data counts can't instigate a crash, and
it makes it much easier to look at changes in the stats.
While changing the code, I also added a lot of comments, and did
a few minor cleanups items.
I also officially added about:tasks as a replacement for about:objects,
as this is really how the service is used in Chrome.
r=mbelshe
Review URL: http://codereview.chromium.org/100297
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@35372 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
entire bitmap memory. To fix up the rowbytes value properly, we simply don't send it via IPC any more, and recalulate it from width and depth in the trusted code. It's a cheap calculation.
Also one bonus fix: don't use an unintialized IconInfo if deserialization fails.
BUG=31307
TEST=Manual; ran with breakpoints on the failure paths.
Review URL: http://codereview.chromium.org/517023
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@35371 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
These vsprops used to be used to setup the branding #defines.
This is now handled in common.gypi.
This is an incremental step toward dropping vsprops entirely for release.
BUG=None
TEST=None
TBR=maruel
Review URL: http://codereview.chromium.org/523020
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@35367 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
| |
This is so that ui tests using UITest have a valid NSAutoreleasePool, which
PlatformTest provides, but testing::Test does not.
Review URL: http://codereview.chromium.org/519026
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@35366 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
| |
with a webkit roll.
TEST=none
BUG=31302
TBR=pam
Review URL: http://codereview.chromium.org/518024
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@35365 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
| |
TBR=scherkus
BUG=none
TEST=none
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@35364 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
| |
that libjingle logging happens on non-Windows platforms.
BUG=none
TEST=manual
Review URL: http://codereview.chromium.org/523012
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@35363 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
| |
particular, fix an integer overflow that might be an issue on Linux. Other changes are to avoid OOB reads and calling front() or [] on an empty or insufficiently sized vector.
BUG=31928
TEST=NONE
Review URL: http://codereview.chromium.org/522024
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@35362 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
| |
TBR=dimich
BUG=none
TEST=layout tests
Review URL: http://codereview.chromium.org/515052
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@35361 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
| |
subsequent allocation.
BUG=NONE
TEST=NONE
Review URL: http://codereview.chromium.org/518022
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@35360 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
| |
Some functions actually return a ShmSeg, not an XID.
Review URL: http://codereview.chromium.org/519024
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@35359 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
| |
BUG=31300
TEST=none
Review URL: http://codereview.chromium.org/521014
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@35358 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Normal buildbot-run tests that need an X display connection
(e.g. ui_tests) are executed by RunPythonCommandInBuildDir() from
scripts/slave/slave_utils.py which takes care of this. However, we
are currently running coverage_posix.py from within a build command
which itself is not run a s a "python command in build dir". Thus, no
Xvfb.
TODO(jrg): transition coverage generation to a distinct buildbot step.
Review URL: http://codereview.chromium.org/524007
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@35357 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
This cleanup was requested by brettw and was started to make it easier for the sync service to post tasks to the WDS thread (now the DB thread). This simplifies the WDS a bit since it no longer has to manage its own thread, and can assume that the DB thread is running throughout its lifetime.
One change in behavior that is significant is that previous to this change, the WDS worker thread would always be joined when Shutdown() was called from Profile::~Profile(). Now the Shutdown() method schedules a task that can extend the lifetime of the WDS past the lifetime of the Profile instance.
Review URL: http://codereview.chromium.org/524003
TBR=skrul@chromium.org
Review URL: http://codereview.chromium.org/521013
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@35356 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
| |
BUG=none
TEST=none
Review URL: http://codereview.chromium.org/465141
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@35355 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Still nothing is being written to disk with this CL,
in-memory SQLite and DiskCaches are being utilized.
Responses are not yet being removed from the DiskCasche
when the should be. Once that's done (in the next CL), we'll
start saving things on disk.
BUG=none
TEST=appcache_database_unittest.cc, appcache_storage_impl_unittest.cc
Review URL: http://codereview.chromium.org/518020
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@35354 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
| |
BUG=none
TEST=Run sync_unit_tests
Review URL: http://codereview.chromium.org/509023
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@35353 0039d316-1c4b-4281-b951-d872f2087c98
|