Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(139)

Issue 2866823002: bluetooth: README files updated to improve cross-linking. (Closed)

Created:
3 years, 7 months ago by scheib
Modified:
3 years, 7 months ago
Reviewers:
ortuno
CC:
blink-reviews, chromium-reviews, darin-cc_chromium.org, fuzzing_chromium.org, haraken, jam, ortuno+watch_chromium.org, scheib+watch_chromium.org
Target Ref:
refs/heads/master
Project:
chromium
Visibility:
Public.

Description

bluetooth: README files updated to improve cross-linking. A few locations used //paths that did not work, they are changed to ../../relative/paths. Additional links are added to other files, and minor improvements to page titles. Also, header style is changed to use '#' characters, per styleguide. R=ortuno@chromium.org Review-Url: https://codereview.chromium.org/2866823002 . Cr-Commit-Position: refs/heads/master@{#470452} Committed: https://chromium.googlesource.com/chromium/src/+/ff0919b6b5e43b9a604bfdc98492a26430b14b1a

Patch Set 1 #

Patch Set 2 : Use '/' for path root more. #

Patch Set 3 : merge TOT #

Unified diffs Side-by-side diffs Delta from patch set Stats (+40 lines, -26 lines) Patch
M content/browser/bluetooth/README.md View 1 2 chunks +11 lines, -9 lines 0 comments Download
M device/bluetooth/README.md View 1 2 chunks +4 lines, -1 line 0 comments Download
M device/bluetooth/public/interfaces/test/README.md View 1 2 chunks +2 lines, -2 lines 0 comments Download
M device/bluetooth/test/README.md View 1 3 chunks +6 lines, -2 lines 0 comments Download
M third_party/WebKit/Source/modules/bluetooth/README.md View 1 1 chunk +15 lines, -10 lines 0 comments Download
M third_party/WebKit/Source/modules/bluetooth/testing/clusterfuzz/README.md View 1 chunk +2 lines, -2 lines 0 comments Download

Messages

Total messages: 20 (9 generated)
scheib
3 years, 7 months ago (2017-05-05 20:53:15 UTC) #2
ortuno
Could it be that we incorrectly used '//'? I see some instances that use a ...
3 years, 7 months ago (2017-05-08 01:24:18 UTC) #3
scheib
Take another look.
3 years, 7 months ago (2017-05-08 20:18:37 UTC) #4
ortuno
lgtm
3 years, 7 months ago (2017-05-08 21:42:16 UTC) #5
commit-bot: I haz the power
CQ is trying da patch. Follow status at: https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.org/2866823002/20001
3 years, 7 months ago (2017-05-08 22:08:55 UTC) #7
commit-bot: I haz the power
Try jobs failed on following builders: win_chromium_compile_dbg_ng on master.tryserver.chromium.win (JOB_FAILED, http://build.chromium.org/p/tryserver.chromium.win/builders/win_chromium_compile_dbg_ng/builds/406145)
3 years, 7 months ago (2017-05-08 22:51:11 UTC) #9
commit-bot: I haz the power
CQ is trying da patch. Follow status at: https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.org/2866823002/40001
3 years, 7 months ago (2017-05-10 00:01:09 UTC) #12
commit-bot: I haz the power
Try jobs failed on following builders: cast_shell_linux on master.tryserver.chromium.linux (JOB_TIMED_OUT, build has not started yet; ...
3 years, 7 months ago (2017-05-10 02:07:51 UTC) #14
commit-bot: I haz the power
CQ is trying da patch. Follow status at: https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.org/2866823002/40001
3 years, 7 months ago (2017-05-10 02:26:21 UTC) #16
commit-bot: I haz the power
Try jobs failed on following builders: chromium_presubmit on master.tryserver.chromium.linux (JOB_TIMED_OUT, build has not started yet; ...
3 years, 7 months ago (2017-05-10 04:28:05 UTC) #18
scheib
3 years, 7 months ago (2017-05-10 04:42:51 UTC) #20
Message was sent while issue was closed.
Committed patchset #3 (id:40001) manually as
ff0919b6b5e43b9a604bfdc98492a26430b14b1a (presubmit successful).

Powered by Google App Engine
This is Rietveld 408576698