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

Side by Side Diff: webkit/plugins/npapi/webplugin_impl.cc

Issue 14307012: webkit: Remove use of ALLOW_THIS_IN_INITIALIZER_LIST. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 7 years, 7 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch | Annotate | Revision Log
OLDNEW
1 // Copyright (c) 2012 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "webkit/plugins/npapi/webplugin_impl.h" 5 #include "webkit/plugins/npapi/webplugin_impl.h"
6 6
7 #include "base/bind.h" 7 #include "base/bind.h"
8 #include "base/debug/crash_logging.h" 8 #include "base/debug/crash_logging.h"
9 #include "base/logging.h" 9 #include "base/logging.h"
10 #include "base/memory/linked_ptr.h" 10 #include "base/memory/linked_ptr.h"
(...skipping 462 matching lines...) Expand 10 before | Expand all | Expand 10 after
473 page_delegate_(page_delegate), 473 page_delegate_(page_delegate),
474 webframe_(webframe), 474 webframe_(webframe),
475 delegate_(NULL), 475 delegate_(NULL),
476 container_(NULL), 476 container_(NULL),
477 plugin_url_(params.url), 477 plugin_url_(params.url),
478 load_manually_(params.loadManually), 478 load_manually_(params.loadManually),
479 first_geometry_update_(true), 479 first_geometry_update_(true),
480 ignore_response_error_(false), 480 ignore_response_error_(false),
481 file_path_(file_path), 481 file_path_(file_path),
482 mime_type_(UTF16ToASCII(params.mimeType)), 482 mime_type_(UTF16ToASCII(params.mimeType)),
483 ALLOW_THIS_IN_INITIALIZER_LIST(weak_factory_(this)) { 483 weak_factory_(this) {
484 DCHECK_EQ(params.attributeNames.size(), params.attributeValues.size()); 484 DCHECK_EQ(params.attributeNames.size(), params.attributeValues.size());
485 StringToLowerASCII(&mime_type_); 485 StringToLowerASCII(&mime_type_);
486 486
487 for (size_t i = 0; i < params.attributeNames.size(); ++i) { 487 for (size_t i = 0; i < params.attributeNames.size(); ++i) {
488 arg_names_.push_back(params.attributeNames[i].utf8()); 488 arg_names_.push_back(params.attributeNames[i].utf8());
489 arg_values_.push_back(params.attributeValues[i].utf8()); 489 arg_values_.push_back(params.attributeValues[i].utf8());
490 } 490 }
491 491
492 // Set subresource URL for crash reporting. 492 // Set subresource URL for crash reporting.
493 base::debug::SetCrashKeyValue("subresource_url", plugin_url_.spec()); 493 base::debug::SetCrashKeyValue("subresource_url", plugin_url_.spec());
(...skipping 878 matching lines...) Expand 10 before | Expand all | Expand 10 after
1372 webframe_->setReferrerForRequest(*request, plugin_url_); 1372 webframe_->setReferrerForRequest(*request, plugin_url_);
1373 break; 1373 break;
1374 1374
1375 default: 1375 default:
1376 break; 1376 break;
1377 } 1377 }
1378 } 1378 }
1379 1379
1380 } // namespace npapi 1380 } // namespace npapi
1381 } // namespace webkit 1381 } // namespace webkit
OLDNEW
« no previous file with comments | « webkit/plugins/npapi/webplugin_delegate_impl_win.cc ('k') | webkit/plugins/ppapi/content_decryptor_delegate.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698