GitLab

BUG=None
TEST=Compile with gcc.

Review URL: https://chromiumcodereview.appspot.com/23935006

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

On Android, the user may disconnect from their Google account and reconnect multiple times within the same session. This case wasn't considered properly when developing the CloudPolicyInvalidator.

When the account is disconnected, the CloudPolicyCore is disconnected before the CloudPolicyStore is reset. When the store is reset, the invalidator unregisters for the policy object and makes a callback to indicate that invalidations are disabled. However, since the CloudPolicyCore is disconnected at this point, a crash is occurring because the code blindly attempts to use objects connected to the core.

Also, when the account becomes reconnected, the refresh scheduler is restarted. This caused the invalidator to be initialized again, which is unexpected.

The solution to these issues is to refactor the CloudPolicyInvalidator to support starting and stopping. The invalidator will listen to events published by the CloudPolicyCore to determine when to start and stop. The invalidator starts when the refresh scheduler starts and stops when it stops.

Also, when CloudPolicyInvalidator::Shutdown is called, outstanding weak pointers held in callbacks should be invalidated. This may not cause any issues since it is likely no messages are pumped on the thread in between the calls to Shutdown and the destructor, but should be fixed.

BUG=280345,272574,280536

Review URL: https://chromiumcodereview.appspot.com/23592017

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

Reverting per the original change description.

> [temporary M30 fix] Disable dragging tabs with web contents modal dialogs displayed
> 
> The full fix for the crashes in the associated bugs in trunk is too
> risky to merge into M30. After discussions among kareng and sky, the
> best approach was to disable dragging of tabs while a web contents modal
> dialog is displayed, to prevent the crashes. This only affects Chrome
> OS and will be reverted after testing in Canary and merging to the
> branch.
> 
> BUG=275274, 274856
> R=sky@chromium.org
> 
> Review URL: https://codereview.chromium.org/23698003

TBR=wittman@chromium.org
BUG=281831

Review URL: https://codereview.chromium.org/23814008

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

Set accept_events to true when we create top level windowsin desktop AURA. This ensures that we don't end
up using WS_EX_TRANSPARENT when the actual window is created. This causes painting and flickering issues in AURA.
Will investigate that later.

BUG=282628
R=sky@chromium.org

Review URL: https://codereview.chromium.org/23493022

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

This removes the commandline used on Android and adds references
to each particular bug in the json workaround list.

BUG=280609
R=piman@chromium.org, zmo@chromium.org

Review URL: https://codereview.chromium.org/23503038

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

This allows the Watchdog app to look up history by day. It also gets rid of a redundant ReadData method (now wholly replaced by ReadFilteredData).

BUG=241672
R=mpcomplete@chromium.org

Review URL: https://chromiumcodereview.appspot.com/23980002

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

R=tonyg@chromium.org
BUG=

Review URL: https://chromiumcodereview.appspot.com/23936003

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

- Computation of dropped frames now correctly accounts for multi-frame drops.
- A welcome side-effect is that variance of the percentage_dropped metric
  is reduced significantly.
- Heads-up for the perf sherrifs: percentage_dropped will increase quite a bit
  with this patch.

R=tonyg@chromium.org, nduca@chromium.org
BUG=264308

Review URL: https://chromiumcodereview.appspot.com/23998002

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

Add UI elements for extension runtime (javascript) errors on the chrome:extensions page.  Include a stack trace and, if possible, links to the source code.

TBR=finnur@chromium.org
BUG=21734

Review URL: https://chromiumcodereview.appspot.com/23624002

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

Move the role of handling browser item status from ChromeLauncherController
to BrowserShortcutLauncherItemController.

R=skuhne@chromium.org
BUG=NONE
TEST=browser_tests, unit_tests

Review URL: https://chromiumcodereview.appspot.com/23534022

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

BUG=272732
TEST=Try visiting any IDN site and confirm no punycode is shown in history.

Review URL: https://chromiumcodereview.appspot.com/22862006

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

R=kalman@chromium.org

Review URL: https://codereview.chromium.org/24054003

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

Depends on: https://codereview.chromium.org/22912008/

R=vandebo

BUG=151701

Review URL: https://chromiumcodereview.appspot.com/23499006

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

The file has exec bit set. This breaks the build.

> Added file to verify domain ownership.
> 
> R=kalman@chromium.org
> 
> Review URL: https://codereview.chromium.org/23583040

TBR=awatson@chromium.org

Review URL: https://codereview.chromium.org/23480047

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

Signature changed with new roll.

BUG=239350
TEST=none
TBR=zhaoqin

Review URL: https://codereview.chromium.org/23626003

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

Using libyuv for all platforms except Android and iOS. The implementation of OnIncomingCapturedFrame is #ifdef'ed.

BUG=282282

Review URL: https://chromiumcodereview.appspot.com/23870002

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

Nothing to show in 'Other Bookmarks' when the folder is empty and not
visible in the bookmarks bar. So, prevent moving selection to the
hidden 'Other Bookmarks' folder to avoid a crash.

BUG=275284

Review URL: https://chromiumcodereview.appspot.com/22887030

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

Configure Views constrained web dialogs to be closed when interstital
WebUI is displayed (e.g. for SSL warnings prior to page load). Affects
these dialogs:

 * print preview
 * SSL certificate viewer (Chrome OS only)
 * profile sign-in confirmation dialog

BUG=240575
R=sky@chromium.org, thestig@chromium.org

Review URL: https://codereview.chromium.org/23819028

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

Review URL: https://chromiumcodereview.appspot.com/23814006

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

BUG=282464
TBR=ygorshenin@chromium.org

Review URL: https://chromiumcodereview.appspot.com/23571009

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

This is direct port of the current content.  No editing yet.

R=eliben@chromium.org

Review URL: https://codereview.chromium.org/23463019

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

Add -W to SHPINXOPTS to make warnings into error.

Add presubmit check that makes sure sphinx is happy
with the content.

Use 2 spaces to indent code.

Add a missing document.

Fix document links in examples.rst.

Use 'out' for build product data to be constent with
chrome and native_client.

Add 'all' target so that 'make' on its own will work
(I'm lazy)

R=binji@chromium.org, eliben@chromium.org

Review URL: https://codereview.chromium.org/23531031

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

R=kalman@chromium.org

Review URL: https://codereview.chromium.org/23583040

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

The "dimmer" cannot be parented by the shelf, but it needs to overlay it. The existing OnBoundsChanged call will only detect size changes since the position relatively to its parent never changes. Therefore the shelf's window needs to be observed as well to get absolute location changes.

BUG=285415
TEST=visual

Review URL: https://chromiumcodereview.appspot.com/24011002

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

These assets where added when working on RequestAutoComplete dialog
and are no longer needed.

NOTRY=true

Review URL: https://chromiumcodereview.appspot.com/23536037

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

We're seeing crashes in history::VisitDatabase::GetStartDate coming from
HistoryBackend::ExpireHistoryBetween where db_ is NULL. This change
mitigates this by two avenues: first_recorded_time_ is cleared when the
DB is closed (so that no other code that checks it blows up), and
GetStartDate is no longer called by ExpireHistoryBetween when db_ is
NULL.

Evidence suggests that this bug is responsible for browser crashes at
shutdown that result in a corrupt profile dialog on the next launch.

BUG=275916, 273832, 165282, 107613, 36588
R=brettw@chromium.org

Review URL: https://chromiumcodereview.appspot.com/23861005

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

Currently, the scrollbar layer animation is triggered at the end of every scroll
gesture. However, as not every scroll gesture actually scrolls a layer, this may
trigger extraneous animations, e.g., when the user overscrolls, or presses and
lifts their finger without scrolling.  Only trigger the scrollbar animation at
the end of a scroll gesture if the gesture caused the layer to scroll.

BUG=285771

Review URL: https://chromiumcodereview.appspot.com/23978008

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

Adding an early return in setSelection to match BaseInputConnection.
Adding clipping in setComposingRegion to match BaseInputConnection.

BUG=286348

Review URL: https://chromiumcodereview.appspot.com/23566019

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

This is a binary-only CL for adding the printer icon, since the commit queue
doesn't handle binary files

BUG=286157
R=vitalybuka@chromium.org

Review URL: https://codereview.chromium.org/23480039

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

- When initializing the WebContents for Android pull the
WindowId from the TabModel and push it to the SessionTabHelper.

BUG=277135
NOTRY=true

Review URL: https://chromiumcodereview.appspot.com/23792002

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

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

This CL adds an interface for DesktopMediaPickerModel
(DesktopMediaPickerModelInterface). It allows the model to be
faked/mocked/stubbed in unittests.

BUG=237907

Review URL: https://chromiumcodereview.appspot.com/23447012

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

> Revert 220228 "FFmpeg roll for M31."
> 
> Causing compile failure on "Google Chrome ChromeOS"
> http://build.chromium.org/p/chromium.chrome/builders/Google%20Chrome%20ChromeOS/builds/57680
> 
> > FFmpeg roll for M31.
> > 
> > BUG=none
> > TEST=asan media_unittests, ffmpeg_regression_tests + manual.
> > 
> > Review URL: https://chromiumcodereview.appspot.com/23172019
> 
> TBR=dalecurtis@chromium.org
> 
> Review URL: https://codereview.chromium.org/23753002

Builder has been upgraded to handle gcc 4.6+ constructurs.

TBR=hashimoto@chromium.org
BUG=263385,281764

Review URL: https://codereview.chromium.org/23703009

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

Reason for revert: Use-after-free heap errors caught by asan.
See bug 286483.

> Make AppShimHostManager a RefCountedThreadSafe.
> 
> This prevents a crash when Chrome shuts down immediately and 
> AppShimHostManager is destructed before InitOnFileThread is run.
> Note this means AppShimHostManager can stay around longer than
> BrowserProcessPlatformPart during initialization.
> 
> Review URL: https://chromiumcodereview.appspot.com/20065004

BUG=286483,242941
TBR=jackhou@chromium.org

Review URL: https://codereview.chromium.org/24052002

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

Configure the collected cookies dialog to be closed when interstital
WebUI is displayed (e.g. for SSL warnings prior to page load).

BUG=240575
R=sky@chromium.org

Review URL: https://codereview.chromium.org/23432007

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

Comment is not applicable anymore.

BUG=none
TEST=none
R=pkotwicz@chromium.org
TBR=pkotwicz@chromium.org

Review URL: https://codereview.chromium.org/24050002

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

BUG=None
TEST=*ResizerTest.*

Review URL: https://chromiumcodereview.appspot.com/23645008

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

This fixes a bug where, when using two thumbnails in the same iframe, they would be reordered if their visibility changed. This would only happen on the second and further NTPs, perhaps on relayout by Chrome.

This changes it so that they are superposed, which is how it should have been all along. Putting position: absolute will make both anchor on body.

BUG=None

Review URL: https://chromiumcodereview.appspot.com/23619019

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

This CL updates the version of PPB_Audio and PPB_Audio_Callback.

BUG=240900
TEST=updated test_audio.{h,cc}

Review URL: https://chromiumcodereview.appspot.com/22320004

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

https://chromiumcodereview.appspot.com/22006002/ moved the apps-v2 class from
the <body> to the <html> node, but did not update the CSS.

BUG=263090
R=lambroslambrou@chromium.org

Review URL: https://codereview.chromium.org/23899005

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