GitLab

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

Cr-Commit-Position: refs/heads/master@{#300053}

From typ commit df1f462ec978d47c7f092e86f4be3fafc7cc446f:

    Rework typ to handle unimportable modules better.

    multiprocessing on windows requires that the __main__ module
    be importable in order to work properly. If developers are mostly
    working and testing on unix, it's possible for them to not realize this
    and attempt to call typ from a wrapper script that may not be
    importable, and then get unexpected errors.

    This change makes the typ entry points check if it is being called
    from an unimportable __main__, and error out if so. Developers
    must then specify several different ways to work around this by passing
    the 'win_multiprocessing' flag to typ.main() or typ.Runner.main().

    The flag takes one of the values specified in the typ.WinMultiprocessing
    class:

      ignore -  The tests are not expected to run on Windows, so ignore
                the fact that __main__ is not importable. (If the tests
                are actually run on Windows, it will err out).

      run_serially - Run serially (one at a time) on Windows, regardless
                of how many cores are available (hence multiprocessing is
                not used).

      spawn -   Typ will convert the args it was passed back into a command
                line, and spawn off an invocation of itself on Windows to actually
                execute the tests in parallel (Since typ itself is importable,
                this works fine). This mode cannot be used in conjunction
                with contexts or setup and teardown functions at this time.

      force -   This is like 'spawn', except that it forces the spawn even
                on non-Windows platforms. This mostly exists for testing.

    As part of this change, we also no longer expose spawn_main() as a public
    entry point (main() handles spawning instead), and merge the cmdline.py
    code directly into runner, as it didn't make sense to split things
    across the two modules once runner had to do the importability checking.

    We also add a host.call_inline() wrapper that is a dumb wrapper
    around subprocess.call(); it does not do the output capturing that
    host.call() does, and is used when spawning the typ subprocess if needed.

TBR=dtu@chromium.org
BUG=402172

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

Cr-Commit-Position: refs/heads/master@{#300052}

Summary of changes available at:
https://chromium.googlesource.com/chromium/blink/+log/cd824db..513e6f3

TBR=cbiesinger@chromium.org,dstockwell@chromium.org

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

Cr-Commit-Position: refs/heads/master@{#300051}

This is to provide a more user friendly error. This:

  AssertionError: Pageset archive_data_file path must be specified.

instead of this:

  AttributeError: 'NoneType' object has no attribute 'AddNewTemporaryRecording'

BUG=

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

Cr-Commit-Position: refs/heads/master@{#300050}

The browser action button on the far right is invisible when user will
be dragging the browser actions container.

Because the browser action buttons, which don't locate at the extension
bar, will be set to be hidden when the dragging finished. So the hidden
buttons must be set to be visible before the dragging starts in the next
time in order to let user know which extension is coming.

BUG=424080
TEST=manually tested.

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

Cr-Commit-Position: refs/heads/master@{#300049}

- FILE_BROWSER_GALLERY_VIDEO_ERROR -> VIDEO_PLAYER_VIDEO_FILE_UNSUPPORTED
- FILE_BROWSER_GALLERY_VIDEO_DECODING_ERROR -> VIDEO_PLAYER_PLAYBACK_ERROR
- FILE_BROWSER_GALLERY_VIDEO_LOOPED_MODE -> IDS_VIDEO_PLAYER_LOOPED_MODE

BUG=none
TEST=none

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

Cr-Commit-Position: refs/heads/master@{#300048}

Create/Get have been replaced by Open which will first attempt to Get() followed by Create() if Get() failed.

CLs in this three-legged-patch:

*1. Add open to browser - https://codereview.chromium.org/664433003/
2. Add open (remove get/create) to Blink - https://codereview.chromium.org/638023003
3. Remove get/create from browser - https://codereview.chromium.org/663503002/

Note that the three CLs have been tested together and pass tests locally.

BUG=423942

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

Cr-Commit-Position: refs/heads/master@{#300047}

BUG=413855

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

Cr-Commit-Position: refs/heads/master@{#300046}

Some of the record_wpr unittests are causing issues. By passing in the --browser
flag with the existing browser_type for these tests, we should be able to avoid
this flakiness (caused by waiting for another browser to finish launching).

BUG=424439

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

Cr-Commit-Position: refs/heads/master@{#300045}

Apparently at one point in time the GetCommandLine methods returned no
value for a missing process. Now it always returns a value or an
exception. So this code should be looking for the exception instead.

BUG=

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

Cr-Commit-Position: refs/heads/master@{#300044}

Similarly to BufferingFileStreamReader, this CL introduces a analogical
BufferingFileStreamWriter class which groups consecutive Write() calls in order
to limit number of IPC calls.

TEST=browser_tests: *FileSystemProvider*BufferingFileStreamWriter*
BUG=398338

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

Cr-Commit-Position: refs/heads/master@{#300043}

Otherwise the decoder attempts to clear the nonexistent framebuffer, which causes an unexpected error and later a crash.

BUG=424024

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

Cr-Commit-Position: refs/heads/master@{#300042}

In this change only attachment payload is stored, support for metadata
will be in separate change.
Also in_memory_attachment_store_unittest is moved into test_template.
This test is now used for both stores.

BUG=
R=maniscalco@chromium.org

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

Cr-Commit-Position: refs/heads/master@{#300041}

BUG=423621

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

Cr-Commit-Position: refs/heads/master@{#300040}

The associated bugs look addressed. In some cases, even the mentioned
classes themselves are not present any more.

BUG=none
TEST=it compiles.

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

Cr-Commit-Position: refs/heads/master@{#300039}

Summary of changes available at:
https://chromium.googlesource.com/chromium/blink/+log/6998a13..cd824db

TBR=cbiesinger@chromium.org,dstockwell@chromium.org

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

Cr-Commit-Position: refs/heads/master@{#300038}

BUG=421393

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

Cr-Commit-Position: refs/heads/master@{#300037}

BUG=419210, 423634
R=cpu@chromium.org, eroman@chromium.org, sky@chromium.org

Committed: https://crrev.com/1aa0fa75b65f403e08ae0f3f2fcb053c02cd9ef2
Cr-Commit-Position: refs/heads/master@{#299737}

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

Cr-Commit-Position: refs/heads/master@{#300036}

The error message is "ERROR:location_bar_view_mac.mm(592)] Not implemented
reached in virtual void LocationBarViewMac::EndOriginChipAnimations(bool)".
It's been spamming for months, so it can't be that important.

BUG=408366

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

Cr-Commit-Position: refs/heads/master@{#300035}

There are still ARRAYSIZE_UNSAFEs in ppapi/tests/test_audio.cc, but it
defines it itself, and it might be compiled by some older NaCl toolchain
that doesn't support C++11. third_party/npapi/npspy/common/format.cpp
also has its own ARRAYSIZE_UNSAFE, but I'm hoping to delete npspy
completely.)

I'll remove ARRAYSIZE_UNSAFE from base/macro.h separately, since it's
quite likely we have deps that use our base (and those deps may use it).

R=ben@chromium.org
TBR=armansito@chromium.org
BUG=423134

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

Cr-Commit-Position: refs/heads/master@{#300034}

BUG=

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

Cr-Commit-Position: refs/heads/master@{#300033}

The image_url, small_image_url, and the button urls are never
set in the current code base. Removing these URLs also means
removing the ImageDownloads of MessageCenterNotificationManager,
and also means removing unnecessary methods of NotificationDelegate.

I think NotificationDelegate::id() can be removed and then
we can completely remove ::NotificationDelegate class for now,
but that would be done in another CL.

This CL depends on https://codereview.chromium.org/554213003/

BUG=None
R=dewittj@chromium.org, peter@chromium.org
TBR=atwilson@chromium.org, vitalybuka@chromium.org, tapted@chromium.org, oshima@chromium.org
TEST=no functional changes, build succeeds

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

Cr-Commit-Position: refs/heads/master@{#300032}

This change enabled fullscreen api for chrome shell

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

Cr-Commit-Position: refs/heads/master@{#300031}

BUG=423621

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

Cr-Commit-Position: refs/heads/master@{#300030}

In certain cases, we don't want to allow encrypting all data
(AKA, the CUSTOM_PASSPHRASE type).

BUG=424187

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

Cr-Commit-Position: refs/heads/master@{#300029}

base: Add discardable memory test that check behavior when creating enough instances that could use up a 32-bit address space.

BUG=422953
TEST=base_unittests --gtest_filter=DiscardableMemoryTests/DiscardableMemoryTest.AddressSpace/*

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

Cr-Commit-Position: refs/heads/master@{#300028}

Currently, when there are two registration and above one is unregistered, console message by above registration is left in top console box though the bottom one goes to the top. Then, the message in each box is set to the message recorded internally when update() is called.

BUG=418568
TEST=N/A

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

Cr-Commit-Position: refs/heads/master@{#300027}

Now that crbug.com/422508 is fixed, we can remove the call that made the
test work.

BUG=422508

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

Cr-Commit-Position: refs/heads/master@{#300026}

There are some DEBUG_CHECKs here verifying that a size_t is >= 0. Since a size_t is unsigned, these are trivially true and with the default warning settings for CrOS builds, the warning generated here will cause a build failure.

BUG=424334
TEST=ensure existing builds pass

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

Cr-Commit-Position: refs/heads/master@{#300025}

Cleanup to make future migration to std::move() easier.

BUG=423621

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

Cr-Commit-Position: refs/heads/master@{#300024}

$ git log f9e401bab..a57d7db40 --date=short --format='%ad %ae %s' | sed 's/@chromium\.org//'
2014-10-15 maruel Add --cache to isolateserver.py download.
2014-10-15 maruel Preparation work to enable isolateserver.py download to use a local cache.

TBR=vadimsh@chromium.org
BUG=

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

Cr-Commit-Position: refs/heads/master@{#300023}

Revert of Revert of Revert of ChromeOS NetworkScreenHandler should not call CheckAndResolveLocale on UI thread. (patchs… (patchset #1 id:1 of https://codereview.chromium.org/658903004/)

Reason for revert:
Speculatively reverting (second time) for https://crbug.com/422703 .

Original issue's description:
> Revert of Revert of ChromeOS NetworkScreenHandler should not call CheckAndResolveLocale on UI thread. (patchs… (patchset #1 id:1 of https://codereview.chromium.org/653123002/)
>
> Reason for revert:
> The speculative revert didn't help.
>
> Original issue's description:
> > Revert of ChromeOS NetworkScreenHandler should not call CheckAndResolveLocale on UI thread. (patchset #17 id:320001 of https://codereview.chromium.org/620563002/)
> >
> > Reason for revert:
> > Speculatively reverting for https://crbug.com/422703
> >
> > Original issue's description:
> > > ChromeOS NetworkScreenHandler should not call CheckAndResolveLocale on UI thread.
> > >
> > > BUG=419030
> > >
> > > Committed: https://crrev.com/d52dffddbf0abe5ca55509a9cdedcd9b0674bb1c
> > > Cr-Commit-Position: refs/heads/master@{#298860}
> >
> > TBR=dzhioev@chromium.org,nkostylev@chromium.org
> > NOTREECHECKS=true
> > NOTRY=true
> > BUG=422703
> >
> > Committed: https://crrev.com/f09329c436f4929c1f9ffd59e49aa4aef441338b
> > Cr-Commit-Position: refs/heads/master@{#299480}
>
> TBR=dzhioev@chromium.org,nkostylev@chromium.org
> NOTREECHECKS=true
> NOTRY=true
> BUG=422703
>
> Committed: https://crrev.com/f4af00363b57344b5d024791b751d966a34b0eb8
> Cr-Commit-Position: refs/heads/master@{#299883}

TBR=dzhioev@chromium.org,nkostylev@chromium.org
NOTREECHECKS=true
NOTRY=true
BUG=422703

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

Cr-Commit-Position: refs/heads/master@{#300022}

Before this change, MojoApplicatioHost was activated only if some
browser-side modules requested it. There was no way for renderer
to activate it.

This CL let MojoApplicationHost always get activated. Now
renderer can request services it needs without its own activation flow.

R=darin@chromium.org,tsepez@chromium.org
TEST=none
BUG=none

Committed: https://crrev.com/bd0aa4dc467c10a4301e44084d8abbba92a35e56
Cr-Commit-Position: refs/heads/master@{#294969}

Committed: https://crrev.com/2c0a563d3019940343e804c7e975bef1bd0f5b54
Cr-Commit-Position: refs/heads/master@{#297551}

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

Cr-Commit-Position: refs/heads/master@{#300021}

BUG=423621

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

Cr-Commit-Position: refs/heads/master@{#300020}

BUG=421695
TEST=gpu_unittests, webgl_conformance_tests
R=bajones@chromium.org

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

Cr-Commit-Position: refs/heads/master@{#300019}

R=mek@chromium.org
BUG=414526

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

Cr-Commit-Position: refs/heads/master@{#300018}

Implemented all the stages of the controller out-of-box.
Known issues:
* Most of the stages are not shown now during the real out-of-box, because C++
  part is not ready yet.
* Some transitions look bad/don't work.
* Spinner is wrong.

BUG=375191
TEST=manually

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

Cr-Commit-Position: refs/heads/master@{#300017}

BUG=423621

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

Cr-Commit-Position: refs/heads/master@{#300016}

This is another followup of crrev.com/297738 and crrev.com/298333.
We need to set 0 to border-top, border-bottom, margin-top, and
margin-bottom in order that a static text element has no height.

BUG=421338

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

Cr-Commit-Position: refs/heads/master@{#300015}

This splits the macro into two:

int a = 1;
ALLOW_UNUSED_LOCAL(a);

and

typedef Foo Bar ALLOW_UNUSED_TYPE;
void foo() ALLOW_UNUSED_TYPE;  // ALLOW_UNUSED_TYPE_OR_FUNC seemed too verbose

This matches changes that have already been made in Blink.

BUG=81439
TEST=none
TBR=ben

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

Cr-Commit-Position: refs/heads/master@{#300014}