GitLab

This reverts commit 84677847.

TEST=ran browser tests on OSX
BUG=103989
TBR=apatrick@chromiu.org

Review URL: http://codereview.chromium.org/8919014

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@114114 0039d316-1c4b-4281-b951-d872f2087c98

BUG=107234
TEST=none
R=csilv

Review URL: http://codereview.chromium.org/8905020

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@114112 0039d316-1c4b-4281-b951-d872f2087c98

BUG=107246
TEST=fixed


Review URL: http://codereview.chromium.org/8921016

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@114111 0039d316-1c4b-4281-b951-d872f2087c98

This reverts commit r112107.

BUG=107269
TEST=as in bug

Review URL: http://codereview.chromium.org/8928001

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@114110 0039d316-1c4b-4281-b951-d872f2087c98

I'm doing this to pick up a naming change for a not-yet-used
D-Bus signal that'll be sent by the power manager
("PowerButtonEvent" becomes "ButtonEvent").

BUG=98328
TEST=ran "gclient sync"
TBR=satorux@chromium.org

Review URL: http://codereview.chromium.org/8919023

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@114108 0039d316-1c4b-4281-b951-d872f2087c98

R=jhawkins@chromium.org
TEST=Use an account with ASP, be able to easily navigate to the ASP input when
prompted on the sync setup overlay and full page promo.
BUG=98928


Review URL: http://codereview.chromium.org/8923008

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@114106 0039d316-1c4b-4281-b951-d872f2087c98

BUG=101099
TEST=
  1. Enable Full Keyboard Access in System Preferences
  2. Create a folder on the bookmarks bar containing many (20+) bookmarks.
  3. Press the Tab key until this folder is selected in the bookmarks bar.
  4. Press the Space key to open the folder.

  After these steps, the folder should look identical to how it looks when you click on it directly.


Review URL: http://codereview.chromium.org/8789010

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@114104 0039d316-1c4b-4281-b951-d872f2087c98

The Active Window is now stored in a property on the RootWindow. Classes wishing to observe changes to this can implement WindowObserver and attach to the RootWindow to be notified of changes in this property.

We provide an ActivationClient interface in Aura for customers to use to set/get the active window, and deactivate a window. This is because setting the active window involves more than just changing the property, there is some additional book-keeping that must be done. The ActivationClient is stored in a property on the RootWindow.

We also provide an ActivationDelegate interface in Aura that window owners can use to be notified of changes in activation state, and to specify whether or not a window can be activated. The ActivationDelegate should be stored on the relevant window in a property.

I moved a lot of Activation-related functionality out of Aura, including all of the unit tests, now on ActivationController, and the associated WindowDelegate implementations which have now become a single TestActivationDelegate implementation.

BUG=none
TEST=unit tests
Review URL: http://codereview.chromium.org/8894018

TBR=ben@chromium.org
Review URL: http://codereview.chromium.org/8926004

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@114101 0039d316-1c4b-4281-b951-d872f2087c98

BUG=107254,107255
TEST=as described in the bug

Review URL: http://codereview.chromium.org/8922014

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@114100 0039d316-1c4b-4281-b951-d872f2087c98

This change causes incognito requests to effectively have a different SSL
session cache from other requests. SSL session information will therefore not
leak into or out of incognito mode.

BUG=30877
TEST=net_unittests

Review URL: http://codereview.chromium.org/8857002

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@114098 0039d316-1c4b-4281-b951-d872f2087c98

The Active Window is now stored in a property on the RootWindow. Classes wishing to observe changes to this can implement WindowObserver and attach to the RootWindow to be notified of changes in this property.

We provide an ActivationClient interface in Aura for customers to use to set/get the active window, and deactivate a window. This is because setting the active window involves more than just changing the property, there is some additional book-keeping that must be done. The ActivationClient is stored in a property on the RootWindow.

We also provide an ActivationDelegate interface in Aura that window owners can use to be notified of changes in activation state, and to specify whether or not a window can be activated. The ActivationDelegate should be stored on the relevant window in a property.

I moved a lot of Activation-related functionality out of Aura, including all of the unit tests, now on ActivationController, and the associated WindowDelegate implementations which have now become a single TestActivationDelegate implementation.

BUG=none
TEST=unit tests
Review URL: http://codereview.chromium.org/8894018

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@114095 0039d316-1c4b-4281-b951-d872f2087c98

BUG=95845
TEST=Print preview a  webpage. Increment the number of copies to 2.
Observe the state of collate checkbox. It should be checked by default.


Review URL: http://codereview.chromium.org/8918013

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@114094 0039d316-1c4b-4281-b951-d872f2087c98

BUG=106381
TEST=NONE
TBR=willchan,acolwell,grt
Review URL: http://codereview.chromium.org/8919018

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@114093 0039d316-1c4b-4281-b951-d872f2087c98

BUG=104393
TEST=tested on aura linux, and chromeos aura


Review URL: http://codereview.chromium.org/8908001

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@114092 0039d316-1c4b-4281-b951-d872f2087c98

Database sizes are low so it is safe to reduce the minimum user text length. This should increase coverage.

To help track potential coverage increases, it is good to remove trivial 0 confidences from the histogram of confidences.

BUG=107212,107213

Review URL: http://codereview.chromium.org/8870006

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@114091 0039d316-1c4b-4281-b951-d872f2087c98

Follow-up to 113090 to see which ModelChangingSyncerCommand triggers
a perf regression.

BUG=97832
TEST=

Review URL: http://codereview.chromium.org/8822017

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@114090 0039d316-1c4b-4281-b951-d872f2087c98

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@114089 0039d316-1c4b-4281-b951-d872f2087c98

Adding metrics to track browser launches per primary/secondary profile. Adding metrics to track number of profiles per machine.

BUG=160771,106774
TEST=none

Review URL: http://codereview.chromium.org/8890054

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@114088 0039d316-1c4b-4281-b951-d872f2087c98

BUG=none
TEST=builds on gcc 32 bit for linux
Review URL: http://codereview.chromium.org/8894028

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@114087 0039d316-1c4b-4281-b951-d872f2087c98

BUG=None.
TEST=Click avatar menu and press Tab key to navigate through the menu.
Selected menu items should have a focus ring.

Review URL: http://codereview.chromium.org/8921003

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@114086 0039d316-1c4b-4281-b951-d872f2087c98

This code was platform independent so could just be copied over into the
platform independent file with no changes.

Review URL: http://codereview.chromium.org/8899015

TBR=brettw@chromium.org
Review URL: http://codereview.chromium.org/8926003

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@114085 0039d316-1c4b-4281-b951-d872f2087c98

Unit tests now instantiate their own AudioManager and can choose to
use the default one or provide their own mock implementation without
having to worry about conflicting with the singleton.

The teardown sequence of the AudioManager and its thread has been cleaned
up significantly and I don't think it has been completely tested before as
the audio thread was terminated before all objects that belonged to the
thread had a chance to do cleanup.  The AudioManager unit tests do not use
the actual audio thread, so this part seems to have been left out.

In Chrome, the AudioManager instance is now owned by BrowserProcessImpl
and always constructed on the UI thread.  This instance is then shared
in the same way that several other 'manager' type objects are shared to
'content' code, via content::ResourceContext.  Audio specific classes
do though receive a direct pointer to the AudioManager and are required
to do proper reference counting if they need to hold onto the instance.
I chose to use the ResourceContext rather than direct use of g_browser_process
to avoid requiring another singleton when writing relatively simple tests
that touch the AudioManager.

I added a couple of safeguards to guard against future regressions:
- Not more than one instance of the AudioManager should be created.
- The AudioManager should not be addrefed by its own thread.  This
  can basically become a circular reference and prevent deterministic
  shutdown.

Reviewers:  Of course you're free to review everything,
but here's the breakdown in terms of the bare minimum from
the standpoint of "Owners approval".  I'm asking Henrik to be the
main reviewer of the entire patch (sorry!).

Henrik: Everything minus the below, but it would be great if you could
        take a look at the whole thing, specifically media/audio.

Pawel:  I'd like you to take a generic look at this approach.
        The key areas as far as the singleton itself goes are in
        media/audio/audio_manager[_base].* and
        chrome/browser/browser_process*.*

Satish: content/browser/speech/*
        media/audio/audio_manager_base.* (new reference counting code)

Andrew: content/browser/renderer_host/media/*
        content/renderer/media/webrtc_audio_device_unittest.cc (Owner)

Avi:    content/browser/renderer_host/render_process_host_impl.cc
        content/browser/resource_context.*

William: chrome/browser/profiles/profile_io_data.cc
	 chrome/browser/browser_process*.*

Robert:  This is basically a heads up. I hope that I didn't break the OpenBSD
         implementation, but unfortunately I have no way of knowing for sure.

Shijing: Please take a look at AudioManagerLinux.  I replaced the set of
         active streams with a simple counter.

BUG=105249
TEST=content_unittests, media_unittests, browser_tests.
Review URL: http://codereview.chromium.org/8818012

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@114084 0039d316-1c4b-4281-b951-d872f2087c98

This application icon index values are being changed from ints to StringOrdinals to decrease the potential number of syncs required when icons are moved, by only needing to change 1 StringOrdinal instead of all the integers.


BUG=61447
TEST=Open up an instance of chromium with multiple application icons and ensure
that icons can still have their page and order changed. There should be no
behaviour change.


Review URL: http://codereview.chromium.org/8198003

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@114083 0039d316-1c4b-4281-b951-d872f2087c98

This bugs in isMark and isLetter lead 'init' feature to be applied
even when a syllable is not at the beginning of a word.

See http://www.microsoft.com/typography/otfntdev/bengalot/features.htm
for 'init' feature application in Bengali.

It turned out that the upstream also fixed this (see https://bugs.freedesktop.org/show_bug.cgi?id=30319) exactly the same way.

Change the upstream reference to 93d8573ddcf97648983a1c1b5cdda8a3ec6d4432
in README.chromium and remove 2 patches in chromium.patch because
they're all landed in the upstream @93d8573ddcf97648
Two removed patches are:

  - Fix OOB access (in Tibetan shaper) @ff0612c2e7df1b86fc
  - Fix crash! (in harfbuzz-gpos.c) @81c8ef785b079980a

In addition, the following upstream fix is included by going to 93d8573ddc.

  - Fix bug in contrib/harfbuzz-unicode.cc @93d8573ddcf9

Webkit bug to add a layout test for 'init' is
   https://bugs.webkit.org/show_bug.cgi?id=74310

BUG=107031
TEST=With Lohit Bengali font installed (the default on CrOS devices), go to http://i18nl10n.com/fonts/beng3.html and the result should be the same as the left in the screenshot attached to the bug instead of the right. A webkit layout test will be added and will be mentioned in the bug later.
Review URL: http://codereview.chromium.org/8895014

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@114082 0039d316-1c4b-4281-b951-d872f2087c98

This is a reland of r113823, hopefully without breaking the Pepper tests.

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@114081 0039d316-1c4b-4281-b951-d872f2087c98

This goes most of the way to re-enabling chrome_frame_net_tests by making it a content::BrowserMainParts.  The tests are still disabled until we are able to test them on IE8 and older; on IE9 all tests currently pass except for a few that hang.

Additionally, this makes chrome_frame_net_tests determine protocol version using the exact same version information as npchrome_frame.dll does, so that version mismatch due to the RC dependency problems we have sometimes seen should not happen any more with this test suite.

While in there, I also updated the poor_mans_trybot.bat script.

BUG=105435

Review URL: http://codereview.chromium.org/8894008

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@114080 0039d316-1c4b-4281-b951-d872f2087c98

The current code is for update time run, so it can generate correct list per OS.  However, we changed it to run locally and check the generated list in, so we need to have OS conditions for tests.

Also, add DEBUG/RELEASE modifier so we can mark test failures better.

BUG=
TEST=gpu_tests built
R=kbr
Review URL: http://codereview.chromium.org/8885003

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@114078 0039d316-1c4b-4281-b951-d872f2087c98

This code was platform independent so could just be copied over into the
platform independent file with no changes.

Review URL: http://codereview.chromium.org/8899015

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@114076 0039d316-1c4b-4281-b951-d872f2087c98

BUG=100026,107181
TEST=none
TBR=vrk

Review URL: http://codereview.chromium.org/8920010

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@114075 0039d316-1c4b-4281-b951-d872f2087c98

If the user had a custom profile name then clicking on "Edit profile" would reset the name to their GAIA name.

The problem was that we were incorrectly detecting that the user had changed their profile picture to their GAIA picture. Clicking on the GAIA picture causes us to set the profile name to the GAIA name.

Fix was to only send profile picture changed events if the user actually changed their profile picture.

BUG=106382
TEST=Reproduced the bug. Applied my fix and verified that it no longer reproduced.


Review URL: http://codereview.chromium.org/8905001

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@114073 0039d316-1c4b-4281-b951-d872f2087c98

base::Bind: Convert chrome_benchmarking_message_filter.cc.

BUG=none
TEST=none
R=csilv,ajwong

Review URL: http://codereview.chromium.org/8897026

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@114072 0039d316-1c4b-4281-b951-d872f2087c98

Test gathers the EPP and TTP metrics for video playback under
constrained network conditions.  The test uses the Constrained Network
Server (CNS) to accomplish this.

 - Starts CNS.
 - Iterates over test matrix of network constraints.
 - Reports results in a manner consumable by chromium.perf_av.
 - Stops CNS.

BUG=106257
TEST=Ran test locally.


Review URL: http://codereview.chromium.org/8802030

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@114071 0039d316-1c4b-4281-b951-d872f2087c98

Also, move some URL constants to url_constants.h/cc.

BUG=103324
TEST=Go to chrome://settings/personal. Sign in to sync. Choose "Sync Everything". The learn more link should go to http://...answer=185277.


Review URL: http://codereview.chromium.org/8589014

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@114070 0039d316-1c4b-4281-b951-d872f2087c98

BUG=77155, 103574
TEST=none

Review URL: http://codereview.chromium.org/8872036

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@114067 0039d316-1c4b-4281-b951-d872f2087c98

BUG=106795
TEST=Virtual Me2Me still works

Review URL: http://codereview.chromium.org/8863010

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@114066 0039d316-1c4b-4281-b951-d872f2087c98

The new truncate mode is needed by my upcoming changes to
rewrite CanvasSkia::DrawText().

Part of the change is changing the binary search code to a
more traditional form that also makes the truncate mode
work correctly. This is covered by the new unit tests.

BUG=105550
TEST=New and existing tests in text_elider_unittest.cc.
Review URL: http://codereview.chromium.org/8917011

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@114063 0039d316-1c4b-4281-b951-d872f2087c98

BUG=105811
TEST=Inspect checkmark position in settings window for all platforms.
Review URL: http://codereview.chromium.org/8918012

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@114060 0039d316-1c4b-4281-b951-d872f2087c98

levels requirested. Also ClearLevel now uses glTexSubImage2D to clear the level if the
texture is immutable.  Finally, rearranged the code in doTexSubImage2D to correctly
bypass clearing a level if the bitmaps supplied matches the dimensions of the texture level.
BUG=106894
Review URL: http://codereview.chromium.org/8872061

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@114059 0039d316-1c4b-4281-b951-d872f2087c98

New in this roll: ASan! (for mac and linux64 for now)

BUG=none
TEST=none
TBR=hans

Review URL: http://codereview.chromium.org/8918007

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@114055 0039d316-1c4b-4281-b951-d872f2087c98

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@114054 0039d316-1c4b-4281-b951-d872f2087c98