diff options
-rw-r--r-- | app/os_exchange_data_provider_win.cc | 3 | ||||
-rw-r--r-- | base/base_drop_target.h | 3 | ||||
-rw-r--r-- | chrome/browser/accessibility_win_browsertest.cc | 2 | ||||
-rw-r--r-- | chrome/browser/autocomplete/autocomplete_edit.h | 3 | ||||
-rw-r--r-- | chrome/browser/browser_process_impl.cc | 2 | ||||
-rw-r--r-- | chrome/browser/login_prompt_uitest.cc | 4 | ||||
-rw-r--r-- | chrome/browser/search_engines/template_url_model.cc | 2 | ||||
-rw-r--r-- | chrome_frame/chrome_frame_npapi.cc | 3 | ||||
-rw-r--r-- | webkit/tools/test_shell/drag_delegate.cc | 3 |
9 files changed, 10 insertions, 15 deletions
diff --git a/app/os_exchange_data_provider_win.cc b/app/os_exchange_data_provider_win.cc index ed33b3c..6e11714 100644 --- a/app/os_exchange_data_provider_win.cc +++ b/app/os_exchange_data_provider_win.cc @@ -303,7 +303,8 @@ void OSExchangeDataProviderWin::SetURL(const GURL& url, data_->contents_.push_back(new DataObjectImpl::StoredDataInfo( ClipboardUtil::GetUrlFormat()->cfFormat, storage)); - // TODO(beng): (http://b/1085501) add CF_HTML... + // TODO(beng): add CF_HTML. + // http://code.google.com/p/chromium/issues/detail?id=6767 // Also add text representations (these should be last since they're the // least preferable). diff --git a/base/base_drop_target.h b/base/base_drop_target.h index 8be0d36..0b9d524 100644 --- a/base/base_drop_target.h +++ b/base/base_drop_target.h @@ -28,9 +28,6 @@ class BaseDropTarget : public IDropTarget { // When suspended is set to |true|, the drop target does not receive drops // from drags initiated within the owning HWND. - // TODO(beng): (http://b/1085385) figure out how we will handle legitimate - // drag-drop operations within the same HWND, such as dragging - // selected text to an edit field. bool suspended() const { return suspended_; } void set_suspended(bool suspended) { suspended_ = suspended; } diff --git a/chrome/browser/accessibility_win_browsertest.cc b/chrome/browser/accessibility_win_browsertest.cc index 06985da..8ebafa2 100644 --- a/chrome/browser/accessibility_win_browsertest.cc +++ b/chrome/browser/accessibility_win_browsertest.cc @@ -20,8 +20,6 @@ namespace { class AccessibilityWinBrowserTest : public InProcessBrowserTest { public: void SetUpCommandLine(CommandLine* command_line) { - // Turns on the accessibility in the renderer. Off by default until - // http://b/issue?id=1432077 is fixed. command_line->AppendSwitch(switches::kEnableRendererAccessibility); } diff --git a/chrome/browser/autocomplete/autocomplete_edit.h b/chrome/browser/autocomplete/autocomplete_edit.h index 66d6b98..41e0d85 100644 --- a/chrome/browser/autocomplete/autocomplete_edit.h +++ b/chrome/browser/autocomplete/autocomplete_edit.h @@ -19,8 +19,9 @@ class AutocompleteEditController; class AutocompleteEditModel; class AutocompleteEditView; -// TODO(pkasting): http://b/1343512 The names and contents of the classes in +// TODO(pkasting): The names and contents of the classes in // this file are temporary. I am in hack-and-slash mode right now. +// http://code.google.com/p/chromium/issues/detail?id=6772 // Embedders of an AutocompleteEdit widget must implement this class. class AutocompleteEditController { diff --git a/chrome/browser/browser_process_impl.cc b/chrome/browser/browser_process_impl.cc index 05ea69f..75a47c2 100644 --- a/chrome/browser/browser_process_impl.cc +++ b/chrome/browser/browser_process_impl.cc @@ -253,7 +253,7 @@ void BrowserProcessImpl::EndSession() { printing::PrintJobManager* BrowserProcessImpl::print_job_manager() { // TODO(abarth): DCHECK(CalledOnValidThread()); - // See <http://b/1287209>. + // http://code.google.com/p/chromium/issues/detail?id=6828 // print_job_manager_ is initialized in the constructor and destroyed in the // destructor, so it should always be valid. DCHECK(print_job_manager_.get()); diff --git a/chrome/browser/login_prompt_uitest.cc b/chrome/browser/login_prompt_uitest.cc index 24cf8f3..e1dc5af 100644 --- a/chrome/browser/login_prompt_uitest.cc +++ b/chrome/browser/login_prompt_uitest.cc @@ -110,10 +110,6 @@ TEST_F(LoginPromptTest, TestTwoAuths) { ASSERT_TRUE( digest_tab->NavigateToURL(server->TestServerPage("auth-digest"))); - // TODO(devint): http://b/1158262 basic_tab is not active, so this logs in to - // a page whose tab isn't active, which isn't actually possible for the user - // to do. I had a fix for this, but I'm reverting it to see if it makes the - // test less flaky. EXPECT_TRUE(basic_tab->NeedsAuth()); EXPECT_TRUE(basic_tab->SetAuth(username_basic_, password_)); EXPECT_TRUE(digest_tab->NeedsAuth()); diff --git a/chrome/browser/search_engines/template_url_model.cc b/chrome/browser/search_engines/template_url_model.cc index 361bfb8..8ad9cfe 100644 --- a/chrome/browser/search_engines/template_url_model.cc +++ b/chrome/browser/search_engines/template_url_model.cc @@ -151,7 +151,7 @@ std::wstring TemplateURLModel::GenerateKeyword(const GURL& url, return std::wstring(); // Strip "www." off the front of the keyword; otherwise the keyword won't work - // properly. See http://b/issue?id=1205573. + // properly. See http://code.google.com/p/chromium/issues/detail?id=6984 . return net::StripWWW(UTF8ToWide(url.host())); } diff --git a/chrome_frame/chrome_frame_npapi.cc b/chrome_frame/chrome_frame_npapi.cc index 5ae267a..f6a0174 100644 --- a/chrome_frame/chrome_frame_npapi.cc +++ b/chrome_frame/chrome_frame_npapi.cc @@ -1399,7 +1399,8 @@ NPObject* ChromeFrameNPAPI::GetWindowObject() const { bool ChromeFrameNPAPI::GetBrowserIncognitoMode() { bool incognito_mode = false; - // Check disabled for Opera due to bug: http://b/issue?id=1815494 + // Check disabled for Opera due to bug: + // http://code.google.com/p/chromium/issues/detail?id=24287 if (GetBrowserType() != BROWSER_OPERA) { // Check whether host browser is in private mode; NPBool private_mode = FALSE; diff --git a/webkit/tools/test_shell/drag_delegate.cc b/webkit/tools/test_shell/drag_delegate.cc index 3b693a9..faa50d8 100644 --- a/webkit/tools/test_shell/drag_delegate.cc +++ b/webkit/tools/test_shell/drag_delegate.cc @@ -14,7 +14,8 @@ namespace { void GetCursorPositions(HWND hwnd, gfx::Point* client, gfx::Point* screen) { // GetCursorPos will fail if the input desktop isn't the current desktop. - // See http://b/1173534. (0,0) is wrong, but better than uninitialized. + // (0,0) is wrong, but better than uninitialized. + // We should clean up all callers to handle failure -- http://b/1208177 . POINT pos; if (!GetCursorPos(&pos)) { pos.x = 0; |