GitLab

BUG=None
R=stevenjb@chromium.org

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

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

BUG=374392
TBR=erg
NOTRY=True

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

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

In HWNDMessageHandler::ShowWindowWitState(), when ui::SHOW_STATE_DEFAULT
is passed as state, the window grabs activation after being shown. The
correct behavior would be to activate the widget if and only if the
can_activate flag is set. HWNDMessageHandler::Show() method already does
this since r244605.
Also, the DefaultWidgetDelegate should honor InitParams::can_activate
flag.

BUG=340736

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

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

Each time we add a new platform in the default build, products that want
only a single platform have to add an entry to GYP_DEFINES to turn it back
off.

This introduces a new setting, ozone_auto_platforms. If this is turned off
in GYP_DEFINES, we'll only build platforms that are explicitly enabled.

TEST=runhooks
BUG=none

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

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

In a multi-monitor setup, when switching from extended mode to mirror mode,
it is possible for the external monitor to receive events between when its
RootWindowController is Shutdown()ed (which destroyes the screen-position
client) and the root-window is destroyed. So make sure the client is there
before trying to use it to convert the event's location to screen space.

BUG=358266
R=oshima@chromium.org

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

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

R=enne@chromium.org, piman@chromium.org, enne, piman
BUG=374287

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

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

this is the biggest chunk I can pull out vaguely cleanly.  It does contain a
couple of fake calls to keep the compiler happy.
This CL also makes SetUpMetadata a member in order to ease testing in a later CL.

This depends on https://codereview.chromium.org/261843004/.

BUG=108012

Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=270016

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

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

R=avi@chromium.org

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

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

There's only one caller of this method now that CompositeAndReadback
is gone, so just inline the code to make things easier to read.

R=enne@chromium.org, enne
BUG=374287

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

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

BUG=none
TBR=ericu@chromium.org, erg@chromium.org
NOTRY=true
NOTREECHECKS=true

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

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

Reason for revert:
May be bad.

Original issue's description:
> Enable QUIC 17.
> 
> Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=270592

TBR=rtenneti@chromium.org
NOTREECHECKS=true
NOTRY=true

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

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

This moves the logic for nexe downloading outside the trusted plugin. This is
part of a series of changes to remove FileDownloader and its associated logic.

I've tried to make this change small so it will be easy to review. A followup
change will clean up some of the accounting logic post-download to make this
path simpler and easier to follow.

BUG=370556
R=bbudge@chromium.org

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

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

This may be too restrictive, but seems like a good start.

BUG=365012
TEST=covered by tests
R=ben@chromium.org

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

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

The original version required 34 float ops (16mul + 11sub + 5add + 2div).
This version removes some of the intermediate dot products, reducing the
max number of operations to 19 (6mul + 2div + 11sub).  It also avoids using
doubles, warranted by the removal of several intermediate operations that might
otherwise incur significance loss.

In local (release build) benchmarks, this reduced the typical CPU cost of a call to
QuadF::Contains(...) by 2/3, both with GCC and Clang on an ARM v7 device.

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

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

BUG= 348437
TEST=Manual, see bug

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

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

Original files: content/public/common/favicon_url.*

BUG=359586

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

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

BUG=374022
R=isherman@chromium.org

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

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

Some cleanup.

BUG=373862
TEST=manual
NOTRY=True

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

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

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

This pulls in the following Native Client changes:

r13168: (mseaborn) Non-SFI NaCl: Enable running more tests with nonsfi_loader
r13169: (dyen) Make toolchain build upload linux 64 bit hosted toolchain.
r13170: (dschuff) Add bot selectors for new PNaCl mac and windows bots
r13171: (dyen) Removed old toolchain management files.
r13172: (mseaborn) PNaCl: Remove unnecessary use of "-Ttext" option when doing bitcode linking
r13173: (petarj) [MIPS] Switch to newer versions of the tools
r13174: (noelallen) Create GROUPS for trusted and untrusted targets to avoid using default toolchain...
r13175: (noelallen) Pin latest bionic version number.
r13176: (jvoung) Remove deprecated NaClDescIoDescMakeFromHandle.
r13177: (jvoung) Add more detailed perf markers to clarify perf of NaCl ELF file load.
r13178: (petarj) [MIPS] Fix incorrect use of assembly macro in the test
r13179: (bradnelson) Use SO_EXCLUSIVEADDRUSE on Windows instead of SO_REUSEADDR.
r13180: (mseaborn) command_tester.py: Don't write output file if the test failed
r13181: (mseaborn) readlink() syscall: Add missing test and fix to work on larger buffer sizes

BUG=none
TEST=browser_tests and nacl_integration

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

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

Fix the "gn check" header errors.

Remove the extra "gles2_utils" target which "gn check" flags as an error. This target is actually inseparable from "command_buffer_common" but exists this way to prevent cycles in GYP files (but not GYP targets). GN doesn't have this problem, so I merged them.

Moved the GPU targets out into build files closer to their files (gpu/command_buffer/client and .../common). This leaves a BUILD.gn file in gpu/ with a comment listing the forwarding locations, since that's where people will look for a BUILD.gn file first.

Delete the duplicate gfx_unittests target, leave a comment for where it is in case somebody does a search for that name.

Make webp compile on Windows

Make ui/gfx compile on Windows. There were some duplicate symbols for ozone. I changed some of the filtering around to do += instead of exclusion. I think in these cases it's easier to follow and maintain.

BUG=
R=piman@chromium.org, urvang@google.com

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

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

Adds a meta-target "//base/allocator" that forwards to tcmallor or nothing depending on the build configuration.

Write tcmalloc target definition.

Write base unit tests prefs, and some support targets from base.gypi.

Fix a bunch of added files in the base target that were exposed by linking the unit tests.

Removes toolkit_uses_gtk which was removed from the GYP build.

BUG=
R=scottmg@chromium.org

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

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

This adds special handling for variables that begin with underscores, which is inspired by Dart. Such variables are not imported when doing an import, which gives .gni files a way to have intermediate private variables that won't pollute the scopes of the files that include them.

This also applies to the root build config, which can have private values now.

Adds some missing unused variable checks. This was disabled because processing imports would mean all imported variables were unused, and files not using all of them would get unused variable errors. This adds the option to mark merged values on a scope as used, which is used for imported values.

BUG=341738
R=cjhopman@chromium.org

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

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

- Window manager provided window borders
- Custom shapes specified via views::Widget::SetShape()

BUG=None
TEST=None

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

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

BUG=371087

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

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

Initial sets of fixes for themes to work well with HiDPI Windows. More changes coming in a separate CL.

oshima, Please review everything.
sky : For owners review.

1. Removed the PlatformScaleImage function from the ResourceBundle class and the corresponding
   code to scale images. With automatic scaling support in ImageSkia this is no longer needed.

2. The other change is in the NewTabButton class to use floating point scales everywhere instead
   of converting from scale factor to scale using different functions which causes problems.

BUG=362979
R=oshima,sky

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

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

This would let us link it to the inspected page in DevTools, as we already
do with other events coming from LTHI.

BUG=361045

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

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

TBR=zhaoqin@chromium.org
BUG=NONE

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

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

BUG=None

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

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

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

NOTRY=true
BUG=369680

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

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

undo https://codereview.chromium.org/288033020

R=mallinath@chromium.org, tomfinegan@chromium.org

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

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

BUG=350233

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

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

For privacy reasons, we want Domain Reliability uploads to be
cookieless. Add a unit test for this.

BUG=356791

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

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

Few changes involved:

1) Using TestBookmarkClient to create BookmarkModel.
2) Removes unnecessary TestBrowserThreadBundle
3) With the above changes TestingProfile and BookmarkModelFactory
goes away.

BUG=367834
TEST=components_unittests --gtest_filter=BookmarkModel*
R=blundell@chromium.org, sdefresne@chromium.org

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

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

Impl side posts callbacks for releasing resources on
SingleThreadProxy::Stop, and there is no reason to have these be real
post tasks.

BUG=344087

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

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

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

Add arm64-specific build rules
https://codereview.chromium.org/290653003

R=tomfinegan@chromium.org

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

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

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

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

BUG=374305
R=boliu@chromium.org

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

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