0
Commit Graph

349 Commits

Author SHA1 Message Date
29a7adabb3 WebView: update outdated docs for test shell
WebView no longer supports the android_webview_apk target for anything
other than automated testing. Keeping docs for building an unsupported
target is misleading, especially since android_webview_apk is not
actually a WebView.

This CL keeps the description about WebView, but removes the parts
about the test shell. It has a pointer to our team's chromium wiki page
for building and installing AOSP WebView.

This renames the section to "Build WebView" instead of "Build WebView
shell" to reflect the update.

BUG=703376

Review-Url: https://codereview.chromium.org/2770983007
Cr-Commit-Position: refs/heads/master@{#459221}
2017-03-23 21:17:08 +00:00
5c66dfe5bd Update vscode document
Add chromium-codesearch extension and update settings.

BUG=

Review-Url: https://codereview.chromium.org/2761363003
Cr-Commit-Position: refs/heads/master@{#458736}
2017-03-22 13:51:45 +00:00
2109641dd6 Android: specify the supported version of Android Studio.
Now that 2.3 has been released, it will be the default version downloaded
by developers. //build/android/gradle/generate_gradle.py doesn't work
with 2.3 yet, and this should be reflected in our docs (we only support
2.2 currently).

BUG=700438

Review-Url: https://codereview.chromium.org/2758053002
Cr-Commit-Position: refs/heads/master@{#458060}
2017-03-20 14:57:02 +00:00
0f156e19b0 Update documentation for base::ResetAndReturn(Callback*)
Added the method to callback.md because it's helpful yet I hadn't discovered it (callback_helpers.h is a more obscure file than callback.h or bind_helpers.h)

BUG=none
R=tzik@chromium.org

Review-Url: https://codereview.chromium.org/2755073002
Cr-Commit-Position: refs/heads/master@{#457939}
2017-03-18 02:49:09 +00:00
136f215ed3 Revert of Android: Support Android Studio 2.3 (patchset id:20001 of https://codereview.chromium.org/2746103004/ )
Reason for revert:
Breaks code editing in Android Studio, see http://crbug/700438 for more info.

Original issue's description:
> Android: Support Android Studio 2.3
>
> Also fixes excludes filters not working. For files whose directory is
> not fully included in the target, add those individually instead.
>
> BUG=700438
>
> Review-Url: https://codereview.chromium.org/2746103004
> Cr-Commit-Position: refs/heads/master@{#457093}
> Committed: f9fca7ca02

TBR=wnwen@chromium.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=700438

Review-Url: https://codereview.chromium.org/2758683003
Cr-Commit-Position: refs/heads/master@{#457866}
2017-03-17 19:59:19 +00:00
f9fca7ca02 Android: Support Android Studio 2.3
Also fixes excludes filters not working. For files whose directory is
not fully included in the target, add those individually instead.

BUG=700438

Review-Url: https://codereview.chromium.org/2746103004
Cr-Commit-Position: refs/heads/master@{#457093}
2017-03-15 15:47:29 +00:00
4a9c82efe7 use namespace base for base::make_optional in docs
BUG=none
TBR=mlamouri@chromium.org

Review-Url: https://codereview.chromium.org/2749983002
Cr-Commit-Position: refs/heads/master@{#456964}
2017-03-15 02:44:26 +00:00
126f704d1c Fix OnceCallback example in docs/callback.md
Fixes the original example, which resulted in an invalid conversion from
base::OnceCallback<void(int)> to base::Closure. Also adds a comment that
this usage is not yet supported.

BUG=none
R=tzik@chromium.org

Review-Url: https://codereview.chromium.org/2735903002
Cr-Commit-Position: refs/heads/master@{#456886}
2017-03-14 23:22:53 +00:00
8c9318ff00 Add basic debugging instructions for Android Studio.
The docs still need to be updated with more detailed instructions and
tips, but currently the docs don't even mention debugging in Android
Studio.

BUG=620034

Review-Url: https://codereview.chromium.org/2740353003
Cr-Commit-Position: refs/heads/master@{#456185}
2017-03-10 22:16:35 +00:00
6694c0e5e6 Remove dependency on scan-build wrapper script for analysis builds.
This CL replaces the Clang 'c++-analyzer' Perl script with logic added to the Python wrapper script. This gives us the hooks we need to run analysis builds with "gomacc" (necessary for distributed builds) and gives us a convenient place to specify analyzer config flags which aren't exposed by scan-build.

Adds a warning suppression rule which prevents the analyzer from raising false positives from stdlib code (code w/namespace "std").

Other changes:
* Modify Goma portions of the GCC toolchain GNI to build paths to
  the Clang static analyzer.
* Create an exception for ASM tool invocations, which don't play
  well with the analysis command line flags.
* Removed 'scan-build' DEP.

BUG=687245
R=thakis@chromium.org,wez@chromium.org

Review-Url: https://codereview.chromium.org/2667853004
Cr-Commit-Position: refs/heads/master@{#456136}
2017-03-10 19:47:21 +00:00
8638ef810c Android: Update docs and install-build-deps-android.sh to require java8
We don't strictly require java8 yet, but will soon.

BUG=662467

Review-Url: https://codereview.chromium.org/2666783004
Cr-Commit-Position: refs/heads/master@{#455459}
2017-03-08 15:44:31 +00:00
ae16e9d846 Clarify that cdb is required to be installed
TBR=brucedawson@chromium.org
BUG=

Review-Url: https://codereview.chromium.org/2735343002
Cr-Commit-Position: refs/heads/master@{#455409}
2017-03-08 09:04:42 +00:00
8cccf2141e Move ios documentation from //docs to //docs/ios.
TBR=cmasso@chromium.org
BUG=none

Review-Url: https://codereview.chromium.org/2731293003
Cr-Commit-Position: refs/heads/master@{#454972}
2017-03-06 22:12:02 +00:00
6fca2b025f change clang-format settings in docs/vscode.md and docs/qtcreator.md
BUG=

Review-Url: https://codereview.chromium.org/2729853008
Cr-Commit-Position: refs/heads/master@{#454640}
2017-03-03 19:10:51 +00:00
eb1ff6b205 Add MANIFEST.json to .gitignore, regenerate it from template in test runner
This solves 2 problems:

1. Regenerating MANIFEST.json is failing on ToT (bug 697207)
- Now that MANIFEST.json is in .gitignore, regeneration works
2. Regenerating adds MANIFEST.json to MANIFEST.json (bug 683485)
- MANIFEST.json no longer shows up in MANIFEST.json

BUG=697207,666957,683485

Change-Id: I09987b66027e1f94055888c8ead58e7e40896d62
Reviewed-on: https://chromium-review.googlesource.com/447959
Commit-Queue: Jeff Carpenter <jeffcarp@chromium.org>
Reviewed-by: Quinten Yearsley <qyearsley@chromium.org>
Cr-Commit-Position: refs/heads/master@{#454454}
2017-03-03 00:48:43 +00:00
ca28511013 Update vscode document
1. Add setting to disable indexing.
2. Add TOCss

BUG=

Review-Url: https://codereview.chromium.org/2725023002
Cr-Commit-Position: refs/heads/master@{#454267}
2017-03-02 15:39:04 +00:00
9e590b21b7 Spruce up "ChromeVox on Desktop Linux" docs
TBR=dmazzoni@chromium.org
BUG=none
NOTRY=true

Review-Url: https://codereview.chromium.org/2719253003
Cr-Commit-Position: refs/heads/master@{#454199}
2017-03-02 07:11:20 +00:00
9cd1712465 Added docs on accessibility tests
BUG=none
NOTRY=true
TBR=nektar@chromium.org

Review-Url: https://codereview.chromium.org/2728663004
Cr-Commit-Position: refs/heads/master@{#454197}
2017-03-02 06:58:46 +00:00
e442e9bea6 Update patts docs to describe editing ebuild too
BUG=none
NOTRY=true
TBR=dtseng@chromium.org

Review-Url: https://codereview.chromium.org/2726173002
Cr-Commit-Position: refs/heads/master@{#454190}
2017-03-02 06:08:08 +00:00
6e560d0309 Create Android Studio chromium style configuration.
The Android Studio Style configuration does not match the checkstyle
configuration, which causes import order to be wrong. The configuration
is specified here:
//tools/android/checkstyle/chromium-style-5.0.xml
under ImportOrder.

This CL copies the style configuration from the Android repository,
but changes it so that it matches what we have in checkstyle.

It also updates the Android Studio documentation to point to the
new file, and adds the new content and the documentation to the
watchlist for Android Studio in addition to updating the error message
from checkstyle to now refer to the new style file.

BUG=None

Review-Url: https://codereview.chromium.org/2710573004
Cr-Commit-Position: refs/heads/master@{#453542}
2017-02-28 07:49:59 +00:00
b4016a52a0 fix a README link
BUG=none

Review-Url: https://codereview.chromium.org/2722543003
Cr-Commit-Position: refs/heads/master@{#453402}
2017-02-28 00:06:07 +00:00
8f65e273d2 Note Options deprecation in more places
Trying to prevent folks from creating large CLs in Options only to
discover we intend to deprecate it shortly. This codifies a few
assumptions we've been making about where new code should go.

BUG=none
R=dbeam@chromium.org
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation

Review-Url: https://codereview.chromium.org/2719093002
Cr-Commit-Position: refs/heads/master@{#453304}
2017-02-27 19:50:56 +00:00
wez
0334393935 Correct some details in the instructions for modifying TTS engine.
BUG=616636
NOTRY=true

Review-Url: https://codereview.chromium.org/2711263005
Cr-Commit-Position: refs/heads/master@{#453043}
2017-02-25 02:01:42 +00:00
2f19855a57 Call out the importance of the --work argument for wpt manifest
R=qyearsley@chromium.org

Review-Url: https://codereview.chromium.org/2717643002
Cr-Commit-Position: refs/heads/master@{#452848}
2017-02-24 16:42:35 +00:00
0ac7dd29d6 Cleanup virtual test suites for origin trials
Remove the 'stable' virtual suite for origin trials layout tests. This
was redundant, as it had the identical expectations as the 'disabled'
virtual suite.

Rename/change the 'disabled' virtual suite to be based strictly on
runtime flags.

BUG=685788

Review-Url: https://codereview.chromium.org/2705643002
Cr-Commit-Position: refs/heads/master@{#452552}
2017-02-23 18:16:17 +00:00
9d63b30c1e Document how to build Chromium for devices.
Document how to build Chromium for devices, how to configure the
Code Signing Identity to use and which Mobile Provisioning Profile
are required to build Chromium and the tests for device.

BUG=694604

Review-Url: https://codereview.chromium.org/2712513003
Cr-Commit-Position: refs/heads/master@{#452459}
2017-02-23 11:24:51 +00:00
3cbcbf54d4 Add iOS VoiceOver doc.
TBR=michaeldo@chromium.org,cmasso@chromium.org
BUG=none

Review-Url: https://codereview.chromium.org/2709623006
Cr-Commit-Position: refs/heads/master@{#452452}
2017-02-23 10:59:32 +00:00
14e3f8a133 Document how to write configs for iOS bots
BUG=686969
R=justincohen@chromium.org, sdefresne@chromium.org

Review-Url: https://codereview.chromium.org/2695813003 .
Cr-Commit-Position: refs/heads/master@{#452274}
2017-02-22 23:38:36 +00:00
8171050893 docs: Change "ChromeOS" to "Chrome OS".
Correct a common typo/meme in the hopes of reducing its
spread.

BUG=none
TBR=brettw@chromium.org

Review-Url: https://codereview.chromium.org/2697323005
Cr-Commit-Position: refs/heads/master@{#452121}
2017-02-22 17:57:55 +00:00
3b70458933 Markdown formatting nit to fix a link in docs.
TBR=sashab@chromium.org

Review-Url: https://codereview.chromium.org/2706063006
Cr-Commit-Position: refs/heads/master@{#452032}
2017-02-22 14:10:48 +00:00
5999edb1de Mention Chromium X-Refs plugin in the sublime text docs
TBR=sashab@chromium.org

Review-Url: https://codereview.chromium.org/2706393003
Cr-Commit-Position: refs/heads/master@{#452019}
2017-02-22 13:02:35 +00:00
e0ee33c5a7 Add a common mistake case for header-only classes in components build doc
R=brettw@chromium.org

Review-Url: https://codereview.chromium.org/2686133003
Cr-Commit-Position: refs/heads/master@{#451390}
2017-02-17 22:41:21 +00:00
eda32ab004 Link to new web-platform-tests.org documentation from our own
Many pages of testthewebforward.org now redirect, so replace links with
something appropriate from the new documentation.

Since the new documentation consistently says "web-platform-tests"
instead of "Web Platform Tests" and this is what folks like jgraham tend
to write as well, align our own usage with that.

BUG=657117

Review-Url: https://codereview.chromium.org/2696323003
Cr-Commit-Position: refs/heads/master@{#451045}
2017-02-16 19:21:58 +00:00
23599b7624 Fix some links in documentation related to layout tests.
Review-Url: https://codereview.chromium.org/2695963006
Cr-Commit-Position: refs/heads/master@{#451037}
2017-02-16 19:10:42 +00:00
2f48975e26 Update accessibility documentation.
Move existing overview to accessibility/ subdirectory and make
accessibility.md a table of contents.

Migrate ChromeVox documentation to markdown.
Add new documentation for PATTS and BRLTTY.
Update accessibility LayoutTests documentation.

BUG=none
TBR=dtseng@chromium.org,nektar@chromium.org
NOTRY=true

Review-Url: https://codereview.chromium.org/2701603002
Cr-Commit-Position: refs/heads/master@{#450865}
2017-02-16 03:39:16 +00:00
df2a863bb9 Update web-platform-tests docs
These updates address a number of issues that have come up in the past
week since the 2-way process was launched. See:
https://groups.google.com/a/chromium.org/d/msg/blink-dev/QIRcSgN9AK4/0JY58rBhCAAJ
https://groups.google.com/a/chromium.org/d/msg/platform-predictability/SBnz6Um7y2A/pDXVSOrpDwAJ

Drive-by: fix section links in other documentation.

BUG=657117
R=rbyers@chromium.org

Review-Url: https://codereview.chromium.org/2695963002
Cr-Commit-Position: refs/heads/master@{#450696}
2017-02-15 15:03:16 +00:00
a7fdb063c5 Update Android junit docs to make it clear how to specify out path.
Previous command-line specified --release which only worked if you
build under out/Release. New instructions make it clear how to
specify a custom one - uses my_build which GN docs use as an
example too.

BUG=none
TBR=tedchoc@chromium.org

Review-Url: https://codereview.chromium.org/2692273004
Cr-Commit-Position: refs/heads/master@{#450464}
2017-02-14 20:57:14 +00:00
58e0b3fed0 Add Chromium documentation guidelines.
BUG=691882
TBR=brettw

Review-Url: https://codereview.chromium.org/2691203002
Cr-Commit-Position: refs/heads/master@{#450229}
2017-02-14 03:55:39 +00:00
ecf5359e68 Fix a command in android build doc
BUG=

Review-Url: https://codereview.chromium.org/2691923003
Cr-Commit-Position: refs/heads/master@{#450184}
2017-02-14 01:20:23 +00:00
ba312ce746 Add doc for vscode
BUG=

Review-Url: https://codereview.chromium.org/2687013006
Cr-Commit-Position: refs/heads/master@{#449883}
2017-02-12 03:38:25 +00:00
d51c79fa34 Remove references to or invocations of vulcanize.py. It's deleted.
Vulcanize now happens dynamically with GN/Ninja. \o/

R=dpapad@chromium.org
BUG=673825
NOTRY=true

Review-Url: https://codereview.chromium.org/2690743003
Cr-Commit-Position: refs/heads/master@{#449875}
2017-02-11 20:58:07 +00:00
bbd0f45dec Link to web_platform_tests.md where appropriate
R=qyearsley@chromium.org,rbyers@chromium.org

Review-Url: https://codereview.chromium.org/2689843002
Cr-Commit-Position: refs/heads/master@{#449799}
2017-02-11 00:09:53 +00:00
9c9781a50e Clarify documentation about importing new WPT directories.
This CL also adds a section about what kinds of changes
will be upstreamed, and the contributing guidelines for
web-platform-tests.

Review-Url: https://codereview.chromium.org/2681293004
Cr-Commit-Position: refs/heads/master@{#449798}
2017-02-11 00:08:39 +00:00
c74a4c9c6b Add documentation for Origin Trials integration
This CL adds documentation for feature authors to integrate with the
origin trials framework.

This moves content that was previously hosted in GitHub:
https://github.com/jpchase/OriginTrials/blob/gh-pages/developer-guide.md

Also updated to reflect recent code changes.

Review-Url: https://codereview.chromium.org/2693503002
Cr-Commit-Position: refs/heads/master@{#449722}
2017-02-10 20:38:09 +00:00
419a91a17b Gtk3: More fixes and refactorings
This CL
* Uses entry colors instead of window colors for the NTP
* Fixes a leak involving ScopedGObject
* Simplifies how background and border colors are calculated
* Adds the ".label" class only on versions of Gtk before 3.20
* Fixes text selection colors on one theme
* Updates the doc to reflect the new way colors are calculated

BUG=132847

Review-Url: https://codereview.chromium.org/2683953005
Cr-Commit-Position: refs/heads/master@{#449709}
2017-02-10 20:05:48 +00:00
6299b9132a Update and slighly expand WPT docs
See https://groups.google.com/a/chromium.org/d/msg/platform-predictability/SBnz6Um7y2A/pDXVSOrpDwAJ
for more discussion.

Review-Url: https://codereview.chromium.org/2690553003
Cr-Commit-Position: refs/heads/master@{#449658}
2017-02-10 18:06:57 +00:00
0227e4a51b Reland "Android: Add owned resources to android studio"
Original Reland: https://codereview.chromium.org/2687453002/

Changes:
- Fix TypeError: 'NoneType' object is not iterable

TBR=agrieve@chromium.org
BUG=682846

Review-Url: https://codereview.chromium.org/2680423005
Cr-Commit-Position: refs/heads/master@{#449408}
2017-02-09 21:14:51 +00:00
2542e2ce6a Revert of Reland "Android: Add owned resources to android studio" (patchset id:40001 of https://codereview.chromium.org/2687453002/ )
Reason for revert:
Nonetype is not iterable.

Original issue's description:
> Reland "Android: Add owned resources to android studio"
>
> Original CL: https://codereview.chromium.org/2667023002/
>
> Changes:
> - Generalize srcjars and resource zips to be generated together.
> - Update documentation
>
> BUG=682846
>
> Review-Url: https://codereview.chromium.org/2687453002
> Cr-Commit-Position: refs/heads/master@{#449345}
> Committed: 5c3f5db756

TBR=estevenson@chromium.org,agrieve@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=682846

Review-Url: https://codereview.chromium.org/2684843006
Cr-Commit-Position: refs/heads/master@{#449375}
2017-02-09 19:34:08 +00:00
d040b0be6a Fix typo in code review documentation.
TBR=jam@chromium.org

Review-Url: https://codereview.chromium.org/2684693007
Cr-Commit-Position: refs/heads/master@{#449369}
2017-02-09 19:11:33 +00:00
5c3f5db756 Reland "Android: Add owned resources to android studio"
Original CL: https://codereview.chromium.org/2667023002/

Changes:
- Generalize srcjars and resource zips to be generated together.
- Update documentation

BUG=682846

Review-Url: https://codereview.chromium.org/2687453002
Cr-Commit-Position: refs/heads/master@{#449345}
2017-02-09 17:48:27 +00:00