Commit 4bbaf6c4 authored by Darwin Huang's avatar Darwin Huang Committed by Commit Bot

Clipboard: Remove note filler word.

Refactoring change only. No functional change.

Change-Id: If333b87c14cf8c9e06cd3bae544c5e71ce998678
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2458825
Commit-Queue: Darwin Huang <huangdarwin@chromium.org>
Auto-Submit: Darwin Huang <huangdarwin@chromium.org>
Reviewed-by: default avatarJarryd Goodman <jarrydg@chromium.org>
Cr-Commit-Position: refs/heads/master@{#815447}
parent 5b8ac9a4
......@@ -4,7 +4,7 @@ This directory contains async clipboard tests automated through use of
Chrome-specific test helper `permissions-helper.js`. Related tests not using
`permissions-helper.js` can be found in
[`web_tests/external/wpt/clipboard-apis/`](https://cs.chromium.org/chromium/src/third_party/blink/web_tests/external/wpt/clipboard-apis/).
Please note that these tests previously could not be run manually, as there
These tests previously could not be run manually, as there
was no exposed way to allow a permission in automated tests.
TODO(https://crbug.com/1076691): Since the
......
......@@ -67,7 +67,7 @@ promise_test(async t => {
}, 'Verify write and read clipboard given platform-specific raw input');
</script>
<p>
Note: This is a manual test because it writes/reads to the shared system
This is a manual test because it writes/reads to the shared system
clipboard and thus cannot be run async with other tests that might interact
with the clipboard.
</p>
......@@ -49,7 +49,7 @@ promise_test(async t => {
}, 'Verify write and read clipboard given 2 platform-neutral raw inputs');
</script>
<p>
Note: This is a manual test because it writes/reads to the shared system
This is a manual test because it writes/reads to the shared system
clipboard and thus cannot be run async with other tests that might interact
with the clipboard.
</p>
......@@ -460,7 +460,7 @@ void ClipboardAndroid::ReadAsciiText(ClipboardBuffer buffer,
*result = g_map.Get().Get(ClipboardFormatType::GetPlainTextType());
}
// Note: |src_url| isn't really used. It is only implemented in Windows.
// |src_url| isn't really used. It is only implemented in Windows.
// |data_dst| is not used. It's only passed to be consistent with other
// platforms.
void ClipboardAndroid::ReadHTML(ClipboardBuffer buffer,
......@@ -606,7 +606,7 @@ void ClipboardAndroid::WriteRTF(const char* rtf_data, size_t data_len) {
NOTIMPLEMENTED();
}
// Note: according to other platforms implementations, this really writes the
// According to other platforms implementations, this really writes the
// URL spec.
void ClipboardAndroid::WriteBookmark(const char* title_data,
size_t title_len,
......
......@@ -24,7 +24,7 @@ jboolean JNI_ClipboardAndroidTestSupport_NativeWriteHtml(
const base::android::JavaParamRef<jstring>& j_html_text) {
{
// Simulate something writing HTML to the clipboard in native.
// NOTE: Android requires both a plaintext and HTML version.
// Android requires both a plaintext and HTML version.
base::string16 html_text;
base::android::ConvertJavaStringToUTF16(env, j_html_text, &html_text);
std::string url;
......
......@@ -263,7 +263,7 @@ const ClipboardFormatType& ClipboardFormatType::GetFileDescriptorType() {
// static
const ClipboardFormatType& ClipboardFormatType::GetFileContentZeroType() {
// Note this uses a storage media type of TYMED_HGLOBAL, which is not commonly
// This uses a storage media type of TYMED_HGLOBAL, which is not commonly
// used with CFSTR_FILECONTENTS (but used in Chromium--see
// OSExchangeDataProviderWin::SetFileContents). Use GetFileContentAtIndexType
// if TYMED_ISTREAM and TYMED_ISTORAGE are needed.
......
......@@ -26,7 +26,7 @@ class ClipboardInternal;
class COMPONENT_EXPORT(UI_BASE_CLIPBOARD) ClipboardNonBacked
: public Clipboard {
public:
// Returns the in-memory clipboard for the current thread. Note that this
// Returns the in-memory clipboard for the current thread. This
// method must *only* be used when the caller is sure that the clipboard for
// the current thread is in fact an instance of ClipboardNonBacked.
static ClipboardNonBacked* GetForCurrentThread();
......
// Copyright (c) 2012 The Chromium Authors. All rights reserved.
// Copyright 2012 The Chromium Authors. All rights reserved.
// Use of this source code is governed by a BSD-style license that can be
// found in the LICENSE file.
//
// Note: This header doesn't use REGISTER_TYPED_TEST_SUITE_P like most
// This header doesn't use REGISTER_TYPED_TEST_SUITE_P like most
// type-parameterized gtests. There are lot of test cases in here that are only
// enabled on certain platforms. However, preprocessor directives in macro
// arguments result in undefined behavior (and don't work on MSVC). Instead,
......@@ -884,7 +884,7 @@ TYPED_TEST(ClipboardTest, PlatformSpecificDataTest) {
this->clipboard().ReadAsciiText(ClipboardBuffer::kCopyPaste,
/* data_dst = */ nullptr, &text_result);
EXPECT_EQ(text_result, text);
// Note: Windows will automatically convert CF_TEXT to its UNICODE version.
// Windows will automatically convert CF_TEXT to its UNICODE version.
EXPECT_TRUE(this->clipboard().IsFormatAvailable(
ClipboardFormatType::GetPlainTextType(), ClipboardBuffer::kCopyPaste,
/* data_dst = */ nullptr));
......
......@@ -822,7 +822,7 @@ SkBitmap ClipboardWin::ReadImageInternal(ClipboardBuffer buffer) const {
// Since Windows uses premultiplied alpha, we scan for instances where
// (R, G, B) > A. If there are any invalid premultiplied colors in the image,
// we assume the alpha channel contains garbage and force the bitmap to be
// opaque as well. Note that this heuristic will fail on a transparent bitmap
// opaque as well. This heuristic will fail on a transparent bitmap
// containing only black pixels...
SkPixmap device_pixels(SkImageInfo::MakeN32Premul(bitmap->bmiHeader.biWidth,
bitmap->bmiHeader.biHeight),
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment