GitLab

BUG=None
TBR=rockot@chromium.org
NOTRY=true
NOPRESUBMIT=true

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

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

TTS initialiation can take around ~150ms to initialize on Android. Trace
this initialization to make it clear if the task is blocking critical
updates on the browser UI thread.

BUG=406597

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

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

BUG=

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

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

Since Write() takes two (or more) steps to finish, it's possible for
CdmFileIOImpl to be destructed after SetLength() is called but before
WriteFile() is called. In that case, since we SetLength(data_size), we
could end up with a corrupted file.

This CL update FileIOTest to explictly cover this case.

BUG=410630
TEST=ECKEncryptedMediaTest.FileIOTest still passes.

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

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

DisplayManager wasn't handling multiply nested things correctly and the
client library wasn't handling uploading to multiple views from the same
application. This fixes those.  There's still some z-ordering issues that
show up pretty visibly. This hacks up texture alpha so you can see
partially through views to try to debug.

BUG=

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

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

Creating a framework for suppressing pollution of the profiler data and applying for know cases of pollution.
See the bug.

The CL introduces TaskStopwatch that has to be used to measure run time for tasks. It takes care of double-counting
run time in the nested-tasks case by subtracting run time of nested tasks from their parents. TaskStopwatch can be
also used for subtracting other nested intervals, such as the time while a nested message pump runs. ThreadData::TallyADeath
now takes a stopwatch parameter instead of start_time and end_time. This helps avoid mistakes when the interval passed up to
the parent for exclusion is different from the interval for the current task duration.

BUG=401560

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

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

Annotate several NavigationController methods used only in testing, and
remove a helper test function that is unused even in tests.

BUG=398263
NOTRY=true

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

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

BUG=409926

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

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

-[NSView setFrame:display:] accepts BOOL as the second argument,
passing nil only worked by accident.

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

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

Changing in the intialization order of WeakPtrFactory such that all
member variables should appear before the WeakPtrFactory to ensure
that any WeakPtrs to Controller are invalidated before its members
variable's destructors are executed, rendering them invalid.

BUG=303818

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

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

We are going deprecate console.timeline/timelineEnd. As the first step they will result only in time/timeEnd events and won't start/stop tracing. This change removes the code that allows to change browser wide tracing recording from the render process, it mostly reverts part of r292578.

BUG=412782
TBR=dcheng

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

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

BUG=NONE

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

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

BUG=387410
R=qyearsley@chromium.org

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

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

Also update the name of the asan dynamic runtime thunk .lib since it was changed in LLVM r217673.
After this change, component ASan build should be able to report NULL derefs and other AVs the same way as the static build does.

BUG=345874

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

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

BUG=341951,371692
R=dmazzoni@chromium.org

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

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

Previously, a scope passed in explicitly via the command line was not applied consistently.

BUG=372381

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

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

This regressed in r294564.

BUG=413850

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

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

https://chromium.googlesource.com/chromium/blink/+log/1588dd41f3260c463323681a0d22a2180b6e6330..01025b523bac1a59fa91a10d5d49f969ed24000d
TBR=loislo@chromium.org,dpranke@chromium.org

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

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

The chrome.app.window API has been moved to src/extensions, so app_shell
no longer needs its custom chrome.shell API to open windows.

* Remove the API implementation -- mostly a manual revert of https://codereview.chromium.org/254473011/
* Remove app_shell_resources.grd since there are no more app_shell resources
* Remove athena's support for ShellAppWindow
* Fix example apps to remove the chrome.shell workaround.

No presubmit because it has a false-positive warning about the enum rename in extension_function_histogram_value.h

No try because no presubmit doesn't seem to work by itself.

BUG=413164
TEST=extensions_unittests, app_shell_unittests, app_shell_browsertests
TBR=isherman@chromium.org for deprecating an extension function histogram
NOPRESUBMIT=true
NOTRY=true

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

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

BUG=404239
NOTRY=True

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

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

Revert of Revert of Change PageTest.WillStartBrowser to depend on platform instead of browser object as it would not ha… (patchset #1 id:1 of https://codereview.chromium.org/562243004/)

Reason for revert:
Reverting the revert. Bot still failed with this CL reverted. Sorry for the noise.

Original issue's description:
> Revert of Change PageTest.WillStartBrowser to depend on platform instead of browser object as it would not ha… (patchset #1 id:1 of https://codereview.chromium.org/554583003/)
>
> Reason for revert:
> It looks like the moto E perf bots are broken. This change looks like it makes sense so I'm reverting.
>
> crbug.com/414267
>
> Original issue's description:
> > Change PageTest.WillStartBrowser to depend on platform instead of browser object as it would not have been created at this point.
> >
> > BUG=413481
> >
> > Committed: https://crrev.com/9bb65868ce3ac1e5aae3a844beacf62281c332b6
> > Cr-Commit-Position: refs/heads/master@{#294703}
>
> TBR=nduca@chromium.org,nednguyen@google.com,tonyg@chromium.org,chrishenry@google.com,wuhu@google.com
> NOTREECHECKS=true
> NOTRY=true
> BUG=413481
>
> Committed: https://crrev.com/5f712b7be45145522f183d683f11bfb243dd89e1
> Cr-Commit-Position: refs/heads/master@{#294816}

TBR=nduca@chromium.org,nednguyen@google.com,tonyg@chromium.org,chrishenry@google.com,wuhu@google.com
NOTREECHECKS=true
NOTRY=true
BUG=413481

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

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

NOTRY=true
BUG=413636

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

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

If you compile without preprocessing (first clang -E then clang, like
ccache does by default without CCACHE_CPP2), you will see warnings like:

you will see warnings like:

../../content/browser/accessibility/browser_accessibility_cocoa.mm:1155:12:error: implicit conversion of NULL constant to 'NSUInteger' (aka
'unsigned long') [-Werror,-Wnull-conversion]
    return __null;
    ~~~~~~ ^~~~~~
           0

It's because we return nil when 0 or NO should be returned instead,
normal clang compile won't catch that for some reason, perhaps because
its checking is less strict for preprocessed code.

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

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

The variable is set in this file but there is no other references to
it in the code base.

BUG=None

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

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

R=sdefresne@chromium.org,cmasone@chromium.org

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

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

https://skia.googlesource.com/skia/+log/5087b2c06762d0ea8b471f63953a587ecafaae0f..a63d5df455e5fb32ac995f753709893221de6147

CQ_EXTRA_TRYBOTS=tryserver.blink:linux_blink_rel,linux_blink_dbg
TBR=stephana@google.com

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

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

This pulls in the following Native Client changes:

r13740: (dschuff) Update revision for PNaCl r13726->r13739
r13741: (dbrazdil) Implemented sysconf(), sbrk(), malloc() in MinSFI.
r13742: (dyen) Added testing for file dup/dup2 functions for irt_ext.
r13743: (jvoung) Add a checkdeps stage to the PNaCl bots/trybots as well.
r13744: (bradnelson) Only allow fastbuild to affect symbol level.
r13745: (jvoung) PNaCl: Update Subzero revision in pnacl/COMPONENT_REVISIONS
r13746: (sbc) Add creat() declaration to newlib fcntl.h

BUG=none
TEST=browser_tests and nacl_integration
CQ_EXTRA_TRYBOTS=tryserver.chromium.linux:linux_rel_precise32,linux_nacl_sdk,linux_chromium_dbg

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

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

We report when:
1, The first time a file is fully read.
2, After the CDM throws an error with a special system code.

BUG=413812
TEST=Tested and both new UMA show in about://histograms.

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

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

We need to make a copy of them first in order to keep Buildbot working.
Only after this lands we can update the Buildbot code to use the new
path.

BUG=338338
TEST=None
R=sky@chromium.org,dpranke@chromium.org

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

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

This is similar to https://codereview.chromium.org/328423002/ where we started archiving browser_tests for Windows.

BUG=

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

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

Upon receiving a notification that network connectivity has been
restored, ASI will clear any backoff delay for uploading attachments.

BUG=411981

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

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

It also adds a Manifest struct in content/public/common.

BUG=409696

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

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

Keeps a vector of cache names in creation order.  They're written to and read from the the index file in the same order.

This simplifies the keys unit tests as the order of the vectors is always the same and they can now be compared directly.

BUG=408787

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

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

Adding overview / layer framework to Activities so that unloaded / sleeping activities can be shown with as little resources as possible

The new |ContentProxy| object is a placeholder for now which only creates a single color view. It can use the |web_view| to generate the image preview in the next step and will work for WebContents as well as App's.

So more work is necessary, this patch is mainly to get the overview mode working again when running out of memory (and avoiding that it gets too big).

Adding sky for the DEPS inclusion.

BUG=408837
TEST=unittests and visual
TBR=sky@chromium.org

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

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

- this-null.patch has landed upstream, remove the patch file
- msvs2013 has a stdint.h, so remove our downstream stdint.h copy in re2
- msvs2013 has va_copy, so don't define it locally

BUG=414049
NOTRY=true

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

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

The new utility function should be used instead of manually flushing the
blocking pool.

This CL replaces BlockingPool::FlushForTesting and RunLoop::RunUntilIdle()
wherever it was obvious. Along the way lint errors have been fixed.

TEST=Refactoring only. All current tests should pass.
BUG=392716

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

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

The Clone() method was added in https://codereview.chromium.org/6670033, but is no longer called anywhere. This CL removes that dead code.

BUG=384873

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

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

TBR=hellner@chromium.org

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

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

typedef std::vector<std::pair<std::string, std::string> > StringPairs;
And use iterator to iterate stl::vector element.

BUG=412250

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

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

internals page tab
This CL fixes that issue by always making sure that at the end
of each test, the current tab is changed from the password
internal page to the test page even if an exception is thrown.

BUG=413273

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

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