GitLab

I can't repro locally. I tried with a release/debug build and with
full/light.

BUG=400495
TEST=none
R=zhaoqin@chromium.org
TBR=zhaoqin@chromium.org

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

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

When signing in from the new avatar bubble, since the inline template has no "choose what to sync" link, thus Chrome shows an inline confirmation in the avatar bubble upon signin completes and blocks sync until the bubble is closed.

However, when some modal warnings show up before signin completes, they steal the focus thus the avatar bubble wont be able to show confirmation for sync settings and sync will be blocked infinitely.

A temporary workaround is to just start sync immediately if a modal warning is shown, since in all cases the dialog mentions that user data will be synced.

I will add a sync settings link to all modal dialogs later so that user could configure sync settings before sync starts.

BUG=435423005

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

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

BUG=394434

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

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

The histograms don't seem to be used anymore (do not show up in UMA). The
hang-detector and about:profiler produce relevant useful data for detecting jank.

BUG=none

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

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

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

http://build.chromium.org/f/chromium/perf/dashboard/ui/changelog_blink.html?url=/trunk&range=179700:179716&mode=html
TBR=jbroman@chromium.org
BUG=

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

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

record & replay to work with AGSA

BUG=396157

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

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

BUG=none

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

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

Depends on https://codereview.chromium.org/441883003/ and an additional
downstream patch

BUG=400694

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

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

When finding a dependency chain, the header check will now prefer chains which
forward direct-dependent configs if the target which provides a header has
one. Previously an error would be produced because the chain which was found
did not forward configs, even though another chain did.

A breadth-first search is used instead of a depth-first search because short
chains are more likely to be intended by the author, and make for both less
work and clearer diagnostic messages.

This reduces the size of 'gn check' output on the chromium tree and the time
to execute. (Output directed to 'wc -l' and /dev/null, respectively.)

  Before: 104685 lines, 542.36s user, 35.573s total
  After: 36977 lines, 7.10s user, 1.526s total

Manual inspection reveals that the line reduction appears to be from the
removal of spurious errors when IsDependencyOf selected an indirect chain
which does not forward direct-dependent configs, even though a direct
dependency existed. (About 7 cases were examined manually; sheer volume makes
full manual verification infeasible.)

BUG=391505
TEST=HeaderCheckerTest, manual

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

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

Additionally, promotes the API to dev channel since it is appropriately
restricted.

BUG=400436

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

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

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

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

> Set the web content's initial size only if the tab isn't selected in session restore.
> 
> BUG=378924
> 
> Review URL: https://codereview.chromium.org/392293002

TBR=oshima@chromium.org

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

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

TBR=sky

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

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

BUG=398337
NOTRY=true
TBR=zmo@chromium.org

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

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

It uses the //net stack, ergo it needs to have a MessageLoop

BUG=395356

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

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

This makes SkPicture::clone() a cheap ref, meaning impl-side painting will render
tiles concurrently from the same source SkPicture rather than from clones.

After testing for a few months in Skia, we believe SkPicture is thread safe enough
to try this out on the Chrome bots.  Please revert this at the slightest sign of
crash, flaky drawing, etc.

CQ_EXTRA_TRYBOTS=tryserver.chromium.linux:linux_layout_rel

BUG=

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

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

Introduce a SystemUIResourceManager class for loading and providing access to
shared UIResourceId's.  Wire this class to the existing OverscrollGlow effect,
using the shared id's with a UIResourceLayer for both glow and edge layers.

This patch is a subset of powei@'s original SystemUIResource patch found at
https://codereview.chromium.org/377013002/.

BUG=326326

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

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

BUG=398888

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

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

TBR=sky

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

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

> Reland r285211: "Debugging RawChannelWin: replace DCHECK with CHECK and record write stats."
> 
> But clang-formatted (and after the changed files were clang-formatted).
> This should be reverted on the same schedule as the original change (no
> later than August 1, 2014).
> 
> > Debugging RawChannelWin: replace DCHECK with CHECK and record write stats.
> >
> > This is a temp change for debugging purpose. It should be reverted once
> > we fix the issue or find out the change doesn't reveal anything. (No later than Aug 1, 2014.)
> >
> > BUG=385795
> > TEST=None
> 
> BUG=385795
> TBR=yzshen@chromium.org
> 
> Review URL: https://codereview.chromium.org/430473004

TBR=viettrungluu@chromium.org

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

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

See VLOG(1) and VLOG(2) logging:
TEST=adb shell setprop log.tag.ChromiumNetwork VERBOSE

See VLOG(1) logging:
TEST=adb shell setprop log.tag.ChromiumNetwork DEBUG

See NO (only FATAL) logging:
TEST=adb shell setprop log.tag.ChromiumNetwork NONE

BUG=400956

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

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

There is only one user of this class. The new implementation is in
base/files/scoped_file.h

R=viettrungluu@chromium.org

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

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

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

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

TBR=sky

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

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

These were only used by one test file, so I moved the implementation there. I
considered updating the test, but the way the test uses wstrings seems
reasonable.

R=rvargas@chromium.org, rvargas

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

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

This CL includes following changes.
- Use system default header, rather than customized Maximize/Minimize/Close buttons.
- Current location is displayed on the left side of toolbar.
- Only the selected file's information is displayed on the preview panel.
- Change-view button is moved from gear menu to the right side of toolbar.
- Search button is placed on toolbar as a button. It expands on click.

BUG=397222,308445
TEST=run browser_tests, manually tested

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

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

Stop honoring the Finch enable/disable groups, but still honor explicit flips
of the flags in chrome://flags.

BUG=383169

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

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

There is a "FIXME: Remove once calls on the Chromium-side have been
removed" for volume() in WebMediaPlayerClient.h. preload() is
similarly strange layering-wise, so get rid of that as well.

Simply store the values when they cannot immediately be forwarded to
the internal object.

In the case of WebMediaPlayerImpl::setVolume(), the value can always
be used immediately, so the setVolume() in DoLoad() appears to simply
have been unnecessary.

BUG=401415

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

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

BUG=401501

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

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

BUG=none
NOTRY=true

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

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

This should prevent problems such as described in

  [chromium-dev] PSA: clobber build needed for IDR_INSPECTOR_*
                 compile errors in blink_platform_impl.cc

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

BUG=400860
TBR=brettw

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

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

ChromeUserSelectionScreen::Init() used to access the |users_| member of
its parent class even though it already knows the user list. This CL
removes the unnecessary access and marks |users_| as private.

BUG=398511
TEST=No compilation errors

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

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

BUG=333201

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

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

The Intent.getClipData() method is only available in JellyBean MR2 and
higher. The code does a correct version check, but the lint checker is
not satisfied with this check.

BUG=401333,348912

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

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

This code is no longer needed as process priority is no longer being changed in-process as this was causing tab switching latency.

BUG=398103,381820

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

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

This CL fixes tombstones.py architecture detection in two ways:
 - If the tombstone contains an ABI line, use this in preference to the devices
   default ABI, since some devices support multiple ABIs (e.g., 'arm' and 'arm64').
 - Map the abi to an arch which the stack tool accepts, e.g., armeabi-v7a -> arm, arm64-v8a -> arm64.

NOTRY=true

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

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

This patch exposes a method to kill a child process service of a given
pid, allowing regression integration test for http://crbug.com/399521 on
the embedder side.

BUG=399521

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

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

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