| Commit message (Collapse) | Author | Age | Files | Lines |
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
disable field trials. I am going to have a need for both soon.
Cleaning up some comments about empty trial names, adding static
method TrialExists() and simplifying many call sites by using this
method.
While I'm in there and needing base/OWNERS approval, add an OWNERS
file for base/metrics that adds jar@chromium.org as an owner for that
directory.
BUG=none
TEST=base_unittests
TBR=jam@chromium.org
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@84197 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
one error, we need to count two when the malformed body is reported,
against the one success reported based on the response code from what
must have been a successful HTTP request.
Updated a unit test that previously should have caught this but was
making incorrect assumptions.
BUG=81587
TEST=net_unittests
Review URL: http://codereview.chromium.org/6932013
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@84196 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
non-virtual one. This is a step along the way to killing it entirely. This also adds a lot of OVERRIDE markers and does some other cleanup in a few places.
The original (stupid) design for the delegate class left subclasses great flexibility in how they mapped infobars to delegates. In practice, no one ever wanted multiple infobars driven off a single delegate, so the mapping was always one-to-one. As a result, it was always correct for InfoBarClosed() to "delete this", but because the base class did not do so, every subclass needed to. Most did; the others leaked memory and failed to run their destructors.
This change forces the base class to delete itself. This fixes the delegate leaks in the couple subclasses that failed to do this. It also eliminates a lot of copy-and-pasted "delete this" implementations.
Ultimately, we'll be moving to a model where the InfoBar "view" class owns the delegate and deletes it directly, which will eliminate InfoBarClosed() completely.
BUG=62154
TEST=none
Review URL: http://codereview.chromium.org/6926001
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@84195 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
logging. VLOG(0) is used only in conditions that are guaranteed to trigger this bug. Other places are now VLOG(1).
There are 2 VLOG(0) that will still print in normal scenario. They are:
1. SyncShare is about to be called.
2. DTM is informing sync configure is done.
I think the above 2 cases are after the bug, meaning the bug happens before either of the above 2 conditions are hit. This will prove that or disprove it.
BUG=
TEST=
Review URL: http://codereview.chromium.org/6933018
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@84194 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
| |
BUG=80902
TEST=Invoke print preview with no printer installed. "Print to PDF" should be
selected and the previewed doc should be in color.
Review URL: http://codereview.chromium.org/6928019
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@84190 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
| |
BUG=80431
TEST=we should not prompt to become the default every time if ~/.local/share/applications/mimeapps.list does not exist
Review URL: http://codereview.chromium.org/6929027
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@84189 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
| |
while.
BUG=none
TEST=PrerenderBrowserTest.PrerenderInfiniteLoopMultiple
Review URL: http://codereview.chromium.org/6930031
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@84188 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
| |
BUG=chromium-os:14770
TEST=manually test to make sure it doesn't crash.
Review URL: http://codereview.chromium.org/6928022
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@84185 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
| |
* The infobar wouldn't tell the browser it was done animating when the height hadn't also changed from the previously-calculated height, so frequently the content wouldn't properly relayout after infobar animation. I think this was the cause of bug 79108.
* A silly bug in ToolbarSizeChanged() meant that we never did re-layout on the content area when the toolbar stopped animating, because we were checking for "call_state == NORMAL" while the AutoReset object was still alive and guaranteeing that the state would not be NORMAL. This caused bug 80142.
* I reversed the enum values for the recursive call state, so recursive ToolbarSizeChanged() calls were all getting "can use fast resize" backwards.
BUG=79108,80142
TEST=Visit a page with a vertical scrollbar. Toggle the bookmark bar on. Once the bar stops animating the scrollbar should redraw so it's once again fully on-screen.
Review URL: http://codereview.chromium.org/6927032
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@84184 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
This adds BrightnessLibrary::IncreaseScreenBrightness()
and DecreaseScreenBrightness().
Contributed by Chris Wolfe <cwolfe@chromium.org>. I am
checking this in on his behalf from
http://codereview.chromium.org/6928011/.
BUG=chromium-os:13281
TEST=cwolfe tested manually in conjunction with other changes that use these functions; i tested that it builds at ToT
TBR=cwolfe
Review URL: http://codereview.chromium.org/6935017
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@84183 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
| |
TBR=tim@chromium.org
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@84182 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
| |
by refreshing the page after adding the data through the DOMUI.
TEST=none
BUG=80682
Review URL: http://codereview.chromium.org/6944002
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@84181 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
| |
BUG=81453
Review URL: http://codereview.chromium.org/6933001
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@84180 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
This patch introduces the notion of non-blocking conflicts. These are conflicts that do not result in the
syncer getting stuck. They affect the HasConflictingUpdates status call, so we attempt to reapply updates
when they occur, but they are not passed to the conflict resolver, and hence do not result in the syncer being stuck. All cases where we have a conflict due to encryption/decryption result in a new
ENCRYPTION_CONFLICT, which are treated as nonblocking conflicts.
Updated sync's protocol_version to 27 so the server can tell which clients understand encryption.
Also, due to this change, it's more likely that local/server changes will be overwritten in the conflict
resolver (since encryption conflicts are kept as unapplied while they can't be decrypted). Added
counters for how often this happens, which will appear on about:sync, and filed follow up bug
crbug.com/76596.
BUG=59242
TEST=new session sync_integration tests, modified apply_updates_command unit tests
Review URL: http://codereview.chromium.org/6714002
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@84177 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
on tab_strip_gtk.cc.
BUG=58030
TEST=None
R=evan@chromium.org
Review URL: http://codereview.chromium.org/6930026
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@84175 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
build:
Autofill phone number enhancements and integration of Phone Number Util Library: part 1
Temporarily the whole library is included, until the patch is upstreamed.
BUG=71443
TEST=Unit-tested
Review URL: http://codereview.chromium.org/6930013
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@84174 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
| |
This pulls in brightness-related code for cwolfe.
BUG=chromium-os:13281
TEST=manual: synced to this revision and checked that the chromeos-chrome package builds and appears to function as before
TBR=cwolfe
Review URL: http://codereview.chromium.org/6930028
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@84172 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
| |
BUG=81427
TEST=PrerenderBrowserTest.*Alert*
Review URL: http://codereview.chromium.org/6915018
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@84170 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
| |
BUG=80895
TEST=Compiles
Review URL: http://codereview.chromium.org/6930014
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@84167 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
| |
were missing them.
BUG=chromium-os:14769, chromium-os:14821
TEST=no crashes on the crash server with these stacks anymore
Review URL: http://codereview.chromium.org/6940001
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@84166 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
| |
Revert GetSyncData() to taking a const Extension& instead of an ID, as
the service may not know about the extension via ID anymore.
BUG=81470
TEST=
Review URL: http://codereview.chromium.org/6931019
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@84165 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
| |
Roll WebKit to 85784.
TBR=morrita
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@84163 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Also remove HandleGoToDashboard, which is also unused.
BUG=none
TEST=none
R=tim@chromium.org
Review URL: http://codereview.chromium.org/6944003
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@84162 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
* Use camelCase for i18n values.
BUG=none
TEST=none
R=csilv@chromium.org
Review URL: http://codereview.chromium.org/6927035
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@84158 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
| |
BUG=(internal i10n bug)
TEST=ran on device
Review URL: http://codereview.chromium.org/6927026
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@84157 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Initial inclusion of Proxy tests
The following tests will test that the Proxy settings are saved properly
using different types of input. It does not test that the fields are actually
functional and that the device is using the proxy. That will come later.
BUG=None
TEST=Run it
Review URL: http://codereview.chromium.org/6914013
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@84155 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
| |
across platfroms.
BUG=none
TEST=none
Review URL: http://codereview.chromium.org/6932021
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@84154 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
| |
tree red.
R=isherman@chromium.org
Review URL: http://codereview.chromium.org/6927033
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@84153 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
| |
BUG=80755
TEST=Install extensions with background pages, make sure View Background Pages wrench menu item has the correct count.
Review URL: http://codereview.chromium.org/6929001
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@84152 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
* Element IDs are dash-form.
BUG=none
TEST=none
R=csilv@chromium.org
Review URL: http://codereview.chromium.org/6929023
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@84151 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
| |
BUG=chromium-os:12641, chromium-os:12851
R=dtu@chromium.org
TEST=
Review URL: http://codereview.chromium.org/6932022
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@84150 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
| |
AutomationProxyMock test. This is
an attempt to fix the flaky ChromeFrame.RequestStart and ChromeFrame.Launch tests
BUG=none
TEST=The ChromeFrame.RequestStart and ChromeFrame.Launch tests should hopefully be more reliable.
Review URL: http://codereview.chromium.org/6927031
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@84149 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
This includes header files for the abstract base classes necessary for lower level parts necessary for video capturing in Chromium as well as an implementation of these classes on Linux. This patch is the first in a series necessary for implementing video capture in the browser process. Tested on Linux.
This patch also includes a fake implementation of VideoCaptureDevice to be used with unittests for testing Video Capture classes.
Patch by perkl@google.com:
http://codereview.chromium.org/6878013/
BUG=none
TEST=none
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@84148 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Close the file browser window in C++, rather than via window.close() in JavaScript
because window.close() is no longer working. This prevents crashes in listeners
that expect to only be called once.
BUG=chromium-os:14524
TEST=Exercise file browser: Hit control-O to open, then cancel the dialog. Hit control-O to open, then open an image. Right-click save an image, cancel dialog. Right-click save an image, save it. Insert USB drive with images, preview individual images, open them, run a slideshow.
Review URL: http://codereview.chromium.org/6935006
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@84146 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
| |
RenderWidgetHostViewWin::UpdateCursorIfOverSelf() sets ExtensionView cursors correctly on WM_SETCURSOR (no mouse capture).
The new View::GetCursor arrow default (crrev.com/83123) conflicted with that, override ExtensionView to no-op.
BUG=81346
TEST=Cursor appearance when mousing over extension bubbles.
Review URL: http://codereview.chromium.org/6929020
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@84145 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
| |
R=dpolukhin@chromium.org
BUG=chromium-os:14618
TEST=Login as a new user, verify that both default image can be selected or photo can be taken and the choice is shown on the login screen/screen lock immediately.
Review URL: http://codereview.chromium.org/6880274
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@84144 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
| |
BUG=chromium-os:14839
TEST=Test sync failures on ChromeOS (shouldn't crash) / check crash reports
Review URL: http://codereview.chromium.org/6927021
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@84143 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Scott's crrev.com/83066 & crrev.com/83776 fix the root crbug.com/80392.
The debug code in question moved the crash callstack from:
views::MenuController::GetMenuPartByScreenCoordinate
ex/ http://crash/reportdetail?reportid=faa16e3bc5833288
Catching the freed object access earlier at:
views::SubmenuView::GetMenuItem
ex/ http://crash/reportdetail?reportid=353971611ff1a9a2
but crash/ doesn't show any similar callstacks at/after 13.0.749.0 (83125).
Please double check crash/, as I don't have much experience with it.
BUG=78792
TEST=none
Review URL: http://codereview.chromium.org/6935005
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@84142 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
| |
BUG=none
TEST=none
Review URL: http://codereview.chromium.org/6933019
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@84141 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
| |
BUG=80056
TEST=Install an extension (e.g. forecastfox), kill the extension from the task manager. The balloon that pops up should show the forecastfox extension.
Review URL: http://codereview.chromium.org/6912034
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@84140 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
| |
prevent it from waiting indefinitely
if the host browser(IE) dies abnormally.
BUG=none
TEST=ChromeFrame net tests should continue to run as before.
Review URL: http://codereview.chromium.org/6930021
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@84139 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
| |
R=eroman@chromium.org
BUG=none
TEST=none
Review URL: http://codereview.chromium.org/6931011
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@84138 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
| |
analysis easier.
BUG=None
TEST=Ran unit tests
Review URL: http://codereview.chromium.org/6931003
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@84137 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
| |
startup
BUG=81438
TEST=none
TBR=glider@chromium.org
Review URL: http://codereview.chromium.org/6933017
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@84136 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
methods.
This was requested by Peter on:
http://codereview.chromium.org/6898053/diff/2001/chrome/browser/automation/testing_automation_provider.cc#newcode1185
BUG=80186
TEST=None
R=pkasting@chromium.org
Review URL: http://codereview.chromium.org/6928004
TBR=tfarina@chromium.org
Review URL: http://codereview.chromium.org/6931016
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@84135 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
| |
TEST=none
BUG=none
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@84134 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
BUG=81396
TEST=tests that use flash plugin
Review URL: http://codereview.chromium.org/6930010
TBR=sunandt@google.com
Review URL: http://codereview.chromium.org/6935007
TBR=maf@chromium.org
Review URL: http://codereview.chromium.org/6927025
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@84133 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
| |
BUG=None
TEST=None
R=atwilson@chromium.org
Review URL: http://codereview.chromium.org/6929019
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@84132 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
window by avoiding descheduling a given command buffer at a couple of
inopportune times. The fact that these fixes work indicates that the
overall IOSurface handling needs work, but these fixes get us back to
a working state.
BUG=80602
TEST=none
Review URL: http://codereview.chromium.org/6927024
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@84131 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
| |
BUG=81485
TEST=See bug description
Review URL: http://codereview.chromium.org/6925003
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@84129 0039d316-1c4b-4281-b951-d872f2087c98
|